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

[Nate Lawson]

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