Re: [PATCH 2/2] xfs: replace xfs_buf_geterror() with bp->b

xfs

Re: [PATCH 2/2] xfs: replace xfs_buf_geterror() with bp->b_error

To:	Chandra Seetharaman <sekharan@xxxxxxxxxx>
Subject:	Re: [PATCH 2/2] xfs: replace xfs_buf_geterror() with bp->b_error
From:	Alex Elder <aelder@xxxxxxx>
Date:	Thu, 11 Aug 2011 17:04:42 -0500
Cc:	<xfs@xxxxxxxxxxx>
In-reply-to:	<20110803021834.29700.40199.sendpatchset@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
References:	<20110803021823.29700.71553.sendpatchset@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx> <20110803021834.29700.40199.sendpatchset@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Reply-to:	<aelder@xxxxxxx>

On Tue, 2011-08-02 at 19:18 -0700, Chandra Seetharaman wrote:
> Since we just checked bp for NULL, it is ok to replace xfs_buf_geterror()
> with bp->b_error in these places.
> 
> Signed-off-by: Chandra Seetharaman <sekharan@xxxxxxxxxx>

Looks good.

Reviewed-by: Alex Elder <aelder@xxxxxxx>

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
[PATCH 0/2] Check return value of xfs_buf_read(), Chandra Seetharaman [PATCH 1/2] xfs: Check the return value of xfs_buf_read() for NULL, Chandra Seetharaman Re: [PATCH 1/2] xfs: Check the return value of xfs_buf_read() for NULL, Alex Elder [PATCH 2/2] xfs: replace xfs_buf_geterror() with bp->b_error, Chandra Seetharaman Re: [PATCH 2/2] xfs: replace xfs_buf_geterror() with bp->b_error, Alex Elder <=

Previous by Date:	Re: [PATCH 1/2] xfs: Check the return value of xfs_buf_read() for NULL, Alex Elder
Next by Date:	Re: frequent kernel BUG and lockups - 2.6.39 + xfs_fsr, Marc Lehmann
Previous by Thread:	[PATCH 2/2] xfs: replace xfs_buf_geterror() with bp->b_error, Chandra Seetharaman
Next by Thread:	Налоговый кодекс от А до Я, Киев 20.08.2011, МBА Tеchnology
Indexes:	[Date] [Thread] [Top] [All Lists]