On 7/23/19 6:26 PM, john.hubbard@xxxxxxxxx wrote: > From: John Hubbard <jhubbard@xxxxxxxxxx> ... > + * 2) This code sees the page as clean, so it calls > + * set_page_dirty(). The page stays dirty, despite being > + * written back, so it gets written back again in the > + * next writeback cycle. This is harmless. > + */ > + if (!PageDirty(page)) > + set_page_dirty_lock(page); > + break; ahem, the above "break" should not be there, it's an artifact, sorry about that. Will correct on the next iteration. thanks, -- John Hubbard NVIDIA > + put_user_page(page); > + } > } > EXPORT_SYMBOL(put_user_pages_dirty_lock); > >