Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 18 Nov 2003 13:34:38 -0800 (PST)
From:      Nate Lawson <nate@root.org>
To:        Robert Watson <rwatson@freebsd.org>
Cc:        current@freebsd.org
Subject:   Re: Updated acpi_cpu patch
Message-ID:  <20031118132905.P64933@root.org>
In-Reply-To: <Pine.NEB.3.96L.1031118161909.92350c-100000@fledge.watson.org>
References:  <Pine.NEB.3.96L.1031118161909.92350c-100000@fledge.watson.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, 18 Nov 2003, Robert Watson wrote:
> On Tue, 18 Nov 2003, Nate Lawson wrote:
>
> > Below you'll find the update patch for acpi_cpu.  Please test this,
> > especially for SMP and laptops with _CST objects in their ASL.
> ...
> > Notes:
> > * Add a detach method that disables entry to acpi_cpu_idle and in the SMP
> > case, IPIs all processors to exit sleeping.  This fixes a panic on
> > shutdown for MP boxes.
>
> Sigh, I appear to have been mistaken about the SMP reboot problem being
> fixed, sorry about that.  Mark's random_harvest panic appears to have
> caused me to miss the other failure mode in my last test.  Stack trace
> attached, and I believe I'm running with your latest patch.

Could you add a printf to the start of acpi_cpu_detach()?  I want to see
if we're being called before or after ACPI is stopped ("Shutting down
ACPI").  Also, please do:
   l *AcpiGetRegister+0x61
I think it's the call to get the bus master status, which is interesting
since this means that cpu_cx_count != 0 which means that acpi_cpu_detach
hasn't run yet.

-Nate



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20031118132905.P64933>