On Mon, Aug 26, 2013 at 3:20 PM, John Spencer <maillist-kmod@xxxxxxxxxxx> wrote: > _FILE_OFFSET_BITS is a glibc internal macro. > musl provides 64bit off_t by default, but > defines stat64 etc as macros to make glibc-centric > programs happy. however the expansion causes > problems with the hack used here to work around > glibc's 32bit past... AFAIK this define comes from AC_SYS_LARGEFILE in autoconf and ends up in our config.h. Checking for the libc in source code is not much future proof and I'd like to avoid it. If this is not the right define to pick, is there any other one? Maybe it would be better to do like this: #ifndef _FILE_OFFSET_BITS #ifndef stat64 WRAP_2ARGS(int, -1, stat64, struct stat64*); #endif ... Because if it's a macro, there's no need to wrap the call. Lucas De Marchi -- To unsubscribe from this list: send the line "unsubscribe linux-modules" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html