SUGGESTED FIX

Event:            putback-to
Parent workspace: /net/jano.sfbay/export/disk05/hotspot/ws/main/gc_baseline
                  (jano.sfbay:/export/disk05/hotspot/ws/main/gc_baseline)
Child workspace:  /net/prt-web.sfbay/prt-workspaces/20061024144131.ysr.gc_baseline/workspace
                  (prt-web:/net/prt-web.sfbay/prt-workspaces/20061024144131.ysr.gc_baseline/workspace)
User:             ysr

Comment:

---------------------------------------------------------

Job ID:                 20061024144131.ysr.gc_baseline
Original workspace:     neeraja:/net/spot/scratch/ysr/gc_baseline
Submitter:              ysr
Archived data:          /net/prt-archiver.sfbay/data/archived_workspaces/main/gc_baseline/2006/20061024144131.ysr.gc_baseline/
Webrev:                 http://analemma.sfbay.sun.com/net/prt-archiver.sfbay/data/archived_workspaces/main/gc_baseline/2006/20061024144131.ysr.gc_baseline/workspace/webrevs/webrev-2006.10.24/index.html

Fixed 6252656: Putative invariant for TLABS _start+_size==_end+alignment_reserve() not being maintained

  Webrev: http://analemma.sfbay/net/spot/scratch/ysr/gc_baseline/webrev

As the email discussion in the bug report noted, the confusion stemmed
mainly from the name of the "_size" field, and the associated comment.
This has been fixed by renaming the field to "_desired_size".
The changes are purely syntactic. If necessary, when the need arises,
one can add an actual_size() method that merely returns the
difference between top() and end().

Reviewed by: John Coomes, Peter Kessler, Tony Printezis

Fix Verified: N/A (not a bug in the traditional sense)

Verification Testing: N/A

Other testing: minimal: build+specjvm 98 run; also PRT

Files:
update: src/share/vm/memory/threadLocalAllocBuffer.cpp
update: src/share/vm/memory/threadLocalAllocBuffer.hpp
update: src/share/vm/memory/threadLocalAllocBuffer.inline.hpp
update: src/share/vm/runtime/thread.hpp
update: src/share/vm/runtime/vmStructs.cpp

Examined files: 3884

Contents Summary:
       5   update
    3879   no action (unchanged)
                                     

EVALUATION

fix will be rolled with the changes for 6483660.
                                     

SUGGESTED FIX

See email in comments section; without having looked closely at the
code at this point, we probably prefer Ross' second alternative:

   "create an method to compute the actual size
    and change the comment and name of the _size
    field."

This field will be updated by RE with actual fix upon putback.
###@###.### 2005-04-08 16:51:33 GMT