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

--- Comment #6 from Lv Zheng <[email protected]> ---
The problem we know is:
The commit in question doesn't match the current kernel ECDT probing order.
It also blocks EC._REG execution invoked in ECDT probing.
And currently, there is no chance for the kernel to re-execute EC._REG.

But this is intended.
According to the spec, on ECDT platforms, EC._REG is not required to be
executed before accessing the EC opregion fields.

In the whole fix series, this problem is covered.
When EC probing happens, EC region handler will be installed without executing
_REG.
After namespace is fully initialized, EC region handler will be removed and
re-installed again using namespace EC, this time _REG will be executed.

Thus the problem we know is about the _REG execution and since it is fixed in
the whole series, I suggested you to try the whole fix series.

=====

But I'm not clear about what has been caused on your platform if the order is
not matched because I don't know what the "Initializing Region" is.
[    0.230469] Executing  Method       \_SB.PCI0.LPCB.EC._REG
[    0.230531] Initializing Region       \GNVS
[    0.230607] Initializing Region       \_SB.PCI0.LPCB.EC.ECOR
[    0.231043] Initializing Region       \_SB.PCI0.IGPU.IGDM
Could you tell me what function was executed for ’Initializing Region", or
paste your debugging patch here? So that I can know if there is something I
don't know.

=====

If there is nothing beyond what I know, then a better regression stopper fix
could apply (I'll post it later).
Which ensures "acpi_gbl_reg_methods_enabled = TRUE" executed after
acpi_ns_initialize_objects(). This is what we want to ensure. If it can work,
we'd better to have this upstreamed instead of the simple fix.

Thanks in advance
-Lv

-- 
You are receiving this mail because:
You are watching the assignee of the bug.
------------------------------------------------------------------------------
Transform Data into Opportunity.
Accelerate data analysis in your applications with
Intel Data Analytics Acceleration Library.
Click to learn more.
http://makebettercode.com/inteldaal-eval
_______________________________________________
acpi-bugzilla mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla

Reply via email to