Re: [PATCH] x86/efi-bgrt: remove the check of the version field

2016-08-24 Thread Dave Young
On 08/22/16 at 04:49pm, Icenowy Zheng wrote: > > > 22.08.2016, 15:28, "Dave Young" : > > On 08/18/16 at 09:41pm, Matt Fleming wrote: > >>  On Wed, 17 Aug, at 01:44:13PM, Dave Young wrote: > >>  > > >>  > Could we add some quirk for these broken hardware instead of changing >

Re: [PATCH] x86/efi-bgrt: remove the check of the version field

2016-08-24 Thread Dave Young
On 08/22/16 at 04:49pm, Icenowy Zheng wrote: > > > 22.08.2016, 15:28, "Dave Young" : > > On 08/18/16 at 09:41pm, Matt Fleming wrote: > >>  On Wed, 17 Aug, at 01:44:13PM, Dave Young wrote: > >>  > > >>  > Could we add some quirk for these broken hardware instead of changing > >>  > the normal

Re: [PATCH] x86/efi-bgrt: remove the check of the version field

2016-08-22 Thread Dave Young
On 08/18/16 at 09:41pm, Matt Fleming wrote: > On Wed, 17 Aug, at 01:44:13PM, Dave Young wrote: > > > > Could we add some quirk for these broken hardware instead of changing > > the normal code? > > I'd prefer not to do that if possible. Due to the way that the BIOS > ecosystem works, this kind

Re: [PATCH] x86/efi-bgrt: remove the check of the version field

2016-08-22 Thread Dave Young
On 08/18/16 at 09:41pm, Matt Fleming wrote: > On Wed, 17 Aug, at 01:44:13PM, Dave Young wrote: > > > > Could we add some quirk for these broken hardware instead of changing > > the normal code? > > I'd prefer not to do that if possible. Due to the way that the BIOS > ecosystem works, this kind

Re: [PATCH] x86/efi-bgrt: remove the check of the version field

2016-08-18 Thread Matt Fleming
On Wed, 17 Aug, at 01:44:13PM, Dave Young wrote: > > Could we add some quirk for these broken hardware instead of changing > the normal code? I'd prefer not to do that if possible. Due to the way that the BIOS ecosystem works, this kind of broken firmware spreads across the industry, appearing

Re: [PATCH] x86/efi-bgrt: remove the check of the version field

2016-08-18 Thread Matt Fleming
On Wed, 17 Aug, at 01:44:13PM, Dave Young wrote: > > Could we add some quirk for these broken hardware instead of changing > the normal code? I'd prefer not to do that if possible. Due to the way that the BIOS ecosystem works, this kind of broken firmware spreads across the industry, appearing

Re: [PATCH] x86/efi-bgrt: remove the check of the version field

2016-08-16 Thread Dave Young
On 08/15/16 at 01:56pm, Matt Fleming wrote: > On Tue, 09 Aug, at 01:25:46PM, Icenowy Zheng wrote: > > Some broken firmwares have a wrongly filled version field in BGRT table. > > (See http://wiki.osdev.org/Broken_UEFI_implementations ) > > > > As we know, these firmwares can also provide correct

Re: [PATCH] x86/efi-bgrt: remove the check of the version field

2016-08-16 Thread Dave Young
On 08/15/16 at 01:56pm, Matt Fleming wrote: > On Tue, 09 Aug, at 01:25:46PM, Icenowy Zheng wrote: > > Some broken firmwares have a wrongly filled version field in BGRT table. > > (See http://wiki.osdev.org/Broken_UEFI_implementations ) > > > > As we know, these firmwares can also provide correct

Re: [PATCH] x86/efi-bgrt: remove the check of the version field

2016-08-15 Thread Josh Triplett
On Mon, Aug 15, 2016 at 01:56:43PM +0100, Matt Fleming wrote: > On Tue, 09 Aug, at 01:25:46PM, Icenowy Zheng wrote: > > Some broken firmwares have a wrongly filled version field in BGRT table. > > (See http://wiki.osdev.org/Broken_UEFI_implementations ) > > > > As we know, these firmwares can

Re: [PATCH] x86/efi-bgrt: remove the check of the version field

2016-08-15 Thread Josh Triplett
On Mon, Aug 15, 2016 at 01:56:43PM +0100, Matt Fleming wrote: > On Tue, 09 Aug, at 01:25:46PM, Icenowy Zheng wrote: > > Some broken firmwares have a wrongly filled version field in BGRT table. > > (See http://wiki.osdev.org/Broken_UEFI_implementations ) > > > > As we know, these firmwares can

Re: [PATCH] x86/efi-bgrt: remove the check of the version field

2016-08-15 Thread Matt Fleming
On Tue, 09 Aug, at 01:25:46PM, Icenowy Zheng wrote: > Some broken firmwares have a wrongly filled version field in BGRT table. > (See http://wiki.osdev.org/Broken_UEFI_implementations ) > > As we know, these firmwares can also provide correct BGRT image, although > the table is wrong. > > After

Re: [PATCH] x86/efi-bgrt: remove the check of the version field

2016-08-15 Thread Matt Fleming
On Tue, 09 Aug, at 01:25:46PM, Icenowy Zheng wrote: > Some broken firmwares have a wrongly filled version field in BGRT table. > (See http://wiki.osdev.org/Broken_UEFI_implementations ) > > As we know, these firmwares can also provide correct BGRT image, although > the table is wrong. > > After

Re: [PATCH] x86/efi-bgrt: remove the check of the version field

2016-08-11 Thread Ingo Molnar
* Icenowy Zheng wrote: > > > 10.08.2016, 20:52, "Ingo Molnar" : > > * Icenowy Zheng wrote: > > > >>  Some broken firmwares have a wrongly filled version field in BGRT table. > >>  (See http://wiki.osdev.org/Broken_UEFI_implementations ) >

Re: [PATCH] x86/efi-bgrt: remove the check of the version field

2016-08-11 Thread Ingo Molnar
* Icenowy Zheng wrote: > > > 10.08.2016, 20:52, "Ingo Molnar" : > > * Icenowy Zheng wrote: > > > >>  Some broken firmwares have a wrongly filled version field in BGRT table. > >>  (See http://wiki.osdev.org/Broken_UEFI_implementations ) > >> > >>  As we know, these firmwares can also provide

Re: [PATCH] x86/efi-bgrt: remove the check of the version field

2016-08-10 Thread Ingo Molnar
* Icenowy Zheng wrote: > Some broken firmwares have a wrongly filled version field in BGRT table. > (See http://wiki.osdev.org/Broken_UEFI_implementations ) > > As we know, these firmwares can also provide correct BGRT image, although > the table is wrong. > > After removing

Re: [PATCH] x86/efi-bgrt: remove the check of the version field

2016-08-10 Thread Ingo Molnar
* Icenowy Zheng wrote: > Some broken firmwares have a wrongly filled version field in BGRT table. > (See http://wiki.osdev.org/Broken_UEFI_implementations ) > > As we know, these firmwares can also provide correct BGRT image, although > the table is wrong. > > After removing the check of the