Re: [PATCH 2.2] i386/dmi_scan updates

[Alan Cox]

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