• Boaz Harrosh's avatar
    exofs: Remove inode->i_count manipulation in exofs_new_inode · fe2fd9ed
    Boaz Harrosh authored
    exofs_new_inode() was incrementing the inode->i_count and
    decrementing it in create_done(), in a bad attempt to make sure
    the inode will still be there when the asynchronous create_done()
    finally arrives. This was very stupid because iput() was not called,
    and if it was actually needed, it would leak the inode.
    
    However all this is not needed, because at exofs_evict_inode()
    we already wait for create_done() by waiting for the
    object_created event. Therefore remove the superfluous ref counting
    and just Thicken the comment at exofs_evict_inode() a bit.
    
    While at it change places that open coded wait_obj_created()
    to call the already available wrapper.
    
    CC: Dave Chinner <dchinner@redhat.com>
    CC: Christoph Hellwig <hch@lst.de>
    CC: Nick Piggin <npiggin@kernel.dk>
    Signed-off-by: default avatarBoaz Harrosh <bharrosh@panasas.com>
    fe2fd9ed
inode.c 32.6 KB