From owner-freebsd-current@FreeBSD.ORG Thu May 22 08:04:05 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 337C237B401 for ; Thu, 22 May 2003 08:04:05 -0700 (PDT) Received: from magic.adaptec.com (magic-mail.adaptec.com [208.236.45.100]) by mx1.FreeBSD.org (Postfix) with ESMTP id 63B0843F75 for ; Thu, 22 May 2003 08:04:04 -0700 (PDT) (envelope-from scott_long@btc.adaptec.com) Received: from redfish.adaptec.com (redfish.adaptec.com [162.62.50.11]) by magic.adaptec.com (8.11.6/8.11.6) with ESMTP id h4MExjZ11924; Thu, 22 May 2003 07:59:45 -0700 Received: from btc.adaptec.com (hollin.btc.adaptec.com [10.100.253.56]) by redfish.adaptec.com (8.8.8p2+Sun/8.8.8) with ESMTP id IAA21286; Thu, 22 May 2003 08:03:51 -0700 (PDT) Message-ID: <3ECCE6D0.1080109@btc.adaptec.com> Date: Thu, 22 May 2003 09:03:44 -0600 From: Scott Long User-Agent: Mozilla/5.0 (X11; U; FreeBSD i386; en-US; rv:1.3) Gecko/20030425 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Bruce Cran References: <200305221305.42579.esn@x123.info> <20030522122913.GA1312@fourtytwo.brucec.backnet> In-Reply-To: <20030522122913.GA1312@fourtytwo.brucec.backnet> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit cc: "Sebastian Yepes \[ESN\]" 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 15:04:05 -0000 Your patch works wonderfully on my Dell 8200. What are the risks of the patch? I'd like to put it into 5.1 if possible. Scott Bruce Cran wrote: > 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 > > > > ------------------------------------------------------------------------ > > --- 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); > } > > > > ------------------------------------------------------------------------ > > _______________________________________________ > freebsd-current@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-current > To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org"