[PATCH 1/3] ath10k: Add debugging for tx-credits usage.
Ben Greear
greearb at candelatech.com
Mon Mar 24 12:50:01 EDT 2014
On 03/24/2014 04:21 AM, Michal Kazior wrote:
> On 24 March 2014 12:12, Kalle Valo <kvalo at qca.qualcomm.com> wrote:
>>> - status = ath10k_htc_send(htc, ATH10K_HTC_EP_0, skb);
>>> + status = ath10k_htc_send(htc, ATH10K_HTC_EP_0, skb, __LINE__);
>>
>> Using line numbers in debug messages is very cumbersome. Some people
>> cherry pick patches, have their own changes and whatnot which will make
>> it more difficult to read the debug logs. Isn't there any better way to
>> do this?
>
> I would prefer to explicitly wait for tx credit replenishment in
> ath10k_wmi_cmd_send() after a command is sent instead of all these
> prints. This way you can get a full call trace if it times out.
Would that be a performance problem (by effectively allowing only a single
item to be sent to the firmware at any given time?)
Thanks,
Ben
--
Ben Greear <greearb at candelatech.com>
Candela Technologies Inc http://www.candelatech.com
More information about the ath10k
mailing list