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

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

Tags

None

Referenced Files

	F106857287: D17144.diff
	Mon, Jan 6, 12:26 PM

	Unknown Object (File)
	Oct 18 2024, 7:59 PM

	Unknown Object (File)
	Oct 4 2024, 8:29 PM

	Unknown Object (File)
	Oct 2 2024, 4:30 AM

	Unknown Object (File)
	Oct 1 2024, 4:35 PM

	Unknown Object (File)
	Sep 23 2024, 6:21 AM

	Unknown Object (File)
	Sep 21 2024, 3:19 PM

	Unknown Object (File)
	Sep 21 2024, 2:48 AM

View All 37 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