Re: [PATCH 17/21] staging: brcm80211: remove struct osl_info parameter from wlc_alloc

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Feb 25, 2011 at 11:31:36AM +0100, Arend van Spriel wrote:
> The functions within wlc_alloc had parameter of struct osl_info type
> but it was never used. As part of osl concept removal this parameter
> has been removed from the function prototypes.
> 

When memory allocations fail it generates a stack dump unless you 
use GFP_NOWARN.  So really nothing would be lost if you just defined
wlc_alloc() like this:

void *wlc_calloc(size_t size)
{
        return kzalloc(size, GFP_ATOMIC);
}

Also it's weird to call the function "wlc_calloc" when it's not simillar
to calloc() at all.  Anyway, that's just a comment.  The patch here is 
fine.

regards,
dan carpenter

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux