> /* no readahead, need to avoid set-but-unused var warnings. */ > #define xfs_buf_readahead(a,d,c,ops) ({ \ > + void *__a = a; \ > xfs_daddr_t __d = d; \ > + __a = __a; \ > __d = __d; /* no set-but-unused warning */ \ What about turning this into an inline function instead of piling more hacks like this onto the existing ones?