On Thu, Jan 07, 2010 at 04:51:10AM +0800, Trond Myklebust wrote: > avail_dirty += bdi_stat(bdi, BDI_DIRTY) + > - bdi_stat(bdi, BDI_UNSTABLE) + > bdi_stat(bdi, BDI_WRITEBACK); > + if (bdi_cap_account_unstable(bdi)) > + avail_dirty += bdi_stat(bdi, BDI_UNSTABLE); It seems that not changing the bdi_stat()s makes more readable code, otherwise looks OK to me. Reviewed-by: Wu Fengguang <fengguang.wu@xxxxxxxxx> -- 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