Is there any good reason for the str routines in afs_util.c to have afs_ prepended to their names? Wouldn't it make more sense to give them their real names, and conditionally compile them depending on what's available in the host kernel? Maybe even with the help of autoconf? (Yes I know autoconf support for kernel code is incomplete).