ZSTDIO: Correctly initialize zstd context with provided 'level'
ClosedPublic
Actions

Authored by cem on Sep 13 2018, 4:17 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Oct 19, 4:35 AM

	Unknown Object (File)
	Sun, Oct 19, 4:35 AM

	Unknown Object (File)
	Sun, Oct 19, 4:35 AM

	Unknown Object (File)
	Sat, Oct 18, 4:35 PM

	Unknown Object (File)
	Aug 11 2025, 5:55 AM

	Unknown Object (File)
	Aug 3 2025, 6:37 PM

	Unknown Object (File)
	Jul 9 2025, 4:46 AM

	Unknown Object (File)
	Jun 22 2025, 6:53 AM

View All 59 Files

Subscribers

imp

Details

Reviewers

markj
vangyzen

Group Reviewers

releng

Commits

rS339497: ZSTDIO: Correctly initialize zstd context with provided 'level'

Summary

Prior to this revision, we allocated sufficient context space for 'level'
but never actually set the compress level parameter, so we would always get
the default '3'.

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 19558
Build 19143: arc lint + arc unit

Event Timeline

cem created this revision.Sep 13 2018, 4:17 AM

Harbormaster completed remote builds in B19558: Diff 47978.Sep 13 2018, 4:17 AM

vangyzen accepted this revision.Sep 13 2018, 1:04 PM

This revision is now accepted and ready to land.Sep 13 2018, 1:04 PM

markj accepted this revision.Sep 13 2018, 3:37 PM

Request for acceptance into head during freeze

Closed by commit rS339497: ZSTDIO: Correctly initialize zstd context with provided 'level' (authored by cem). · Explain WhyOct 20 2018, 9:50 PM

This revision was automatically updated to reflect the committed changes.

Herald added a subscriber: imp. · View Herald TranscriptOct 20 2018, 9:50 PM

Revision Contents
Changeset List

Path

Size

sys/

kern/

subr_compressor.c

23 lines

Diff 47978

View Options

sys/kern/subr_compressor.c

ZSTDIO: Correctly initialize zstd context with provided 'level'ClosedPublicActions