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

Tom Rodriguez Thomas.Rodriguez at Sun.COM
Tue Mar 9 17:56:32 PST 2010

Testing all looked good so I've pushed this.  Thanks!


On Mar 3, 2010, at 7:18 AM, Andrew Haley wrote:

> 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
>>>> 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
> OK to commit?  I guess I'll need a bug ID in the Sun database too.
> This is
> Andrew.

More information about the hotspot-dev mailing list