On Tue, May 05, 2015 at 08:01:52AM -0700, Linus Torvalds wrote: > Ugh. Could we aim to *not* add code like this. > > Instead, just declare (but don't define) the > ext4_encrypted_symlink_inode_operations thing, so that *if* somebody > uses it they get a link error, and make sure that > "ext4_encrypted_inode()" ends up always returning zero when encryption > isn't enabled, so that the compiler will actually optimize the whole > thing out (which apparently is already the case, judging by the > build-bug-on. Sure, no problem. BUILD_BUG_ON was used only because it triggered an error earlier, I simply forgot to rip it out afterwards. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html