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
Unknown Object (File)
Fri, Dec 6, 3:36 AM
Unknown Object (File)
Fri, Dec 6, 3:36 AM
Unknown Object (File)
Fri, Dec 6, 3:36 AM
Unknown Object (File)
Fri, Dec 6, 3:35 AM
Unknown Object (File)
Fri, Dec 6, 3:35 AM
Unknown Object (File)
Thu, Dec 5, 2:35 PM
Unknown Object (File)
Tue, Dec 3, 6:52 PM
Unknown Object (File)
Tue, Dec 3, 6:30 PM

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 Not Applicable
Unit
Tests Not Applicable

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