Hi,
The subject for the V2 was messed up before I posted patches for review.
Please review my re-sent v2 at
https://lists.libvirt.org/archives/list/devel@lists.libvirt.org/thread/6TMMK4FQ7GVJM7EWZD2NWUEUFRUWFW24/
Regards,
Pavan
On Thu, Jul 04, 2024 at 11:13:30AM +0000, Purna Pavan Chandra wrote:
> * Add version checks in save/restore validations
> * Add use_timeout in chSocketRecv
> * Address Praveen Paladugu's comments
>
> v1: https://lists.libvirt.org/archives/list/devel@lists.libvirt.org/thread/PT77VAI7RF2BUNR2HUIFM46WCP5DVEKV/
>
> ch: support restore with network devices
>
> Current ch driver supports restore only for domains without any network
> configuration defined. This was because libvirt explicitly passes network fds
> and CH did not had support to restore with new net FDS. This support has been
> added recently, https://github.com/cloud-hypervisor/cloud-hypervisor/pull/6402
> The changes in this patch series includes moving to socket communication for
> restore api, create new net fds and pass them via SCM_RIGHTS to CH.
>
> Purna Pavan Chandra (8):
> ch: report response message instead of just code
> ch: Pass net ids explicitly during vm creation
> ch: refactor chProcessAddNetworkDevices
> ch: support poll with -1 in chSocketRecv
> ch: use monitor socket fd to send restore request
> ch: refactor virCHMonitorSaveVM
> ch: support restore with net devices
> ch: kill CH process if restore fails
>
> src/ch/ch_capabilities.c | 6 +
> src/ch/ch_capabilities.h | 1 +
> src/ch/ch_driver.c | 29 +++--
> src/ch/ch_monitor.c | 62 +++++++----
> src/ch/ch_monitor.h | 6 +-
> src/ch/ch_process.c | 233 +++++++++++++++++++++++++++++++--------
> 6 files changed, 254 insertions(+), 83 deletions(-)
>
> --
> 2.34.1