Relocate crc32 functions from libkern.h to gsb_crc32.h/c

[delphij]

Summary

This is to avoid API conflict of crc32() with zlib.h.
ABI conflict can be avoided with -DZ_PREFIX but that requires header file split.
Use and honor Gary S. Brown as filename prefix.

Test Plan

No functionality changes are made.
% make buildworld buildkernel TARGET_ARCH=<arch>
% make installkernel
... reboot etc.
% make installworld
... reboot etc
% make -C .../tests/sys/kern
with nooptions SCTP and options SCTP for i386, amd64 and aach64/arm64

And some/few regression tests.

[ota_j.email.ne.jp]

Combine SCTP cases.

[kib]

sys/sys/crc32.h

61 ↗

(On Diff #57161)

There should be no line split in the function declaration after the return type.

[kib]

sys/sys/crc32.h

40 ↗

(On Diff #57161)

Should this extern declaration conditionalized for _KERNEL ? Is this header supposed to be useful for usermode at all ?

[ota_j.email.ne.jp]

Fixed function prototype style.

[ota_j.email.ne.jp]

sys/sys/crc32.h

40 ↗

(On Diff #57161)

crc32_tab[] is used in crc32_raw() below.

Do you mean to surround all of crc32 functions with ifdef _KERNEL?
crc32.c doesn't have _KERNEL.
It looks easy to compile with sys/crc32.h header but there is no library created for user land usage and also some architectures have assembly code as well. I don't think linking with kernel crc32 is easy.

zlib provides crc32() function as well and easy to link with -lz.

[ota_j.email.ne.jp]

I'd like to rename libkern/crc32.c to libkern/kern_crc32.c so that we don't get object filename conflict with zlib/crc32.c as well.
I'm having difficulties with arc diff...

[ota_j.email.ne.jp]

Rename libkern/crc32.c to libkern/kern_crc32.c to avoid having same object
filename as zlib/crc32.c.

I wonder if we'd better rename the header to sys/kern_crc32.h as well.

[delphij]

sys/libkern/x86/crc32_sse42.c

32

Unrelated to your change, but maybe we can change this to #if(n)def _KERNEL (along with the comment above) and simply drop USERSPACE_TESTING in tests/sys/kern/Makefile?

[ota_j.email.ne.jp]

Replace ifdef USERSPACE_TESTING to ifndef _KERNEL.

[ota_j.email.ne.jp]

Remove one more extra prototype.

[ota_j.email.ne.jp]

sys/libkern/x86/crc32_sse42.c

32

Now, I get kib's _KERNEL question at last.

I adjusted and ran 'make -c tests/sys/kern' to test.

[ota_j.email.ne.jp]

sys/libkern/crc32.c

1

By the way, there are files named sys/kern/kern_xxx.c and sys/libkern/xxx.c.
sys/libkern/kern_crc32.c format doesn't follow any of existing filename format.

What's the policy between kern and libkern directory?

[ota_j.email.ne.jp]

More _KERNEL adjustments.

[ota_j.email.ne.jp]

r347984 creates conflicts for lio_bsd.h and if_cdce.c.
I need to svn up and rebase.

[ota_j.email.ne.jp]

Svn up and resolved conflicts.

[ota_j.email.ne.jp]

After research, I found that our crc32 functions are based on the code written
by Gary S Brown. We also have many other *crc32.[hc] files, too. Rather than,
prefixing with kern, prefix with his initial to honor his work.
It looks kern prefix is used for sub-systems.

[ota_j.email.ne.jp]

Fix KERN to GSB.

[ota_j.email.ne.jp]

Xin committed this change with minor adjustments and https://svnweb.freebsd.org/base?view=revision&revision=349151 was the revision.

[delphij]

Committed as rS349151