> +static void inode_wb_requeue(struct inode *inode, struct bdi_writeback *wb, > + struct writeback_control *wbc) I'd just call this requeue_inode to fit the naming scheme in elsewhere in the file. I'd also suggest adding a small top of function comment explaining it. Otherwise looks good, 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