On Tue, Sep 9, 2014, Haggai Eran <haggaie@xxxxxxxxxxxx> wrote: > On 04/09/2014, Roland Dreier wrote: >> Have you done any review or testing of these changes? If so can you >> share the results? > We have tested this feature thoroughly inside Mellanox. We ran random > tests that performed MR registrations, memory mappings and unmappings, > calls to madvise with MADV_DONTNEED for invalidations, sending and > receiving of data, and RDMA operations. The test validated the integrity > of the data, and we verified the integrity of kernel memory by running > the tests under a debugging kernel. Hi Roland, Per your request we provided the information on tests conducted with the patches. Note that the patches can't really disrupt existing applications that don't set the new IB_ACCESS_ON_DEMAND MR flag when they register memory. Also the whole set of changes to the umem area is dependent on building with CONFIG_INFINIBAND_ON_DEMAND_PAGING -- all in all, everything is in place for protecting against potential regression that this series could introduce. As you didn't provide any feedback for > six months, and we have all the above in place (report on stability tests, performance data and mechanics to avoid regressions) I think it would be fair to get this picked for the coming merge window, thoughts? Or. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html