Date: Mon, 05 Jan 2004 13:33:15 -0700 (MST) From: "M. Warner Losh" <imp@bsdimp.com> To: obrien@FreeBSD.org Cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/contrib/dev/acpica dsmthdat.c Message-ID: <20040105.133315.89067481.imp@bsdimp.com> In-Reply-To: <20040105184940.GA29017@dragon.nuxi.com> References: <20040105090152.GG74113@pcwin002.win.tue.nl> <20040105091847.S22517@root.org> <20040105184940.GA29017@dragon.nuxi.com>
next in thread | previous in thread | raw e-mail | index | archive | help
In message: <20040105184940.GA29017@dragon.nuxi.com>
"David O'Brien" <obrien@FreeBSD.org> writes:
: 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.
The patched AML doesn't work as well for me as does this change.
Warner
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040105.133315.89067481.imp>