RFR: 8339148: Make os::Linux::active_processor_count() public

Julian Waters jwaters at openjdk.org
Wed Sep 4 12:08:20 UTC 2024


On Wed, 4 Sep 2024 11:23:22 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:

>> Please review this Linux-only change which makes `os::Linux::active_processor_count()` public (from currently protected). This allows us to clean up some `friend class` usages and will also make some container related patches less clunky.
>> 
>> Testing:
>> - [ ] GHA
>> - [x] Container tests on linux x86_64
>> 
>> Thoughts?
>
> Do I need a second review for this or is this trivial enough to go in with one review?

> @jerboaa The label `@tstuefe` is not a valid label. These labels are valid:
> 
> * `graal`
> * `serviceability`
> * `hotspot`
> * `hotspot-compiler`
> * `ide-support`
> * `kulla`
> * `i18n`
> * `shenandoah`
> * `jdk`
> * `javadoc`
> * `security`
> * `hotspot-runtime`
> * `jmx`
> * `build`
> * `nio`
> * `client`
> * `core-libs`
> * `compiler`
> * `net`
> * `hotspot-gc`
> * `hotspot-jfr`

Bahahahahaha - Probably the funniest thing I've seen in a while

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

PR Comment: https://git.openjdk.org/jdk/pull/20833#issuecomment-2328794748


More information about the hotspot-runtime-dev mailing list