From: Len Brown <lenb@kernel.org>
To: Zhao Yakui <yakui.zhao@intel.com>
Cc: "Wu, Fengguang" <fengguang.wu@intel.com>,
"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>
Subject: Re: [PATCH 2.6.28-rc6] ACPICA: don't cond_resched() when irqs_disabled()
Date: Fri, 19 Dec 2008 03:38:18 -0500 (EST) [thread overview]
Message-ID: <alpine.LFD.2.00.0812190326000.6312@localhost.localdomain> (raw)
In-Reply-To: <1229670120.3990.11.camel@yakui_zhao.sh.intel.com>
> The issue on Fengguang's laptop is different with that on the box of
> T61. On Fengguang's laptop after evaluating one ACPI object(GPE
> _L18object), the irq is disabled mystically. After some investigations
> we find that it is caused by SMI(The SMI will be triggered while
> evaluating the ACPI object. And this issue is also related with Video).
Interesting.
I hadn't realized you root-caused that failure.
This is good to know, but the fact that SMM can
leave interrupts disabled is frightening.
If the BIOS SMM on that box disables interrupts when
random AML is run, then it is going to run into
all kinds of other problems in addition to this one...
Hopefully the BIOS bugs is fixed by a BIOS update...
> If the boot option of "noapic" is added on the box of bug12252, the
> irq_router_resume will be called, in which the _CRS/_SRS object will be
> evaluated. But unfortunately the irq is disabled when irq_router_resume
> is called.
Right. I forced the T61 to run into this problem by forcing
it to use PIC mode and suspending -- for it doesn't use the links
in IOAPIC mode. Many other systems do use the links in default
mode, however, so I'm surprised that we had not heard more about
this failure.
Now that I search for acpi_ps_complete_op() on kerneloops.org,
I see that just a few people have run into it.
thanks,
-- Len Brown, Intel Open Source Technology Center>
next prev parent reply other threads:[~2008-12-19 8:38 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20081126063522.GA23611@localhost>
[not found] ` <20081126135508.GA1891@localhost>
[not found] ` <20081126152043.c92ae839.akpm@linux-foundation.org>
2008-12-02 2:45 ` [PATCH 2.6.28-rc6] ACPICA: don't cond_resched() when irqs_disabled() Wu Fengguang
2008-12-19 5:22 ` Len Brown
2008-12-19 7:02 ` Zhao Yakui
2008-12-19 8:38 ` Len Brown [this message]
2008-12-19 11:01 ` Wu Fengguang
2008-12-19 18:24 ` Len Brown
2008-12-20 0:28 ` Wu Fengguang
2008-12-19 13:51 ` Wu Fengguang
2008-12-19 18:35 ` Len Brown
2008-12-20 0:34 ` Wu Fengguang
2009-01-01 19:39 ` Robert Hancock
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=alpine.LFD.2.00.0812190326000.6312@localhost.localdomain \
--to=lenb@kernel.org \
--cc=fengguang.wu@intel.com \
--cc=linux-acpi@vger.kernel.org \
--cc=yakui.zhao@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