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

[David O'Brien]

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 at FreeBSD.org)