lkml.org 
[lkml]   [2012]   [Nov]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [Pv-drivers] [PATCH 01/12] VMCI: context implementation.
From
Date
On Wed, 2012-11-21 at 13:29 -0800, Dmitry Torokhov wrote:
> Hi Joe,

Howdy.

> On Wed, Nov 21, 2012 at 01:04:46PM -0800, Joe Perches wrote:
> > On Wed, 2012-11-21 at 12:31 -0800, George Zhang wrote:
> > > + context = kzalloc(sizeof(*context), GFP_KERNEL);
> > > + if (!context) {
> > > + pr_warn("Failed to allocate memory for VMCI context.\n");
> >
> > OOM logging messages aren't necessary as alloc failures
> > are already logged with a stack trace.
> >
>
> Are we sure we are going to keep this policy forever?

As in geological eras? Hard to say. It's been there awhile.
If more OOM logging messages are removed, it's more likely
for the core facility to stay.

There is an __GFP_NOWARN override to avoid the stack trace.

> I'd rather keep the OOM warnings.

Your choice.

cheers, Joe



\
 
 \ /
  Last update: 2012-11-22 22:21    [W:0.049 / U:22.124 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site