On Tue, Sep 19, 2023 at 02:17:15AM +0200, Thomas Gleixner wrote:

On Thu, Sep 07 2023 at 03:54, Gregory Price wrote:

quoted

+	/* 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