From owner-freebsd-current@FreeBSD.ORG Thu May 22 05:29:27 2003 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 3155D37B401 for ; Thu, 22 May 2003 05:29:27 -0700 (PDT) Received: from grassmarket.ucs.ed.ac.uk (grassmarket.ucs.ed.ac.uk [129.215.166.64]) by mx1.FreeBSD.org (Postfix) with ESMTP id D863843F85 for ; Thu, 22 May 2003 05:29:25 -0700 (PDT) (envelope-from s9905155@sms.ed.ac.uk) Received: from fourtytwo.brucec.backnet (12266209.resnet.ed.ac.uk [10.6.0.100] (may be forged))h4MCTHT10432; Thu, 22 May 2003 13:29:18 +0100 (BST) Received: from fourtytwo.brucec.backnet (localhost.brucec.backnet [127.0.0.1]) h4MCTDQ1001380; Thu, 22 May 2003 13:29:13 +0100 (BST) (envelope-from bruce@fourtytwo.brucec.backnet) Received: (from bruce@localhost)h4MCTDxu001379; Thu, 22 May 2003 13:29:13 +0100 (BST) Date: Thu, 22 May 2003 13:29:13 +0100 From: Bruce Cran To: "Sebastian Yepes [ESN]" Message-ID: <20030522122913.GA1312@fourtytwo.brucec.backnet> References: <200305221305.42579.esn@x123.info> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="+HP7ph2BbKc20aGI" Content-Disposition: inline In-Reply-To: <200305221305.42579.esn@x123.info> User-Agent: Mutt/1.5.4i cc: current@freebsd.org Subject: Re: FBSD 5.1b2 Inst. Results on Dell i8500 X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 22 May 2003 12:29:27 -0000 --+HP7ph2BbKc20aGI Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Thu, May 22, 2003 at 01:05:42PM +0200, Sebastian Yepes [ESN] wrote: > I have installed on my Dell Inspiron 5.0R, 5.1b2 & 5-Current ihave found some > good stuff on the 5.1b2 and some bad stuff.. her we go > > > dmseg -> http://www.x123.info/src/i8500/dmesg.5.1b2 > > > Good Stuff:: > --------------------- > Well it look like the Intel ICH4 UDMA100 controller is working now i well make > some benchmarks to see if this is true.. > this was not working on the 5.0R > > > Bad Stuff:: > --------------------- > ACPI:: on the console i get a constante flood of this msg > ..... > ACPI-0293: *** Warning: Buffer created with zero length in AML > -0166: *** Error: UtAllocate: Attempt to allocate zero bytes > ACPI-0293: *** Warning: Buffer created with zero length in AML > -0166: *** Error: UtAllocate: Attempt to allocate zero bytes > ACPI-0293: *** Warning: Buffer created with zero length in AML > -0166: *** Error: UtAllocate: Attempt to allocate zero bytes > ..... > > In the 5.0R the Battery satus was working ok, > but now on the 5-Corrent it's not ;( The attached patch fixes the battery status and stops the messages on my i8500. Sleep states S1 and S5 work, as do the lid, power and sleep buttons. -- Bruce Cran --+HP7ph2BbKc20aGI Content-Type: text/plain; charset=us-ascii Content-Disposition: attachment; filename="acpi_dell.diff" --- sys/contrib/dev/acpica/dsmthdat.c Mon May 12 01:44:41 2003 +++ sys/contrib/dev/acpica/dsmthdat.c Wed May 14 16:00:50 2003 @@ -392,8 +393,6 @@ { ACPI_STATUS Status; ACPI_NAMESPACE_NODE *Node; - ACPI_OPERAND_OBJECT *NewDesc = Object; - ACPI_FUNCTION_TRACE ("DsMethodDataSetValue"); @@ -411,32 +409,14 @@ return_ACPI_STATUS (Status); } - /* - * If the object has just been created and is not attached to anything, - * (the reference count is 1), then we can just store it directly into - * the arg/local. Otherwise, we must copy it. - */ - if (Object->Common.ReferenceCount > 1) - { - Status = AcpiUtCopyIobjectToIobject (Object, &NewDesc, WalkState); - if (ACPI_FAILURE (Status)) - { - return_ACPI_STATUS (Status); - } - - ACPI_DEBUG_PRINT ((ACPI_DB_EXEC, "Object Copied %p, new %p\n", - Object, NewDesc)); - } - else - { /* Increment ref count so object can't be deleted while installed */ - AcpiUtAddReference (NewDesc); - } + AcpiUtAddReference (Object); /* Install the object */ - Node->Object = NewDesc; + Node->Object = Object; + return_ACPI_STATUS (Status); } --+HP7ph2BbKc20aGI--