gmane.comp.java.openjdk.hotspot.gc.devel

hg: hsx/hotspot-gc/langtools: 6 new changesets

2012-05-25T23:19:27

Changeset: d10db3576c08
Author:    ksrini
Date:      2012-05-04 07:55 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/langtools/rev/d10db3576c08

7166010: (javac) JavacMessager incorrectly restores log source file
Reviewed-by: jjg
Contributed-by: jan.lahoda-QHcLZuEGTsvQT0dZR+AlfA< at >public.gmane.org

! src/share/classes/com/sun/tools/javac/processing/JavacMessager.java
+ test/tools/javac/processing/messager/MessagerDiags.java

Changeset: 833bab705918
Author:    ksrini
Date:      2012-05-10 12:32 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/langtools/rev/833bab705918

7159445: (javac) emits inaccurate diagnostics for enhanced for-loops
Reviewed-by: jjg
Contributed-by: jan.lahoda-QHcLZuEGTsvQT0dZR+AlfA< at >public.gmane.org

! src/share/classes/com/sun/tools/javac/parser/JavacParser.java
! src/share/classes/com/sun/tools/javac/resources/compiler.properties
+ test/tools/javac/diags/examples/ForeachBadInitialization.java
! test/tools/javac/parser/JavacParserTest.java

Changeset: 96a8278e323c
Aut

Request for review (XXS): 7171936: LOG_G incorrectly defined inglobalDefinitions.hpp

Bengt Rutisson — 2012-05-25T20:47:27

Hi all,

Can I have a couple of reviews for this really small change:
http://cr.openjdk.java.net/~brutisso/7171936-LOG_G/webrev.00/

Background:
The constant LOG_G in globalDefinitions.hpp is incorrectly definied as 
40. It should be 30. LOG_G is not being used anywhere, so rather than 
fixing it we should remove it.

LOG_K is also not used and LOG_M is only used in one single place. I 
suggest that we remove LOG_K and move LOG_M to where it is being used.

Thanks Ulf Zibis for catching the wrongly defined constant.

Bengt

hg: hsx/hotspot-gc/jdk: 71 new changesets

2012-05-25T03:49:35

Changeset: 7c4eed4b6c19
Author:    bae
Date:      2012-05-21 14:04 +0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/7c4eed4b6c19

7124400: [macosx] CGraphicsDevice.getConfigurations() returns reference to member (does not copy configs)
Reviewed-by: anthony, kizune

! src/macosx/classes/sun/awt/CGraphicsDevice.java
! test/java/awt/GraphicsDevice/CloneConfigsTest.java

Changeset: 416b3a498e71
Author:    bae
Date:      2012-05-21 14:53 +0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/416b3a498e71

7154088: [macosx] Regression: Component.createImage do not inherits component attributes
Reviewed-by: art, kizune

! src/macosx/classes/sun/lwawt/LWComponentPeer.java

Changeset: 1b90a0113359
Author:    lana
Date:      2012-05-21 11:20 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/1b90a0113359

Merge


Changeset: c31eeeda3ed1
Author:    serb
Date:      2012-05-03 18:29 +0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/c31eeeda3ed1

