Re: RFR(xs): 8193909: Obsolete(remove) Co-operative Memory Management (CMM)

Hi Thomas, On 02/15/2018 03:32 AM, Thomas Schatzl wrote: Hi, On Wed, 2018-02-14 at 13:45 -0800, sangheon.kim wrote: Hi all, Could I have some reviews for CMM removal? This is closed CR but some public codes also need small modifications. This CR is for removing stuff related to an Oracle JDK

Re: RFR(xs): 8193909: Obsolete(remove) Co-operative Memory Management (CMM)

Hi, On Wed, 2018-02-14 at 13:45 -0800, sangheon.kim wrote: > Hi all, > > Could I have some reviews for CMM removal? > This is closed CR but some public codes also need small > modifications. > This CR is for removing stuff related to an Oracle JDK > module/package. > Changes are just removing

Re: RFR(xs): 8193909: Obsolete(remove) Co-operative Memory Management (CMM)

Hi Mandy, Thank you for the review! Sangheon On 02/14/2018 03:45 PM, mandy chung wrote: +1 Mandy On 2/14/18 1:45 PM, sangheon.kim wrote: Hi all, Could I have some reviews for CMM removal? This is closed CR but some public codes also need small modifications. This CR is for removing

Re: RFR(xs): 8193909: Obsolete(remove) Co-operative Memory Management (CMM)

+1 Mandy On 2/14/18 1:45 PM, sangheon.kim wrote: Hi all, Could I have some reviews for CMM removal? This is closed CR but some public codes also need small modifications. This CR is for removing stuff related to an Oracle JDK module/package. Changes are just removing CMM from lists or in a

Re: RFR(xs): 8193909: Obsolete(remove) Co-operative Memory Management (CMM)

Hi David and Jesper, Thank you for the review. Sangheon On 02/14/2018 02:53 PM, jesper.wilhelms...@oracle.com wrote: Looks good! /Jesper On 14 Feb 2018, at 22:45, sangheon.kim wrote: Hi all, Could I have some reviews for CMM removal? This is closed CR but some

Re: RFR(xs): 8193909: Obsolete(remove) Co-operative Memory Management (CMM)

Looks good! /Jesper > On 14 Feb 2018, at 22:45, sangheon.kim wrote: > > Hi all, > > Could I have some reviews for CMM removal? > This is closed CR but some public codes also need small modifications. This > CR is for removing stuff related to an Oracle JDK

Re: RFR(xs): 8193909: Obsolete(remove) Co-operative Memory Management (CMM)

That all seems trivially fine. Thanks, David On 15/02/2018 7:45 AM, sangheon.kim wrote: Hi all, Could I have some reviews for CMM removal? This is closed CR but some public codes also need small modifications. This CR is for removing stuff related to an Oracle JDK module/package. Changes are

RFR(xs): 8193909: Obsolete(remove) Co-operative Memory Management (CMM)

Hi all, Could I have some reviews for CMM removal? This is closed CR but some public codes also need small modifications. This CR is for removing stuff related to an Oracle JDK module/package. Changes are just removing CMM from lists or in a test to skip the testing logic. CR: