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

Danny Kukawka danny.kukawka at web.de
Thu Dec 6 11:08:41 PST 2007


On Donnerstag, 6. Dezember 2007, David Zeuthen wrote:
> 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.

Commited,

Danny




More information about the hal mailing list