RFR: JDK-8197534: [TESTBUG] compiler/gcbarriers/PreserveFPRegistersTest.java fails with Conflicting collector combinations

[Vladimir Kozlov]

Reviewed.

Vaibhav, please, don't forget to use original 8153194 bug id in your changeset for 8u as Tobias 
suggested.

Thanks,
Vladimir

On 2/15/18 12:33 AM, Tobias Hartmann wrote:
> Hi Vaibhav,
> 
> looks good to me. Unfortunately, I'm not a JDK 8u reviewer.
> 
> Thanks,
> Tobias
> 
> On 14.02.2018 16:22, Vaibhav Choudhary wrote:
>> Thank you again.
>>
>> This issue is not a clean backport (mean to say “as it is”). There is a trivial change from 9 to 8u code, which is :-
>>
>> In 8u, * @library /testlibrary /testlibrary/whitebox
>> In 9,  * @library /testlibrary /test/lib
>>
>> Rest of the code is same.
>>
>> Thanks,
>> Vaibhav Choudhary
>> vaibhav.x.choudhary at oracle.com
>> https://blogs.oracle.com/vaibhav
>>
>>
>>
>>> On 14-Feb-2018, at 7:55 PM, Tobias Hartmann <tobias.hartmann at oracle.com> wrote:
>>>
>>> Hi Vaibhav,
>>>
>>> If the backport applies cleanly, you don't need a new review but request approval on jdk8u-dev according to:
>>> http://openjdk.java.net/projects/jdk8u/approval-template.html
>>>
>>> You also don't need a new bug but the backport will be automatically created once you push with 8153194 as bug id.
>>> In this case, you can close 8197534 as duplicate of 8153194.
>>>
>>> Thanks,
>>> Tobias
>>>
>>>
>>> On 14.02.2018 15:15, Vaibhav Choudhary wrote:
>>>> Thanks Tobias.
>>>>
>>>> In that case, I need to change the scope of the bug. Here is the new webrev :-
>>>>
>>>> http://cr.openjdk.java.net/~rpatil/8197534/webrev.01/
>>>>
>>>> Thanks,
>>>> Vaibhav Choudhary
>>>> vaibhav.x.choudhary at oracle.com <mailto:vaibhav.x.choudhary at oracle.com>
>>>> https://blogs.oracle.com/vaibhav
>>>>
>>>>
>>>>
>>>>> On 13-Feb-2018, at 2:50 PM, Tobias Hartmann <tobias.hartmann at oracle.com <mailto:tobias.hartmann at oracle.com>> wrote:
>>>>>
>>>>> Hi Vaibhav,
>>>>>
>>>>> You linked the wrong webrev. Here's the correct one:
>>>>> http://cr.openjdk.java.net/~rpatil/8197534/webrev.00
>>>>>
>>>>> Wouldn't it make more sense to backport the full fix for JDK-8153194 [1]?
>>>>>
>>>>> Thanks,
>>>>> Tobias
>>>>>
>>>>> [1] https://bugs.openjdk.java.net/browse/JDK-8153194
>>>>>
>>>>>
>>>>> On 13.02.2018 09:49, Vaibhav Choudhary wrote:
>>>>>> Hi All,
>>>>>> Please review the fix for bug: https://bugs.openjdk.java.net/browse/JDK-8197534
>>>>>> Webrev: http://cr.openjdk.java.net/~rpatil/8197534/webrev.00/ <http://cr.openjdk.java.net/~rpatil/8141243/webrev.00>
>>>>>>
>>>>>> Thanks,
>>>>>> Vaibhav C
>>>>>>
>>>>
>>