On Wed, Nov 28, 2012 at 10:30 PM, Al Viro <viro@xxxxxxxxxxxxxxxxxx> wrote: > > Note that sync_blockdev() a few lines prior to that is good only if we > have no other processes doing write(2) (or dirtying the mmapped pages, > for that matter). The window isn't too wide, but... So with Mikulas' patches, the write actually would block (at write level) due to the locking. The mmap'ed patches may be around and flushed, but the logic to not allow currently *active* mmaps (with the rather nasty random -EBUSY return value) should mean that there is no race. Or rather, there's a race, but it results in that EBUSY thing. With my simplfied locking, the sync_blockdev() is right before (not a few lines prior) to the kill_bdev(), and in a perfect world they'd actually be one single operation ("write back and invalidate pages with the wrong block-size"). But they aren't. Linus -- 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