lkml.org 
[lkml]   [2009]   [May]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [patch 06/11 -mmotm] oom: move oom_adj value from task_struct to mm_struct
On Mon, 11 May 2009 01:43:10 -0700 (PDT)
David Rientjes <rientjes@google.com> wrote:

> On Mon, 11 May 2009, KOSAKI Motohiro wrote:
>
> > From technical view, your explanation seems good although I haven't
> > review your patch.
> > but this patch break staging driver rule. I don't like it.
> >
> > IOW, if greg pull this patch into his staging tree, it might make
> > patch conflict against -mm tree in nearly future.
> >
>
> If I am going to break code that is in mainline (like the Android low
> memory killer is) by moving a member of a data structure, then I'd like to
> provide a fix for it so that it works after my change. I'll leave the
> merging to Andrew and Greg and if they'd prefer I do this patch in two
> parts, that's fine by me as long as they merge them at the same time. I
> thought since this code is mainline that it would be easier to go through
> in one series.

Yep, this all has to be done as a single commit to avoid creating
points in the commit sequence where the kernel doesn't build.

As I'm cherrypicking my way around the android driver changes I
reworked this patch to apply on plain old mainline.



\
 
 \ /
  Last update: 2009-05-11 23:29    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site