On Fri, Jun 17, 2016 at 04:34:23AM -0700, Christoph Hellwig wrote:
> > diff --git a/fs/xfs/xfs_buf.c b/fs/xfs/xfs_buf.c
> > index efa2a73..2333db7 100644
> > --- a/fs/xfs/xfs_buf.c
> > +++ b/fs/xfs/xfs_buf.c
> > @@ -947,7 +947,8 @@ xfs_buf_trylock(
> > if (locked)
> > XB_SET_OWNER(bp);
> >
> > - trace_xfs_buf_trylock(bp, _RET_IP_);
> > + locked ? trace_xfs_buf_trylock(bp, _RET_IP_) :
> > + trace_xfs_buf_trylock_fail(bp, _RET_IP_);
> > return locked;
>
> I think this should be something like:
>
> if (locked) {
> XB_SET_OWNER(bp);
> trace_xfs_buf_trylock(bp, _RET_IP_);
> } else {
> trace_xfs_buf_trylock_fail(bp, _RET_IP_);
> }
>
> otherwise this looks good and can go in without the rest of the series.
I'll fix that up on commit.
Cheers,
Dave.
--
Dave Chinner
david@xxxxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
