Re: [PATCH] Staging: hv: fix sleeping while atomic issue

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

 



On Fri, Dec 10, 2010 at 05:00:01PM +0200, Timo Teräs wrote:
> osd_schedule_callback() is called from VmbusOnMsgDPC() which runs
> in a tasklet. Avoid possible sleeping by using GFP_ATOMIC for the
> memory allocation.
> 
> Seems to fix #16701.
> 
> Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=16701
> Cc: Haiyang Zhang <haiyangz@xxxxxxxxxxxxx>
> Cc: Hank Janssen <hjanssen@xxxxxxxxxxxxx>
> Signed-off-by: Timo Teräs <timo.teras@xxxxxx>
> ---
>  drivers/staging/hv/osd.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/staging/hv/osd.c b/drivers/staging/hv/osd.c
> index 8c3eb27..eb9b20d 100644
> --- a/drivers/staging/hv/osd.c
> +++ b/drivers/staging/hv/osd.c
> @@ -214,7 +214,7 @@ int osd_schedule_callback(struct workqueue_struct *wq,
>  {
>  	struct osd_callback_struct *cb;
>  
> -	cb = kmalloc(sizeof(*cb), GFP_KERNEL);
> +	cb = kmalloc(sizeof(*cb), GFP_ATOMIC);

How about just fixing the one place where this is needed?  You don't
need this for all of the osd_schedule_callback() functions, right?

Actually, we should unroll this function entirely and use the "real"
functions the kernel provides us.  I'd much rather see that fix than
this one which just papers over the problem.

thanks,

greg k-h
_______________________________________________
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