Re: [PATCH] fix spurious EBUSY on memory cgroup removal

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

 



> On Tue, 25 Mar 2008 14:47:13 +0900 (JST) yamamoto@xxxxxxxxxxxxx (YAMAMOTO Takashi) wrote:
> 
> > [ resending with To: akpm.  Andrew, can you include this in -mm tree? ]
> 
> Shouldn't it be in 2.6.25?

yes, probably.

(i'm not sure about linux development model.)

YAMAMOTO Takashi

> 
> > hi,
> > 
> > the following patch is to fix spurious EBUSY on cgroup removal.
> > 
> > YAMAMOTO Takashi
> > 
> > 
> > call mm_free_cgroup earlier.
> > otherwise a reference due to lazy mm switching can prevent cgroup removal.
> > 
> > Signed-off-by: YAMAMOTO Takashi <yamamoto@xxxxxxxxxxxxx>
> > Acked-by: Balbir Singh <balbir@xxxxxxxxxxxxxxxxxx>
> > ---
> > 
> > --- linux-2.6.24-rc8-mm1/kernel/fork.c.BACKUP	2008-01-23 14:43:29.000000000 +0900
> > +++ linux-2.6.24-rc8-mm1/kernel/fork.c	2008-01-31 17:26:31.000000000 +0900
> > @@ -393,7 +393,6 @@ void __mmdrop(struct mm_struct *mm)
> >  {
> >  	BUG_ON(mm == &init_mm);
> >  	mm_free_pgd(mm);
> > -	mm_free_cgroup(mm);
> >  	destroy_context(mm);
> >  	free_mm(mm);
> >  }
> > @@ -415,6 +414,7 @@ void mmput(struct mm_struct *mm)
> >  			spin_unlock(&mmlist_lock);
> >  		}
> >  		put_swap_token(mm);
> > +		mm_free_cgroup(mm);
> >  		mmdrop(mm);
> >  	}
> >  }
> _______________________________________________
> Containers mailing list
> Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx
> https://lists.linux-foundation.org/mailman/listinfo/containers
_______________________________________________
Containers mailing list
Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linux-foundation.org/mailman/listinfo/containers

[Index of Archives]     [Cgroups]     [Netdev]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux