Yumin,

The FIELDINFO_TAG_MASK part in InstanceKlass is already fixed in JDK-8006403. It should be sync'd to the dev repos soon (or has it already?)
I fell in that trap of duplicating the fix as JDK-8006641 already...

- Kris

On 01/24/2013 02:14 PM, Yumin Qi wrote:
Hi,

  Can I have your comments on fix for
  8005278: Serviceability Agent: jmap -heap and jstack -m fail

http://cr.openjdk.java.net/~minqi/8005278/

Problems: 1) In JVM, BinaryTreeDictionary is typedef'ed as AFLBinaryTreeDictionary and this name carried to type library for SA. In SA we still use olde name for that; 2) FreeList now is template based which is not reflected in SA; 3) There is a misuse of FIELFINFO_TAG_MASK(which is not in SA code), in SA code FIELDINFO_TAG_SIZE was wrongly used as FIELDINFO_TAG_MASK and lead to not able to find correct field info.

  Thanks
  Yumin




Reply via email to