On Thu, Apr 11, 2013 at 05:47:53PM +0200, Paolo Bonzini wrote: > Il 11/04/2013 17:46, Michael S. Tsirkin ha scritto: > > > Ok, let's keep it simple. The only two things we need are: > > > > > > 1) remove the patch to disable is_dup_page > > > > > > 2) rename the transport to "x-rdma" (just in migration.c) > > > > > > Both things together let us keep it safe for a release or two. Let's > > > merge this thing. > > > > I would drop the chunk based thing too. Besides being slow, it turns > > out that it pins all memory anyway. So no memory overcommit. > > It doesn't pin zero pages. Those are never transmitted (it's a recent > change). So pages that are ballooned at the beginning of migration, and > remain ballooned throughout, will never be pinned. > > Paolo
Of course Michael says it's slow unless you disable zero page detection, and then I'm guessing it does? -- MST