linux-kernel - 答复: [PATCH] irqchip/gic-v3: Check SRE bit for GICv2 legacy support

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, 6 Jan 2021 09:22:18 +0000
From:   wanghuiqiang <wanghuiqiang@...wei.com>
To:     Shameerali Kolothum Thodi <shameerali.kolothum.thodi@...wei.com>,
        "'Ard Biesheuvel'" <ardb@...nel.org>
CC:     'Marc Zyngier' <maz@...nel.org>,
        "'eric.auger@...hat.com'" <eric.auger@...hat.com>,
        "'linux-kernel@...r.kernel.org'" <linux-kernel@...r.kernel.org>,
        "'linux-arm-kernel@...ts.infradead.org'" 
        <linux-arm-kernel@...ts.infradead.org>,
        Linuxarm <linuxarm@...wei.com>, "xuwei (O)" <xuwei5@...wei.com>
Subject: 答复: [PATCH] irqchip/gic-v3: Check SRE bit for GICv2 legacy support

Hi Ard and all,

The issue is root caused, it is introduced by BIOS new feature implemented. 
With old BIOS，we use static MADT table and the GICV/GICH is set to 0 and reported this table to OS. But we added new features which will dynamic update MADT table based on some external input, the developer is set GICV/GICH as what we have done like previous generation chipset code did. But in fact, there is different compared with old generation chipset code.
I'll let my internal team know this and fix this issue in later BIOS release.

Thanks!  

-----邮件原件-----
发件人: wanghuiqiang 
发送时间: 2020年12月15日 15:49
收件人: Shameerali Kolothum Thodi <shameerali.kolothum.thodi@...wei.com>; Ard Biesheuvel <ardb@...nel.org>
抄送: Marc Zyngier <maz@...nel.org>; eric.auger@...hat.com; linux-kernel@...r.kernel.org; linux-arm-kernel@...ts.infradead.org; Linuxarm <linuxarm@...wei.com>; xuwei (O) <xuwei5@...wei.com>
主题: 答复: [PATCH] irqchip/gic-v3: Check SRE bit for GICv2 legacy support

Sorry response late.
Hi Shameer & Ard,

Could you let me know which firmware you are using? If the difference is Madt table vGIC your pointed , they are the same. We changed the vGIC memory base address at very early design stage.

Thanks！ 

-----邮件原件-----
发件人: Shameerali Kolothum Thodi
发送时间: 2020年12月2日 16:23
收件人: Ard Biesheuvel <ardb@...nel.org>
抄送: Marc Zyngier <maz@...nel.org>; eric.auger@...hat.com; linux-kernel@...r.kernel.org; linux-arm-kernel@...ts.infradead.org; Linuxarm <linuxarm@...wei.com>; wanghuiqiang <wanghuiqiang@...wei.com>; xuwei (O) <xuwei5@...wei.com>
主题: RE: [PATCH] irqchip/gic-v3: Check SRE bit for GICv2 legacy support

[+]

> -----Original Message-----
> From: Ard Biesheuvel [mailto:ardb@...nel.org]
> Sent: 30 November 2020 18:32
> To: Shameerali Kolothum Thodi <shameerali.kolothum.thodi@...wei.com>
> Cc: Marc Zyngier <maz@...nel.org>; eric.auger@...hat.com; 
> linux-kernel@...r.kernel.org; linux-arm-kernel@...ts.infradead.org;
> Linuxarm <linuxarm@...wei.com>
> Subject: Re: [PATCH] irqchip/gic-v3: Check SRE bit for GICv2 legacy 
> support
> 
...

> 
> Any clue why production D06 firmware deviates from the D06 port that 
> exists in Tianocore's edk2-platforms repository? Because that version 
> does not have this bug, and I wonder why that code was upstreamed at 
> all if a substantially different version gets shipped with production 
> hardware.

Ok. Thanks for pointing this out. I have informed our UEFI team about this.
They will check Internally and clarify.

Regards,
Shameer