Messages in this thread | | | Subject | Re: [PATCH -next] orangefs: use vzalloc() instead of vmalloc()/memset(0) | From | Joe Perches <> | Date | Wed, 06 Jul 2016 11:11:23 -0700 |
| |
On Wed, 2016-07-06 at 14:00 -0400, Mike Marshall wrote: > Hi Joe...
Hi Mike.
> I looked around at other code to see how they were using vzalloc... > even if it is just the caller of the function that uses vzalloc, most > fs's that use vzalloc print an error message when vzalloc fails...
That's incorrect.
There are 72 v.alloc calls in fs/ 2 of those calls have a specific OOM message. There's 1 in reiserfs (old and unsupported) and 1 in orangefs
> I sure do > like to follow the flow of our kernel module all in one place > with the gossip statements...
I don't see how that matters here as there's already an OOM on v.alloc failure.
cheers, Joe
| |