On 06/14/2016 04:59 PM, Benjamin Herrenschmidt wrote:

On Tue, 2016-06-14 at 16:17 +0530, Shreyas B Prabhu wrote:

quoted

I ignored adding this check because this is part of initcall and we are
unlikely to run out of memory at this state. But I'll add the check in
next version.

Why do you malloc the u64 array and not the string pointer array ?
Shouldn't you either have both on stack or both allocated ?

Yes. I'll make this consistent.

Thanks,
Shreyas