> +static int __writepage(struct page *page, struct writeback_control *wbc, > + void *data) Can we give this a somewhat more descriptive name, e.g. ext4_writepage_cb or so? Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>
> +static int __writepage(struct page *page, struct writeback_control *wbc, > + void *data) Can we give this a somewhat more descriptive name, e.g. ext4_writepage_cb or so? Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>