FS#68470 - [mkinitcpio] zstd -19 is slow

Attached to Project: Arch Linux
Opened by Oleksandr Natalenko (post-factum) - Thursday, 29 October 2020, 22:57 GMT
Last edited by Giancarlo Razzolini (grazzolini) - Tuesday, 05 January 2021, 18:48 GMT
Task Type Bug Report
Category Packages: Core
Status Closed
Assigned To No-one
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 4
Private No

Details

mkinitcpio v29 came with zstd support (yay \o/), but the default compression level is -19, which is horribly slow even on my 12c/24t Zen 2.

Please consider lowering the compression level at least to -10.

Thanks.
This task depends upon

Closed by  Giancarlo Razzolini (grazzolini)
Tuesday, 05 January 2021, 18:48 GMT
Reason for closing:  Implemented
Additional comments about closing:  Implemented by https://github.com/archlinux/mkinitcpio/ issues/40
Comment by Oleksandr Natalenko (post-factum) - Thursday, 29 October 2020, 23:12 GMT
FWIW, COMPRESSION_OPTIONS doesn't seem to work since it appends, not replaces zstd cmdline. I get something like "zstd -T0 -19 -10" if I specify COMPRESSION_OPTIONS=(-10).
Comment by Oleksandr Natalenko (post-factum) - Friday, 30 October 2020, 16:52 GMT
https://github.com/archlinux/mkinitcpio/pull/39
Comment by Oleksandr Natalenko (post-factum) - Thursday, 05 November 2020, 10:33 GMT
https://github.com/archlinux/mkinitcpio/issues/40
Comment by Ben (root) - Wednesday, 02 December 2020, 11:10 GMT
Hello, I was to propose the same fix as pull/39: prepending the default options instead of appending. So, that is a +1.

Although in my opinion the compression level should not be preset in mkinitcpio, because zstd is meant for speed and -19 is a lot slower than gzip, even with -T0. Anyway, the proposed fix allows to override the default, whatever it is. So with the fix the default -19 can be kept.

Loading...