From owner-cvs-all@FreeBSD.ORG Mon Jan 5 11:17:04 2004 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 50A3D16A4D0 for ; Mon, 5 Jan 2004 11:17:04 -0800 (PST) Received: from root.org (root.org [67.118.192.226]) by mx1.FreeBSD.org (Postfix) with SMTP id C41F243D5D for ; Mon, 5 Jan 2004 11:16:57 -0800 (PST) (envelope-from nate@root.org) Received: (qmail 23237 invoked by uid 1000); 5 Jan 2004 19:16:58 -0000 Date: Mon, 5 Jan 2004 11:16:58 -0800 (PST) From: Nate Lawson To: David O'Brien In-Reply-To: <20040105184940.GA29017@dragon.nuxi.com> Message-ID: <20040105111536.A23188@root.org> References: <200401040635.i046Z1e2086058@repoman.freebsd.org> <20040105090152.GG74113@pcwin002.win.tue.nl> <20040105091847.S22517@root.org> <20040105184940.GA29017@dragon.nuxi.com> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII cc: cvs-src@freebsd.org cc: src-committers@freebsd.org cc: cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/contrib/dev/acpica dsmthdat.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 05 Jan 2004 19:17:04 -0000 On Mon, 5 Jan 2004, David O'Brien wrote: > On Mon, Jan 05, 2004 at 09:19:51AM -0800, Nate Lawson wrote: > > > > Modified files: (Branch: INTEL) > > > > sys/contrib/dev/acpica dsmthdat.c > > > > Log: > > > > Fix for "UtAllocate: Attempt to allocate zero bytes" warnings on Dell > > > > laptops that resulted in problems reading battery status. Don't > > > > copy Buffers, Packages, or Regions in DsStoreObjectToLocal(). > > > > > > > > Tested by: scottl, sam > > > > Submitted by: Luming Yu (Intel) > > > > > > Does this obsolete Mark Santcroos' patch available from my page > > > (http://sandcat.nl/~stijn/freebsd/dell.php)? If so, I'll add a note to > > > that effect to the page. Unfortunately I won't have time to test this myself, > > > so I'd appreciate it if someone else can verify this. > > > > Yes, I believe it does. However, it breaks my T23's battery state due to > > the refcount change. So I don't think this fix will stay in this form. > > A loader tunable? That's certainly easier for Dell owners than patching > their AML. No tunable. We'll find the right fix shortly, and then this will work for everyone. Until then, I believe there are more Dell users than T23 users so I committed this workaround. Other Thinkpads (R40, for example) should not be affected. -Nate