Jens Axboe <jens.axboe@xxxxxxxxxx> writes: >> This should be ((rw & RW_MASK) == WRITE) too? Anyway, this seems change >> behavior of submit_bh(WRITE_BARRIER) (maybe reiserfs only), it wouldn't >> be your intent... > > Yes, I believe the simpler and more correct fix is: > > diff --git a/fs/buffer.c b/fs/buffer.c > index 38653e3..16b2263 100644 > --- a/fs/buffer.c > +++ b/fs/buffer.c > @@ -2926,14 +2926,13 @@ int submit_bh(int rw, struct buffer_head * bh) > BUG_ON(!buffer_mapped(bh)); > BUG_ON(!bh->b_end_io); > > - if (buffer_ordered(bh) && (rw == WRITE)) > + if (buffer_ordered(bh) && (rw & WRITE)) > rw = WRITE_BARRIER; I see. But, umm..., this means WRITE_SYNC with barrier was deprecated? Or typo? -- OGAWA Hirofumi <hirofumi@xxxxxxxxxxxxxxxxxx> -- 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