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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

Attachment: signature.asc
Description: OpenPGP digital signature

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

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]