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

[Daniel Veillard]

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 at veillard.com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/