[PATCH] read dmi info from sysfs instead of call dmidecode

David Zeuthen david at fubar.dk
Thu Dec 6 10:46:52 PST 2007


On Thu, 2007-12-06 at 18:40 +0100, Danny Kukawka wrote:
> On Donnerstag, 6. Dezember 2007, David Zeuthen wrote:
> > On Tue, 2007-11-13 at 20:08 +0100, Danny Kukawka wrote:
> > > Hi,
> > >
> > > this patch makes HAL to read dmi/smbios information on Linux from sysfs
> > > (/sys/devices/virtual/dmi/id) instead of use a prober to call and parse
> > > dmidecode.
> >
> > I'm pretty sure this was discussed some (many maybe) months ago and I
> > think what I said was that this wasn't going to work as some of the
> > files are read-only for root.... Doesn't this patch suffer from the same
> > problems?
> 
> Can't remember any discussion about this. Yes, there are some (3) files which 
> are readable only for root. Solved this by read these files before drop the 
> privileges. Tested it and the new code do the same as the old one.

Sounds like a good idea then. Thanks.

      David




More information about the hal mailing list