LKML Archive mirror
 help / color / mirror / Atom feed
From: "Zheng, Lv" <lv.zheng@intel.com>
To: Chris Bainbridge <chris.bainbridge@gmail.com>
Cc: "Moore, Robert" <robert.moore@intel.com>,
	"Wysocki, Rafael J" <rafael.j.wysocki@intel.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [BUG] ODEBUG: assert_init not available (active state 0)
Date: Fri, 5 Feb 2016 02:52:00 +0000	[thread overview]
Message-ID: <1AE640813FDE7649BE1B193DEA596E883BB4450C@SHSMSX101.ccr.corp.intel.com> (raw)
In-Reply-To: <20160203103636.GA4932@localhost>

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@gmail.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@intel.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@intel.com>
>     Signed-off-by: Bob Moore <robert.moore@intel.com>
>     Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.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 ]---

  reply	other threads:[~2016-02-05  2:52 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-03 10:36 [BUG] ODEBUG: assert_init not available (active state 0) Chris Bainbridge
2016-02-05  2:52 ` Zheng, Lv [this message]
2016-02-14 10:59   ` Chris Bainbridge
2016-02-23  2:18     ` Zheng, Lv
2016-02-27 19:08       ` Chris Bainbridge
2016-03-05 15:11         ` [PATCH] ACPICA: Events: Execute some _REG methods in early boot Chris Bainbridge
2016-03-07  6:36           ` Zheng, Lv
2016-03-07 10:19             ` Chris Bainbridge
2016-03-08  0:54               ` Zheng, Lv

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1AE640813FDE7649BE1B193DEA596E883BB4450C@SHSMSX101.ccr.corp.intel.com \
    --to=lv.zheng@intel.com \
    --cc=chris.bainbridge@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rafael.j.wysocki@intel.com \
    --cc=robert.moore@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).