PING: RFR: 8194249: SA: G1HeapRegionTable#getByAddress() returns incorrect HeapRegion

[Thomas Schatzl]

Hi all,

On Tue, 2018-01-30 at 11:06 +0100, Thomas Schatzl wrote:
> Hi,
> 
> On Tue, 2018-01-30 at 07:00 +1000, David Holmes wrote:
> > Added in hotspot-gc-dev. Although this is in the SA it is about the
> > SA interaction with G1 and so likely needs someone familiar with G1
> > to review it.
> > 
> > David
> > 
> > On 28/01/2018 10:41 PM, Yasumasa Suenaga wrote:
> > > PING: Could you review it?
> > > 
> > > > >  http://cr.openjdk.java.net/~ysuenaga/JDK-8194249/webrev.01/
> > > 
> > > This webrev has been reviewed by Jini.
> > > I need a Reviewer and sponsor.
> 
>   looks good to me - however there is another (pre-existing) bug: the
> shift in that code should be a logical shift, not an arithmetic
> shift.
> 
> I.e. ">>" instead of ">>>".
> 
> I will run the patch through testing and report back in a few hours.
> Should be okay.

  is good. Do you want to fix the issue with the shift operator too
here, or use another CR?

Thanks,
  Thomas