diff options
| author | mdodd <mdodd@FreeBSD.org> | 2004-08-31 21:45:30 +0000 |
|---|---|---|
| committer | mdodd <mdodd@FreeBSD.org> | 2004-08-31 21:45:30 +0000 |
| commit | d8deb0d121da0d84add30ea7ae0039943756b0bc (patch) | |
| tree | a8ee252c98db959ae69eba2b3df2abeb8143ac42 /sys/i386/bios | |
| parent | c4c3664bf7184430fffaf30bc032fc6db454f413 (diff) | |
| download | FreeBSD-src-d8deb0d121da0d84add30ea7ae0039943756b0bc.zip FreeBSD-src-d8deb0d121da0d84add30ea7ae0039943756b0bc.tar.gz | |
Fix checksum calculation.
Submitted by: Jean Delvare <khali@linux-fr.org>
Diffstat (limited to 'sys/i386/bios')
| -rw-r--r-- | sys/i386/bios/vpd.c | 19 |
1 files changed, 7 insertions, 12 deletions
diff --git a/sys/i386/bios/vpd.c b/sys/i386/bios/vpd.c index 0e22abe..735efe7 100644 --- a/sys/i386/bios/vpd.c +++ b/sys/i386/bios/vpd.c @@ -153,11 +153,8 @@ vpd_probe (device_t dev) goto bad; } - if (vpd_cksum(RES2VPD(res))) { - device_printf(dev, "VPD checksum failed.\n"); - error = ENXIO; - goto bad; - } + if (vpd_cksum(RES2VPD(res))) + device_printf(dev, "VPD checksum failed. BIOS update may be required.\n"); bad: if (res) @@ -281,6 +278,10 @@ static driver_t vpd_driver = { DRIVER_MODULE(vpd, nexus, vpd_driver, vpd_devclass, vpd_modevent, 0); MODULE_VERSION(vpd, 1); +/* + * Perform a checksum over the VPD structure, starting with + * the BuildID. (Jean Delvare <khali@linux-fr.org>) + */ static int vpd_cksum (struct vpd *v) { @@ -290,13 +291,7 @@ vpd_cksum (struct vpd *v) ptr = (u_int8_t *)v; cksum = 0; - for (i = 0; i < v->Length ; i++) { + for (i = offsetof(struct vpd, BuildID); i < v->Length ; i++) cksum += ptr[i]; - } - - /* XXX: For some reason this isn't right. */ - printf("VPD cksum: 0x%02x\n", cksum); - cksum = 0; - return (cksum); } |
