Re: [PATCH 5/7] Add auditing of start/stop events to the QEMU driver

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

 



On Wed, Oct 27, 2010 at 12:36:15PM +0100, Daniel P. Berrange wrote:
> Add audit hooks to report all start and stop events on QEMU
> guest domains.
> 
> * src/qemu/qemu_driver.c: Audit start/stop events
> ---
>  src/qemu/qemu_driver.c |   59 ++++++++++++++++++++++++++++++++++++++++++++++-
>  1 files changed, 57 insertions(+), 2 deletions(-)

  patch 1-4 trivial ACKs

One of the differences if we lock down in the driver (beside the
redundancy that will be needed) is that we end up writing to the
audit system deep in the driver with all the locks needed for operation.
Is there a risk of being blocked while writing to the audit system ?
This could potentially be a problem because all operations on the
domain would be stopped during that time.

  beside that generic issue, patch looks fine, ACK

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel@xxxxxxxxxxxx  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]