Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Vladimir Kozlov
On Sun, 27 Apr 2025 01:37:33 GMT, Igor Veresov wrote: >> I thought if we can not use AOT cache we issue warning and continue >> execution without it. >> Unless you are checking for damaged AOT cache which may affect execution >> without it. >> >> @iklam what do you think? > > But this runs on

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Igor Veresov
On Sun, 27 Apr 2025 02:00:48 GMT, Vladimir Kozlov wrote: >> @vnkozlov Are you ok with Ioi's code change proposal? > > Yes, I am ok. I assume you will change accordingly all other similar empty > constructors in your code: `MethodCounters()` and `*TrainingData()`. Yes, will do. - P

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Ioi Lam
On Sun, 27 Apr 2025 01:11:39 GMT, Vladimir Kozlov wrote: >> I think it's OK to test without these two additional flags. This will make >> sure that the two diagnostic flags don't have any bad side effect even if >> AOT class linking is disabled (due to flags like -XX:+UseZGC). > > I see them in

About compiling on macOS

2025-04-26 Thread mitani
I have just compiled in the master branch. As a result, I am struggling with the following compilation errors. Configuration summary: * Name: macosx-aarch64-server-release * Debug level:release * HS debug level: product * JVM variants: server -n * JVM features: -n server: 'cds

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Igor Veresov
On Sun, 27 Apr 2025 01:20:45 GMT, Vladimir Kozlov wrote: >> But this is a verification code. That seems to be the usual strategy, is it >> not? > > I thought if we can not use AOT cache we issue warning and continue execution > without it. > Unless you are checking for damaged AOT cache which m

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Vladimir Kozlov
On Sun, 27 Apr 2025 01:49:20 GMT, Igor Veresov wrote: >> src/hotspot/share/oops/methodData.cpp line 1296: >> >>> 1294: >>> 1295: MethodData::MethodData() { >>> 1296: assert(CDSConfig::is_dumping_static_archive() || UseSharedSpaces, >>> "only for CDS"); >> >> 1. Should its code be guarded by

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Igor Veresov
On Sat, 26 Apr 2025 21:30:25 GMT, Vladimir Kozlov wrote: >> Improve warm-up time by making profile data from a previous run of an >> application instantly available, when the HotSpot Java Virtual Machine >> starts. Specifically, enhance the [AOT cache](https://openjdk.org/jeps/483) >> to store

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Vladimir Kozlov
On Fri, 25 Apr 2025 20:18:41 GMT, Igor Veresov wrote: > Improve warm-up time by making profile data from a previous run of an > application instantly available, when the HotSpot Java Virtual Machine > starts. Specifically, enhance the [AOT cache](https://openjdk.org/jeps/483) > to store method

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Igor Veresov
On Sat, 26 Apr 2025 21:07:04 GMT, Vladimir Kozlov wrote: >> Improve warm-up time by making profile data from a previous run of an >> application instantly available, when the HotSpot Java Virtual Machine >> starts. Specifically, enhance the [AOT cache](https://openjdk.org/jeps/483) >> to store

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Igor Veresov
On Sat, 26 Apr 2025 21:28:12 GMT, Vladimir Kozlov wrote: >> Improve warm-up time by making profile data from a previous run of an >> application instantly available, when the HotSpot Java Virtual Machine >> starts. Specifically, enhance the [AOT cache](https://openjdk.org/jeps/483) >> to store

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Igor Veresov
On Sat, 26 Apr 2025 22:09:24 GMT, Vladimir Kozlov wrote: >> Improve warm-up time by making profile data from a previous run of an >> application instantly available, when the HotSpot Java Virtual Machine >> starts. Specifically, enhance the [AOT cache](https://openjdk.org/jeps/483) >> to store

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Igor Veresov
On Sat, 26 Apr 2025 22:15:27 GMT, Vladimir Kozlov wrote: >> Improve warm-up time by making profile data from a previous run of an >> application instantly available, when the HotSpot Java Virtual Machine >> starts. Specifically, enhance the [AOT cache](https://openjdk.org/jeps/483) >> to store

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Igor Veresov
On Sat, 26 Apr 2025 22:42:02 GMT, Vladimir Kozlov wrote: >> Improve warm-up time by making profile data from a previous run of an >> application instantly available, when the HotSpot Java Virtual Machine >> starts. Specifically, enhance the [AOT cache](https://openjdk.org/jeps/483) >> to store

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Igor Veresov
On Sat, 26 Apr 2025 21:30:25 GMT, Vladimir Kozlov wrote: >> Improve warm-up time by making profile data from a previous run of an >> application instantly available, when the HotSpot Java Virtual Machine >> starts. Specifically, enhance the [AOT cache](https://openjdk.org/jeps/483) >> to store

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Igor Veresov
On Sat, 26 Apr 2025 22:32:00 GMT, Vladimir Kozlov wrote: >> Improve warm-up time by making profile data from a previous run of an >> application instantly available, when the HotSpot Java Virtual Machine >> starts. Specifically, enhance the [AOT cache](https://openjdk.org/jeps/483) >> to store

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Igor Veresov
On Sat, 26 Apr 2025 22:11:58 GMT, Vladimir Kozlov wrote: >> Improve warm-up time by making profile data from a previous run of an >> application instantly available, when the HotSpot Java Virtual Machine >> starts. Specifically, enhance the [AOT cache](https://openjdk.org/jeps/483) >> to store

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Ioi Lam
On Sat, 26 Apr 2025 23:35:01 GMT, Igor Veresov wrote: >> test/hotspot/jtreg/runtime/cds/appcds/aotProfile/AOTProfileFlags.java line >> 30: >> >>> 28: * @requires vm.cds >>> 29: * @comment work around JDK-8345635 >>> 30: * @requires !vm.jvmci.enabled >> >> Consider adding: >> >> * @require

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Vladimir Kozlov
On Sun, 27 Apr 2025 00:52:37 GMT, Ioi Lam wrote: >> Could you please explain why? @iklam, what do you think? > > I think it's OK to test without these two additional flags. This will make > sure that the two diagnostic flags don't have any bad side effect even if AOT > class linking is disabled

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Ioi Lam
On Sun, 27 Apr 2025 00:02:41 GMT, Igor Veresov wrote: >> src/hotspot/share/cds/dumpAllocStats.hpp line 151: >> >>> 149: } >>> 150: >>> 151: void record_dynamic_proxy_class() { >> >> This is not called. This code seems not related. > > True. @iklam, this came with a change you wanted me to

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Vladimir Kozlov
On Sun, 27 Apr 2025 00:24:31 GMT, Igor Veresov wrote: >> src/hotspot/share/ci/ciInstanceKlass.hpp line 47: >> >>> 45: friend class ciField; >>> 46: friend class ciReplay; >>> 47: friend class CompileTrainingData; >> >> Not referenced here > > It allows `CompileTrainingData` to peek into t

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling

2025-04-26 Thread Vladimir Kozlov
On Sat, 26 Apr 2025 23:58:04 GMT, Igor Veresov wrote: >> src/hotspot/share/oops/trainingData.cpp line 76: >> >>> 74: >>> 75: static void verify_archived_entry(TrainingData* td, const >>> TrainingData::Key* k) { >>> 76: guarantee(TrainingData::Key::can_compute_cds_hash(k), ""); >> >> Should