On Tue, Jul 17, Olaf Hering wrote: To make this robust against allocation errors I will change it to do { > + /* Allocate new mfn for previous pfn */ > + rc = HYPERVISOR_memory_op(XENMEM_populate_physmap, &reservation); } while (rc == 0); > + > + /* Make sure the previous pfn is really connected to a (new) mfn */ > + BUG_ON(rc != 1); Olaf