On Wed, Jul 27, 2011 at 12:38:03AM +0200, Jan Kara wrote: > In principle, a filesystem may want to have ->sync_fs() called during sync(1) > although it does not have a bdi (i.e. s_bdi is set to noop_backing_dev_info). > Only writeback code really needs bdi set to something reasonable. So move the > checks where they are more logical. Looks good, although as mentioned this would be better done before patch 1. Reviewed-by: Christoph Hellwig <hch@xxxxxx> -- 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