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

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



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


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