Re: [PATCH 1/1] New driver: rtl8xxxu (mac80211)

[Johannes Berg <johannes@sipsolutions.net>]

On Sun, 2015-08-30 at 14:41 -0400, Jes Sorensen wrote:
> 
> > > +> > > 	> > > urb = usb_alloc_urb(0, GFP_KERNEL);
> > 
> > The above statement generated a "scheduling while atomic" splat. The
> > gfp_t argument needs to be GFP_KERNEL.
> 
> You are seeing scheduling while atomic in the TX path? That just seems
> wrong to me - Johannes is the mac80211 TX path not meant to allow
> sleeping?

The TX path is one of the few things where you can't sleep - that's
needed for performance reasons since the network stack doesn't allow
sleeping in the TX path.

> I have never seen this on x86 and I have been running the driver for a
> long time. It is puzzling this causes a problem on PowerPC. It there
> something special in the config for it? I am inclined to say there is
> something wrong with the PPC32 setup rather than with my usage of
> GFP_KERNEL in the TX path.

No, using GFP_KERNEL is wrong here. You're perhaps not enabling enough
options for it to come to a warning on x86.

> johannes