cgroup cleanup | Patchew

[libvirt] [PATCH 7/7] vircgroup: Remove obsolete sa_assert

Posted by Pavel Hrdina 7 years, 5 months ago

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
---
 src/util/vircgroup.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/src/util/vircgroup.c b/src/util/vircgroup.c
index 02de33f74f..64507bf8aa 100644
--- a/src/util/vircgroup.c
+++ b/src/util/vircgroup.c
@@ -1054,10 +1054,6 @@ virCgroupMakeGroup(virCgroupPtr parent,
         if (virCgroupPathOfController(group, i, "", &path) < 0)
             goto error;
 
-        /* As of Feb 2011, clang can't see that the above function
-         * call did not modify group. */
-        sa_assert(group->controllers[i].mountPoint);
-
         VIR_DEBUG("Make controller %s", path);
         if (!virFileExists(path)) {
             if (!create ||
-- 
2.17.1

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

Re: [libvirt] [PATCH 7/7] vircgroup: Remove obsolete sa_assert

Posted by Ján Tomko 7 years, 5 months ago

On Fri, Aug 24, 2018 at 03:45:57PM +0200, Pavel Hrdina wrote:
>Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
>---
> src/util/vircgroup.c | 4 ----
> 1 file changed, 4 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

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