On 4/15/21 1:04 AM, Qu Wenruo wrote:
In __process_pages_contig() we update page status according to page_ops.That update process is a bunch of if () {} branches, which lies inside two loops, this makes it pretty hard to expand for later subpage operations. So this patch will extract this operations into its own function, process_one_pages(). Also since we're refactoring __process_pages_contig(), also move the new helper and __process_pages_contig() before the first caller of them, to remove the forward declaration. Signed-off-by: Qu Wenruo <[email protected]>
The refactor is fine, I still have questions about the pages_processed thing, but that can be addressed in the other patch and then will trickle down to here, so you can add
Reviewed-by: Josef Bacik <[email protected]> Thanks, Josef
