I've a crash extension called 'pci', its a direct replacement for the 'dev' command but contains its own (huge) internal database of pci information. I thought I'd finished integrating oui.txt from ieee and pci.ids from sourceforge but looking at the code today, I have *not* done that yet.... hmmm thought I'd finished that stuff. But, its on the todo list. I wrote it about a year ago and haven't really done anything with the sources since then. It only uses the pci information from www.pcidatabase.com at present. I'll see if I can get this up on the net for use if there is interest. There are two parts, you download the source pci id information from the web and run a pre-processor on it, and then compile the extension with the header file you've created. The ext-pci.so is currently ~1.1MB in size. Another todo list item was to shrink the size of the database via restructuring it but I don't expect more than a 25%-33% reduction in size at most. Bud Brown Red Hat, Inc Westford, MA SEG/Storage Team ------------------------------ Message: 4 Date: Tue, 27 Apr 2010 16:36:31 +0530 From: Sharyathi Nagesh <sharyath@xxxxxxxxxx> To: Dave Anderson <anderson@xxxxxxxxxx> Cc: Crash-utility@xxxxxxxxxx, huachenl@xxxxxxxxxx Subject: Message-ID: <4BD6C537.9030102@xxxxxxxxxx> Content-Type: text/plain; charset=UTF-8 Thanks for the reply Dave. Even I feel it is better to change the error message to what you have suggested. That is simplest way to address the issue as well. It gives a clearer message to the user. I am curious to hear from Bud on the alternate scheme and progress as well. Bud your views ? Thanks Sharyathi On 04/23/2010 08:01 PM, Dave Anderson wrote: > > ----- "Dave Anderson" <anderson@xxxxxxxxxx> wrote: > >>> With this I wanted to know, if you intend to keep dev -p behavior >>> as it is now or there is any plan to change it to display actual >>> values? >>> >>> Thank You Sharyathi N >> >> I (personally) have no plans to change it. If I remember >> correctly, Bud Brown came up with an alternate scheme, but the >> imported data from the kernel proper required to accomplish it was >> enormous (bordering on absurd), so I suggested that it would be >> more appropriate as an extension module. >> >> Bud -- feel free to chime in here... ;-) >> >> For that matter, even the "old" way required the import of ~1000 >> lines of kernel #define's -- which always bugged me -- and was >> pretty much the only crash command that had to do such a thing. >> >> Dave > > Actually, at a minimum, I should change this: > > if (!symbol_exists("pci_devices")) error(FATAL, "no PCI devices found > on this system.\n"); > > to the generic option_not_supported() message. > > Dave ------------------------------ -- Crash-utility mailing list Crash-utility@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/crash-utility End of Crash-utility Digest, Vol 55, Issue 11 ********************************************* -- Crash-utility mailing list Crash-utility@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/crash-utility