RFR(XS): 8078243: Fix include of stack.inline.hpp in taskqueue.hpp.

[Stefan Karlsson]

Hi Goetz,

On 2015-04-21 10:20, Lindenmaier, Goetz wrote:
>
> Hi,
>
> another include that’s missing.  This time it breaks aix only, would 
> not have been
>
> detected by non-pch build J
>
> taskqueue.hpp calls Stack::push(), thus it must include stack.inline.hpp.
>
> Compilation of parallelScavenge/vmPSOperations.cpp failed.
>
> Please review this tiny change.  I also please need a sponsor.
>
> http://cr.openjdk.java.net/~goetz/webrevs/8078243-pchBld2/webrev.01/ 
> <http://cr.openjdk.java.net/%7Egoetz/webrevs/8078243-pchBld2/webrev.01/>
>

I understand that this is an easy fix, but I don't want stack.inline.hpp 
to be added to the hpp file. The preferred fix would be extract the 
offending push function out of the taskqueue.hpp file and put it in a 
new file called taskqueue.inline.hpp. I created a RFE for that a couple 
of weeks ago:

https://bugs.openjdk.java.net/browse/JDK-8076177 - Remove usage of 
stack.inline.hpp functions from taskqueue.hpp

Until someone takes care about 8076177, could you workaround the compile 
error by including stack.inline.hpp in vmPSOperations.cpp? Do you know 
what in vmPSOperations.cpp depends on taskqueue.hpp? I can't find 
anything, but I see that we include psScavenge.inline.hpp, which 
includes taskqueue.hpp via psPromotionManager.inline.hpp and 
psPromotionManager.hpp. Maybe you could workaround the problem by 
removing the "#include .../psScavenge.inline.hpp" line, which isn't 
needed by vmPSOperations.cpp?

Thanks,
StefanK

> Best regards,
>
>   Goetz.
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20150421/e97e5a38/attachment.htm>