Right, but introducing new APIs is not free.
Let's leave is_dup_page unconditionally in now. We can always remove it
later if it turns out to be useful.
The important thing is to have the code in early to give it wider
exposure. Once it is in, people can test it more, benchmark
with/without is_dup_page, etc. We can declare it experimental, and
break the protocol later if it turns out to be bad.
I think all that's needed is:
1) benchmark the various chunk sizes (with is_dup_page disabled and your
current stress test -- better than nothing). Please confirm that the
source can modify the chunk size and the destination will just pick it up.
2) remove the patch to disable is_dup_page
3) rename the transport to "x-rdma" (just in migration.c).
And that's it. The patches should be ready.
We have converged on a good interface between RDMA and the generic
migration code, and that's the important thing because later
implementations will not throw away that work.
Paolo