Page MenuHomeFreeBSD

loader: remove libsa/crc32.c and use version from zlib
ClosedPublic

Authored by tsoome on Mar 14 2020, 3:48 PM.
Tags
None
Referenced Files
F109276731: D24068.diff
Sun, Feb 2, 10:49 PM
Unknown Object (File)
Sat, Jan 25, 7:16 PM
Unknown Object (File)
Fri, Jan 24, 5:26 PM
Unknown Object (File)
Tue, Jan 21, 9:17 AM
Unknown Object (File)
Sat, Jan 18, 9:55 PM
Unknown Object (File)
Sat, Jan 18, 1:39 PM
Unknown Object (File)
Fri, Jan 17, 10:40 PM
Unknown Object (File)
Dec 6 2024, 3:36 AM

Details

Summary

we have crc32(const void *, size_t) in libsa. Unfortunately zlib has crc32(long, const unigned char *, unsigned) and we have conflict.

Since we do build libsa with zlib, we can use zlib version instead.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 29936
Build 27753: arc lint + arc unit

Event Timeline

missed stand/uboot/lib/glue.c

I’d suggest using the zlib one unless it takes excessive space (bigger precomputed tables, for ex.).

In D24068#529184, @cem wrote:

I’d suggest using the zlib one unless it takes excessive space (bigger precomputed tables, for ex.).

I did that in illumos:) I was not sure about the preferred way with freebsd; it is larger, but it is present anyhow because we do link zlib and it is referred.

I did that in illumos:) I was not sure about the preferred way with freebsd; it is larger, but it is present anyhow because we do link zlib and it is referred.

Ah, if we already link zlib I see no reason to include a second copy.

drop crc32 from libsa and use version provided by zlib.

tsoome retitled this revision from loader: rename crc32 to crc32_libsa to loader: remove libsa/crc32.c and use version from zlib.Mar 14 2020, 8:42 PM
tsoome edited the summary of this revision. (Show Details)
allanjude added a subscriber: allanjude.

Reviewed By: allanjude

This revision is now accepted and ready to land.Mar 16 2020, 8:28 PM