Date:	Fri, 5 Feb 2016 02:52:00 +0000
From:	"Zheng, Lv" <lv.zheng@...el.com>
To:	Chris Bainbridge <chris.bainbridge@...il.com>
CC:	"Moore, Robert" <robert.moore@...el.com>,
	"Wysocki, Rafael J" <rafael.j.wysocki@...el.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [BUG] ODEBUG: assert_init not available (active state 0)

Hi,

The below change is spec compliant.

I think you should first test again after the RefOrName stuffs reverted.

Then, if you still can see the issues, the story is:
===
We are enabling correct initialization order and table parsing facilities.
These patches are part of them.
===

So you could wait for just several days to test it again.
And report here if you can still see issues.

Thanks and best regards
-Lv

> From: Chris Bainbridge [mailto:chris.bainbridge@...il.com]
> Sent: Wednesday, February 3, 2016 6:37 PM
> 
> v4.5-rc2.
> Macbook 2012 (IVB).
> 
> ODEBUG reports an error or two at shutdown. Apart from the ODEBUG
> error(s), this bug also seems to cause some graphical corruption when
> typing in Firefox text fields.
> 
> 
> efaed9be998b5ae0afb7458e057e5f4402b43fa0 is the first bad commit
> commit efaed9be998b5ae0afb7458e057e5f4402b43fa0
> Author: Lv Zheng <lv.zheng@...el.com>
> Date:   Tue Dec 29 14:03:08 2015 +0800
> 
>     ACPICA: Events: Enhance acpi_ev_execute_reg_method() to ensure no _REG
> evaluations can happen during OS early boot stages
> 
>     ACPICA commit 31178590dde82368fdb0f6b0e466b6c0add96c57
> 
>     We can ensure no early _REG evaluations by ensuring the following rules in
>     acpi_ev_execute_reg_method():
>     1. If an address space handler is installed during early stage,
>        _REG(CONNECT) evaluations are blocked. This is achieved using
>        acpi_gbl_reg_methods_enabled which is renamed from
>        acpi_gbl_reg_methods_executed.
>     2. If _REG(CONNECT) has never been evalauted for the region object,
>        _REG(DISCONNECT) evaluations are blocked. This is achieved by a new
>        region object flag: AOPOBJ_REG_CONNECTED.
>     Note that, after applying this patch, we can ensure _REG(DISCONNECT) is
>     always paired to _REG(CONNECT). Lv Zheng
> 
>     Link: https://github.com/acpica/acpica/commit/31178590
>     Signed-off-by: Lv Zheng <lv.zheng@...el.com>
>     Signed-off-by: Bob Moore <robert.moore@...el.com>
>     Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@...el.com>
> 
> :040000 040000 357c50e20ee06ad557c77a2a03d09719dae9dc52
> 3e0610826dc9cb86454baf06de9392aa31e0aa91 M	drivers
> 
> 
> [   34.512758] ------------[ cut here ]------------
> [   34.512765] WARNING: CPU: 0 PID: 4975 at lib/debugobjects.c:263
> debug_print_object+0x85/0xa0()
> [   34.512770] ODEBUG: assert_init not available (active state 0) object type:
> timer_list hint: stub_timer+0x0/0x20
> [   34.512772] Modules linked in:
> [   34.512774] CPU: 0 PID: 4975 Comm: systemd Not tainted 4.4.0-rc7+ #353
> [   34.512776] Hardware name: Apple Inc. MacBookPro10,2/Mac-
> AFD8A9D944EA4843, BIOS MBP102.88Z.0106.B0A.1509130955 09/13/2015
> [   34.512779]  ffffffff81f9b41c ffff880227a1bdb0 ffffffff814ec829
> ffff880227a1bdf8
> [   34.512782]  ffff880227a1bde8 ffffffff810cd831 ffff880227a1be90
> ffffffff822514c0
> [   34.512785]  ffffffff81f9b4c2 ffffffff8327e288 00007f29190ba700
> ffff880227a1be48
> [   34.512786] Call Trace:
> [   34.512790]  [<ffffffff814ec829>] dump_stack+0x4b/0x72
> [   34.512793]  [<ffffffff810cd831>] warn_slowpath_common+0x81/0xc0
> [   34.512795]  [<ffffffff810cd8b7>] warn_slowpath_fmt+0x47/0x50
> [   34.512798]  [<ffffffff811398c2>] ? do_init_timer+0x52/0x60
> [   34.512800]  [<ffffffff8150a015>] debug_print_object+0x85/0xa0
> [   34.512802]  [<ffffffff81139830>] ?
> trace_event_raw_event_tick_stop+0x100/0x100
> [   34.512805]  [<ffffffff8150ac38>] debug_object_assert_init+0xf8/0x130
> [   34.512807]  [<ffffffff8111a5bd>] ? trace_hardirqs_on+0xd/0x10
> [   34.512810]  [<ffffffff8113afdf>] del_timer+0x1f/0x70
> [   34.512813]  [<ffffffff811c2331>] laptop_sync_completion+0x61/0x100
> [   34.512815]  [<ffffffff811c22d0>] ? laptop_io_completion+0x30/0x30
> [   34.512819]  [<ffffffff812569cf>] sys_sync+0x7f/0x90
> [   34.512824]  [<ffffffff81b88e17>] entry_SYSCALL_64_fastpath+0x12/0x6f
> [   34.512825] ---[ end trace 8fe52cbaccc47e66 ]---
> [   34.512830] ------------[ cut here ]------------
> [   34.512833] WARNING: CPU: 0 PID: 4975 at lib/debugobjects.c:263
> debug_print_object+0x85/0xa0()
> [   34.512835] ODEBUG: assert_init not available (active state 0) object type:
> timer_list hint: stub_timer+0x0/0x20
> [   34.512836] Modules linked in:
> [   34.512838] CPU: 0 PID: 4975 Comm: systemd Tainted: G        W       4.4.0-rc7+
> #353
> [   34.512839] Hardware name: Apple Inc. MacBookPro10,2/Mac-
> AFD8A9D944EA4843, BIOS MBP102.88Z.0106.B0A.1509130955 09/13/2015
> [   34.512842]  ffffffff81f9b41c ffff880227a1bdb0 ffffffff814ec829
> ffff880227a1bdf8
> [   34.512845]  ffff880227a1bde8 ffffffff810cd831 ffff880227a1be90
> ffffffff822514c0
> [   34.512847]  ffffffff81f9b4c2 ffffffff8331f388 00007f29190ba700
> ffff880227a1be48
> [   34.512848] Call Trace:
> [   34.512850]  [<ffffffff814ec829>] dump_stack+0x4b/0x72
> [   34.512853]  [<ffffffff810cd831>] warn_slowpath_common+0x81/0xc0
> [   34.512855]  [<ffffffff810cd8b7>] warn_slowpath_fmt+0x47/0x50
> [   34.512857]  [<ffffffff811398c2>] ? do_init_timer+0x52/0x60
> [   34.512859]  [<ffffffff8150a015>] debug_print_object+0x85/0xa0
> [   34.512861]  [<ffffffff81139830>] ?
> trace_event_raw_event_tick_stop+0x100/0x100
> [   34.512864]  [<ffffffff8150ac38>] debug_object_assert_init+0xf8/0x130
> [   34.512865]  [<ffffffff8111a5bd>] ? trace_hardirqs_on+0xd/0x10
> [   34.512868]  [<ffffffff8113afdf>] del_timer+0x1f/0x70
> [   34.512869]  [<ffffffff811c2331>] laptop_sync_completion+0x61/0x100
> [   34.512871]  [<ffffffff811c22d0>] ? laptop_io_completion+0x30/0x30
> [   34.512873]  [<ffffffff812569cf>] sys_sync+0x7f/0x90
> [   34.512876]  [<ffffffff81b88e17>] entry_SYSCALL_64_fastpath+0x12/0x6f
> [   34.512877] ---[ end trace 8fe52cbaccc47e67 ]---

Powered by blists - more mailing lists