https://bugzilla.kernel.org/show_bug.cgi?id=187311

--- Comment #22 from Lv Zheng <lv.zh...@intel.com> ---
(In reply to Pavel Machek from comment #20)
> Seems to do the trick... but it still changes behaviour for header.length >
> 0x0f4 and < 0x100. Are you sure that's a good idea?

Why will ">= V3_SIZE && < V4_SIZE" matter a regression fix?

The bisected commit did only the following functional change:
-    if (acpi_gbl_FADT.header.length <= ACPI_FADT_V2_SIZE) {
+    if (acpi_gbl_FADT.header.length <= ACPI_FADT_V3_SIZE) {
As V2_SIZE actually doesn't exist in the world (according to Bob, it should
only be used by IA64 test platforms), IMO, changing the following could be a
sufficient regression fix.
-    if (acpi_gbl_FADT.header.length <= ACPI_FADT_V3_SIZE) {
+    if (acpi_gbl_FADT.header.length < ACPI_FADT_V3_SIZE) {

-- 
You are receiving this mail because:
You are watching the assignee of the bug.

------------------------------------------------------------------------------
_______________________________________________
acpi-bugzilla mailing list
acpi-bugzilla@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla

Reply via email to