From: Felix Fietkau <nbd@nbd.name>
To: Tim Shepard <shep@alum.mit.edu>
Cc: "Toke Høiland-Jørgensen" <toke@toke.dk>,
linux-wireless@vger.kernel.org,
make-wifi-fast@lists.bufferbloat.net,
ath9k-devel@lists.ath9k.org
Subject: Re: [Make-wifi-fast] [PATCH 1/2] ath9k: use mac80211 intermediate software queues
Date: Fri, 17 Jun 2016 14:35:54 -0000 [thread overview]
Message-ID: <b6a2f65f-193b-755a-0781-d7eaeb0354dc@nbd.name> (raw)
In-Reply-To: <E1bDu1d-0007mR-00@www.xplot.org>
On 2016-06-17 15:41, Tim Shepard wrote:
>> > diff --git a/drivers/net/wireless/ath/ath9k/ath9k.h b/drivers/net/wireless/ath/ath9k/ath9k.h
>> > index 93b3793..caeae10 100644
>> > --- a/drivers/net/wireless/ath/ath9k/ath9k.h
>> > +++ b/drivers/net/wireless/ath/ath9k/ath9k.h
>> > @@ -145,8 +145,6 @@ int ath_descdma_setup(struct ath_softc *sc, struct ath_descdma *dd,
>> > #define BAW_WITHIN(_start, _bawsz, _seqno) \
>> > ((((_seqno) - (_start)) & 4095) < (_bawsz))
>> >
>> > -#define ATH_AN_2_TID(_an, _tidno) (&(_an)->tid[(_tidno)])
>> > -
>> > #define IS_HT_RATE(rate) (rate & 0x80)
>> > #define IS_CCK_RATE(rate) ((rate >= 0x18) && (rate <= 0x1e))
>> > #define IS_OFDM_RATE(rate) ((rate >= 0x8) && (rate <= 0xf))
>> > @@ -232,8 +230,10 @@ struct ath_buf {
>> >
>> > struct ath_atx_tid {
>> > struct list_head list;
>> > + struct sk_buff_head i_q;
>> Do we really need a third queue here? Instead of adding yet another
>> layer of queueing here, I think we should even get rid of buf_q.
>>
>> Channel context based queue handling can be dealt with by
>> stopping/starting relevant queues on channel context changes.
>>
>> buf_q becomes unnecessary when you remove all code in the drv_tx
>> codepath that moves frames to the intermediate queue.
>>
>> Any frame that was pulled from the intermediate queue and prepared for
>> tx, but which can't be sent right now can simply be queued to retry_q.
>>
>> This will also help with getting the diffstat insertion/deletion ratio
>> under control ;)
>>
>> > struct sk_buff_head buf_q;
>> > struct sk_buff_head retry_q;
>> > + struct ieee80211_txq *swq;
>> No need for this pointer, you can use container_of.
>
>
> Felix, great to hear from you and thanks for your feedback. I will
> try to work on this.
>
> I was struggling to understand the channel context stuff, and I have
> no idea how to test it. (Is there anyone else listening who might be
> able to help with testing the channel context stuff as we improve this
> patch and simplify the ath9k driver's use of the new mac80211
> intermediate queues?)
>
>
> Felix, do you have any thoughts on the renaming of txq to hwx that I
> had done in my original version of this patch? I had a good e-mail
> discussion with Toke a week or two ago (cc these same various lists)
> and I believe he came to understand that perhaps the renaming I had
> done in the original version of this patch was worth doing.
>
> Now in Toke's version of my patch he calls the ieee80211 txq a "swq"
> and the ath9k hardware queue is called a "txq". (I had called the
> ieee80211 txq a "txq" and I renamed the ath9k hardware queue "hwq"
> throught all the ath9k driver code. This also made ath9k's names of
> things more similar to mt76 which I was looking at as an example of a
> driver that uses your new ieee80211 txq mechanism.
>
> I think the renaming is worth doing, but I also understand the
> renaming can be disruptive to others actively working on ath9k.
> It would be nice to have another opinion on this.
I think we should finish intermediate queues support first and then look
into the rename later.
- Felix
next parent reply other threads:[~2016-06-17 14:35 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <E1bDu1d-0007mR-00@www.xplot.org>
2016-06-17 14:35 ` Felix Fietkau [this message]
[not found] <E1bDxpG-0004Ic-00@www.xplot.org>
2016-06-17 19:15 ` Toke Høiland-Jørgensen
[not found] <20160617090929.31606-1-toke@toke.dk>
[not found] ` <20160617090929.31606-2-toke@toke.dk>
2016-06-17 13:28 ` Felix Fietkau
2016-06-17 13:43 ` Toke Høiland-Jørgensen
2016-06-17 13:48 ` Felix Fietkau
2016-06-17 16:33 ` Felix Fietkau
2016-06-17 9:17 [Make-wifi-fast] [PATCH 0/2] ath9k: Add airtime fairness scheduler Toke Høiland-Jørgensen
2016-06-17 9:17 ` [Make-wifi-fast] [PATCH 1/2] ath9k: use mac80211 intermediate software queues Toke Høiland-Jørgensen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://lists.bufferbloat.net/postorius/lists/make-wifi-fast.lists.bufferbloat.net/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b6a2f65f-193b-755a-0781-d7eaeb0354dc@nbd.name \
--to=nbd@nbd.name \
--cc=ath9k-devel@lists.ath9k.org \
--cc=linux-wireless@vger.kernel.org \
--cc=make-wifi-fast@lists.bufferbloat.net \
--cc=shep@alum.mit.edu \
--cc=toke@toke.dk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox