From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-15.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,UNPARSEABLE_RELAY, USER_AGENT_SANE_2 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id DEE83C433E0 for ; Sat, 27 Feb 2021 13:35:59 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 95FB164E7A for ; Sat, 27 Feb 2021 13:35:59 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 95FB164E7A Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=mediatek.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To:Date:To:From: Subject:Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=xrZC0B0AfMDKuZV8ub6BvL7cYX8T/6VMhocOfoJBqjo=; b=d5Csjwvar9/UrysPbnKEeyLXn dshsyOFFvx5FOeSuhY0JxoLC1UtRasUkklPN7NULOnl57Px5qSK8a/t8Ai4FmCxWE9faxF2dcgD7G 5+5II4hHHWMUd29zVBeWgKDjHrJY2zKZDu0wYqClmIOR0BFp09rKQ7L8tP7TaiKmkx/v+g/5xX17i DAXx3gFOoWzEvlSpQbjJfSf71OYV2V4spPnNsBoergbeQzs+9hxKZpI4/iuWHi3+UTPOICE1juvv/ IxLVbZbyHqpxyHYPNIHkkyQn11tKELfGDIl6SjE+BR+vwfPIJ/DdG2RNHwJFKhLnbHL8dXOPP37L6 D57LGhFvg==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1lFzl5-00060H-1B; Sat, 27 Feb 2021 13:35:47 +0000 Received: from mailgw01.mediatek.com ([216.200.240.184]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1lFzl1-0005zl-BX for linux-mediatek@lists.infradead.org; Sat, 27 Feb 2021 13:35:45 +0000 X-UUID: 8b103354a25d4d79b09ad6d6c123fc65-20210227 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=mediatek.com; s=dk; h=Content-Transfer-Encoding:MIME-Version:Content-Type:References:In-Reply-To:Date:CC:To:From:Subject:Message-ID; bh=+fyyoOPtYIfaBWS2oMwL/SOuUE5ToRW6IY5SE9Eij2Y=; b=ih5XHJwLwbBuNlzTxx/8BdyjY+QCbLCHkatvQkty11coaV/MrP707FdC0aigDsPH5JM3tUnlRNG5FGZenYQM+9YY/MNxrWC6NSnnTquskG7FF7TeJQW3dA+wCsgT6hPJUcQC7kH2NRK7FDHody9cjzs/ifEUtGvLUI/SLj9oAeQ=; X-UUID: 8b103354a25d4d79b09ad6d6c123fc65-20210227 Received: from mtkcas68.mediatek.inc [(172.29.94.19)] by mailgw01.mediatek.com (envelope-from ) (musrelay.mediatek.com ESMTP with TLSv1.2 ECDHE-RSA-AES256-SHA384 256/256) with ESMTP id 586855807; Sat, 27 Feb 2021 05:35:33 -0800 Received: from MTKMBS06N2.mediatek.inc (172.21.101.130) by MTKMBS62N1.mediatek.inc (172.29.193.41) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Sat, 27 Feb 2021 05:35:32 -0800 Received: from mtkcas07.mediatek.inc (172.21.101.84) by mtkmbs06n2.mediatek.inc (172.21.101.130) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Sat, 27 Feb 2021 21:35:30 +0800 Received: from [172.21.77.33] (172.21.77.33) by mtkcas07.mediatek.inc (172.21.101.73) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Sat, 27 Feb 2021 21:35:30 +0800 Message-ID: <1614432930.14551.7.camel@mtkswgap22> Subject: Re: [PATCH 2/5] mt76: mt7615: stop ext_phy queue when mac reset happens From: Ryder Lee To: Lorenzo Bianconi Date: Sat, 27 Feb 2021 21:35:30 +0800 In-Reply-To: References: <3794a3ffadc4a08548a1223e6955671977e78e49.1614411256.git.ryder.lee@mediatek.com> X-Mailer: Evolution 3.2.3-0ubuntu6 MIME-Version: 1.0 X-TM-SNTS-SMTP: 0FACCB4C644AB259B6BB76140C4710DC3A2830871372B2E1334688B423EE6B1E2000:8 X-MTK: N X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210227_083544_165540_1AA5710A X-CRM114-Status: GOOD ( 15.06 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-mediatek@lists.infradead.org, Lorenzo Bianconi , Shayne Chen , linux-wireless@vger.kernel.org, Felix Fietkau Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org On Sat, 2021-02-27 at 11:23 +0100, Lorenzo Bianconi wrote: > > Stop Tx burst for ext_phy after mac reset. > > > > Signed-off-by: Ryder Lee > > --- > > drivers/net/wireless/mediatek/mt76/mt7615/mac.c | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/drivers/net/wireless/mediatek/mt76/mt7615/mac.c b/drivers/net/wireless/mediatek/mt76/mt7615/mac.c > > index df2dc77a323a..0f613063c1ff 100644 > > --- a/drivers/net/wireless/mediatek/mt76/mt7615/mac.c > > +++ b/drivers/net/wireless/mediatek/mt76/mt7615/mac.c > > @@ -2085,6 +2085,7 @@ void mt7615_mac_reset_work(struct work_struct *work) > > del_timer_sync(&dev->phy.roc_timer); > > cancel_work_sync(&dev->phy.roc_work); > > if (phy2) { > > + set_bit(MT76_RESET, &phy2->mt76->state); > > Hi Ryder, > > since we run mt76_txq_schedule_all then and mt76_worker_disable few lines below, what is the point > of setting the state to RESET? Just in case mt76_txq_schedule_* still has the possibility to be called by some other place at the time reset happens. Also, I think MT76_RESET is more reliable to break Tx scheduling (from inside of the burst loop). Ryder > > > cancel_delayed_work_sync(&phy2->mt76->mac_work); > > del_timer_sync(&phy2->roc_timer); > > cancel_work_sync(&phy2->roc_work); > > @@ -2118,6 +2119,8 @@ void mt7615_mac_reset_work(struct work_struct *work) > > > > clear_bit(MT76_MCU_RESET, &dev->mphy.state); > > clear_bit(MT76_RESET, &dev->mphy.state); > > + if (phy2) > > + clear_bit(MT76_RESET, &phy2->mt76->state); > > > > mt76_worker_enable(&dev->mt76.tx_worker); > > napi_enable(&dev->mt76.tx_napi); > > -- > > 2.18.0 > > _______________________________________________ Linux-mediatek mailing list Linux-mediatek@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-mediatek From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-17.2 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,UNPARSEABLE_RELAY, USER_AGENT_SANE_2 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id BEDA4C433E0 for ; Sat, 27 Feb 2021 13:36:35 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 8DCC964EAF for ; Sat, 27 Feb 2021 13:36:35 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230014AbhB0NgT (ORCPT ); Sat, 27 Feb 2021 08:36:19 -0500 Received: from mailgw01.mediatek.com ([210.61.82.183]:48335 "EHLO mailgw01.mediatek.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S229912AbhB0NgS (ORCPT ); Sat, 27 Feb 2021 08:36:18 -0500 X-UUID: 293a586b2d7b46f8875db4c04388ea09-20210227 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=mediatek.com; s=dk; h=Content-Transfer-Encoding:MIME-Version:Content-Type:References:In-Reply-To:Date:CC:To:From:Subject:Message-ID; bh=+fyyoOPtYIfaBWS2oMwL/SOuUE5ToRW6IY5SE9Eij2Y=; b=ih5XHJwLwbBuNlzTxx/8BdyjY+QCbLCHkatvQkty11coaV/MrP707FdC0aigDsPH5JM3tUnlRNG5FGZenYQM+9YY/MNxrWC6NSnnTquskG7FF7TeJQW3dA+wCsgT6hPJUcQC7kH2NRK7FDHody9cjzs/ifEUtGvLUI/SLj9oAeQ=; X-UUID: 293a586b2d7b46f8875db4c04388ea09-20210227 Received: from mtkmrs01.mediatek.inc [(172.21.131.159)] by mailgw01.mediatek.com (envelope-from ) (Cellopoint E-mail Firewall v4.1.14 Build 0819 with TLSv1.2 ECDHE-RSA-AES256-SHA384 256/256) with ESMTP id 1094444110; Sat, 27 Feb 2021 21:35:32 +0800 Received: from mtkcas07.mediatek.inc (172.21.101.84) by mtkmbs06n2.mediatek.inc (172.21.101.130) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Sat, 27 Feb 2021 21:35:30 +0800 Received: from [172.21.77.33] (172.21.77.33) by mtkcas07.mediatek.inc (172.21.101.73) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Sat, 27 Feb 2021 21:35:30 +0800 Message-ID: <1614432930.14551.7.camel@mtkswgap22> Subject: Re: [PATCH 2/5] mt76: mt7615: stop ext_phy queue when mac reset happens From: Ryder Lee To: Lorenzo Bianconi CC: Felix Fietkau , Lorenzo Bianconi , Shayne Chen , , Date: Sat, 27 Feb 2021 21:35:30 +0800 In-Reply-To: References: <3794a3ffadc4a08548a1223e6955671977e78e49.1614411256.git.ryder.lee@mediatek.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.2.3-0ubuntu6 MIME-Version: 1.0 X-TM-SNTS-SMTP: 0FACCB4C644AB259B6BB76140C4710DC3A2830871372B2E1334688B423EE6B1E2000:8 X-MTK: N Content-Transfer-Encoding: base64 Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org T24gU2F0LCAyMDIxLTAyLTI3IGF0IDExOjIzICswMTAwLCBMb3JlbnpvIEJpYW5jb25pIHdyb3Rl Og0KPiA+IFN0b3AgVHggYnVyc3QgZm9yIGV4dF9waHkgYWZ0ZXIgbWFjIHJlc2V0Lg0KPiA+IA0K PiA+IFNpZ25lZC1vZmYtYnk6IFJ5ZGVyIExlZSA8cnlkZXIubGVlQG1lZGlhdGVrLmNvbT4NCj4g PiAtLS0NCj4gPiAgZHJpdmVycy9uZXQvd2lyZWxlc3MvbWVkaWF0ZWsvbXQ3Ni9tdDc2MTUvbWFj LmMgfCAzICsrKw0KPiA+ICAxIGZpbGUgY2hhbmdlZCwgMyBpbnNlcnRpb25zKCspDQo+ID4gDQo+ ID4gZGlmZiAtLWdpdCBhL2RyaXZlcnMvbmV0L3dpcmVsZXNzL21lZGlhdGVrL210NzYvbXQ3NjE1 L21hYy5jIGIvZHJpdmVycy9uZXQvd2lyZWxlc3MvbWVkaWF0ZWsvbXQ3Ni9tdDc2MTUvbWFjLmMN Cj4gPiBpbmRleCBkZjJkYzc3YTMyM2EuLjBmNjEzMDYzYzFmZiAxMDA2NDQNCj4gPiAtLS0gYS9k cml2ZXJzL25ldC93aXJlbGVzcy9tZWRpYXRlay9tdDc2L210NzYxNS9tYWMuYw0KPiA+ICsrKyBi L2RyaXZlcnMvbmV0L3dpcmVsZXNzL21lZGlhdGVrL210NzYvbXQ3NjE1L21hYy5jDQo+ID4gQEAg LTIwODUsNiArMjA4NSw3IEBAIHZvaWQgbXQ3NjE1X21hY19yZXNldF93b3JrKHN0cnVjdCB3b3Jr X3N0cnVjdCAqd29yaykNCj4gPiAgCWRlbF90aW1lcl9zeW5jKCZkZXYtPnBoeS5yb2NfdGltZXIp Ow0KPiA+ICAJY2FuY2VsX3dvcmtfc3luYygmZGV2LT5waHkucm9jX3dvcmspOw0KPiA+ICAJaWYg KHBoeTIpIHsNCj4gPiArCQlzZXRfYml0KE1UNzZfUkVTRVQsICZwaHkyLT5tdDc2LT5zdGF0ZSk7 DQo+IA0KPiBIaSBSeWRlciwNCj4gDQo+IHNpbmNlIHdlIHJ1biBtdDc2X3R4cV9zY2hlZHVsZV9h bGwgdGhlbiBhbmQgbXQ3Nl93b3JrZXJfZGlzYWJsZSBmZXcgbGluZXMgYmVsb3csIHdoYXQgaXMg dGhlIHBvaW50DQo+IG9mIHNldHRpbmcgdGhlIHN0YXRlIHRvIFJFU0VUPw0KDQpKdXN0IGluIGNh c2UgbXQ3Nl90eHFfc2NoZWR1bGVfKiBzdGlsbCBoYXMgdGhlIHBvc3NpYmlsaXR5IHRvIGJlIGNh bGxlZA0KYnkgc29tZSBvdGhlciBwbGFjZSBhdCB0aGUgdGltZSByZXNldCBoYXBwZW5zLiBBbHNv LCBJIHRoaW5rIE1UNzZfUkVTRVQNCmlzIG1vcmUgcmVsaWFibGUgdG8gYnJlYWsgVHggc2NoZWR1 bGluZyAoZnJvbSBpbnNpZGUgb2YgdGhlIGJ1cnN0IGxvb3ApLg0KDQpSeWRlcg0KPiANCj4gPiAg CQljYW5jZWxfZGVsYXllZF93b3JrX3N5bmMoJnBoeTItPm10NzYtPm1hY193b3JrKTsNCj4gPiAg CQlkZWxfdGltZXJfc3luYygmcGh5Mi0+cm9jX3RpbWVyKTsNCj4gPiAgCQljYW5jZWxfd29ya19z eW5jKCZwaHkyLT5yb2Nfd29yayk7DQo+ID4gQEAgLTIxMTgsNiArMjExOSw4IEBAIHZvaWQgbXQ3 NjE1X21hY19yZXNldF93b3JrKHN0cnVjdCB3b3JrX3N0cnVjdCAqd29yaykNCj4gPiAgDQo+ID4g IAljbGVhcl9iaXQoTVQ3Nl9NQ1VfUkVTRVQsICZkZXYtPm1waHkuc3RhdGUpOw0KPiA+ICAJY2xl YXJfYml0KE1UNzZfUkVTRVQsICZkZXYtPm1waHkuc3RhdGUpOw0KPiA+ICsJaWYgKHBoeTIpDQo+ ID4gKwkJY2xlYXJfYml0KE1UNzZfUkVTRVQsICZwaHkyLT5tdDc2LT5zdGF0ZSk7DQo+ID4gIA0K PiA+ICAJbXQ3Nl93b3JrZXJfZW5hYmxlKCZkZXYtPm10NzYudHhfd29ya2VyKTsNCj4gPiAgCW5h cGlfZW5hYmxlKCZkZXYtPm10NzYudHhfbmFwaSk7DQo+ID4gLS0gDQo+ID4gMi4xOC4wDQo+ID4g DQoNCg==