* Regression: OOPs on boot due to "wlcore: Add support for optional wakeirq"
@ 2018-10-25 17:04 John Stultz
2018-10-25 17:15 ` John Stultz
2018-10-25 17:26 ` Tony Lindgren
0 siblings, 2 replies; 5+ messages in thread
From: John Stultz @ 2018-10-25 17:04 UTC (permalink / raw)
To: Tony Lindgren, Kalle Valo; +Cc: Eyal Reizer, lkml, Anders Roxell
Hey Tony,
In testing linus/master on my hikey board, I'm hitting the following
OOPS on bootup:
[ 1.870279] Unable to handle kernel read from unreadable memory at
virtual address 0000000000000010
[ 1.870283] Mem abort info:
[ 1.870287] ESR = 0x96000005
[ 1.870292] Exception class = DABT (current EL), IL = 32 bits
[ 1.870296] SET = 0, FnV = 0
[ 1.870299] EA = 0, S1PTW = 0
[ 1.870302] Data abort info:
[ 1.870306] ISV = 0, ISS = 0x00000005
[ 1.870309] CM = 0, WnR = 0
[ 1.870312] [0000000000000010] user address but active_mm is swapper
[ 1.870318] Internal error: Oops: 96000005 [#1] PREEMPT SMP
[ 1.870327] CPU: 0 PID: 5 Comm: kworker/0:0 Not tainted
4.19.0-05129-gb3d1e8e #48
[ 1.870331] Hardware name: HiKey Development Board (DT)
[ 1.870350] Workqueue: events_freezable mmc_rescan
[ 1.870358] pstate: 60400005 (nZCv daif +PAN -UAO)
[ 1.870366] pc : wl1271_probe+0x210/0x350
[ 1.870371] lr : wl1271_probe+0x210/0x350
[ 1.870374] sp : ffffff80080739b0
[ 1.870377] x29: ffffff80080739b0 x28: 0000000000000000
[ 1.870384] x27: 0000000000000000 x26: 0000000000000000
[ 1.870391] x25: 0000000000000036 x24: ffffffc074ecb598
[ 1.870398] x23: ffffffc07ffdce78 x22: ffffffc0744ed808
[ 1.870404] x21: ffffffc074ecbb98 x20: ffffff8008ff9000
[ 1.870411] x19: ffffffc0744ed800 x18: ffffff8008ff9a48
[ 1.870418] x17: 0000000000000000 x16: 0000000000000000
[ 1.870425] x15: ffffffc074ecb503 x14: ffffffffffffffff
[ 1.870431] x13: ffffffc074ecb502 x12: 0000000000000030
[ 1.870438] x11: 0101010101010101 x10: 0000000000000040
[ 1.870444] x9 : ffffffc075400248 x8 : ffffffc075400270
[ 1.870451] x7 : 0000000000000000 x6 : 0000000000000000
[ 1.870457] x5 : 0000000000000000 x4 : 0000000000000000
[ 1.870463] x3 : 0000000000000000 x2 : 0000000000000000
[ 1.870469] x1 : 0000000000000028 x0 : 0000000000000000
[ 1.870477] Process kworker/0:0 (pid: 5, stack limit = 0x(____ptrval____))
[ 1.870480] Call trace:
[ 1.870485] wl1271_probe+0x210/0x350
[ 1.870491] sdio_bus_probe+0x100/0x128
[ 1.870500] really_probe+0x1a8/0x2b8
[ 1.870506] driver_probe_device+0x58/0x100
[ 1.870511] __device_attach_driver+0x94/0xd8
[ 1.870517] bus_for_each_drv+0x70/0xc8
[ 1.870522] __device_attach+0xe0/0x140
[ 1.870527] device_initial_probe+0x10/0x18
[ 1.870532] bus_probe_device+0x94/0xa0
[ 1.870537] device_add+0x374/0x5b8
[ 1.870542] sdio_add_func+0x60/0x88
[ 1.870546] mmc_attach_sdio+0x1b0/0x358
[ 1.870551] mmc_rescan+0x2cc/0x390
[ 1.870558] process_one_work+0x12c/0x320
[ 1.870563] worker_thread+0x48/0x458
[ 1.870569] kthread+0xf8/0x128
[ 1.870575] ret_from_fork+0x10/0x18
[ 1.870583] Code: 92400c21 b2760021 a90687a2 97e95bf9 (f9400803)
[ 1.870587] ---[ end trace 1e15f81d3c139ca9 ]---
I've bisected it down to 3c83dd577c7f ("wlcore: Add support for
optional wakeirq").
It seems since we don't have a wakeirq value in the dts, the wakeirq
value in wl1271_probe() is zero, which then causes trouble in
irqd_get_trigger_type(irq_get_irq_data(wakeirq)).
Should we check wakeirq before we set the second elements of res[]?
thanks
-john
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Regression: OOPs on boot due to "wlcore: Add support for optional wakeirq"
2018-10-25 17:04 Regression: OOPs on boot due to "wlcore: Add support for optional wakeirq" John Stultz
@ 2018-10-25 17:15 ` John Stultz
2018-10-25 17:31 ` Tony Lindgren
2018-10-25 17:33 ` Kalle Valo
2018-10-25 17:26 ` Tony Lindgren
1 sibling, 2 replies; 5+ messages in thread
From: John Stultz @ 2018-10-25 17:15 UTC (permalink / raw)
To: Tony Lindgren, Kalle Valo; +Cc: Eyal Reizer, lkml, Anders Roxell
On Thu, Oct 25, 2018 at 10:04 AM, John Stultz <john.stultz@linaro.org> wrote:
> Hey Tony,
> In testing linus/master on my hikey board, I'm hitting the following
> OOPS on bootup:
>
> [ 1.870279] Unable to handle kernel read from unreadable memory at
> virtual address 0000000000000010
> [ 1.870283] Mem abort info:
> [ 1.870287] ESR = 0x96000005
> [ 1.870292] Exception class = DABT (current EL), IL = 32 bits
> [ 1.870296] SET = 0, FnV = 0
> [ 1.870299] EA = 0, S1PTW = 0
> [ 1.870302] Data abort info:
> [ 1.870306] ISV = 0, ISS = 0x00000005
> [ 1.870309] CM = 0, WnR = 0
> [ 1.870312] [0000000000000010] user address but active_mm is swapper
> [ 1.870318] Internal error: Oops: 96000005 [#1] PREEMPT SMP
> [ 1.870327] CPU: 0 PID: 5 Comm: kworker/0:0 Not tainted
> 4.19.0-05129-gb3d1e8e #48
> [ 1.870331] Hardware name: HiKey Development Board (DT)
> [ 1.870350] Workqueue: events_freezable mmc_rescan
> [ 1.870358] pstate: 60400005 (nZCv daif +PAN -UAO)
> [ 1.870366] pc : wl1271_probe+0x210/0x350
> [ 1.870371] lr : wl1271_probe+0x210/0x350
> [ 1.870374] sp : ffffff80080739b0
> [ 1.870377] x29: ffffff80080739b0 x28: 0000000000000000
> [ 1.870384] x27: 0000000000000000 x26: 0000000000000000
> [ 1.870391] x25: 0000000000000036 x24: ffffffc074ecb598
> [ 1.870398] x23: ffffffc07ffdce78 x22: ffffffc0744ed808
> [ 1.870404] x21: ffffffc074ecbb98 x20: ffffff8008ff9000
> [ 1.870411] x19: ffffffc0744ed800 x18: ffffff8008ff9a48
> [ 1.870418] x17: 0000000000000000 x16: 0000000000000000
> [ 1.870425] x15: ffffffc074ecb503 x14: ffffffffffffffff
> [ 1.870431] x13: ffffffc074ecb502 x12: 0000000000000030
> [ 1.870438] x11: 0101010101010101 x10: 0000000000000040
> [ 1.870444] x9 : ffffffc075400248 x8 : ffffffc075400270
> [ 1.870451] x7 : 0000000000000000 x6 : 0000000000000000
> [ 1.870457] x5 : 0000000000000000 x4 : 0000000000000000
> [ 1.870463] x3 : 0000000000000000 x2 : 0000000000000000
> [ 1.870469] x1 : 0000000000000028 x0 : 0000000000000000
> [ 1.870477] Process kworker/0:0 (pid: 5, stack limit = 0x(____ptrval____))
> [ 1.870480] Call trace:
> [ 1.870485] wl1271_probe+0x210/0x350
> [ 1.870491] sdio_bus_probe+0x100/0x128
> [ 1.870500] really_probe+0x1a8/0x2b8
> [ 1.870506] driver_probe_device+0x58/0x100
> [ 1.870511] __device_attach_driver+0x94/0xd8
> [ 1.870517] bus_for_each_drv+0x70/0xc8
> [ 1.870522] __device_attach+0xe0/0x140
> [ 1.870527] device_initial_probe+0x10/0x18
> [ 1.870532] bus_probe_device+0x94/0xa0
> [ 1.870537] device_add+0x374/0x5b8
> [ 1.870542] sdio_add_func+0x60/0x88
> [ 1.870546] mmc_attach_sdio+0x1b0/0x358
> [ 1.870551] mmc_rescan+0x2cc/0x390
> [ 1.870558] process_one_work+0x12c/0x320
> [ 1.870563] worker_thread+0x48/0x458
> [ 1.870569] kthread+0xf8/0x128
> [ 1.870575] ret_from_fork+0x10/0x18
> [ 1.870583] Code: 92400c21 b2760021 a90687a2 97e95bf9 (f9400803)
> [ 1.870587] ---[ end trace 1e15f81d3c139ca9 ]---
>
>
> I've bisected it down to 3c83dd577c7f ("wlcore: Add support for
> optional wakeirq").
>
> It seems since we don't have a wakeirq value in the dts, the wakeirq
> value in wl1271_probe() is zero, which then causes trouble in
> irqd_get_trigger_type(irq_get_irq_data(wakeirq)).
>
> Should we check wakeirq before we set the second elements of res[]?
Here's my first swing at doing the above. It seems to work ok. Let me
know if it looks reasonable and I'll submit it as a proper patch.
https://git.linaro.org/people/john.stultz/android-dev.git/commit/?h=dev/hikey-mainline-WIP&id=383fb6e3abb71b8a721f196120a2e3a9da3315ac
thanks
-john
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Regression: OOPs on boot due to "wlcore: Add support for optional wakeirq"
2018-10-25 17:04 Regression: OOPs on boot due to "wlcore: Add support for optional wakeirq" John Stultz
2018-10-25 17:15 ` John Stultz
@ 2018-10-25 17:26 ` Tony Lindgren
1 sibling, 0 replies; 5+ messages in thread
From: Tony Lindgren @ 2018-10-25 17:26 UTC (permalink / raw)
To: John Stultz; +Cc: Kalle Valo, Eyal Reizer, lkml, Anders Roxell
* John Stultz <john.stultz@linaro.org> [181025 17:04]:
> Hey Tony,
> In testing linus/master on my hikey board, I'm hitting the following
> OOPS on bootup:
>
> [ 1.870279] Unable to handle kernel read from unreadable memory at
...
> [ 1.870485] wl1271_probe+0x210/0x350
> [ 1.870491] sdio_bus_probe+0x100/0x128
...
> I've bisected it down to 3c83dd577c7f ("wlcore: Add support for
> optional wakeirq").
>
> It seems since we don't have a wakeirq value in the dts, the wakeirq
> value in wl1271_probe() is zero, which then causes trouble in
> irqd_get_trigger_type(irq_get_irq_data(wakeirq)).
>
> Should we check wakeirq before we set the second elements of res[]?
Oh sorry about that, yeah I'll take a look and post a patch ASAP.
Hmm for me value 0 there causes no errors looks like.
Regards,
Tony
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Regression: OOPs on boot due to "wlcore: Add support for optional wakeirq"
2018-10-25 17:15 ` John Stultz
@ 2018-10-25 17:31 ` Tony Lindgren
2018-10-25 17:33 ` Kalle Valo
1 sibling, 0 replies; 5+ messages in thread
From: Tony Lindgren @ 2018-10-25 17:31 UTC (permalink / raw)
To: John Stultz; +Cc: Kalle Valo, Eyal Reizer, lkml, Anders Roxell
Hi,
* John Stultz <john.stultz@linaro.org> [181025 17:15]:
> On Thu, Oct 25, 2018 at 10:04 AM, John Stultz <john.stultz@linaro.org> wrote:
> > Should we check wakeirq before we set the second elements of res[]?
>
> Here's my first swing at doing the above. It seems to work ok. Let me
> know if it looks reasonable and I'll submit it as a proper patch.
>
> https://git.linaro.org/people/john.stultz/android-dev.git/commit/?h=dev/hikey-mainline-WIP&id=383fb6e3abb71b8a721f196120a2e3a9da3315ac
Yes thanks for fixing it, looks good to me:
Acked-by: Tony Lindgren <tony@atomide.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Regression: OOPs on boot due to "wlcore: Add support for optional wakeirq"
2018-10-25 17:15 ` John Stultz
2018-10-25 17:31 ` Tony Lindgren
@ 2018-10-25 17:33 ` Kalle Valo
1 sibling, 0 replies; 5+ messages in thread
From: Kalle Valo @ 2018-10-25 17:33 UTC (permalink / raw)
To: John Stultz
Cc: Tony Lindgren, Eyal Reizer, lkml, Anders Roxell, linux-wireless
+ linux-wireless
John Stultz <john.stultz@linaro.org> writes:
> On Thu, Oct 25, 2018 at 10:04 AM, John Stultz <john.stultz@linaro.org> wrote:
>> Hey Tony,
>> In testing linus/master on my hikey board, I'm hitting the following
>> OOPS on bootup:
>>
>> [ 1.870279] Unable to handle kernel read from unreadable memory at
>> virtual address 0000000000000010
>> [ 1.870283] Mem abort info:
>> [ 1.870287] ESR = 0x96000005
>> [ 1.870292] Exception class = DABT (current EL), IL = 32 bits
>> [ 1.870296] SET = 0, FnV = 0
>> [ 1.870299] EA = 0, S1PTW = 0
>> [ 1.870302] Data abort info:
>> [ 1.870306] ISV = 0, ISS = 0x00000005
>> [ 1.870309] CM = 0, WnR = 0
>> [ 1.870312] [0000000000000010] user address but active_mm is swapper
>> [ 1.870318] Internal error: Oops: 96000005 [#1] PREEMPT SMP
>> [ 1.870327] CPU: 0 PID: 5 Comm: kworker/0:0 Not tainted
>> 4.19.0-05129-gb3d1e8e #48
>> [ 1.870331] Hardware name: HiKey Development Board (DT)
>> [ 1.870350] Workqueue: events_freezable mmc_rescan
>> [ 1.870358] pstate: 60400005 (nZCv daif +PAN -UAO)
>> [ 1.870366] pc : wl1271_probe+0x210/0x350
>> [ 1.870371] lr : wl1271_probe+0x210/0x350
>> [ 1.870374] sp : ffffff80080739b0
>> [ 1.870377] x29: ffffff80080739b0 x28: 0000000000000000
>> [ 1.870384] x27: 0000000000000000 x26: 0000000000000000
>> [ 1.870391] x25: 0000000000000036 x24: ffffffc074ecb598
>> [ 1.870398] x23: ffffffc07ffdce78 x22: ffffffc0744ed808
>> [ 1.870404] x21: ffffffc074ecbb98 x20: ffffff8008ff9000
>> [ 1.870411] x19: ffffffc0744ed800 x18: ffffff8008ff9a48
>> [ 1.870418] x17: 0000000000000000 x16: 0000000000000000
>> [ 1.870425] x15: ffffffc074ecb503 x14: ffffffffffffffff
>> [ 1.870431] x13: ffffffc074ecb502 x12: 0000000000000030
>> [ 1.870438] x11: 0101010101010101 x10: 0000000000000040
>> [ 1.870444] x9 : ffffffc075400248 x8 : ffffffc075400270
>> [ 1.870451] x7 : 0000000000000000 x6 : 0000000000000000
>> [ 1.870457] x5 : 0000000000000000 x4 : 0000000000000000
>> [ 1.870463] x3 : 0000000000000000 x2 : 0000000000000000
>> [ 1.870469] x1 : 0000000000000028 x0 : 0000000000000000
>> [ 1.870477] Process kworker/0:0 (pid: 5, stack limit = 0x(____ptrval____))
>> [ 1.870480] Call trace:
>> [ 1.870485] wl1271_probe+0x210/0x350
>> [ 1.870491] sdio_bus_probe+0x100/0x128
>> [ 1.870500] really_probe+0x1a8/0x2b8
>> [ 1.870506] driver_probe_device+0x58/0x100
>> [ 1.870511] __device_attach_driver+0x94/0xd8
>> [ 1.870517] bus_for_each_drv+0x70/0xc8
>> [ 1.870522] __device_attach+0xe0/0x140
>> [ 1.870527] device_initial_probe+0x10/0x18
>> [ 1.870532] bus_probe_device+0x94/0xa0
>> [ 1.870537] device_add+0x374/0x5b8
>> [ 1.870542] sdio_add_func+0x60/0x88
>> [ 1.870546] mmc_attach_sdio+0x1b0/0x358
>> [ 1.870551] mmc_rescan+0x2cc/0x390
>> [ 1.870558] process_one_work+0x12c/0x320
>> [ 1.870563] worker_thread+0x48/0x458
>> [ 1.870569] kthread+0xf8/0x128
>> [ 1.870575] ret_from_fork+0x10/0x18
>> [ 1.870583] Code: 92400c21 b2760021 a90687a2 97e95bf9 (f9400803)
>> [ 1.870587] ---[ end trace 1e15f81d3c139ca9 ]---
>>
>>
>> I've bisected it down to 3c83dd577c7f ("wlcore: Add support for
>> optional wakeirq").
>>
>> It seems since we don't have a wakeirq value in the dts, the wakeirq
>> value in wl1271_probe() is zero, which then causes trouble in
>> irqd_get_trigger_type(irq_get_irq_data(wakeirq)).
>>
>> Should we check wakeirq before we set the second elements of res[]?
>
> Here's my first swing at doing the above. It seems to work ok. Let me
> know if it looks reasonable and I'll submit it as a proper patch.
>
> https://git.linaro.org/people/john.stultz/android-dev.git/commit/?h=dev/hikey-mainline-WIP&id=383fb6e3abb71b8a721f196120a2e3a9da3315ac
If you submit that remember to CC linux-wireless so that patchwork sees
it.
--
Kalle Valo
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-10-25 17:33 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-10-25 17:04 Regression: OOPs on boot due to "wlcore: Add support for optional wakeirq" John Stultz
2018-10-25 17:15 ` John Stultz
2018-10-25 17:31 ` Tony Lindgren
2018-10-25 17:33 ` Kalle Valo
2018-10-25 17:26 ` Tony Lindgren
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).