Re: RFR: 8293499: Provide jmod compression level option

2022-09-12 Thread Mark Reinhold
On Mon, 12 Sep 2022 11:48:50 GMT, Claes Redestad wrote: > > Meaning, we allow `--compress=zip-[0..9]` and default to `zip-6`. This has > > the added benefit of clearly stating that `zip-0` is still ZIP, but without > > the compression applied. This is a bit ambiguous with > > `--compression-le

Re: RFR: 8293499: Provide jmod compression level option

2022-09-12 Thread Claes Redestad
On Mon, 12 Sep 2022 11:20:14 GMT, Aleksey Shipilev wrote: > Meaning, we allow `--compress=zip-[0..9]` and default to `zip-6`. This has > the added benefit of clearly stating that `zip-0` is still ZIP, but without > the compression applied. This is a bit ambiguous with `--compression-level=0`.

Re: RFR: 8293499: Provide jmod compression level option

2022-09-12 Thread Aleksey Shipilev
On Mon, 12 Sep 2022 10:48:53 GMT, Claes Redestad wrote: > The part about being open-ended about alternate compression algorithms still > applies, but since the compression for `jmod` is currently always zip we can > defer such wording for if/when that day comes. Well, we can future-proof it by

Re: RFR: 8293499: Provide jmod compression level option [v2]

2022-09-12 Thread Jaikiran Pai
On Mon, 12 Sep 2022 10:48:16 GMT, Aleksey Shipilev wrote: >> I have been looking into `make clean-images images` performance, and >> realized jmod keeps compressing files with default compression level. Tuning >> that toward lighter compression levels improves build performance >> considerably

Re: RFR: 8293499: Provide jmod compression level option [v2]

2022-09-12 Thread Jaikiran Pai
On Mon, 12 Sep 2022 10:48:16 GMT, Aleksey Shipilev wrote: >> I have been looking into `make clean-images images` performance, and >> realized jmod keeps compressing files with default compression level. Tuning >> that toward lighter compression levels improves build performance >> considerably

Re: RFR: 8293499: Provide jmod compression level option

2022-09-12 Thread Claes Redestad
On Mon, 12 Sep 2022 10:26:56 GMT, Aleksey Shipilev wrote: > Um, this is `jmod`, it does not have `--compress=...` option? `jlink` does. > So, I don't quite understand this comment :) Doh! Yes, I mixed up the two and assumed you'd be considering this for `jlink` as well. The part about being op

Re: RFR: 8293499: Provide jmod compression level option

2022-09-12 Thread Aleksey Shipilev
On Mon, 12 Sep 2022 10:18:40 GMT, Jaikiran Pai wrote: > Should we instead complain/fail when this option is used for anything other > than `create`? Good idea, see new commit. - PR: https://git.openjdk.org/jdk/pull/10213

Re: RFR: 8293499: Provide jmod compression level option [v2]

2022-09-12 Thread Aleksey Shipilev
> I have been looking into `make clean-images images` performance, and realized > jmod keeps compressing files with default compression level. Tuning that > toward lighter compression levels improves build performance considerably, > without a heavy loss in *.jmod sizes. > > This PR allows JMO

Re: RFR: 8293499: Provide jmod compression level option

2022-09-12 Thread Aleksey Shipilev
On Mon, 12 Sep 2022 10:15:26 GMT, Jaikiran Pai wrote: >> I have been looking into `make clean-images images` performance, and >> realized jmod keeps compressing files with default compression level. Tuning >> that toward lighter compression levels improves build performance >> considerably, wi

Re: RFR: 8293499: Provide jmod compression level option

2022-09-12 Thread Aleksey Shipilev
On Mon, 12 Sep 2022 10:13:12 GMT, Claes Redestad wrote: > Could we word this so that the accepted values is compression-dependent? The > value appears to be ignored for `--compress=0|1` (should it throw if > specified or not specified to, say, 0?). We might also add other compressions > in the

Re: RFR: 8293499: Provide jmod compression level option

2022-09-12 Thread Jaikiran Pai
On Thu, 8 Sep 2022 07:57:36 GMT, Aleksey Shipilev wrote: > I have been looking into `make clean-images images` performance, and realized > jmod keeps compressing files with default compression level. Tuning that > toward lighter compression levels improves build performance considerably, > wit

Re: RFR: 8293499: Provide jmod compression level option

2022-09-12 Thread Jaikiran Pai
On Thu, 8 Sep 2022 07:57:36 GMT, Aleksey Shipilev wrote: > I have been looking into `make clean-images images` performance, and realized > jmod keeps compressing files with default compression level. Tuning that > toward lighter compression levels improves build performance considerably, > wit

Re: RFR: 8293499: Provide jmod compression level option

2022-09-12 Thread Claes Redestad
On Thu, 8 Sep 2022 07:57:36 GMT, Aleksey Shipilev wrote: > I have been looking into `make clean-images images` performance, and realized > jmod keeps compressing files with default compression level. Tuning that > toward lighter compression levels improves build performance considerably, > wit

Re: RFR: 8293499: Provide jmod compression level option

2022-09-12 Thread Aleksey Shipilev
On Thu, 8 Sep 2022 07:57:36 GMT, Aleksey Shipilev wrote: > I have been looking into `make clean-images images` performance, and realized > jmod keeps compressing files with default compression level. Tuning that > toward lighter compression levels improves build performance considerably, > wit

Re: RFR: 8293499: Provide jmod compression level option

2022-09-09 Thread Alan Bateman
On Thu, 8 Sep 2022 07:57:36 GMT, Aleksey Shipilev wrote: > I have been looking into `make clean-images images` performance, and realized > jmod keeps compressing files with default compression level. Tuning that > toward lighter compression levels improves build performance considerably, > wit

Re: RFR: 8293499: Provide jmod compression level option

2022-09-09 Thread Alan Bateman
On Thu, 8 Sep 2022 07:57:36 GMT, Aleksey Shipilev wrote: > I have been looking into `make clean-images images` performance, and realized > jmod keeps compressing files with default compression level. Tuning that > toward lighter compression levels improves build performance considerably, > wit

Re: RFR: 8293499: Provide jmod compression level option

2022-09-08 Thread Mandy Chung
On Thu, 8 Sep 2022 07:57:36 GMT, Aleksey Shipilev wrote: > I have been looking into `make clean-images images` performance, and realized > jmod keeps compressing files with default compression level. Tuning that > toward lighter compression levels improves build performance considerably, > wit

Re: RFR: 8293499: Provide jmod compression level option

2022-09-08 Thread Alan Bateman
On Thu, 8 Sep 2022 07:57:36 GMT, Aleksey Shipilev wrote: > I have been looking into `make clean-images images` performance, and realized > jmod keeps compressing files with default compression level. Tuning that > toward lighter compression levels improves build performance considerably, > wit

RFR: 8293499: Provide jmod compression level option

2022-09-08 Thread Aleksey Shipilev
I have been looking into `make clean-images images` performance, and realized jmod keeps compressing files with default compression level. Tuning that toward lighter compression levels improves build performance considerably, without a heavy loss in *.jmod sizes. This PR allows JMOD to select