On Tue, Sep 19, 2023 at 02:17:15AM +0200, Thomas Gleixner wrote: > On Thu, Sep 07 2023 at 03:54, Gregory Price wrote: > > + /* All tasks mapping each page is checked in phys_page_migratable */ > > + nodes_setall(target_nodes); > > How is the comment related to nodes_setall() and why is nodes_setall() > unconditional when target_nodes is only used in the @nodes != NULL case? > Short follow up, sorry for the spam. I realized there is a better way to do this by simply bypassing the task node check in do_pages_moves if the mm_struct is NULL. This removes the need for this struct all-together. Will simplify down for a v2 and add comments to do_pages_move accordingly. ~Gregory
