Re: [RFC 2.6.11-rc2-mm2 7/7] mm: manual page migration -- sys_page_migrate

Robin Holt
2005-02-15T18:59:43Z

On Tue, Feb 15, 2005 at 08:35:29AM -0800, Paul Jackson wrote:
> What about the suggestion I had that you sort of skipped over, which
> amounted to changing the system call from a node array to just one
> node:
>
> sys_page_migrate(pid, va_start, va_end, count, old_nodes, new_nodes);
>
> to:
>
> sys_page_migrate(pid, va_start, va_end, old_node, new_node);
>
> Doesn't that let you do all you need to? Is it insane too?

Migration could be done in most cases and would only fall apart when
there are overlapping node lists and no nodes available as temp space
and we are not moving large chunks of data.

What is the fundamental concern with passing in an array of integers?
That seems like a fairly easy to verify item with very little chance
of breaking. I don't feel the concern that others seem to.

I do see the benefit to those arrays as being a single pass through the
page tables, the ability to migrate without using a temporary node, and
reducing the number of times data is copied when there are overlapping
nodes. To me, those seem to be very compelling reasons when compared
to the potential for a possible problem with an array of integers.

Thanks,
Robin