On Mon, Nov 14, 2016 at 01:34:49PM +0100, David Sterba wrote: > On Sat, Nov 12, 2016 at 10:14:49PM +0000, Sergei Trofimovich wrote: > > > > Basically gcc tries to say us BUILD_ASSERT is not visible. > > > > > > > > BUILD_ASSERT lives in kerncompat.h which this change adds. > > > > > > I think including the kerncompat.h is too intrusive here, I've fixed by > > > providing an empty macro if it's not defined. I'll release 4.8.2 soon. > > > > Apologies. I did not test your fix right afterwards. Seems now header is > > incomplete > > due to missing NULL (gcc-6): > > > > btrfs-progs-v4.8.3 $ gcc -c ioctl.h -o /tmp/a.o > > ioctl.h: In function 'btrfs_err_str': > > ioctl.h:711:11: error: 'NULL' undeclared (first use in this function) > > return NULL; > > ^~~~ > > ioctl.h:711:11: note: each undeclared identifier is reported only once for > > each function it appears in > > The ioctl.h file can be included in both C and C++ code, I'd rahter > avoid to ifdef the right definition of NULL, so s/NULL/0/ seems as a > best fix to me.
Or #include <stddef.h>, now committed to devel. -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html