Re: [PATCH 2/9] xfs: xfs_buf_ioend and xfs_buf_iodone_work duplicate functionality

Christoph Hellwig
2014-08-29T00:22:14Z

> STATIC void

needs the static removed as pointed out by Brian.
> - bool read = !!(bp->b_flags & XBF_READ);
> + bool read = !!(bp->b_flags & XBF_READ);

We don't really need the double negation here.

Otherwise this looks reasonable to me.