[PATCH 24/23] xfs: convert buffer verifiers to an ops structure.

Christoph Hellwig hch at infradead.org
Tue Oct 23 07:36:47 CDT 2012


On Thu, Oct 18, 2012 at 03:50:28PM +1100, Dave Chinner wrote:
> Here's a patch that does this. It adds on top of the series, and
> points out more changes needed to growfs to support CRC checking
> properly.

I like this a lot, and I'm fine with having this at the end of the
series if it makes your life easier.

> -xfs_agfl_write_verify(
> +xfs_agfl_read_verify(
>  	struct xfs_buf	*bp)
>  {
>  	xfs_agfl_verify(bp);
>  }
>  
>  static void
> -xfs_agfl_read_verify(
> +xfs_agfl_write_verify(
>  	struct xfs_buf	*bp)
>  {
>  	xfs_agfl_verify(bp);
> -	bp->b_pre_io = xfs_agfl_write_verify;
> -	bp->b_iodone = NULL;
> -	xfs_buf_ioend(bp, 0);
>  }

Any good reason to keep all these no-op wrappers?



More information about the xfs mailing list