[PATCH 2/4] ath10k: fix HTC endpoint worker starvation

Kalle Valo kvalo at qca.qualcomm.com
Mon Aug 12 10:56:58 EDT 2013


Michal Kazior <michal.kazior at tieto.com> writes:

>> @@ -602,7 +602,8 @@ static void ath10k_htc_reset_endpoint_states(struct ath10k_htc *htc)
>>                 skb_queue_head_init(&ep->tx_queue);
>>                 ep->htc = htc;
>>                 ep->tx_credit_flow_enabled = true;
>> -               INIT_WORK(&ep->send_work, ath10k_htc_send_work);
>> +               tasklet_init(&ep->send_task, ath10k_htc_send_task,
>> +                             (unsigned long)ep);
>>         }
>>  }
>>
>
> Indentation is off by one here. Is it okay if I just resend this
> single patch or is it easier for applying if I resend the whole
> patchset?

It's errorprone if I start to pick patches from different sets. So
please resend the whole series. But take a look at my other comment
first.

-- 
Kalle Valo



More information about the ath10k mailing list