build failed in Linux 3.13 #87

Merged
merged 1 commit into from Apr 3, 2014

Conversation

Projects
None yet
2 participants
@tadokoro
Contributor

tadokoro commented Mar 21, 2014

build failed in Linux 3.13 because inet6_hash_frag of net/ipv6/reassembly.c has been static since linux 3.13.

$ make
make -C /lib/modules/3.13-1-amd64/build M=$PWD;
make[1]: Entering directory `/usr/src/linux-headers-3.13-1-amd64'
  CC [M]  /home/tadokoro/src/NAT64/mod/fragment_db.o
/home/tadokoro/src/NAT64/mod/fragment_db.c: In function 'hash_function':
/home/tadokoro/src/NAT64/mod/fragment_db.c:156:3: error: implicit declaration of function 'inet6_hash_frag' [-Werror=implicit-function-declaration]
   result = inet6_hash_frag(key->ipv6.identification, &key->ipv6.src_addr,
   ^
cc1: some warnings being treated as errors
make[4]: *** [/home/tadokoro/src/NAT64/mod/fragment_db.o] Error 1
make[3]: *** [_module_/home/tadokoro/src/NAT64/mod] Error 2
make[2]: *** [sub-make] Error 2
make[1]: *** [all] Error 2
make[1]: Leaving directory `/usr/src/linux-headers-3.13-1-amd64'
make: *** [all] Error 2

I copied the code from reassembly.c.

Implimenting inet6_hash_frag because it has been static since Linux 3…
….13.

Its implementation is a copy of net/ipv6/reassembly.c's function of the same name.

@ydahhrk ydahhrk added this to the 3.1.4 milestone Mar 26, 2014

@ydahhrk ydahhrk self-assigned this Mar 26, 2014

@ydahhrk ydahhrk merged commit 373c705 into NICMx:master Apr 3, 2014

@ydahhrk

This comment has been minimized.

Show comment
Hide comment
@ydahhrk

ydahhrk Apr 3, 2014

Member

Thank you; done at last.

nf_hook.c also had some incompatibilities with 3.13+, but they were warnings that actually didn't wreck any havoc. Regardless, I silenced them in commit ff26b76.

Member

ydahhrk commented Apr 3, 2014

Thank you; done at last.

nf_hook.c also had some incompatibilities with 3.13+, but they were warnings that actually didn't wreck any havoc. Regardless, I silenced them in commit ff26b76.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment