mbox series

[V4,0/4] Enable virtual time-based airtime scheduler support on ath10k

Message ID 1576221593-1086-1-git-send-email-yiboz@codeaurora.org (mailing list archive)
Headers show
Series Enable virtual time-based airtime scheduler support on ath10k | expand

Message

Yibo Zhao Dec. 13, 2019, 7:19 a.m. UTC
This series fix some issues when enabling virtual time-based airtime scheduler on ath10k.

Changes since v3:
  Change schedule_pos to previous node once it has chance to be moved/removed
  from current position in the tree in loop scenario and bring back schedule_round
  in case that same node is to be scheduled again in the mean time.

  Increase airtime grace period to 2000 us in the first patch.

  Put per-AC station weight checking in its lock during configuration from application.

Changes since v2:
  Changes station airtime weight to be per-AC based to avoid sync issue
  Remove Co-developed-by and Toke's sign-off as Toke suggested

Changes since v1:
  Modify the author of Co-developed-by as Johannes suggested

Toke Høiland-Jørgensen (1):
  mac80211: Switch to a virtual time-based airtime scheduler

Yibo Zhao (3):
  mac80211: fix issue in loop scenario
  mac80211: fix low throughput in multi-clients situation
  mac80211: Sync airtime weight sum with per AC synced sta airtime
    weight together

 net/mac80211/cfg.c         |  29 +++++-
 net/mac80211/debugfs.c     |  48 +++++++++-
 net/mac80211/debugfs_sta.c |  18 ++--
 net/mac80211/ieee80211_i.h |  12 ++-
 net/mac80211/main.c        |   2 +-
 net/mac80211/sta_info.c    |  25 +++--
 net/mac80211/sta_info.h    |   8 +-
 net/mac80211/tx.c          | 234 +++++++++++++++++++++++++++++++--------------
 8 files changed, 278 insertions(+), 98 deletions(-)

Comments

Justin Capella Dec. 13, 2019, 9:03 a.m. UTC | #1
Would it make sense to consider skb->priority / QoS in the Access
Classifier selection?

On Thu, Dec 12, 2019 at 11:26 PM Yibo Zhao <yiboz@codeaurora.org> wrote:
>
> This series fix some issues when enabling virtual time-based airtime scheduler on ath10k.
>
> Changes since v3:
>   Change schedule_pos to previous node once it has chance to be moved/removed
>   from current position in the tree in loop scenario and bring back schedule_round
>   in case that same node is to be scheduled again in the mean time.
>
>   Increase airtime grace period to 2000 us in the first patch.
>
>   Put per-AC station weight checking in its lock during configuration from application.
>
> Changes since v2:
>   Changes station airtime weight to be per-AC based to avoid sync issue
>   Remove Co-developed-by and Toke's sign-off as Toke suggested
>
> Changes since v1:
>   Modify the author of Co-developed-by as Johannes suggested
>
> Toke Høiland-Jørgensen (1):
>   mac80211: Switch to a virtual time-based airtime scheduler
>
> Yibo Zhao (3):
>   mac80211: fix issue in loop scenario
>   mac80211: fix low throughput in multi-clients situation
>   mac80211: Sync airtime weight sum with per AC synced sta airtime
>     weight together
>
>  net/mac80211/cfg.c         |  29 +++++-
>  net/mac80211/debugfs.c     |  48 +++++++++-
>  net/mac80211/debugfs_sta.c |  18 ++--
>  net/mac80211/ieee80211_i.h |  12 ++-
>  net/mac80211/main.c        |   2 +-
>  net/mac80211/sta_info.c    |  25 +++--
>  net/mac80211/sta_info.h    |   8 +-
>  net/mac80211/tx.c          | 234 +++++++++++++++++++++++++++++++--------------
>  8 files changed, 278 insertions(+), 98 deletions(-)
>
> --
> 1.9.1
Toke Høiland-Jørgensen Dec. 13, 2019, 10:06 a.m. UTC | #2
Justin Capella <justincapella@gmail.com> writes:

> Would it make sense to consider skb->priority / QoS in the Access
> Classifier selection?

Erm, what? Not sure I understand what you're asking here...

-Toke