[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] libxc/restore: Don't duplicate state in process_vcpu_basic()

  • To: Andrew Cooper <Andrew.Cooper3@xxxxxxxxxx>
  • From: Ian Jackson <ian.jackson@xxxxxxxxxx>
  • Date: Fri, 20 Dec 2019 18:46:40 +0000
  • Authentication-results: esa1.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none; spf=None smtp.pra=ian.jackson@xxxxxxxxxx; spf=Pass smtp.mailfrom=Ian.Jackson@xxxxxxxxxx; spf=None smtp.helo=postmaster@xxxxxxxxxxxxxxx
  • Cc: Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Wei Liu <wl@xxxxxxx>
  • Delivery-date: Fri, 20 Dec 2019 18:46:52 +0000
  • Ironport-sdr: LUy6aOAnLuvgyY+hTqTmOdoXVEHNX8HAhUUvJdg40Q0AXPZLx0l+3GPLOtuctM+TDLTOpBGAo9 ShdYfagGwh8OZ8mQnimxhTluKAPBdb+/qBmMvDy//a8HgGimUmulPFDdTbgF8QcmSkF47u3p1a GKGLQyM+FfQAYab+2zy8RegtCUGjqDLYK3NzJLMvrpKxGsbIi58vTTtKRsLLF0e32RJWn3/NG1 jGNwDeLs/BrVyHouioiSIkj28QWuxJgrVoZYnYOL5RzAOo4on0Z9vBGti+iRE9UDkY3ZyaF4AV V7Q=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

Andrew Cooper writes ("Re: [PATCH] libxc/restore: Don't duplicate state in 
> On 20/12/2019 18:22, Ian Jackson wrote:
> > It is not obvious to me that nothing uses the modified fields after
> > process_vcpu_basic has edited things.  The ctx of which the vcpu is
> > part is passed to process_vcpu_<various> by update_vcpu_context.
> Yes.  It was a consequence of adding Remus support (which happened on
> xen-devel, post actually-getting-migration-v2-working).
> Originally, this was copied out of the incoming xc_sr_record* pointer
> which was const.  Adding Remus support required buffering these records
> on each checkpoint, which is why they are stashed sideways now.
> By the time we get process_vcpu_basic(), we are committed to completing
> state restoration and unpausing the guest.
> In particular, it is not possible to process any further memory pages
> because the hypercall used here changes typerefs on pagetables, which
> makes them immutable from the toolstacks point of view (technically not,
> but we can't map them and memcpy them any more).

Thanks for the explanation.  Can you add that to the commit message
please ?  That way if this turns out to be wrong, or someone wants to
revert it again, we'll have some record of why this was thought to be

With that done,

Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.