From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.toke.dk (mail.toke.dk [52.28.52.200]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by lists.bufferbloat.net (Postfix) with ESMTPS id 4EC2E3B2A4 for ; Mon, 21 Jan 2019 11:51:46 -0500 (EST) From: Toke =?utf-8?Q?H=C3=B8iland-J=C3=B8rgensen?= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=toke.dk; s=20161023; t=1548089505; bh=KQ/tcdtTr7DtI1/bxd8J/JVBRZ35X46rbjowDG2uFIA=; h=From:To:Cc:Subject:In-Reply-To:References:Date:From; b=MYFAuSqLEF0Yi5a6qqOGU+MbAQUi9LHQhsoQ9dNokKQhpMaGguJYyGgDVWFi1ZBeQ pB5stgycOnJ/KXVj7fKSZX+7DikXMR0QN/sff4T4hzh9uVFL3B1uqBjrh8FedhpYKs utzdIT0Wma8Kkg26xI0pSt/752xjdoxGcrxfKgv2WzrCbLusgVf+ludOeUVmO/cwV5 gHc7xIWOl8VikaHiolhNj5q3FS0uo6RxTeAb+Z36KvpHZgV97otWfcj39KuH261mjv 4/Smh4E8XUAZnj9q50OZbJRDlMp0Dp7RHONgEJZPlYVRK6CaOxY7bwROV4FsXovECR 0EKKnCVbcpeBQ== To: Rajkumar Manoharan , linux-wireless@vger.kernel.org, ath10k@lists.infradead.org Cc: make-wifi-fast@lists.bufferbloat.net, Rajkumar Manoharan In-Reply-To: <1545181331-5288-5-git-send-email-rmanohar@codeaurora.org> References: <1545181331-5288-1-git-send-email-rmanohar@codeaurora.org> <1545181331-5288-5-git-send-email-rmanohar@codeaurora.org> Date: Mon, 21 Jan 2019 17:51:44 +0100 X-Clacks-Overhead: GNU Terry Pratchett Message-ID: <87pnsqkl7j.fsf@toke.dk> MIME-Version: 1.0 Content-Type: text/plain Subject: Re: [Make-wifi-fast] [PATCH v5 4/6] ath9k: Switch to mac80211 TXQ scheduling and airtime APIs X-BeenThere: make-wifi-fast@lists.bufferbloat.net X-Mailman-Version: 2.1.20 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 21 Jan 2019 16:51:46 -0000 Just discovered this while working on my follow-up: > void ath_tx_queue_tid(struct ath_softc *sc, struct ath_atx_tid *tid) > { > - struct ath_vif *avp = (struct ath_vif *) tid->an->vif->drv_priv; > - struct ath_chanctx *ctx = avp->chanctx; > - struct ath_acq *acq; > + struct ieee80211_txq *queue = > + container_of((void *)tid, struct ieee80211_txq, drv_priv); > > - if (!ctx || !list_empty(&tid->list)) > - return; > - > - acq = &ctx->acq[TID_TO_WME_AC(tid->tidno)]; > - spin_lock_bh(&acq->lock); > - __ath_tx_queue_tid(sc, tid); > - spin_unlock_bh(&acq->lock); > + ieee80211_return_txq(sc->hw, queue); > } After we evolved the API, this is now wrong, as ieee80211_return_txq() should only be called while holding the right lock. I'll post a fixed version tomorrow. -Toke