RFR: 8312132: Add tracking of multiple address spaces in NMT [v46]

Johan Sjölen jsjolen at openjdk.org
Tue May 7 12:04:59 UTC 2024


On Thu, 25 Apr 2024 10:23:35 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

>> src/hotspot/share/nmt/vmatree.cpp line 242:
>> 
>>> 240:   }
>>> 241:   return diff;
>>> 242: }
>> 
>> Would be nice if we can break this function into some smaller sub-functions. It is 200+ line now and little hard to track the logic. Thanks!
>
> Sure, I think there are a couple of cases which are actual functions (taking input, producing output, nothing else), those can be converted.

I've cleaned up the code a bit. I don't think there's much of a point in hiding the computations behind subfunctions, as it reads from top to bottom.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18289#discussion_r1592364938


More information about the hotspot-dev mailing list