Messages in this thread | | | Date | Wed, 1 Jul 2015 14:29:57 -0700 (PDT) | From | David Rientjes <> | Subject | Re: [patch 1/3] mm, oom: organize oom context into struct |
| |
On Wed, 1 Jul 2015, Sergey Senozhatsky wrote:
> On (06/30/15 15:46), David Rientjes wrote: > > > > There are essential elements to an oom context that are passed around to > > > > multiple functions. > > > > > > > > Organize these elements into a new struct, struct oom_context, that > > > > specifies the context for an oom condition. > > > > > > > > > > s/oom_context/oom_control/ ? > > > > > > > I think it would be confused with the existing memory.oom_control for > > memcg. > > > > Hello David, > > Sorry, I meant that in commit message you say > > :Organize these elements into a new struct, struct oom_context, that > :specifies the context for an oom condition. > > but define and use `struct oom_control' (not `struct oom_context') >
Oh, point very well taken, thank you.
| |