cvs commit: src/sys/contrib/dev/acpica dsmthdat.c

[M. Warner Losh]

In message: <20040105184940.GA29017 at dragon.nuxi.com>
            "David O'Brien" <obrien at 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