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

[Xen-devel] [PATCH 02/12] libxc/restore: Introduce functionality to simplify blob handling


  • To: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • From: Ian Jackson <ian.jackson@xxxxxxxxxx>
  • Date: Tue, 14 Jan 2020 16:50:40 +0000
  • Authentication-results: esa4.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: Tue, 14 Jan 2020 16:50:48 +0000
  • Ironport-sdr: Svo2mHMIQXlHuropfqEnDHrKhf6rwxoYhwTRH8ACwSDnZwekXhmElP4xDstmpEMuw2iey5gQIB XniWGtkutAdAK2SD5hZ3wbxfiBynciXYiRuvEK933bV2eexgPxXAU1/jDqjOf+5yZbufLxvoOk /VPNH1UqDDY2oVk6M4kKHks839Yr11pGDE5E/x5EnPpvK7RdPN4cEwB1jtQy+j5g7cglDbUOAI /GmQkubTYpJnXsBcErwQ7LdZN5WYT1ISkYFmZ7YpDWFY2n+Fe+RnYeNUq7gjrIUEVU3GYKlfRV 5WY=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

Andrew Cooper writes ("[PATCH 02/12] libxc/restore: Introduce functionality to 
simplify blob handling"):
> During migration, we buffer several blobs of data which ultimately need
> handing back to Xen at an appropriate time.
> 
> Currently, this is all handled in an ad-hoc manner, but more blobs are soon
> going to be added.  Introduce xc_sr_blob to encapsulate a ptr/size pair, and
> update_blob() to handle the memory management aspects.
> 
> Switch the HVM_CONTEXT and the four PV_VCPU_* blobs over to this new
> infrastructure.

I like the new approach.

I was expecting to see "no functional change in the commit message".

If you think that is true, and with that added,
Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

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