2004-03-22 17:47:24

by Meelis Roos

[permalink] [raw]
Subject: PnPBIOS: Unknown tag '0x82'

Since the beginning of its existence, the pnpbios driver talks about
unknown tag '0x82' on one of my computers. The computer has Intel
D815EEA2 mainboard, BIOS has been updated quite recently. I added the
tag dump to printout and here it is:

PnPBIOS: Unknown tag '0x82', length '18': 82 12 00 49 6e 74 65 6c 20 46 69 72 6d 77 61 72 65 20

This 0x82 0x12 0x00 and then 'Intel Firmware'.

Anything to worry about? Are the next tags still correctly parsed? The
full dmesg is now

PnPBIOS: Scanning system for PnP BIOS support...
PnPBIOS: Found PnP BIOS installation structure at 0xc00f2480
PnPBIOS: PnP BIOS version 1.0, entry 0xf0000:0x1d2a, dseg 0xf0000
pnp: 00:09: ioport range 0x4d0-0x4d1 has been reserved
pnp: 00:09: ioport range 0xcf8-0xcff could not be reserved
PnPBIOS: Unknown tag '0x82', length '18': 82 12 00 49 6e 74 65 6c 20 46 69 72 6d 77 61 72 65 20 .
pnp: 00:0b: ioport range 0x800-0x87f has been reserved
PnPBIOS: 20 nodes reported by PnP BIOS; 20 recorded by driver

--
Meelis Roos ([email protected])



2004-03-22 21:07:41

by Adam Belay

[permalink] [raw]
Subject: Re: PnPBIOS: Unknown tag '0x82'

On Mon, Mar 22, 2004 at 07:47:21PM +0200, Meelis Roos wrote:
> Since the beginning of its existence, the pnpbios driver talks about
> unknown tag '0x82' on one of my computers. The computer has Intel
> D815EEA2 mainboard, BIOS has been updated quite recently. I added the
> tag dump to printout and here it is:
>
> PnPBIOS: Unknown tag '0x82', length '18': 82 12 00 49 6e 74 65 6c 20 46 69 72 6d 77 61 72 65 20
>
> This 0x82 0x12 0x00 and then 'Intel Firmware'.
>
> Anything to worry about? Are the next tags still correctly parsed? The
> full dmesg is now
>
> PnPBIOS: Scanning system for PnP BIOS support...
> PnPBIOS: Found PnP BIOS installation structure at 0xc00f2480
> PnPBIOS: PnP BIOS version 1.0, entry 0xf0000:0x1d2a, dseg 0xf0000
> pnp: 00:09: ioport range 0x4d0-0x4d1 has been reserved
> pnp: 00:09: ioport range 0xcf8-0xcff could not be reserved
> PnPBIOS: Unknown tag '0x82', length '18': 82 12 00 49 6e 74 65 6c 20 46 69 72 6d 77 61 72 65 20 .
> pnp: 00:0b: ioport range 0x800-0x87f has been reserved
> PnPBIOS: 20 nodes reported by PnP BIOS; 20 recorded by driver
>
> --
> Meelis Roos ([email protected])

In this case it should be harmless. Typically when one tag is
corrupted (or incorrectly interpreted) it will also complain
about the following tag because of size checks. Where did the
unknown tag occur? Perhaps in pnpbios_parse_resource_option_data?

Thanks,
Adam

2004-03-23 09:50:17

by Meelis Roos

[permalink] [raw]
Subject: Re: PnPBIOS: Unknown tag '0x82'

> > PnPBIOS: Scanning system for PnP BIOS support...
> > PnPBIOS: Found PnP BIOS installation structure at 0xc00f2480
> > PnPBIOS: PnP BIOS version 1.0, entry 0xf0000:0x1d2a, dseg 0xf0000
> > pnp: 00:09: ioport range 0x4d0-0x4d1 has been reserved
> > pnp: 00:09: ioport range 0xcf8-0xcff could not be reserved
> > PnPBIOS: Unknown tag '0x82', length '18': 82 12 00 49 6e 74 65 6c 20 46 69 72 6d 77 61 72 65 20 .
> > pnp: 00:0b: ioport range 0x800-0x87f has been reserved
> > PnPBIOS: 20 nodes reported by PnP BIOS; 20 recorded by driver
>
> In this case it should be harmless. Typically when one tag is
> corrupted (or incorrectly interpreted) it will also complain
> about the following tag because of size checks. Where did the
> unknown tag occur? Perhaps in pnpbios_parse_resource_option_data?

