On Mon, Oct 05, 2009 at 04:00:02PM -0500, Jamie Strandboge wrote: > Attached is an updated patch_2_apparmor_driver_updated.patch and > patch_3_docs_updated.patch. Thanks for the review! These two patches > along with the previous patch_1_reenable-nonfile-labels.patch pass > syntax-check, make check (for the tests I added) and introduce no > regressions over the previous patch. See below for inline comments. In > addition to implenting your suggestions, the patch also now supports > <readonly/> for disks and I defensively quote the pid, monitor and > logfile. I also added ReserveSecurityLabel (a noop) along with stubs > for SetSecurityHostdevLabel() and RestoreSecurityHostdevLabel(). Okay, both patches are now commited, thanks ! There is just the few cleanups I suggested earlier but this can be done when you have time :-) 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