Re: [PATCH 2.2] i386/dmi_scan updates

From: Alan Cox (alan@lxorguk.ukuu.org.uk)
Date: Sat Oct 05 2002 - 14:52:59 EST


On Sat, 2002-10-05 at 22:36, Jean Delvare wrote:
> - Stop skipping DMI entries when type is less than those of the
> previous entry. I could see no reason for doing this.

Fixes crashes on certain vendors hardware. It shouldnt be needed,
however in the real world it proves to be a rather essential heuristic.
DMidecode doesnt do it because in userspace I dont mind spewing crap to
show a user a problem.

> - Verify the DMI entry point structure checksum.
> - Start looking for the DMI entry point from 0xF0000, not 0xE0000.

Looks ok

> - Fix an off-by-one error causing the last address scanned being
> 0x100000, not 0xFFFF0 as it should.

Yep

> - Do not display the DMI version if it would be 0.0.
> - Remove senseless tests in dump (debug) code.

These are also not senseless. Not everyone seems to use the proper null
string, sometimes you get spaces too

The technical changes look right, and in theory all of it does. In
practice I'd rather see a patch that kept the rule of thumb about order
and the ' ' check

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Mon Oct 07 2002 - 22:00:51 EST