On Tue, 2014-02-11 at 14:18 +0000, Matt Fleming wrote:
> (Entering the realm of ACPI, pulling in more folks)
>
> On Tue, 11 Feb, at 09:47:57PM, Madper Xie wrote:
> > Thanks for your reply. I attached the all the dmesg outputs for you.
> > [ 0.000000] ACPI: BGRT 00000000d9bb5100 00003C (v00 HPQOEM SLIC-BPC 01072009 AMI 00010013)
>
> Your BGRT table has an extra 4 bytes for some reason, (0x3C should be
> 0x38) and doesn't match the defintion in include/acpi/actbl3.h,
>
>
> struct acpi_table_bgrt {
> struct acpi_table_header header; /* Common ACPI table header */
> u16 version;
> u8 status;
> u8 image_type;
> u64 image_address;
> u32 image_offset_x;
> u32 image_offset_y;
> };
>
>
> which explains why when we dereference ->image_address we don't get a
> sensible value.
>
> Guys, what do you make of this? Firmware bug? Full thread here,
>
> http://article.gmane.org/gmane.linux.kernel.efi/3499
>
Hi Matt,
Yes, I agree that the table size should be 0x38. However, ACPI spec
states that bit0 of status indicates if the boot image graphic is valid.
This bit is set to 0 (invalid) on the system. Can you check this bit
and return when invalid?
Thanks,
-Toshi
--
To unsubscribe from this list: send the line "unsubscribe linux-efi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
