Re: [PATCH 12/14] audit: Simplify locking around untag_chunk()

> if (chunk->dead || !(entry->flags & FSNOTIFY_MARK_FLAG_ATTACHED)) { > - mutex_unlock(&entry->group->mark_mutex); > - if (new) > - fsnotify_put_mark(new->mark); > - goto out; > + mutex_unlock(&audit_tree_group->mark_mutex); > + return; This isn't an error, but more a question of style. Since the end of the function has been simplified, this could just be "goto out_mutex", or remove the one remaining "goto out_mutex" after the next patch and do the same as here since other exits aren't so clean.