Messages in this thread | | | Subject | Re: [PATCH RFC v2] slab: introduce kmalloc_array | From | Xi Wang <> | Date | Fri, 10 Feb 2012 08:55:02 -0500 |
| |
On Feb 10, 2012, at 8:11 AM, Alexey Dobriyan wrote: >> It should be named kaalloc(), I think.
I like this shorter name. Let's see what others think. ;-)
>> Why it is ULONG_MAX, when size_t is used?
Is there a SIZE_MAX or something similar?
> Also, it could be written more "robust" against people who will make > sizeof() the first argument with __builtin_constant_p().
Do you mean something like this?
BUILD_BUG_ON(__builtin_constant_p(n));
or
BUILD_BUG_ON(__builtin_constant_p(n) && !__builtin_constant_p(size));
- xi
| |