Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 27 Mar 2005 14:46:22 -0800
From:      Nate Lawson <nate@root.org>
To:        vova@fbsd.ru
Cc:        cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/dev/acpica acpi_perf.c
Message-ID:  <424737BE.9000808@root.org>
In-Reply-To: <1111516858.1198.2.camel@localhost>
References:  <200503200351.j2K3pIC8068507@repoman.freebsd.org> <1111516858.1198.2.camel@localhost>

next in thread | previous in thread | raw e-mail | index | archive | help
Vladimir Grebenschikov wrote:
> В вс, 20/03/2005 в 03:51 +0000, Nate Lawson пишет:
> 
>>njl         2005-03-20 03:51:18 UTC
>>
>> FreeBSD src repository
>>
>> Modified files:
>>   sys/dev/acpica       acpi_perf.c 
>> Log:
>> Attach acpi_perf early.  Especially when it is being used to provide info
>> to other devices, it needs to be attached first.  (Multi-pass newbus
>> probes would be a better solution.)
>> 
>> Revision  Changes    Path
>> 1.16      +6 -1      src/sys/dev/acpica/acpi_perf.c
> 
> 
> I guess this patch makes panic on my notebook:
> 
> cpu0: <ACPI CPU (2 Cx states)> on acpi0
> acpi_perf0: <ACPI CPU Frequency Control> on cpu0
> acpi_perf0: failed in PERF_STATUS attach
> device_attach: acpi_perf0 attach returned 6
> panic: resource_list_add: resource entry is busy

I believe the patch I just committed fully cleans up resources for the 
error case correctly.  This should fix your panic.  The other question 
is why your status register fails to attach but the control register 
succeeds.  Can you send me your acpidump:

acpidump -t -d > vova.asl

-- 
Nate



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