Re: [libvirt] [PATCH 3/7] Make cgroups a little more efficient

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

 



On Fri, Jul 17, 2009 at 04:28:51PM +0200, Daniel Veillard wrote:
> On Fri, Jul 17, 2009 at 09:04:25AM -0400, Daniel P. Berrange wrote:
> >  
> >      mounts = fopen("/proc/mounts", "r");
> >      if (mounts == NULL) {
> > -        DEBUG0("Unable to open /proc/mounts: %m");
> > -        goto err;
> > +        VIR_ERROR0("Unable to open /proc/mounts");
> > +        return -ENOENT;
> >      }
> >  
> >      while (getmntent_r(mounts, &entry, buf, sizeof(buf)) != NULL) {
> > -        if (STREQ(entry.mnt_type, "cgroup") &&
> > -            (strstr(entry.mnt_opts, controller))) {
> > -            root->path = strdup(entry.mnt_dir);
> > -            break;
> > -        }
> > -    }
> > -
> > -    DEBUG("Mount for %s is %s\n", controller, root->path);
> 
>   Hum, we should keep that debug output in some ways, and report if not
> found either.


You missed the code later on which does this :-)

+            if (!group->controllers[i].placement) {
+                VIR_ERROR("Could not find placement for controller %s at %s",
+                          virCgroupControllerTypeToString(i),
+                          group->controllers[i].placement);
+                rc = -ENOENT;
+                break;
+            }
+            VIR_DEBUG("Detected mount/mapping %i:%s at %s in %s", i,
+                      virCgroupControllerTypeToString(i),
+                      group->controllers[i].mountPoint,
+                      group->controllers[i].placement);



Daniel
-- 
|: Red Hat, Engineering, London   -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org  -o-  http://virt-manager.org  -o-  http://ovirt.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-  F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|

--
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]