7160623: [mac

hg: hsx/hotspot-gc/jaxws: Added tag jdk8-b40 for changeset09a0ddda03cb

2012-05-25T03:48:25

Changeset: f2072b164b05
Author:    katleman
Date:      2012-05-24 16:15 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jaxws/rev/f2072b164b05

Added tag jdk8-b40 for changeset 09a0ddda03cb

! .hgtags

hg: hsx/hotspot-gc/jaxp: Added tag jdk8-b40 for changeset 9ecfdbd6aed4

2012-05-25T03:48:09

Changeset: 6f5c0e17415d
Author:    katleman
Date:      2012-05-24 16:15 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jaxp/rev/6f5c0e17415d

Added tag jdk8-b40 for changeset 9ecfdbd6aed4

! .hgtags

hg: hsx/hotspot-gc/corba: Added tag jdk8-b40 for changeset56d030e5035f

2012-05-25T03:48:00

Changeset: 113f0d5f0a08
Author:    katleman
Date:      2012-05-24 16:15 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/corba/rev/113f0d5f0a08

Added tag jdk8-b40 for changeset 56d030e5035f

! .hgtags

hg: hsx/hotspot-gc: Added tag jdk8-b40 for changeset a2b2d435f1d2

2012-05-25T03:47:54

Changeset: 1a8c7c530f8a
Author:    katleman
Date:      2012-05-24 16:15 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/rev/1a8c7c530f8a

Added tag jdk8-b40 for changeset a2b2d435f1d2

! .hgtags

Re: RFR(XS): 7143858: G1: Back to back young GCs with the second GChavinga minimally sized eden

Charlie Hunt — 2012-05-23T17:17:27

Looks great John!

I knew you'd get this (annoying behavior) figured out.

charlie ...

On May 21, 2012, at 6:12 PM, John Cuthbertson wrote:

Re: RFR(XS): 7143858: G1: Back to back young GCs with the second GChaving a minimally sized eden

Bengt Rutisson — 2012-05-23T14:28:30

John,

Looks good!

Nice catch.

Bengt

22 maj 2012 kl. 01:12 skrev John Cuthbertson <john.cuthbertson-QHcLZuEGTsvQT0dZR+AlfA< at >public.gmane.org>:

RFR(XS): 7143858: G1: Back to back young GCs with the second GC havinga minimally sized eden

John Cuthbertson — 2012-05-21T23:12:36

Hi Everyone,

Can I have a couple of volunteers review the fix for this CR? The webrev 
can be found at: http://cr.openjdk.java.net/~johnc/7143858/webrev.0/

Description:

In some recent G1 logs we have seen some evacuation pauses that looked 
premature, i.e. the eden occupancy was much less than the target 
capacity - usually only one or two regions. These premature pauses 
always (almost immediately) followed a normal evacuation pause with 
little or no application activity in-between. Bengt's recent changes to 
display the GC cause indicated that the premature pauses were always 
GCLocker Initiated GCs.

What seemed to have been happening was that as the last left a JNI 
critical region, before it scheduled the GCLocker Initiated GC, another 
thread would attempt an allocation. The allocating thread would see that 
GCLocker was no longer active and successfully schedule the evacuation 
pause. As part of this GC operation, a mutator alloc region would get 
allocated and the object allocation request would

hg: hsx/hotspot-gc/hotspot: 7168848: Add test to check that humongousobject allocation path also checks the heap occupancy.

2012-05-21T16:04:11

Changeset: 9a344d88dc22
Author:    mnunez
Date:      2012-05-21 14:59 +0200
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/9a344d88dc22

7168848: Add test to check that humongous object allocation path also checks the heap occupancy.
Summary: Added test that checks humongous object allocation path also check the heap occupancy and initiate a marking cycle when / if needed.
Reviewed-by: brutisso, jwilhelm

+ test/gc/7168848/HumongousAlloc.java

hg: hsx/hotspot-gc/hotspot: 7169062: CMS: Assertion failed with-XX:+ObjectAlignmentInBytes=64

2012-05-21T05:18:40

Changeset: c92a79900986
Author:    brutisso
Date:      2012-05-17 21:53 +0200
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/c92a79900986

7169062: CMS: Assertion failed with -XX:+ObjectAlignmentInBytes=64
Summary: Removed the assert in CompactibleFreeListSpace::check_free_list_consistency() since it was too strict. Simplified CompactibleFreeListSpace::set_cms_values() to reducde the need for asserts.
Reviewed-by: jcoomes, stefank

! src/share/vm/gc_implementation/concurrentMarkSweep/compactibleFreeListSpace.cpp

hg: hsx/hotspot-gc/hotspot: 9 new changesets

2012-05-19T07:30:42

Changeset: 26423ef693ac
Author:    katleman
Date:      2012-05-17 06:20 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/26423ef693ac

Added tag jdk8-b39 for changeset 96a403721094

! .hgtags

Changeset: 35e504cb49a6
Author:    collins
Date:      2012-05-11 11:30 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/35e504cb49a6

7167625: Adjustments for SE-Embedded build process
Summary: Simple change to the SE-Embedded build rules that should not affect any other OpenJDK users.
Reviewed-by: kvn, dholmes

! make/linux/makefiles/vm.make
! src/share/vm/runtime/arguments.cpp

Changeset: fada85d11d92
Author:    jprovino
Date:      2012-05-16 13:33 -0400
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/fada85d11d92

Merge


Changeset: 8f972594effc
Author:    kvn
Date:      2012-05-14 09:36 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/hotspot/rev/8f972594effc

6924259: Remove String.count/String.offset
Summary: Allow a version of String class th

Re: Request for review (S): 7169062 CMS: Assertion failed with-XX:+ObjectAlignmentInBytes=64

Bengt Rutisson — 2012-05-18T08:24:02

John,

Thanks for the review!

Stefan also looked at this, so I'm all set to push.

Thanks!
Bentg

On 2012-05-18 01:07, John Coomes wrote:

hg: hsx/hotspot-gc/langtools: Added tag jdk8-b39 for changeseta9f547c218d9

2012-05-18T03:57:03

Changeset: 8b869afd2eb4
Author:    katleman
Date:      2012-05-17 06:20 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/langtools/rev/8b869afd2eb4

Added tag jdk8-b39 for changeset a9f547c218d9

! .hgtags

hg: hsx/hotspot-gc/jdk: 4 new changesets

2012-05-18T03:54:28

Changeset: c2d9166f3284
Author:    ihse
Date:      2012-05-11 08:21 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/c2d9166f3284

7168208: Change use of < at > in one sed command involving paths to different character
Reviewed-by: ohair

! make/common/Release.gmk

Changeset: 8d665b69ebf1
Author:    mfang
Date:      2012-05-15 11:46 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/8d665b69ebf1

7157855: jvisualvm.1 not included in binaries
Reviewed-by: katleman, thurka

! make/common/Release.gmk

Changeset: b6f529117521
Author:    katleman
Date:      2012-05-16 22:07 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/b6f529117521

Merge


Changeset: 47cd90bf0f66
Author:    katleman
Date:      2012-05-17 06:20 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jdk/rev/47cd90bf0f66

Added tag jdk8-b39 for changeset b6f529117521

! .hgtags

hg: hsx/hotspot-gc/jaxws: Added tag jdk8-b39 for changeset7f6b44fd3034

2012-05-18T03:54:14

Changeset: 09a0ddda03cb
Author:    katleman
Date:      2012-05-17 06:20 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jaxws/rev/09a0ddda03cb

Added tag jdk8-b39 for changeset 7f6b44fd3034

! .hgtags

hg: hsx/hotspot-gc/jaxp: Added tag jdk8-b39 for changeset f95fdbe525c8

2012-05-18T03:54:01

Changeset: 9ecfdbd6aed4
Author:    katleman
Date:      2012-05-17 06:20 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/jaxp/rev/9ecfdbd6aed4

Added tag jdk8-b39 for changeset f95fdbe525c8

! .hgtags

hg: hsx/hotspot-gc/corba: Added tag jdk8-b39 for changeset785af00e2827

2012-05-18T03:53:54

Changeset: 56d030e5035f
Author:    katleman
Date:      2012-05-17 06:20 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/corba/rev/56d030e5035f

Added tag jdk8-b39 for changeset 785af00e2827

! .hgtags

hg: hsx/hotspot-gc: 4 new changesets

2012-05-18T03:53:49

Changeset: 955a3e8ed4f0
Author:    ohair
Date:      2012-05-10 08:26 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/rev/955a3e8ed4f0

7167593: Changed get_source.sh to allow for getting full oracle jdk repo forest
Reviewed-by: erikj, asaha, chegar, sla, dholmes, mbykov, coleenp

! get_source.sh
! make/scripts/hgforest.sh

Changeset: 8a4e92c10a9a
Author:    ohair
Date:      2012-05-11 17:52 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/rev/8a4e92c10a9a

7167976: Fix broken get_source.sh script
Reviewed-by: tbell

! make/scripts/hgforest.sh

Changeset: 8927dd68aee3
Author:    katleman
Date:      2012-05-16 22:06 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/rev/8927dd68aee3

Merge


Changeset: a2b2d435f1d2
Author:    katleman
Date:      2012-05-17 06:20 -0700
URL:       http://hg.openjdk.java.net/hsx/hotspot-gc/rev/a2b2d435f1d2

Added tag jdk8-b39 for changeset 8927dd68aee3

! .hgtags

Re: Request for review (S): 7169062 CMS: Assertion failed with-XX:+ObjectAlignmentInBytes=64

John Coomes — 2012-05-17T23:07:00

Looks good to me.

-John