Re: [PATCH 07/18] exofs: use iput() for inode reference countdecrements

From: Nick Piggin
Date: Sat Oct 16 2010 - 03:56:58 EST


On Fri, Oct 08, 2010 at 04:21:21PM +1100, Dave Chinner wrote:
> From: Dave Chinner <dchinner@xxxxxxxxxx>
>
> Direct modification of the inode reference count is a no-no. Convert
> the exofs decrements to call iput() instead of acting directly on
> i_count.

Could this go to exofs maintainer and get merged as a bugfix?

>
> Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx>
> ---
> fs/exofs/inode.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/fs/exofs/inode.c b/fs/exofs/inode.c
> index b631ff3..0fb4d4c 100644
> --- a/fs/exofs/inode.c
> +++ b/fs/exofs/inode.c
> @@ -1101,7 +1101,7 @@ static void create_done(struct exofs_io_state *ios, void *p)
>
> set_obj_created(oi);
>
> - atomic_dec(&inode->i_count);
> + iput(inode);
> wake_up(&oi->i_wq);
> }
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/