[libvirt] [PATCH 01/13] Introduce the function virCgroupForHypervisor

[Eric Blake]

The subject should include 'v2' to mention that this is a respin; you
can do that with 'git send-email --subject-prefix=PATCHv2'.

On 07/10/2012 03:11 AM, tangchen wrote:
> Introduce the function virCgroupForHypervisor() to create sub directory
> for hypervisor thread(include I/O thread, vhost-net thread)

So if I understand correctly, the intent is to create this hierarchy:

cgroup mount point(s) (top-level controllers)
- libvirt (subdirectory for all libvirt cgroups)
--- qemu (all groups tied to a driver, virCgroupForDriver)
----- $name (all groups tied to a VM, virCgroupForDomain)
------- $vcpuN (one group per vcpu, virCgroupForVcpu)
------- hypervisor (one catchall group for non-vcpu, virCgroupForHypervisor)

where hypervisor and vcpuN directories are siblings, both adding up to
the sum total of $name for the VM totals.

If so, please document that in the commit message.

> 
> Signed-off-by: Wen Congyang <wency at cn.fujitsu.com>

Still doesn't have the correct git authorship; git send-email would
automatically insert a line:

From: Wen Congyang <wency at cn.fujitsu.com>

if you had correctly attributed authorship.  I can fix that as needed,
though.

> ---
>  .gnulib                  |    2 +-

Oops; gnulib submodule updates should generally be an independent patch,
it looks like it was an accident that you bumped it on this patch.

Other than that, this patch looks okay.

-- 
Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org



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