Re: [PATCH v2 0/8] New changes in v2:

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/6T... 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/PT... > > 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