Prev: oom: filter tasks not sharing the same cpuset
Next: oom: kill useless debug print
From: David Rientjes on 8 Jun 2010 15:10
On Tue, 8 Jun 2010, KOSAKI Motohiro wrote:

> badness() is wrong name because it's too generic name.
>
> rename it.
>

This is already renamed in my heuristic rewrite which we all agree, in
principle, is needed. I'm really confused that you're trying to take
little bits and pieces of my work like this, using my name for the
function in this case, for example, and not working with other developers
in reviewing their work.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Minchan Kim on 16 Jun 2010 10:50
On Wed, Jun 16, 2010 at 08:31:20PM +0900, KOSAKI Motohiro wrote:
>
> badness() is wrong name because it's too generic name. rename it.
>
> Signed-off-by: KOSAKI Motohiro <kosaki.motohiro(a)jp.fujitsu.com>
Reviewed-by: Minchan Kim <minchan.kim(a)gmail.com>

--
Kind regards,
Minchan Kim
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: David Rientjes on 16 Jun 2010 17:50
On Wed, 16 Jun 2010, KOSAKI Motohiro wrote:

>
> badness() is wrong name because it's too generic name. rename it.
>

This is already done in my badness heuristic rewrite, can we please focus
on its review?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: KOSAKI Motohiro on 16 Jun 2010 22:00
> On Wed, 16 Jun 2010, KOSAKI Motohiro wrote:
>
> >
> > badness() is wrong name because it's too generic name. rename it.
> >
>
> This is already done in my badness heuristic rewrite, can we please focus
> on its review?

Please resend that as individual patches.



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
 | 
Pages: 1
Prev: oom: filter tasks not sharing the same cpuset
Next: oom: kill useless debug print