dajhorn/pkg-zfs — comment on commit 810633b6bc4f41e36eabbd1a4d66d5d9f8a67c04
File:  line 0
By behlendorf:

Just a few nits:

* Let's use HAVE\_SET\_NLINK instead of HAVE\_FS\_SET\_NLINK for consistency. The rest of the checks basically just use HAVE\_ and then the function name in caps.

* For the sake of readability I've tried to keep the #ifdef usage to a minimum in the actual code. In cases like this I've introduced a compatibility macro in include/linux/vfs\_compat.h, see HAVE\_INSERT\_INODE\_LOCKED in that header as an example. The code in zfs_znode.c can then just seamlessly use the update interface, in this case set\_nlink().