Page MenuHomeFreeBSD

GZIO to update zlib version 1.0.4 to 1.2.11.
ClosedPublic

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

Details

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
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

markj added inline comments.Aug 25 2019, 1:38 AM
sys/kern/subr_compressor.c
121 ↗(On Diff #61245)

Why did you change the level?

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

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 ↗(On Diff #61245)

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
This revision was automatically updated to reflect the committed changes.