From owner-freebsd-acpi@FreeBSD.ORG Mon Mar 28 06:50:00 2005 Return-Path: Delivered-To: freebsd-acpi@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 4A9A016A4CE; Mon, 28 Mar 2005 06:50:00 +0000 (GMT) Received: from www.cryptography.com (li-22.members.linode.com [64.5.53.22]) by mx1.FreeBSD.org (Postfix) with ESMTP id ECFB543D1F; Mon, 28 Mar 2005 06:49:59 +0000 (GMT) (envelope-from nate@root.org) Received: from [10.0.5.51] (adsl-64-171-184-204.dsl.snfc21.pacbell.net [64.171.184.204]) by www.cryptography.com (8.12.8/8.12.8) with ESMTP id j2S6nmZi028929 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NOT); Sun, 27 Mar 2005 22:49:50 -0800 Message-ID: <4247A90C.5050000@root.org> Date: Sun, 27 Mar 2005 22:49:48 -0800 From: Nate Lawson User-Agent: Mozilla Thunderbird 1.0 (Windows/20041206) X-Accept-Language: en-us, en MIME-Version: 1.0 To: vova@fbsd.ru References: <200503200351.j2K3pIC8068507@repoman.freebsd.org> <1111516858.1198.2.camel@localhost> <424737BE.9000808@root.org> <1111985287.1113.0.camel@localhost> In-Reply-To: <1111985287.1113.0.camel@localhost> Content-Type: text/plain; charset=KOI8-R; format=flowed Content-Transfer-Encoding: 8bit cc: acpi@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/acpica acpi_perf.c X-BeenThere: freebsd-acpi@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: ACPI and power management development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 28 Mar 2005 06:50:00 -0000 Vladimir Grebenschikov wrote: > В вс, 27/03/2005 в 14:46 -0800, Nate Lawson пишет: > >>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: on acpi0 >>>acpi_perf0: 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: > > > Thank you, will try fresh kernel later today. > > > Attached > > > Method (_PCT, 0, NotSerialized) > { > If (LEqual (And (PDC0, One), One)) > { > ... > } > Else > { > Return (Package (0x02) > { > ResourceTemplate () > { > Register (SystemIO, 0x10, 0x00, 0x00000000000000B2) > }, > > ResourceTemplate () > { > Register (SystemIO, 0x08, 0x00, 0x00000000000000B3) > } > }) > } > } I've moved this to the acpi@ list now... Are you sure there isn't a BIOS update for your system available? It appears this is a Dell since a number of them have the same problem. The issue is that the control and status registers overlap. To fix, change this line: Register (SystemIO, 0x10, 0x00, 0x00000000000000B2) to this: Register (SystemIO, 0x08, 0x00, 0x00000000000000B2) Then, recompile your asl as described in the handbook and load it at boot. I'll try to figure out a way to work around this but it looks difficult to detect since 16 bits is a valid register width. I seem to remember imp@ had a means for detecting that a resource alloc failed due to an overlapping register and then split the previous resource (since this driver owns both). I'll ask. -- Nate