Date: Mon, 5 Jan 2004 11:16:58 -0800 (PST) From: Nate Lawson <nate@root.org> To: David O'Brien <obrien@freebsd.org> Cc: cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/contrib/dev/acpica dsmthdat.c Message-ID: <20040105111536.A23188@root.org> In-Reply-To: <20040105184940.GA29017@dragon.nuxi.com> References: <200401040635.i046Z1e2086058@repoman.freebsd.org> <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
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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040105111536.A23188>