On Thu, Mar 18, 2010 at 02:58:14PM -0400, Bob Copeland wrote:
> On Tue, Mar 16, 2010 at 11:45:52PM +0100, Benoit PAPILLAULT wrote:
> > One solution is to let the TX descriptor in the TX queue if the current
> > value of TXDP is currently pointing to it.
>
> Makes sense, just need to be careful w
On Tue, Mar 16, 2010 at 11:45:52PM +0100, Benoit PAPILLAULT wrote:
> One solution is to let the TX descriptor in the TX queue if the current
> value of TXDP is currently pointing to it.
Makes sense, just need to be careful we don't exit if the TX queue is
also empty, otherwise we'd never be able
Bob Copeland a écrit :
> On Sun, Mar 14, 2010 at 08:43:02PM +0200, Maxim Levitsky wrote:
>
>> One thing I noticed is racy behaviour of ath5k_txbuf_setup.
>>
>> ..
>>
>> spin_lock_bh(&txq->lock);
>> list_add_tail(&bf->list, &txq->q);
>> if (txq->link == NULL) /* is this first packe
On Sun, Mar 14, 2010 at 08:43:02PM +0200, Maxim Levitsky wrote:
> One thing I noticed is racy behaviour of ath5k_txbuf_setup.
>
> ..
>
> spin_lock_bh(&txq->lock);
> list_add_tail(&bf->list, &txq->q);
> if (txq->link == NULL) /* is this first packet? */
> ath5k_hw_s
On Sun, Mar 14, 2010 at 08:43:02PM +0200, Maxim Levitsky wrote:
> One thing I noticed is racy behaviour of ath5k_txbuf_setup.
>
> ..
>
> spin_lock_bh(&txq->lock);
> list_add_tail(&bf->list, &txq->q);
> if (txq->link == NULL) /* is this first packet? */
> ath5k_hw_s
One thing I noticed is racy behaviour of ath5k_txbuf_setup.
...
spin_lock_bh(&txq->lock);
list_add_tail(&bf->list, &txq->q);
if (txq->link == NULL) /* is this first packet? */
ath5k_hw_set_txdp(ah, txq->qnum, bf->daddr);
else /* no, so only link it