Request for approval: Allow Java's ELF symtab reader to use separate debuginfo files

Andrew Haley aph at redhat.com
Wed Mar 3 07:18:26 PST 2010


On 01/20/2010 05:35 PM, Tom Rodriguez wrote:
> 
> On Jan 20, 2010, at 2:16 AM, Andrew Haley wrote:
> 
>> On 01/19/2010 07:43 PM, Tom Rodriguez wrote:
>>
>>> So do you want to pursue making the SA work with a stripped
>>> libjvm.so or is requiring debug info to be installed acceptable?
>>> Sorry if I made this more complicated than you wanted.  I think your
>>> original change is fine, but if the norm is being stripped then
>>> making these tools work in that situation would be good.
>>
>> Clearly it would be.  Java programmers on Linux systems aren't used to
>> installing debuginfo, so it would be better all round if these tools
>> were to work without it.  (And besides, the idea that the Windows port
>> might be better in any way irks me!)  I'll have a look.
> 
> So the brute force approach is to make all vtables visible in the
> dynamic symbol table.

This patch does everything, I hope.  It adds all vtables to the
dynamic symtab, it changes the ELF reader to use the dynamic symtab,
and it adds a debuginfo reader that works with every GNU/Linux system
I know of.

Webrev at http://cr.openjdk.java.net/~aph/100126-hotspot-webrev-3/

OK to commit?  I guess I'll need a bug ID in the Sun database too.

This is https://bugs.openjdk.java.net/show_bug.cgi?id=100126

Andrew.


More information about the hotspot-dev mailing list