Besides moving this much earlier in the series as mentioned before the most important thing is giving the helpers a different name as the iget name is already used for a different purpose, even if got rid of the original iget and only have iget_locked. I think iref/iref_locked would be good enough names for the helpers. One other things is that a lot of the current igrab users could use this helper instead - there's very few places where we look at inodes that don't already have a permanent reference when we try to acquire another one. -- 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