Request for review:

Joseph Provino joseph.provino at
Wed Jul 3 12:53:17 PDT 2013

On 07/03/2013 03:27 PM, Vladimir Kozlov wrote:
> I don't like to have renaming done together with the fix. Is renaming 
> required?
> Thanks,
> Vladimir

Renaming isn't required but if I keep PLATFORM_NMT_DETAIL_SUPPORTED
I would need to add the new flag PLATFORM_NATIVE_STACK_WALKING_SUPPORTED.

I could use PLATFORM_NMT_DETAIL_SUPPORTED but NMT doesn't have anything 
to do
with this bug 8017473.

What happened is that 8011064 got reported and fixed first.  The fix was 
to disallow
NMT_detail in some cases so PLATFORM_NMT_DETAIL_SUPPORTED made sense.

Bug 8017473 makes it clear that there are times when native stack 
walking can't be done.
PLATFORM_NATIVE_STACK_WALKING_SUPPORTED is more general and makes sense for
8011064 and 8017473.

Do you think it would be better to use a new name and then file another 
bug to change


> On 7/3/13 12:10 PM, Joseph Provino wrote:
>> Bug report:
>> This is for SE_8 but will be backported to 7u.
>> Webrev:
>> <>
>> to make the name more general.
>> Added -DPLATFORM_NMT_DETAIL_SUPPORTED=1 to linux/makefiles/debug.make
>> because
>> with the low optimization for debug builds, -fno-omit-frame-pointer is
>> set and stack walking
>> is always permissible.
>> Changed vm.make to optionally include an architecture specific makefile
>> in case some files
>> need to be compiled with special options such as 
>> -fno-omit-frame-pointer.
>> Thanks.
>> joe

More information about the hotspot-dev mailing list