RFR: 8311661: Resolve duplicate symbol of StringTable::StringTable with JDK static linking [v4]

[Jiangli Zhou]

On Thu, 13 Jul 2023 18:01:47 GMT, Ioi Lam <iklam at openjdk.org> wrote:

> Hi Jiangli,
> 
> Putting the HotSpot code in a namespace is a fundamental change that should be done in a JEP. The idea of using namespaces for HotSpot has been around for a while, but so far there hasn't been a strong motivation for doing it. Perhaps static linking would finally give us a go-ahead reason.
> 
> The JEP should discuss the goals, design choices, risks and alternatives. For example:
> 
> * Other than static linking, what other problems can we solve with namespaces? Knowing the other goals may help us in choosing a design.
> * Do we want a single namespace, or multiple namespaces (one for GC, one for JIT, etc)
> * Do we want to change over incrementally (as in this PR), or in a single step
> * Do we want to enable namespaces optionally (e.g., only for static linking)?
> * With namespaces, the debug symbols will be much bigger, and we could also run into issues with debuggers and other tools.
> * There may be alternatives for static linking that may have less impact than namespaces.
> 
> The [hotspot-dev mailing list](https://mail.openjdk.org/mailman/listinfo/hotspot-dev) would be the best place to have such discussions. Also, at this time, many OpenJDK developers who are interested on this topic are on vacation, so we probably need to wait till the end of the summer so everyone has a chance to chime in.
> 
> JEP may feel like a lot of process, but for this change, I think it's the best avenue for exploration.
> 
> Thanks Ioi

Thanks for the response, @iklam. Going through the JEP for the hotspot namespace makes a lot of sense. Waiting until the end of summer also sounds good to me.

Maybe it's still early to ask the question, would you be willing to drive the JEP for the namespace changes? I'm happy to write up the JEP otherwise.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/14808#issuecomment-1634731589