On Wed, Oct 22, 2008 at 09:42:37AM +0200, Rami Rosen wrote: > Do we want to continue as usual if dev_alloc_skb() in theis method fails? I think we should continue and as such, I would rather not see the patch that proposes to abort here go in. The allocated skb is not used at this point anyway; it is only used as an optimization to prepare a ProbeRsp frame for possible future use. This could be done at the time when ProbeReq is received (and we sent the last Beacon), i.e., only when needed. Or if the optimization of generating this only once is considered desirable, that place could try to allocate a new skb if the one here failed (or do it on first need and cache the result). -- Jouni Malinen PGP id EFC895FA -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
