Page MenuHomeFreeBSD

Remove zlib 1.0.4 from kernel.
ClosedPublic

Authored by delphij on Aug 23 2019, 7:32 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 20, 5:13 AM
Unknown Object (File)
Feb 16 2024, 7:15 AM
Unknown Object (File)
Feb 16 2024, 7:15 AM
Unknown Object (File)
Feb 16 2024, 7:15 AM
Unknown Object (File)
Feb 16 2024, 7:15 AM
Unknown Object (File)
Feb 16 2024, 4:21 AM
Unknown Object (File)
Jan 22 2024, 11:10 AM
Unknown Object (File)
Jan 17 2024, 8:12 AM
Subscribers

Details

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 26084
Build 24618: arc lint + arc unit

Event Timeline

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

PTAL.

sys/conf/files
4002–4003

Yes.

delphij marked an inline comment as done.

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

@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.
sys/kern/subr_compressor.c
178

It doesn't look crc32_raw() and zlib's crc32() isn't inter-exchangeable.
I created and tested in D21408.

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:"??)

phabricator didn't count that for closing this review

Looks like the Differential Revision: string and the URL were on different lines.