Re: [PATCH 20/25] wimax/i2400m: fix i2400m->wake_tx_skb handling

On Fri, Jan 04, 2013 at 03:19:55PM -0600, Dan Williams wrote: > On Fri, 2012-12-21 at 17:57 -0800, Tejun Heo wrote: > > i2400m_net_wake_tx() sets ->wake_tx_skb with the given skb if > > ->wake_tx_ws is not pending; however, i2400m_wake_tx_work() could have > > just started execution and haven't fet

Re: [PATCH 20/25] wimax/i2400m: fix i2400m->wake_tx_skb handling

On Fri, 2012-12-21 at 17:57 -0800, Tejun Heo wrote: > i2400m_net_wake_tx() sets ->wake_tx_skb with the given skb if > ->wake_tx_ws is not pending; however, i2400m_wake_tx_work() could have > just started execution and haven't fetched -> previous packet will be leaked. > > Update ->wake_tx_skb hand

RE: [PATCH 20/25] wimax/i2400m: fix i2400m->wake_tx_skb handling

> From: Tejun Heo [mailto:hte...@gmail.com] On Behalf Of Tejun Heo 0m: fix i2400m->wake_tx_skb handling > ... > > Only compile tested. I don't have access to hardware to test this--might want to contact Dan Williams, as he has been more actively using it. -- To unsubscribe from this list: send

[PATCH 20/25] wimax/i2400m: fix i2400m->wake_tx_skb handling

i2400m_net_wake_tx() sets ->wake_tx_skb with the given skb if ->wake_tx_ws is not pending; however, i2400m_wake_tx_work() could have just started execution and haven't fetched ->wake_tx_skb handling. * i2400m_net_wake_tx() now tests whether the previous ->wake_tx_skb has been consumed by ->wake_