From: Felix Fietkau <nbd@nbd.name>
To: Lorenzo Bianconi <lorenzo@kernel.org>
Cc: linux-wireless@vger.kernel.org, Sean Wang <sean.wang@mediatek.com>
Subject: Re: [PATCH] mt76: only iterate over initialized rx queues
Date: Sun, 24 May 2020 19:15:25 +0200 [thread overview]
Message-ID: <88f94882-e000-5747-263e-d54e1416950f@nbd.name> (raw)
In-Reply-To: <20200524161300.GA3032@localhost.localdomain>
On 2020-05-24 18:13, Lorenzo Bianconi wrote:
>> On 2020-05-24 15:45, Lorenzo Bianconi wrote:
>> >> Fixes the following reported crash:
>> >>
>> >> [ 2.361127] BUG: spinlock bad magic on CPU#0, modprobe/456
>> >> [ 2.361583] lock: 0xffffa1287525b3b8, .magic: 00000000, .owner: <none>/-1, .owner_cpu: 0
>> >> [ 2.362250] CPU: 0 PID: 456 Comm: modprobe Not tainted 4.14.177 #5
>> >> [ 2.362751] Hardware name: HP Meep/Meep, BIOS Google_Meep.11297.75.0 06/17/2019
>> >> [ 2.363343] Call Trace:
>> >> [ 2.363552] dump_stack+0x97/0xdb
>> >> [ 2.363826] ? spin_bug+0xa6/0xb3
>> >> [ 2.364096] do_raw_spin_lock+0x6a/0x9a
>> >> [ 2.364417] mt76_dma_rx_fill+0x44/0x1de [mt76]
>> >> [ 2.364787] ? mt76_dma_kick_queue+0x18/0x18 [mt76]
>> >> [ 2.365184] mt76_dma_init+0x53/0x85 [mt76]
>> >> [ 2.365532] mt7615_dma_init+0x3d7/0x546 [mt7615e]
>> >> [ 2.365928] mt7615_register_device+0xe6/0x1a0 [mt7615e]
>> >> [ 2.366364] mt7615_mmio_probe+0x14b/0x171 [mt7615e]
>> >> [ 2.366771] mt7615_pci_probe+0x118/0x13b [mt7615e]
>> >> [ 2.367169] pci_device_probe+0xaf/0x13d
>> >> [ 2.367491] driver_probe_device+0x284/0x2ca
>> >> [ 2.367840] __driver_attach+0x7a/0x9e
>> >> [ 2.368146] ? driver_attach+0x1f/0x1f
>> >> [ 2.368451] bus_for_each_dev+0xa0/0xdb
>> >> [ 2.368765] bus_add_driver+0x132/0x204
>> >> [ 2.369078] driver_register+0x8e/0xcd
>> >> [ 2.369384] do_one_initcall+0x160/0x257
>> >> [ 2.369706] ? 0xffffffffc0240000
>> >> [ 2.369980] do_init_module+0x60/0x1bb
>> >> [ 2.370286] load_module+0x18c2/0x1a2b
>> >> [ 2.370596] ? kernel_read_file+0x141/0x1b9
>> >> [ 2.370937] ? kernel_read_file_from_fd+0x46/0x71
>> >> [ 2.371320] SyS_finit_module+0xcc/0xf0
>> >> [ 2.371636] do_syscall_64+0x6b/0xf7
>> >> [ 2.371930] entry_SYSCALL_64_after_hwframe+0x3d/0xa2
>> >> [ 2.372344] RIP: 0033:0x7da218ae4199
>> >> [ 2.372637] RSP: 002b:00007fffd0608398 EFLAGS: 00000246 ORIG_RAX: 0000000000000139
>> >> [ 2.373252] RAX: ffffffffffffffda RBX: 00005a705449df90 RCX: 00007da218ae4199
>> >> [ 2.373833] RDX: 0000000000000000 RSI: 00005a7052e73bd8 RDI: 0000000000000006
>> >> [ 2.374411] RBP: 00007fffd06083e0 R08: 0000000000000000 R09: 00005a705449d540
>> >> [ 2.374989] R10: 0000000000000006 R11: 0000000000000246 R12: 0000000000000000
>> >> [ 2.375569] R13: 00005a705449def0 R14: 00005a7052e73bd8 R15: 0000000000000000
>> >>
>> >> Reported-by: Sean Wang <sean.wang@mediatek.com>
>> >> Fixes: d3377b78cec6 ("mt76: add HE phy modes and hardware queue")
>> >> Signed-off-by: Felix Fietkau <nbd@nbd.name>
>> >
>> > Just a couple of nitpicks inline, for the rest:
>> >
>> > Acked-by: Lorenzo Bianconi <lorenzo@kernel.org>
>> >
>> >> ---
>> >> drivers/net/wireless/mediatek/mt76/debugfs.c | 2 +-
>> >> drivers/net/wireless/mediatek/mt76/dma.c | 4 ++--
>> >> drivers/net/wireless/mediatek/mt76/mt76.h | 4 ++++
>> >> drivers/net/wireless/mediatek/mt76/mt7603/mac.c | 3 ++-
>> >> drivers/net/wireless/mediatek/mt76/mt7615/mac.c | 3 ++-
>> >> drivers/net/wireless/mediatek/mt76/mt7615/pci.c | 8 +++++---
>> >> drivers/net/wireless/mediatek/mt76/mt76x02_mmio.c | 3 ++-
>> >> drivers/net/wireless/mediatek/mt76/mt7915/mac.c | 3 ++-
>> >> 8 files changed, 20 insertions(+), 10 deletions(-)
>> >>
>> >> diff --git a/drivers/net/wireless/mediatek/mt76/mt76.h b/drivers/net/wireless/mediatek/mt76/mt76.h
>> >> index afb1ccf61b74..78378dcf430a 100644
>> >> --- a/drivers/net/wireless/mediatek/mt76/mt76.h
>> >> +++ b/drivers/net/wireless/mediatek/mt76/mt76.h
>> >> @@ -671,6 +671,10 @@ static inline u16 mt76_rev(struct mt76_dev *dev)
>> >> #define mt76_queue_tx_cleanup(dev, ...) (dev)->mt76.queue_ops->tx_cleanup(&((dev)->mt76), __VA_ARGS__)
>> >> #define mt76_queue_kick(dev, ...) (dev)->mt76.queue_ops->kick(&((dev)->mt76), __VA_ARGS__)
>> >>
>> >> +#define mt76_for_each_q_rx(dev, i) \
>> >> + for (i = 0; i < ARRAY_SIZE((dev)->q_rx) && \
>> >> + (dev)->q_rx[i].desc; i++)
>
> reviewing the patch I guess this will not work for usb code since we do not
> allocate q_rx[].desc pointer there. I guess we can use q_rx[].ndesc instead
Thanks, I will send v2 with that change included.
- Felix
prev parent reply other threads:[~2020-05-24 17:15 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-24 13:07 [PATCH] mt76: only iterate over initialized rx queues Felix Fietkau
2020-05-24 13:45 ` Lorenzo Bianconi
2020-05-24 16:05 ` Felix Fietkau
2020-05-24 16:13 ` Lorenzo Bianconi
2020-05-24 17:15 ` Felix Fietkau [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=88f94882-e000-5747-263e-d54e1416950f@nbd.name \
--to=nbd@nbd.name \
--cc=linux-wireless@vger.kernel.org \
--cc=lorenzo@kernel.org \
--cc=sean.wang@mediatek.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox