Date: Mon, 5 Jan 2004 10:49:40 -0800 From: "David O'Brien" <obrien@freebsd.org> To: Nate Lawson <nate@root.org> Cc: cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/contrib/dev/acpica dsmthdat.c Message-ID: <20040105184940.GA29017@dragon.nuxi.com> In-Reply-To: <20040105091847.S22517@root.org> References: <200401040635.i046Z1e2086058@repoman.freebsd.org> <20040105090152.GG74113@pcwin002.win.tue.nl> <20040105091847.S22517@root.org>
next in thread | previous in thread | raw e-mail | index | archive | help
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. -- -- David (obrien@FreeBSD.org)
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040105184940.GA29017>