{"comment":{"url":"https://api.github.com/repos/tuxoko/zfs/comments/13749078","html_url":"https://github.com/tuxoko/zfs/commit/bd7947a04ecf4113e7fb94bd785bdd93c7a1d9cc#commitcomment-13749078","id":13749078,"user":{"login":"behlendorf","id":148917,"avatar_url":"https://avatars.githubusercontent.com/u/148917?v=3","gravatar_id":"","url":"https://api.github.com/users/behlendorf","html_url":"https://github.com/behlendorf","followers_url":"https://api.github.com/users/behlendorf/followers","following_url":"https://api.github.com/users/behlendorf/following{/other_user}","gists_url":"https://api.github.com/users/behlendorf/gists{/gist_id}","starred_url":"https://api.github.com/users/behlendorf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/behlendorf/subscriptions","organizations_url":"https://api.github.com/users/behlendorf/orgs","repos_url":"https://api.github.com/users/behlendorf/repos","events_url":"https://api.github.com/users/behlendorf/events{/privacy}","received_events_url":"https://api.github.com/users/behlendorf/received_events","type":"User","site_admin":false},"position":13,"line":451,"path":"module/zfs/zfs_znode.c","commit_id":"bd7947a04ecf4113e7fb94bd785bdd93c7a1d9cc","created_at":"2015-10-13T18:48:30Z","updated_at":"2015-10-13T18:48:30Z","body":"@tuxoko your right.  I was confused because this function used to use `iget_locked()` instead of `new_inode()` which did set I_NEW.  Which is why I originally added the `unlock_new_inode()` call in the error path.  Things have changed since then!  Thanks for keeping me honest, I withdraw my objection!  This LGTM.  If you can refresh it with a good commit comment I'll get it merged."}}