DB> Should use lxcError() here - as things are setup by virExec(),
DB> calls to lxcError() call to virRaiseError() which ultimately calls
DB> fprintf(). By using lxcError() can we more easily change logging
DB> later if needed.

Indeed.

DB> I think we need to virCgroupRemove() in the error path to cleanup
DB> a partially constructed cgroup ?

Yep, I'll fix that too.

Thanks!

-- 
Dan Smith
IBM Linux Technology Center
Open Hypervisor Team
email: [EMAIL PROTECTED]

Attachment: pgpCrdzpJFR1t.pgp
Description: PGP signature

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

Reply via email to