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

--- Comment #26 from Lv Zheng <[email protected]> ---
(In reply to Chris Bainbridge from comment #24)
> > attachment 208611 [details]
> 
> After this patch (ACPI 2.0 / AML: Fix
> module level execution by correctly parsing table as TermList) is
> applied all of the "Initializing Region" lines are missing.

Sounds like a wrong rebase result of attachment 208591.
As we changed the region initialization code in the regression fix (attachment
208201)...
It's hard to say if this is a problem because all object initialization code
will be invoked again before the object is accessed.
But anyway, this is too aggressive to have this cleanup done in this patch.
So I rebased attachment 208591 to attachment 209221
And here is the refreshed test request:

1. Please download the linux-next branch of linux-pm.git:
https://git.kernel.org/cgit/linux/kernel/git/rafael/linux-pm.git/?h=linux-next

2. Please apply the following patches and try to see if the problem still
exists:
attachment 208201
attachment 208151
attachment 208161
attachment 208171
attachment 208181
attachment 208191
attachment 208591 -> attachment 209221
attachment 208601
attachment 208611
Please help to confirm if anything wrong with the addtional 3 patches applied.
Please also tell me the acpi_ut_display_init_pathname() order for this test.

> 
> Before:
> 
> [    0.000000] ACPI: ECDT 0x0000000088D87000 000053 (v01 APPLE  Apple00 
> 00000001 Loki 0000005F)
> [    0.201717] ACPI : EC: EC description table is found, configuring boot EC
> [    0.201729] ACPI : EC: EC started
> [    0.234444] Initializing Region       \_PR.PPMT
...
> [    0.244677] Initializing Region       \RCRB
> [    0.250763] Executing  Method       \_SB._STA
> [    0.250773] Executing  Method       \_SB._INI
> [    0.250904] Executing  Method       \_SB.PCI0._STA
> [    0.250914] Executing  Method       \_SB.PCI0._INI
> [    0.251067] Executing  Method       \_SB.PCI0.XHC1._STA
> [    0.251076] Executing  Method       \_SB.PCI0.XHC1._INI
> [    0.311848] ACPI : EC: EC stopped
> [    0.312125] ACPI : EC: GPE = 0x17, I/O: command/status = 0x66, data = 0x62
> [    0.312131] ACPI : EC: EC started
> [    0.312216] Executing  Method       \_SB.PCI0.LPCB.EC._REG
> [    0.407484] ACPI: SBS HC: EC = 0xffff880260b24800, offset = 0x20,
> query_bit = 0x10

Good indications.

> 
> After:
> 
> [    0.000000] ACPI: ECDT 0x0000000088D87000 000053 (v01 APPLE  Apple00 
> 00000001 Loki 0000005F)
> [    0.204426] ACPI : EC: EC description table is found, configuring boot EC
> [    0.204438] ACPI : EC: EC started
> [    0.242853] Executing  Method       \_SB._STA
> [    0.242864] Executing  Method       \_SB._INI
> [    0.242998] Executing  Method       \_SB.PCI0._STA
> [    0.243008] Executing  Method       \_SB.PCI0._INI
> [    0.243163] Executing  Method       \_SB.PCI0.XHC1._STA
> [    0.243171] Executing  Method       \_SB.PCI0.XHC1._INI
> [    0.301474] ACPI : EC: EC stopped
> [    0.301768] ACPI : EC: GPE = 0x17, I/O: command/status = 0x66, data = 0x62
> [    0.301775] ACPI : EC: EC started
> [    0.301859] Executing  Method       \_SB.PCI0.LPCB.EC._REG
> [    0.397768] ACPI: SBS HC: EC = 0xffff88026099c800, offset = 0x20,
> query_bit = 0x10

As I said above, the region initialization can be done in a different place.
Can you see strange things happening there without this object initialization
step executed?
For example, still suffer from the ODEBUG issue?

Thanks
-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://pubads.g.doubleclick.net/gampad/clk?id=278785231&iu=/4140
_______________________________________________
acpi-bugzilla mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla

Reply via email to