[8u] RFR 8217785: Padding ParallelTaskTerminator::_offered_termination variable

[Aleksey Shipilev]

On 7/30/19 4:50 PM, Zhengyu Gu wrote:
> 8u backport:
> Webrev: http://cr.openjdk.java.net/~zgu/JDK-8217785-8u/webrev.00/

I believe most of 8u code uses just this:

 char _pad_before[DEFAULT_CACHE_LINE_SIZE]
 ...
 char _pad_after[DEFAULT_CACHE_LINE_SIZE]

Therefore, I see no need in doing new defines, which also have potential to leak into global
symbols, especially from the frequently used header like taskqueue.hpp. The change becomes
three-liner then. You need to do #include "utilities/globalDefinitions.hpp" and two new field
definitions.

-- 
Thanks,
-Aleksey