> This PR guards usage of FileForce, SocketRead, and SocketWrite events with 
> `jfrTracing` to prevent those classes from being loaded when JFR is not in 
> use. This is the same technique as what's currently used with exception 
> events and FileRead/FileWrite events.
> 
> I used NMT and a simple test app that exercises file force and socket IO 
> paths to check for a difference in memory usage.
> **NMT Before:** 
> Classes=1188, Metadata used=966064 B, Class space used=93008 B
> **NMT After:**
> Classes=1182, Metadata used=943728 B, Class space used=89456 B
> Note that the difference in amount used doesn't actually change amount 
> committed because the backing memory is pre-allocated in chunks with larger 
> granularity.
> 
> Testing:
> - new test test/jdk/jdk/jfr/event/io/TestEventsNotLoadedWithoutJfr.java to 
> check the guards work properly
> - tier 1
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Robert Toyonaga has updated the pull request incrementally with two additional 
commits since the last revision:

 - use feedback to improve test
 - update copywrite headers

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/30948/files
  - new: https://git.openjdk.org/jdk/pull/30948/files/3394e886..d28d941a

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=30948&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=30948&range=01-02

  Stats: 100 lines in 10 files changed: 6 ins; 51 del; 43 mod
  Patch: https://git.openjdk.org/jdk/pull/30948.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/30948/head:pull/30948

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

Reply via email to