[libvirt] [PATCH] cgroup: Don't fail to start a VM when DBus isn't compiled in or running

Peter Krempa pkrempa at redhat.com
Fri Aug 30 22:20:17 UTC 2013


On 08/30/13 22:31, Eric Blake wrote:
> On 08/14/2013 07:39 AM, Peter Krempa wrote:
>> The systemd code to create cgroups has fallback means built in, but
>> fails to engage them if a host isn't running DBus or libvirtd is
>> compiled without DBus (yes, you still don't need to run DBus).
>>
>> This patch changes the return value in case DBus isn't available to so
>> that the fallback code can be activated and cgroups are created
>> manually.
>> ---
>>  src/util/virsystemd.c | 6 ++++--
>>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> ACK, and worth having in 1.1.2.

This patch was overridden by a series of patches (
http://www.redhat.com/archives/libvir-list/2013-August/msg00845.html and
the previous versions) as I found more issues with DBus while playing
with the build after sending the initial patch, but I forgot to
self-NACK this version :/

Sorry for wasting your time with reviewing this. The series in question
is already pushed.

Peter

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 901 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20130831/5dd1d8c7/attachment-0001.sig>


More information about the libvir-list mailing list