EVALUATION

See main CR
                                     

EVALUATION

http://hg.openjdk.java.net/hsx/hotspot-main/hotspot/rev/d425748f2203
                                     

SUGGESTED FIX

Minor rename done during code review round 0:

$ diff -c src/share/vm/prims/jvmtiTagMap.cpp{.cr0,}
*** src/share/vm/prims/jvmtiTagMap.cpp.cr0      Thu Jun 23 07:49:40 2011
--- src/share/vm/prims/jvmtiTagMap.cpp  Thu Jun 23 16:07:05 2011
***************
*** 1656,1662 ****
    static inline void mark(oop o);           // mark an object
    static inline bool visited(oop o);        // check if object has been visited
  
!   static inline bool get_needs_reset()        { return _needs_reset; }
    static inline void set_needs_reset(bool v)  { _needs_reset = v; }
  };
  
--- 1656,1662 ----
    static inline void mark(oop o);           // mark an object
    static inline bool visited(oop o);        // check if object has been visited
  
!   static inline bool needs_reset()            { return _needs_reset; }
    static inline void set_needs_reset(bool v)  { _needs_reset = v; }
  };
  
***************
*** 1685,1691 ****
    // iterate over all objects and restore the mark bits to
    // their initial value
    RestoreMarksClosure blk;
!   if (get_needs_reset()) {
      Universe::heap()->object_iterate(&blk);
    } else {
      // We don't need to reset mark bits on this call, but reset the
--- 1685,1691 ----
    // iterate over all objects and restore the mark bits to
    // their initial value
    RestoreMarksClosure blk;
!   if (needs_reset()) {
      Universe::heap()->object_iterate(&blk);
    } else {
      // We don't need to reset mark bits on this call, but reset the
                                     

EVALUATION

http://hg.openjdk.java.net/hsx/hotspot-rt/hotspot/rev/d425748f2203
                                     

EVALUATION

The attached patch actually contains two changes:

- removes unnecessary reset of a "visited" flag on Java heap objects
- calls collect_stack_roots() before collect_simple_roots()

Ashok and I have been refining the two optimizations, running tests
and trying to characterize the results.
                                     

EVALUATION

This bug will be used to handle the:

- removes unnecessary reset of a "visited" flag on Java heap objects

optimization.

The other optimization:

- calls collect_stack_roots() before collect_simple_roots()

will be handled by the following bug:

    6951623 3/3 possible performance problems in FollowReferences()
                and GetObjectsWithTags()
                                     

SUGGESTED FIX

Please see the attached 7043987-webrev-cr0.tgz for
the proposed fix (code review round 0).