I think i was a bit quick/excited about getting this info in there..Tested on my 2 machines here (both running 2.6.24 though) without issues.. Will have to look into it further (after taking your advice) and rework this patch.
On Mon, 2008-06-30 at 17:42 +0100, Mark Brown wrote:
On Tue, Jul 01, 2008 at 02:31:37AM +1000, Travis Place wrote:
+if [ -d $SYSFS ] +then +BOARD_VENDOR=`cat /sys/devices/virtual/dmi/id/board_vendor` +PRODUCT_VERSION=`cat /sys/devices/virtual/dmi/id/product_version` +PRODUCT_NAME=`cat /sys/devices/virtual/dmi/id/product_name`
This produces visible errors if the information isn't present (which would be reasonable enough). It might be better to check for errors and include something about being unable to read DMI information in the report - that'd be a bit less scary.
Also, on my system the DMI information is under /sys/class/dmi rather than /sys/devices/virtual (this is with 2.6.25).
This will certainly be a pain, if the info keep moving around.. unless we do some quick 'search' for the 'dmi' dir.
+fi
How about bios_vendor, bios_date and bios_version as well?
How useful is the BIOS information going to be, in most cases ?
For embedded devices the relevant information is often in /proc/cpuinfo so it might be worth adding that too.
Can i get an example output or 'cpuinfo' on an embedded device (if you have one available) so i can see what im looking for.
Thanks, Travis Place