Cc ACPI people for thoughts..
On 06/01/17 at 03:17am, Maniaxx wrote:
> On 27.05.2017 at 07:34 wrote Dave Young:
> >> Can you double check it works before the commit of below?
> linux-git-4.10rc5.r115.gc4c39c70c5fe is fine (1 commit back)
> linux-git-4.10rc5.r116.g7b0a911478c7 is not (early init BGRT patch)
>
> >> If machine
> >> boots fine then check /sys/firmware/acpi/bgrt/image see if it exists or
> >> not.
> Not sure. /sys/firmware/acpi/bgrt/ doesn't exist here but i have
> /sys/firmware/acpi/tables/ that contains all tables. BGRT is 60 bytes
> and holds the same binary content as pr_info hex_dump (in dmesg log below).
This means the efi_bgrt_init failed out originally before the early init BGRT
patch. Checking the code the only difference is current code we have no
below code:
status = acpi_get_table("BGRT", 0,
(struct acpi_table_header **)&bgrt_tab);
if (ACPI_FAILURE(status))
return;
So probably acpi_get_table has some more sanity checking and it failed
early. Can you add a printk before above return to confirm it? Just test
the old kernel without early init BGRT patch.
>
> >> if you can manange to save full kernel log it would be also helpful.
> Here is a log of linux-git-4.10rc5.r115.gc4c39c70c5fe with Sabrina
> Dubroca's test code applied:
> https://pastebin.com/ZkBNejrC
>
> This is how i applied the patch. I hope its correct:
> https://pastebin.com/see6V0Bk
>
Thanks
Dave
--
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
