Remove zlib 1.0.4 from kernel.
ClosedPublic
Actions

Authored by delphij on Aug 23 2019, 7:32 AM.

Details

Reviewers

emaste
ota_j.email.ne.jp
markj

Commits

rS351480: Remove zlib 1.0.4 from kernel.

Summary

Remove zlib 1.0.4 from kernel.

PR: 229763

Test Plan

make tinderbox

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 26093
Build 24626: arc lint + arc unit

Event Timeline

delphij created this revision.Aug 23 2019, 7:32 AM

Harbormaster completed remote builds in B26027: Diff 61149.Aug 23 2019, 7:32 AM

emaste accepted this revision.Aug 23 2019, 3:58 PM

emaste added inline comments.

sys/conf/files
4002–4003	Makes no material difference, but presumably this list could have been stripped down with each individual removal?

This revision is now accepted and ready to land.Aug 23 2019, 3:58 PM

We have a last item - sys/kern/subr_compressor.c includes sys/zutil.h.
I attempted and saw crc32() API conflict between gsb_crc32.h and zutil.h.
I haven't got a chance this week, though.

This revision now requires changes to proceed.Aug 23 2019, 11:51 PM

Fix subr_compressor.c.

Harbormaster completed remote builds in B26084: Diff 61240.Aug 24 2019, 7:02 PM

PTAL.

sys/conf/files
4002–4003	Yes.

@markj -- do you have some suggestion on how should we test subr_compressor.c changes?

In D21375#465721, @delphij wrote:

@markj -- do you have some suggestion on how should we test subr_compressor.c changes?

That code is used for user and kernel core dump compression. So:

Set kern.compress_user_cores=1 (a value of 2 means "use zstd") and trigger a user core dump. I usually just run "cat" and enter Ctrl-\ to send SIGQUIT. Verify that you can decompress the core dump and open it in gdb.
Enable kernel core dump compression using "dumpon -z" (see the dumpon(8) man page) and trigger a kernel core dump using sysctl debug.kdb.panic=1. Verify that you can gunzip the resulting vmcore (it should appear under /var/crash following a reboot) and open it in kgdb.

ota_j.email.ne.jp mentioned this in D21408: GZIO to update zlib version 1.0.4 to 1.2.11..Aug 24 2019, 10:44 PM

ota_j.email.ne.jp added inline comments.Aug 24 2019, 10:47 PM

sys/kern/subr_compressor.c
178 ↗	(On Diff #61240)	It doesn't look crc32_raw() and zlib's crc32() isn't inter-exchangeable. I created and tested in D21408.

Remove D21408 portion of change.

Harbormaster completed remote builds in B26093: Diff 61258.Aug 25 2019, 7:52 AM

ota_j.email.ne.jp accepted this revision.Aug 25 2019, 2:07 PM

This revision is now accepted and ready to land.Aug 25 2019, 2:07 PM

Looks like this was merged as rS351480 but phabricator didn't count that for closing this review (full URL shouldn't be in "Differential Revision:"??)