RFR: 8357086: os::xxx functions returning memory size should return size_t [v17]

[Anton Artemov]

> Hi,
> 
> in this PR the output value types for functions which return memory are changed, namely:
> 
> 
> static julong available_memory(); --> static bool available_memory(size_t& value);
> static julong used_memory(); --> static bool used_memory(size_t& value);
> static julong free_memory(); --> static bool free_memory(size_t& value); 
> static jlong total_swap_space(); --> static bool total_swap_space(size_t& value);
> static jlong free_swap_space(); --> static bool free_swap_space(size_t& value); 
> static julong physical_memory(); --> static bool physical_memory(size_t& value); 
> 
> 
> The return boolean value indicates success, whereas the actual value is assigned to the input argument. The following recommended usage pattern is introduced: where applicable, and unsuccessful call is logged. 
> 
> Later, the return value can be attributed with `[[nodiscard]]` to enforce the pattern.
> 
> Tested in GHA and Tiers 1-5.

Anton Artemov has updated the pull request incrementally with one additional commit since the last revision:

  8357086: Addressed reviewer's comments

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/25450/files
  - new: https://git.openjdk.org/jdk/pull/25450/files/0ed9b2e1..bbfedaf0

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=25450&range=16
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=25450&range=15-16

  Stats: 49 lines in 7 files changed: 0 ins; 33 del; 16 mod
  Patch: https://git.openjdk.org/jdk/pull/25450.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/25450/head:pull/25450

PR: https://git.openjdk.org/jdk/pull/25450