Comment 7 for bug 1211521

Revision history for this message
Daniel Manrique (roadmr) wrote :

Hi, now I'm comparing the device data as obtained with the checkbox_parser script and dumped to json format. All I'm changing is the parser version.

The differences from the old (2234) parser and the new (2235) parser are as follows:

- A bunch of devices gained product and vendor ids, which is good.
- A bunch of devices changed from category: null to category: OTHER. The criterion used for this is that any product with a valid product_id and no other category will be categorized as OTHER. This is not necessarily good, to be addressed in a following merge request.
- 82579 Gigabit Network device has wrong bus (net vs. pci), and no product or vendor id. This is bad.
- WIreless device (Centrino Advanced-N 6205) has wrong product_id (0x1C12, which matches the PCI controller, not the device itself).

Now, the differences from the 2235 parser (bad) and the one with my fixes are as follows:

- 82579 Gigabit Network device has correct bus (pci), as well as product id 0x1502 and vendor id 0x8086 (correct).
- WIreless device (Centrino Advanced-N 6205) has the correct product_id (0x0133)

This means that the fixed parser fixes everything but the category:OTHER problem (it's another bug so it'll be fixed separately).

I'll attach the dumps to provide evidence of this :)