On Wed, Jun 20, 2007 at 11:13:39PM +0200, Johannes Berg wrote: > Mohamed, > > > + padding = calc_pad_len(subframe_len); > > + /* the last MSDU has no padding */ > > + if (subframe_len > remaining) > > + return TXRX_DROP; > > + > > + frame = dev_alloc_skb(local->hw.extra_tx_headroom + > > + subframe_len); > > + > > + if (frame == NULL) > > + return TXRX_DROP; > > + > > + memcpy(skb_put(frame, subframe_len), (u8*)eth, subframe_len); > > + frame->mac.raw = frame->data; > > + skb2 = NULL; > > Here you allocate a new frame which can be sent back to the device when > one of the aggregated frames was a multicast frame and we're an AP > device. You also correctly add extra_tx_headroom, but it seems that this > is missing some skb_reserve(extra_tx_headroom), no? Could you make a > patch adding it? I'm not sure I fully understand the code there. Was there ever any reply to this? John -- John W. Linville linville@xxxxxxxxxxxxx - 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
