[PATCH 02/12 v3] xfs: Remove the macro XFS_BUF_ZEROFLAGS
Alex Elder
aelder at sgi.com
Mon Jul 25 10:57:32 CDT 2011
On Sun, 2011-07-24 at 07:39 -0400, Christoph Hellwig wrote:
> > @@ -1247,6 +1247,7 @@ xlog_bdstrat(
> > {
> > struct xlog_in_core *iclog = bp->b_fspriv;
> >
> > + bp->b_flags &= ~(XBF_FUA|XBF_FLUSH);
>
> I don't think this is correct, we set these before calling into
> xlog_bdstrat, so this means we'll always stip it off after that.
>
> Given that this area is more complicated than the pure macro removal
> how about letting it as-is for now? I have some bigger refactoring
> in this area in my queue, and I'll just incorporate the
> XFS_BUF_ZEROFLAGS into it.
Christoph, are you suggesting that this one hunk just
be excluded from the series? Or the entire patch?
-Alex
More information about the xfs
mailing list