On Fri, 2006-09-15 at 18:30 +0200, Björn Steinbrink wrote:
> On 2006.09.14 18:41:26 -0700, Rohit Seth wrote:
> > +static int __init configfs_container_init(void)
> > +{
> > +   int ret;
> > +
> > +   config_group_init(&containerfs_group.cs_subsys.su_group);
> > +   init_MUTEX(&containerfs_group.cs_subsys.su_sem);
> > +   ret = configfs_register_subsystem(&containerfs_group.cs_subsys);
> > +
> > +   if (ret) 
> > +           printk(KERN_ERR "Error %d while registering container 
> > subsystem\n", ret);
> > +   else {
> > +           container_wq = create_workqueue("Kcontainerd");
> > +           if (container_wq == NULL) {
> > +                   ret = -ENOMEM;
> > +                   printk(KERN_ERR "Unable to create Container 
> > controllers");
> 
> Shouldn't the subsystem be unregistered here?
> 

Yes. Indeed.  I'll fix it.

Thanks,
-rohit

> > +           }
> > +   }
> > +   return ret;
> > +}
> 
> Regards,
> Björn


-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
ckrm-tech mailing list
https://lists.sourceforge.net/lists/listinfo/ckrm-tech

Reply via email to