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
_______________________________________________
Virtualization mailing list
Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linuxfoundation.org/mailman/listinfo/virtualization
