linux-kernel - Re: [PATCH v5] ACPI/IORT: Fix PMCG node single ID mapping handling

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Wed, 20 May 2020 18:43:56 +0100
From:   Will Deacon <will@...nel.org>
To:     Lorenzo Pieralisi <lorenzo.pieralisi@....com>
Cc:     Tuan Phan <tuanphan@...amperecomputing.com>,
        patches@...erecomputing.com, Hanjun Guo <guohanjun@...wei.com>,
        Sudeep Holla <sudeep.holla@....com>,
        "Rafael J. Wysocki" <rjw@...ysocki.net>,
        Len Brown <lenb@...nel.org>,
        Robin Murphy <robin.murphy@....com>,
        Shameer Kolothum <shameerali.kolothum.thodi@...wei.com>,
        linux-acpi@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v5] ACPI/IORT: Fix PMCG node single ID mapping handling

On Wed, May 20, 2020 at 06:27:36PM +0100, Lorenzo Pieralisi wrote:
> On Wed, May 20, 2020 at 10:13:07AM -0700, Tuan Phan wrote:
> > An IORT PMCG node can have no ID mapping if its overflow interrupt is
> > wire based therefore the code that parses the PMCG node can not assume
> > the node will always have a single mapping present at index 0.
> > 
> > Fix iort_get_id_mapping_index() by checking for an overflow interrupt
> > and mapping count.
> > 
> > Fixes: 24e516049360 ("ACPI/IORT: Add support for PMCG")
> > 
> > Acked-by: Lorenzo Pieralisi <lorenzo.pieralisi@....com>
> > Reviewed-by: Hanjun Guo <guoahanjun@...wei.com>
> > Signed-off-by: Tuan Phan <tuanphan@...amperecomputing.com>
> > ---
> > v1 -> v2:
> > - Use pmcg node to detect wired base overflow interrupt.
> > 
> > v2 -> v3:
> > - Address Hanjun and Robin's comments.
> > 
> > v3 -> v4:
> > - Update the title and description as mentioned by Lorenzo.
> > 
> > v4 -> v5:
> > - Remove period in the title and commit references.
> > 
> >  drivers/acpi/arm64/iort.c | 5 +++++
> 
> Hi Will,
> 
> is there a chance we can get this patch into v5.8 ? I understand
> we are very late in the cycle but I wanted to ask (it applies cleanly
> to for-next/acpi).

Sorry, Lorenzo -- I didn't notice that this had been Acked already. I was
waiting for somebody to chime in! I'll queue it ASAP for 5.8.

Will