RFR: 8253418: ZGC: Use pd_ prefix to denote platform dependent code

Per Lidén pliden at openjdk.java.net
Mon Sep 21 11:36:39 UTC 2020


On Mon, 21 Sep 2020 11:06:45 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

> Rename ZGC's platform dependent code.

Changes requested by pliden (Reviewer).

src/hotspot/os/posix/gc/z/zInitialize_posix.cpp line 27:

> 25: #include "gc/z/zInitialize.hpp"
> 26:
> 27: void ZInitialize::os_initialize() {

Shouldn't this be pd_initialize?

src/hotspot/os/posix/gc/z/zVirtualMemory_posix.cpp line 32:

> 30: #include <sys/types.h>
> 31:
> 32: void ZVirtualMemoryManager::os_initialize() {

Same here.

src/hotspot/os/windows/gc/z/zInitialize_windows.cpp line 28:

> 26: #include "gc/z/zSyscall_windows.hpp"
> 27:
> 28: void ZInitialize::os_initialize() {

And here.

src/hotspot/os/windows/gc/z/zVirtualMemory_windows.cpp line 90:

> 88: }
> 89:
> 90: void ZVirtualMemoryManager::os_initialize() {

And here.

src/hotspot/share/gc/z/zInitialize.cpp line 55:

> 53:   ZBarrierSet::set_barrier_set(barrier_set);
> 54:
> 55:   os_initialize();

And here.

src/hotspot/share/gc/z/zInitialize.hpp line 33:

> 31: class ZInitialize {
> 32: private:
> 33:   void os_initialize();

And here.

src/hotspot/share/gc/z/zVirtualMemory.cpp line 52:

> 50:
> 51:   // Initialize OS specific parts
> 52:   os_initialize();

And here.

src/hotspot/share/gc/z/zVirtualMemory.hpp line 54:

> 52:
> 53:   // OS specific implementation
> 54:   void os_initialize();

And here.

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

PR: https://git.openjdk.java.net/jdk/pull/283



More information about the hotspot-gc-dev mailing list