Code Review for 7019819

Prepared by:never on Tue Feb 15 15:45:18 PST 2011
Compare against: ssh://
Summary of changes: 2 lines changed: 0 ins; 0 del; 2 mod; 362 unchg
Patch of changes: 7019819.patch
Author comments:
7019819: bare oop in ciField

This appears to be a long standing issue though it was previously
unlikely to occur. There's a bare oop in initialize_from over a call
to ciField::type which may call compute_type, which make call into
ciEnv::get_klass_by_name_impl where a safepoint could potentially
occur. The fix for 6354181 added acquire of the Compile_lock which
made it more likely that we might safepoint here. The fix is properly
handleize the oop over the call. Tested with failing test from nightly.

Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/share/vm/ci/ciField.cpp

2 lines changed: 0 ins; 0 del; 2 mod; 362 unchg

This code review page was prepared using /never/bin/webrev (vers 23.18-hg-never).