[PATCH v2] kernel/fork: fix CLONE_CHILD_CLEARTID regression in nscd

2016-08-23 Thread Michal Hocko
From: Michal Hocko fec1d0115240 ("[PATCH] Disable CLONE_CHILD_CLEARTID for abnormal exit") has caused a subtle regression in nscd which uses CLONE_CHILD_CLEARTID to clear the nscd_certainly_running flag in the shared databases, so that the clients are notified when nscd is restarted. Now, when n

Re: [PATCH v2] kernel/fork: fix CLONE_CHILD_CLEARTID regression in nscd

2016-08-30 Thread Michal Hocko
On Wed 24-08-16 17:37:16, Michal Hocko wrote: > On Wed 24-08-16 17:32:00, Oleg Nesterov wrote: > > On 08/24, Michal Hocko wrote: > > > > > > Sounds better? > > > diff --git a/kernel/fork.c b/kernel/fork.c > > > index b89f0eb99f0a..ddde5849df81 100644 > > > --- a/kernel/fork.c > > > +++ b/kernel/for

Re: [PATCH v2] kernel/fork: fix CLONE_CHILD_CLEARTID regression in nscd

2016-08-24 Thread Oleg Nesterov
On 08/24, Michal Hocko wrote: > > Sounds better? > diff --git a/kernel/fork.c b/kernel/fork.c > index b89f0eb99f0a..ddde5849df81 100644 > --- a/kernel/fork.c > +++ b/kernel/fork.c > @@ -914,7 +914,8 @@ void mm_release(struct task_struct *tsk, struct mm_struct > *mm) > > /* >* Signa

Re: [PATCH v2] kernel/fork: fix CLONE_CHILD_CLEARTID regression in nscd

2016-08-24 Thread Michal Hocko
On Wed 24-08-16 17:32:00, Oleg Nesterov wrote: > On 08/24, Michal Hocko wrote: > > > > Sounds better? > > diff --git a/kernel/fork.c b/kernel/fork.c > > index b89f0eb99f0a..ddde5849df81 100644 > > --- a/kernel/fork.c > > +++ b/kernel/fork.c > > @@ -914,7 +914,8 @@ void mm_release(struct task_struct

Re: [PATCH v2] kernel/fork: fix CLONE_CHILD_CLEARTID regression in nscd

2016-08-23 Thread Oleg Nesterov
On 08/23, Michal Hocko wrote: > > --- a/kernel/fork.c > +++ b/kernel/fork.c > @@ -913,14 +913,11 @@ void mm_release(struct task_struct *tsk, struct > mm_struct *mm) > deactivate_mm(tsk, mm); > > /* > - * If we're exiting normally, clear a user-space tid field if > - * reque

Re: [PATCH v2] kernel/fork: fix CLONE_CHILD_CLEARTID regression in nscd

2016-08-24 Thread Michal Hocko
On Tue 23-08-16 18:32:34, Oleg Nesterov wrote: > On 08/23, Michal Hocko wrote: > > > > --- a/kernel/fork.c > > +++ b/kernel/fork.c > > @@ -913,14 +913,11 @@ void mm_release(struct task_struct *tsk, struct > > mm_struct *mm) > > deactivate_mm(tsk, mm); > > > > /* > > -* If we're exiti