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: danms(a)us.ibm.com