It's in pnpbios_parse_compatible_ids.

--
Meelis Roos ([email protected])

2004-03-24 21:34:44

by Adam Belay

[permalink] [raw]
Subject: Re: PnPBIOS: Unknown tag '0x82'

On Tue, Mar 23, 2004 at 11:50:04AM +0200, Meelis Roos wrote:
> > > PnPBIOS: Scanning system for PnP BIOS support...
> > > PnPBIOS: Found PnP BIOS installation structure at 0xc00f2480
> > > PnPBIOS: PnP BIOS version 1.0, entry 0xf0000:0x1d2a, dseg 0xf0000
> > > pnp: 00:09: ioport range 0x4d0-0x4d1 has been reserved
> > > pnp: 00:09: ioport range 0xcf8-0xcff could not be reserved
> > > PnPBIOS: Unknown tag '0x82', length '18': 82 12 00 49 6e 74 65 6c 20 46 69 72 6d 77 61 72 65 20 .
> > > pnp: 00:0b: ioport range 0x800-0x87f has been reserved
> > > PnPBIOS: 20 nodes reported by PnP BIOS; 20 recorded by driver
> >
> > In this case it should be harmless. Typically when one tag is
> > corrupted (or incorrectly interpreted) it will also complain
> > about the following tag because of size checks. Where did the
> > unknown tag occur? Perhaps in pnpbios_parse_resource_option_data?
>
> It's in pnpbios_parse_compatible_ids.

Could you please try this patch.

Thanks,
Adam

--- a/drivers/pnp/pnpbios/rsparser.c 2004-03-11 02:55:25.000000000 +0000
+++ b/drivers/pnp/pnpbios/rsparser.c 2004-03-24 16:24:17.000000000 +0000
@@ -505,6 +505,11 @@

switch (tag) {

+ case LARGE_TAG_ANSISTR:
+ strncpy(dev->name, p + 3, len >= PNP_NAME_LEN ? PNP_NAME_LEN - 2 : len);
+ dev->name[len >= PNP_NAME_LEN ? PNP_NAME_LEN - 1 : len] = '\0';
+ break;
+
case SMALL_TAG_COMPATDEVID: /* compatible ID */
if (len != 4)
goto len_err;

2004-03-27 05:00:18

by Adam Belay

[permalink] [raw]
Subject: Re: PnPBIOS: Unknown tag '0x82'

On Thu, Mar 25, 2004 at 01:19:53PM +0200, Meelis Roos wrote:
> > Could you please try this patch.
>
> It works, thanks!

Great. I'll submit this with my next set of changes.

>
> dmesg now reports
>
> PnPBIOS: Scanning system for PnP BIOS support...
> PnPBIOS: Found PnP BIOS installation structure at 0xc00f2480
> PnPBIOS: PnP BIOS version 1.0, entry 0xf0000:0x1d2a, dseg 0xf0000
> pnp: 00:09: ioport range 0x4d0-0x4d1 has been reserved
> pnp: 00:09: ioport range 0xcf8-0xcff could not be reserved
> pnp: 00:0b: ioport range 0x800-0x87f has been reserved
> PnPBIOS: 20 nodes reported by PnP BIOS; 20 recorded by driver
>
> I don't have the previous output of lspnp at hand but now it reports
> among other things
>
> 0a INT0800 memory controller: flash
> flags: [no disable] [no config] [static]
> allocated resources:
> mem 0xffb00000-0xffbfffff [32 bit] [r/o]
> possible resources:
> compatible devices:
> identifier 'Intel Firmware Hub'
>
> This might be it showing up.

Yes, I think it's the same tag.

Thanks,
Adam