[Crash-utility] dev -p command fails post linux 2.6.25

[bud.brown at redhat.com]

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 at in.ibm.com>
To: Dave Anderson <anderson at redhat.com>
Cc: Crash-utility at redhat.com, huachenl at cn.ibm.com
Subject: 
Message-ID: <4BD6C537.9030102 at in.ibm.com>
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 at redhat.com> 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 at redhat.com
https://www.redhat.com/mailman/listinfo/crash-utility


End of Crash-utility Digest, Vol 55, Issue 11
*********************************************