* [PATCH] mwifiex: usb: unlock on error in mwifiex_usb_tx_aggr_tmo()
@ 2017-07-10 7:21 Dan Carpenter
2017-07-17 3:12 ` Xinming Hu
2017-07-28 14:51 ` Kalle Valo
0 siblings, 2 replies; 3+ messages in thread
From: Dan Carpenter @ 2017-07-10 7:21 UTC (permalink / raw)
To: Amitkumar Karwar, Xinming Hu
Cc: Nishant Sarmukadam, Ganapathi Bhat, Kalle Valo, linux-wireless,
kernel-janitors
We need to unlock if mwifiex_usb_prepare_tx_aggr_skb() fails.
Fixes: a2ca85ad721d ("mwifiex: usb: add timer to flush aggregation packets")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
diff --git a/drivers/net/wireless/marvell/mwifiex/usb.c b/drivers/net/wireless/marvell/mwifiex/usb.c
index cb1753e43ef4..880ef1cb4088 100644
--- a/drivers/net/wireless/marvell/mwifiex/usb.c
+++ b/drivers/net/wireless/marvell/mwifiex/usb.c
@@ -1112,7 +1112,7 @@ static void mwifiex_usb_tx_aggr_tmo(unsigned long context)
if (err) {
mwifiex_dbg(adapter, ERROR,
"prepare tx aggr skb failed, err=%d\n", err);
- return;
+ goto unlock;
}
if (atomic_read(&port->tx_data_urb_pending) >=
@@ -1133,6 +1133,7 @@ static void mwifiex_usb_tx_aggr_tmo(unsigned long context)
done:
if (err == -1)
mwifiex_write_data_complete(adapter, skb_send, 0, -1);
+unlock:
spin_unlock_irqrestore(&port->tx_aggr_lock, flags);
}
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] mwifiex: usb: unlock on error in mwifiex_usb_tx_aggr_tmo()
2017-07-10 7:21 [PATCH] mwifiex: usb: unlock on error in mwifiex_usb_tx_aggr_tmo() Dan Carpenter
@ 2017-07-17 3:12 ` Xinming Hu
2017-07-28 14:51 ` Kalle Valo
1 sibling, 0 replies; 3+ messages in thread
From: Xinming Hu @ 2017-07-17 3:12 UTC (permalink / raw)
To: Dan Carpenter, Amitkumar Karwar
Cc: Nishant Sarmukadam, Ganapathi Bhat, Kalle Valo, linux-wireless,
kernel-janitors, Cathy Luo
SGksDQoNCj4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gRnJvbTogRGFuIENhcnBlbnRl
ciBbbWFpbHRvOmRhbi5jYXJwZW50ZXJAb3JhY2xlLmNvbV0NCj4gU2VudDogMjAxN8TqN9TCMTDI
1SAxNToyMQ0KPiBUbzogQW1pdGt1bWFyIEthcndhcjsgWGlubWluZyBIdQ0KPiBDYzogTmlzaGFu
dCBTYXJtdWthZGFtOyBHYW5hcGF0aGkgQmhhdDsgS2FsbGUgVmFsbzsNCj4gbGludXgtd2lyZWxl
c3NAdmdlci5rZXJuZWwub3JnOyBrZXJuZWwtamFuaXRvcnNAdmdlci5rZXJuZWwub3JnDQo+IFN1
YmplY3Q6IFtFWFRdIFtQQVRDSF0gbXdpZmlleDogdXNiOiB1bmxvY2sgb24gZXJyb3IgaW4NCj4g
bXdpZmlleF91c2JfdHhfYWdncl90bW8oKQ0KPiANCj4gRXh0ZXJuYWwgRW1haWwNCj4gDQo+IC0t
LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t
LS0tLS0tLS0tLS0NCj4gV2UgbmVlZCB0byB1bmxvY2sgaWYgbXdpZmlleF91c2JfcHJlcGFyZV90
eF9hZ2dyX3NrYigpIGZhaWxzLg0KPiANCj4gRml4ZXM6IGEyY2E4NWFkNzIxZCAoIm13aWZpZXg6
IHVzYjogYWRkIHRpbWVyIHRvIGZsdXNoIGFnZ3JlZ2F0aW9uIHBhY2tldHMiKQ0KPiBTaWduZWQt
b2ZmLWJ5OiBEYW4gQ2FycGVudGVyIDxkYW4uY2FycGVudGVyQG9yYWNsZS5jb20+DQo+IA0KPiBk
aWZmIC0tZ2l0IGEvZHJpdmVycy9uZXQvd2lyZWxlc3MvbWFydmVsbC9td2lmaWV4L3VzYi5jDQo+
IGIvZHJpdmVycy9uZXQvd2lyZWxlc3MvbWFydmVsbC9td2lmaWV4L3VzYi5jDQo+IGluZGV4IGNi
MTc1M2U0M2VmNC4uODgwZWYxY2I0MDg4IDEwMDY0NA0KPiAtLS0gYS9kcml2ZXJzL25ldC93aXJl
bGVzcy9tYXJ2ZWxsL213aWZpZXgvdXNiLmMNCj4gKysrIGIvZHJpdmVycy9uZXQvd2lyZWxlc3Mv
bWFydmVsbC9td2lmaWV4L3VzYi5jDQo+IEBAIC0xMTEyLDcgKzExMTIsNyBAQCBzdGF0aWMgdm9p
ZCBtd2lmaWV4X3VzYl90eF9hZ2dyX3Rtbyh1bnNpZ25lZCBsb25nDQo+IGNvbnRleHQpDQo+ICAJ
aWYgKGVycikgew0KPiAgCQltd2lmaWV4X2RiZyhhZGFwdGVyLCBFUlJPUiwNCj4gIAkJCSAgICAi
cHJlcGFyZSB0eCBhZ2dyIHNrYiBmYWlsZWQsIGVycj0lZFxuIiwgZXJyKTsNCj4gLQkJcmV0dXJu
Ow0KPiArCQlnb3RvIHVubG9jazsNCj4gIAl9DQo+IA0KPiAgCWlmIChhdG9taWNfcmVhZCgmcG9y
dC0+dHhfZGF0YV91cmJfcGVuZGluZykgPj0gQEAgLTExMzMsNiArMTEzMyw3DQo+IEBAIHN0YXRp
YyB2b2lkIG13aWZpZXhfdXNiX3R4X2FnZ3JfdG1vKHVuc2lnbmVkIGxvbmcgY29udGV4dCkNCj4g
IGRvbmU6DQo+ICAJaWYgKGVyciA9PSAtMSkNCj4gIAkJbXdpZmlleF93cml0ZV9kYXRhX2NvbXBs
ZXRlKGFkYXB0ZXIsIHNrYl9zZW5kLCAwLCAtMSk7DQo+ICt1bmxvY2s6DQo+ICAJc3Bpbl91bmxv
Y2tfaXJxcmVzdG9yZSgmcG9ydC0+dHhfYWdncl9sb2NrLCBmbGFncyk7ICB9DQo+IA0KDQpDaGFu
Z2UgbG9va3MgZmluZSB0byBtZS4gVGhhbmtzLg0KDQpSZWdhcmRzLA0KU2ltb24NCg==
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: mwifiex: usb: unlock on error in mwifiex_usb_tx_aggr_tmo()
2017-07-10 7:21 [PATCH] mwifiex: usb: unlock on error in mwifiex_usb_tx_aggr_tmo() Dan Carpenter
2017-07-17 3:12 ` Xinming Hu
@ 2017-07-28 14:51 ` Kalle Valo
1 sibling, 0 replies; 3+ messages in thread
From: Kalle Valo @ 2017-07-28 14:51 UTC (permalink / raw)
To: Dan Carpenter
Cc: Amitkumar Karwar, Xinming Hu, Nishant Sarmukadam, Ganapathi Bhat,
linux-wireless, kernel-janitors
Dan Carpenter <dan.carpenter@oracle.com> wrote:
> We need to unlock if mwifiex_usb_prepare_tx_aggr_skb() fails.
>
> Fixes: a2ca85ad721d ("mwifiex: usb: add timer to flush aggregation packets")
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
>
> diff --git a/drivers/net/wireless/marvell/mwifiex/usb.c b/drivers/net/wireless/marvell/mwifiex/usb.c
> index cb1753e43ef4..880ef1cb4088 100644
> --- a/drivers/net/wireless/marvell/mwifiex/usb.c
> +++ b/drivers/net/wireless/marvell/mwifiex/usb.c
> @@ -1112,7 +1112,7 @@ static void mwifiex_usb_tx_aggr_tmo(unsigned long context)
> if (err) {
> mwifiex_dbg(adapter, ERROR,
> "prepare tx aggr skb failed, err=%d\n", err);
> - return;
> + goto unlock;
> }
>
> if (atomic_read(&port->tx_data_urb_pending) >=
> @@ -1133,6 +1133,7 @@ static void mwifiex_usb_tx_aggr_tmo(unsigned long context)
> done:
> if (err == -1)
> mwifiex_write_data_complete(adapter, skb_send, 0, -1);
> +unlock:
> spin_unlock_irqrestore(&port->tx_aggr_lock, flags);
> }
Patch applied to wireless-drivers-next.git, thanks.
fe0c94be77a0 mwifiex: usb: unlock on error in mwifiex_usb_tx_aggr_tmo()
--
https://patchwork.kernel.org/patch/9832397/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-07-28 14:51 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-07-10 7:21 [PATCH] mwifiex: usb: unlock on error in mwifiex_usb_tx_aggr_tmo() Dan Carpenter
2017-07-17 3:12 ` Xinming Hu
2017-07-28 14:51 ` Kalle Valo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).