Re: [PATCH v3 0/5] ch: handle events from cloud-hypervisor

On 10/23/24 10:02, Purna Pavan Chandra Aekkaladevi wrote: > changes from v2->v3: > * Remove patch 'utils: Implement virFileIsNamedPipe' as it is no more needed. > * Remove the eventmonitorpath only if it exists > * Added domain name as a prefix to logs from ch_events.c. This will make > debugging easier. > * Simplified event parsing logic by reserving a byte for null char. > > changes from v1->v2: > > * Rebase on latest master > * Use /* */ for comments > * Remove fifo file if already exists > * Address other comments from Praveen Paladugu > > cloud-hypervisor raises various events, including VM lifecylce operations > such as boot, shutdown, pause, resume, etc. Libvirt will now read these > events and take the necessary actions, such as correctly updating the > domain state. A FIFO file is passed to `--event-monitor` option of > cloud-hypervisor. Libvirt creates a new thread that acts as the reader > of the fifo file and continuously monitors for new events. Currently, > shutdown events are handled by updating the domain state appropriately. > > Purna Pavan Chandra Aekkaladevi (5): > ch: pass --event-monitor option to cloud-hypervisor > ch: start a new thread for handling ch events > ch: events: Read and parse cloud-hypervisor events > ch: events: facilitate lifecycle events handling > NEWS: Mention event handling support in ch driver > > NEWS.rst | 7 + > po/POTFILES | 1 + > src/ch/ch_events.c | 329 ++++++++++++++++++++++++++++++++++++++++++++ > src/ch/ch_events.h | 54 ++++++++ > src/ch/ch_monitor.c | 52 ++++++- > src/ch/ch_monitor.h | 11 ++ > src/ch/meson.build | 2 + > 7 files changed, 449 insertions(+), 7 deletions(-) > create mode 100644 src/ch/ch_events.c > create mode 100644 src/ch/ch_events.h > Sorry for late review. I was side tracked. But I think we need v4.