Messages in this thread |  | | | Date | Fri, 29 Aug 2008 15:53:19 -0700 | | From | Andrew Morton <> | | Subject | Re: [PATCH] dyn_array: using %pF instead of print_fn_descriptor_symbol |
| |
> ptr = __alloc_bootmem_nopanic(total_size, max_align, 0); > if (!ptr) > panic("Can not alloc dyn_alloc\n");
Why duplicate the panic()? Just call __alloc_bootmem().
> #ifdef CONFIF_GENERIC_HARDIRQS
That doesn't appear to have been very well tested?
The code has a few coding-style glitches which checkpatch can detect.
|  |