CFV: New JDK 9/10 Reviewer: Martin Doerr

Thu Mar 16 13:41:38 UTC 2017

Vote: yes.

Best regards,

Zoltan

On 03/15/2017 08:40 PM, Volker Simonis wrote:
> I hereby nominate Martin Doerr (mdoerr) to JDK 9 and JDK 10 Reviewer.
>
> Martin is a long standing member of the JVM team at SAP. He's a JIT export
> with a strong focus on the PowerPC/s390/Itanium architectures and one of
> the initial authors of the ppc64/s390x ports. During the last years he has
> contributed 46 changesets across the hotspot(45) [3] and jdk (1) [4]
> repositories (see below for a complete list).
>
> Votes are due by 29 March 2017, 22:00 CET.
>
> Only current JDK 9/10 Reviewers [1] are eligible to vote on this
> nomination.  Votes must be cast in the open by replying to this mailing
> list.
>
> Since we are in the borderland between JDK 9 and JDK 10 now I'm not sure if
> we need a vote per project so I'm sending this CFV to both the JDK 9 and
> JDK 10 lists. Unless someone can verify that it is enough to vote for one
> of these projects I kindly ask everybody who is on both lists to reply to
> both mails and treat this as two separate votes.
>
> For Three-Vote Consensus voting instructions, see [2].
>
> Volker Simonis
>
> [1] http://openjdk.java.net/census
> [2] http://openjdk.java.net/projects/#reviewer-vote
> [3]
> http://hg.openjdk.java.net/jdk10/hs/hotspot/log?revcount=60&rev=(author(%22mdoerr%22)+or+desc(%22Contributed-by%3A+martin.doerr%40sap.com%22))+and+not+merge()
> [4]
> http://hg.openjdk.java.net/jdk10/hs/jdk/log?revcount=50&rev=(author(%22mdoerr%22)+or+desc(%22Contributed-by%3A+martin.doerr%40sap.com%22))+and+not+merge()
>
> hotspot:
>
> 8050972: Concurrency problem in PcDesc cache
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/82cd02bbfc3a>
> 8050973: CMS/G1 GC: add missing Resource and Handle mark
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/8842d38f4016>
> 8062950: Bug in locking code when UseOptoBiasInlining is disabled:
> assert(dmw->is_neutral()) failed: invariant
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/ef2e397e8b7b>
> 8068724: ppc64: update assembler: SPR access, CR logic, HTM
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/4374388bcbbb>
> 8136525: Generate interpreter entries only once and avoid unnecessary jump
> to jump <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/17efe8fc4f48>
> 8139421: PPC64LE: MacroAssembler::bxx64_patchable kill register R12
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/c6a1e7983723>
> 8139734: ppc: fix build after "8078556: Runtime: implement ranges..."
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/09c316072f18>
> 8138890: C1: Ambiguous operator delete
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/a0c5acb7c322>
> 8138894: C1: Support IRIW on weak memory platforms
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/50bd312dedb1>
> 8138892: C1: Improve counter overflow checking
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/3df67250a63a>
> 8138895: C1: PPC64 Port needs special register for Locks in synchronization
> code <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/260779207e00>
> 8138952: C1: Distinguish between PPC32 and PPC64
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/b946bb90c515>
> 8138896: C1: NativeGeneralJump is mixed up with NativeCall in C1 patching
> code <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/a5909414bbe4>
> 8144847: PPC64: Update Transactional Memory and Atomic::cmpxchg code
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/1fd6f49c4b51>
> 8144019: PPC64 C1: Introduce Client Compiler
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/4a24de859a87>
> 8136445: Performance issue with Nashorn and C2's global code motion
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/07cbc9ee9cd0>
> 8144822: PPC64: Fix build after 8072008
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/0e212b4de71a>
> 8143817: C1: Platform dependent stack space not preserved for all runtime
> calls <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/eeaef3c57176>
> 8144850: C1: operator delete needs an implementation
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/2c44727165a3>
> 8146231: ppc64/gcc 4.1.2: fix build after "8143072: [JVMCI] Port JVMCI to
> AArch64" <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/78ffbdb49065>
> 8145913: PPC64: add Montgomery multiply intrinsic
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/ce0dacc26f3d>
> 8146612: C2: Precedence edges specification violated
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/f6615ec051d9>
> 8146613: PPC64: C2 does no longer respect int to long conversion for stub
> calls <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/555c4d3f2fa5>
> 8146978: PPC64: Fix build after integration of C++ interpreter removal
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/7c1c2a79f981>
> 8149655: PPC64: Implement CompactString intrinsics
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/f1c5937e76a2>
> 8151818: C1: LIRGenerator::move_to_phi can't deal with illegal phi
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/4226e949ec95>
> 8154867: PPC64: Better byte behavior
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/de92190be5e1>
> 8153267: nmethod's exception cache not multi-thread safe
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/95b8ce0f1228>
> 8154836: VM crash due to "Base pointers must match"
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/003924155da8>
> 8141635: Implement VarHandles/Unsafe intrinsics on POWER
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/c8325c5c290d>
> 8159976: PPC64: Add missing intrinsics for sub-word atomics
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/84d6afe30a53>
> 8165014: Unaligned unsafe access should throw InternalError on Solaris
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/ee428b2585cd>
> 8165018: Missing memory barrier for PPC64 in Unsafe_GetObjectVolatile
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/6ed5d1d6f24c>
> 8165489: Missing G1 barrier in Unsafe_GetObjectVolatile
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/a696583f5ddb>
> 8166140: C1: Possible integer overflow in LIRGenerator::generate_address on
> several platforms <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/f6c1ea29110e>
> 8155729: C2: Skip transformation of LoadConP for heap-based compressed oops
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/5baf89bc0a4b>
> 8166689: PPC64: Race condition between stack bang and non-entrant patching
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/32b7bb18922d>
> 8166970: Adapt mutex padding according to DEFAULT_CACHE_LINE_SIZE
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/036f46dcf262>
> 8168083: PPC64: Cleanup template interpreter after 8154580 and 8154867
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/8cd9f7147af1>
> 8170991: PPC64: Bad code for initialization of short arrays
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/545ddc149651> 8171244: PPC64:
> Make interpreter's math entries consistent with C1 and C2 and support FMA
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/9d5a9f0570e6>
> 8171398: s390x: Make interpreter's math entries consistent with C1 and C2
> and support FMA <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/66107bf60a17>
> 8172145: C2: anti dependence missed because store hidden by membar
> <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/9102f200c421>
> 8172199: s390: Use same get_key_start_from_aescrypt_object implementation
> as PPC64 <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/96db752884e3>
> 8029101: PPC64 (part 211): ordering of Independent Reads of Independent
> Writes <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/c6d7e7406136>
>
> jdk:
>
> 8053931: (fc) FileDispatcherImpl.lock0 does not handle ERROR_IO_PENDING
> [win] <http://hg.openjdk.java.net/jdk9/dev/jdk/rev/79b2e88250a9>