On Tue, 8 Jun 2010, KOSAKI Motohiro wrote:
> > diff --git a/mm/oom_kill.c b/mm/oom_kill.c
> > --- a/mm/oom_kill.c
> > +++ b/mm/oom_kill.c
> > @@ -184,14 +184,6 @@ unsigned long badness(struct task_struct *p, unsigned long uptime)
> > points /= 4;
> >
> > /*
> > - * If p's nodes don't overlap ours, it may still help to kill p
> > - * because p may have allocated or otherwise mapped memory on
> > - * this node before. However it will be less likely.
> > - */
> > - if (!has_intersects_mems_allowed(p))
> > - points /= 8;
> > -
> > - /*
> > * Adjust the score by oom_adj.
> > */
> > if (oom_adj) {
> > @@ -277,6 +269,8 @@ static struct task_struct *select_bad_process(unsigned long *ppoints,
> > continue;
> > if (mem && !task_in_mem_cgroup(p, mem))
> > continue;
> > + if (!has_intersects_mems_allowed(p))
> > + continue;
> >
> > /*
> > * This task already has access to memory reserves and is
>
> pulled. but I'll merge my fix. and append historical remark.
>
Andrew, are you the maintainer for these fixes or is KOSAKI?
I've been posting this particular patch for at least three months with
five acks:
Acked-by: Rik van Riel <riel@xxxxxxxxxx>
Acked-by: Nick Piggin <npiggin@xxxxxxx>
Acked-by: Balbir Singh <balbir@xxxxxxxxxxxxxxxxxx>
Acked-by: KOSAKI Motohiro <kosaki.motohiro@xxxxxxxxxxxxxx>
Reviewed-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx>
and now he's saying he'll merge his own fix and rewrite the changelog and
pull it?
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxxx For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>