On Fri, 2019-12-13 at 15:19 +0800, Yibo Zhao wrote: > In a loop txqs dequeue scenario, if the first txq in the rbtree gets > removed from rbtree immediately in the ieee80211_return_txq(), the > loop will break soon in the ieee80211_next_txq() due to schedule_pos > not leading to the second txq in the rbtree. Thus update schedule_pos > to previous node once the node of schedule_pos is either removed from > rbtree or move to other location in rbtree due to airtime update. For my understanding - this is a fix to the first patch in the series? I guess you didn't squash it because that's Toke's patch or something? I tend to think you still should, and annotate the changes, but I wanted to understand it first. johannes
