Differential D21408

GZIO to update zlib version 1.0.4 to 1.2.11.
ClosedPublic
Actions

Authored by ota_j.email.ne.jp on Aug 24 2019, 10:44 PM.

Tags

None

Referenced Files

	F81672774: D21408.id61250.diff
	Fri, Apr 19, 6:06 PM

	F81672765: D21408.id.diff
	Fri, Apr 19, 6:06 PM

	F81672762: D21408.id61245.diff
	Fri, Apr 19, 6:06 PM

	F81672758: D21408.id61257.diff
	Fri, Apr 19, 6:06 PM

	Unknown Object (File)
	Wed, Apr 17, 5:45 AM

	Unknown Object (File)
	Mon, Apr 8, 6:56 AM

	Unknown Object (File)
	Mar 20 2024, 5:13 AM

	Unknown Object (File)
	Mar 14 2024, 8:41 AM

View All 44 Files

Subscribers

Details

Reviewers

markj
delphij
emaste

Commits

rS351477: GZIO: Update to use zlib 1.2.11.

Summary

Use ZLIB's crc32() function.

We need this change before D21375.

Test Plan

% sysctl kern.compress_user_cores=1
% top
% kill -6 <top's pid>
% gzip -d top.core.gz

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 26091
Build 24625: arc lint + arc unit

Event Timeline

ota_j.email.ne.jp created this revision.Aug 24 2019, 10:44 PM

Herald added a subscriber: imp. · View Herald TranscriptAug 24 2019, 10:44 PM

Harbormaster completed remote builds in B26088: Diff 61245.Aug 24 2019, 10:44 PM

ota_j.email.ne.jp mentioned this in D21375: Remove zlib 1.0.4 from kernel..Aug 24 2019, 10:47 PM

markj added inline comments.Aug 25 2019, 1:38 AM

sys/kern/subr_compressor.c
121	Why did you change the level?

Include zutil.h instead of zlib.h in order to pick up OS_CODE and DEF_MEM_LEVEL.

Harbormaster completed remote builds in B26091: Diff 61250.Aug 25 2019, 2:58 AM

ota_j.email.ne.jp edited the test plan for this revision. (Show Details)Aug 25 2019, 2:59 AM

ota_j.email.ne.jp marked an inline comment as done.Aug 25 2019, 3:02 AM

ota_j.email.ne.jp added inline comments.

sys/kern/subr_compressor.c
121	This changeset was brought from my initial changeset. Back in that time, due to some API name conflict and few others, I couldn't include zutil.h. Our zlib migration committed in CURRENT doesn't have this restriction and I was able to revert back.

delphij accepted this revision.Aug 25 2019, 7:47 AM

This revision is now accepted and ready to land.Aug 25 2019, 7:47 AM

Closed by commit rS351477: GZIO: Update to use zlib 1.2.11. (authored by delphij). · Explain WhyAug 25 2019, 7:51 AM

This revision was automatically updated to reflect the committed changes.

delphij added a commit: rS351477: GZIO: Update to use zlib 1.2.11..

Revision Contents
Changeset List

Path

Size

sys/

kern/

subr_compressor.c

10 lines

Diff 61250

sys/kern/subr_compressor.c

Loading...