| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <CAKdam56QEXY55SYC7YND4BP=tQmiyAa=avVCADTj3Ym6X9cCvg@mail.gmail.com>
Date: Mon, 30 Jul 2012 16:45:10 +0530
From: "Poddar, Sourav" <sourav.poddar@...com>
To: "DebBarma, Tarun Kanti" <tarun.kanti@...com>
Cc: lo <linux-omap@...r.kernel.org>,
Santosh Shilimkar <santosh.shilimkar@...com>,
Benoit Cousson <b-cousson@...com>,
linux-kernel@...r.kernel.org, Felipe Balbi <balbi@...com>
Subject: Re: [GPIO] Crashed when not using
On Mon, Jul 30, 2012 at 3:04 PM, DebBarma, Tarun Kanti
<tarun.kanti@...com> wrote:
> Sourav,
>
> On Mon, Jul 30, 2012 at 2:13 PM, Poddar, Sourav <sourav.poddar@...com> wrote:
>> Hi All,
>>
>> I tried using gpio as an interrupt line for my driver
>> (drivers/staging/iio/light/tsl2x7x_core.c) for omap5.
>> The interrupt line number was directly passed to the driver using
>> device tree. But what I observed
>> is the following crash..
>>
>>
>> [ 1.599273] mousedev: PS/2 mouse device common for all mice
>> [ 1.607513] i2c /dev entries driver
>> [ 1.613739] Driver for 1-wire Dallas network protocol.
>> [ 1.622650] usbcore: registered new interface driver usbhid
>> [ 1.628540] usbhid: USB HID core driver
>> [ 1.633728] Unhandled fault: imprecise external abort (0x1406) at 0x00000000
>> [ 1.641113] Internal error: : 1406 [#1] SMP ARM
>> [ 1.645874] Modules linked in:
>> [ 1.649078] CPU: 0 Not tainted (3.5.0-02045-g0b474d6-dirty #415)
>> [ 1.655761] PC is at _set_gpio_triggering+0x44/0x264
>> [ 1.660980] LR is at gpio_irq_type+0xb8/0x160
>> [ 1.665527] pc : [<c02dbb68>] lr : [<c02dc2dc>] psr: 60000093
>> [ 1.665527] sp : dc851df0 ip : c07a8f00 fp : 00000000
>> [ 1.677581] r10: 00000081 r9 : 00000000 r8 : dc8ffc10
>> [ 1.683074] r7 : 20000093 r6 : 00000000 r5 : 00000001 r4 : fa05b000
>> [ 1.689910] r3 : dc8ffc10 r2 : 00000002 r1 : 00000002 r0 : 00000140
>> [ 1.696746] Flags: nZCv IRQs off FIQs on Mode SVC_32 ISA ARM
>> Segment kernel
>> [ 1.704528] Control: 10c53c7d Table: 8000406a DAC: 00000017
>> [ 1.710540] Process swapper/0 (pid: 1, stack limit = 0xdc8502f8)
>> [ 1.716857] Stack: (0xdc851df0 to 0xdc852000)
>> [ 1.721405] 1de0: 00000002
>> c0731280 dc8ffc6c 20000093
>> [ 1.730010] 1e00: c02dc224 c0731280 dc8d4980 00000000 c07a8e50
>> 00000141 00000002 c00a37b8
>> [ 1.738586] 1e20: 00002002 c0731280 dc8d4980 00000141 c07312d4
>> c07312b4 40000013 c00a3d7c
>> [ 1.747161] 1e40: 00000141 c06728c0 00000000 c0090578 c247f000
>> c00a2d5c c0412258 00000141
>> [ 1.755737] 1e60: c0731280 c0537818 dc8d4980 c00a3fe4 c07bf6a0
>> 00002002 c071bed8 c247f000
>> [ 1.764312] 1e80: dc9df000 00000004 c07bf6a0 dc9df020 c247f3c0
>> c04ea344 c06728c0 c247f000
>> [ 1.772888] 1ea0: 00000000 09090578 c07bf6a0 dc9df004 dc9df000
>> c04ea194 00000091 c071bed8
>> [ 1.781494] 1ec0: c06f89f4 c03e13c0 c03e1314 dc9df020 c0d1afd8
>> dc9df054 c07bf60c c03272ac
>> [ 1.790069] 1ee0: dc9df020 c07bf60c dc9df054 00000000 00000091
>> c03274c0 c07bf60c dc851f08
>> [ 1.798645] 1f00: c032742c c0325b10 dc8f26a8 dc9ef790 00000000
>> c07bf60c c07bcea8 c243d6c0
>> [ 1.807220] 1f20: 00000000 c0326240 c0672958 c02bce08 c07bf5e4
>> c07bf60c c03e12a0 00000000
>> [ 1.815795] 1f40: 00000091 c071bed8 c06f89f4 c0327ab0 c07bf5e4
>> 00000007 c07c8c40 00000000
>> [ 1.824401] 1f60: 00000091 c03e1738 00000000 dc850000 00000007
>> c0008648 00000000 c112c9f0
>> [ 1.832977] 1f80: c06a6d58 c06f89f4 00000001 60000013 c06694c8
>> 00000000 00000006 00000006
>> [ 1.841552] 1fa0: 60000013 c0700a20 00000007 c07c8c40 c06d020c
>> 00000091 c071bed8 c0700a28
>> [ 1.850128] 1fc0: 00000000 c06d0380 00000006 00000006 c06d020c
>> 00000000 00000000 c06d028c
>> [ 1.858703] 1fe0: c001548c 00000013 00000000 00000000 00000000
>> c001548c 3f3f3f3f 3f3f3f3f
>> [ 1.867309] [<c02dbb68>] (_set_gpio_triggering+0x44/0x264) from
>> [<c02dc224>] (gpio_irq_type+0x0/0x160)
>> [ 1.877075] [<c02dc224>] (gpio_irq_type+0x0/0x160) from [<00002002>] (0x2002)
>> [ 1.884552] Code: e3120008 11816006 01c66001 e7846000 (e593c0fc)
>> [ 1.890960] ------------[ cut here ]------------
>> [ 1.895812] WARNING: at arch/arm/mach-omap2/omap_l3_noc.c:113
>> l3_interrupt_handler+0x184/0x1bc()
>> [ 1.905029] L3 custom error: MASTER:MPU TARGET:L4 PER2
>> [ 1.910430] Modules linked in:
>> [ 1.913635] [<c001bd5c>] (unwind_backtrace+0x0/0xf4) from
>> [<c00414c4>] (warn_slowpath_common+0x4c/0x64)
>> [ 1.923492] [<c00414c4>] (warn_slowpath_common+0x4c/0x64) from
>> [<c0041570>] (warn_slowpath_fmt+0x30/0x40)
>> [ 1.933563] [<c0041570>] (warn_slowpath_fmt+0x30/0x40) from
>> [<c0035de4>] (l3_interrupt_handler+0x184/0x1bc)
>> [ 1.943786] [<c0035de4>] (l3_interrupt_handler+0x184/0x1bc) from
>> [<c00a28b8>] (handle_irq_event_percpu+0x64/0x24c)
>> [ 1.954650] [<c00a28b8>] (handle_irq_event_percpu+0x64/0x24c) from
>> [<c00a2adc>] (handle_irq_event+0x3c/0x5c)
>> [ 1.964965] [<c00a2adc>] (handle_irq_event+0x3c/0x5c) from
>> [<c00a56a8>] (handle_fasteoi_irq+0x98/0x13c)
>> [ 1.974822] [<c00a56a8>] (handle_fasteoi_irq+0x98/0x13c) from
>> [<c00a235c>] (generic_handle_irq+0x34/0x44)
>> [ 1.984863] [<c00a235c>] (generic_handle_irq+0x34/0x44) from
>> [<c00153a0>] (handle_IRQ+0x4c/0xac)
>> [ 1.994110] [<c00153a0>] (handle_IRQ+0x4c/0xac) from [<c0008480>]
>> (gic_handle_irq+0x2c/0x60)
>> [ 2.002960] [<c0008480>] (gic_handle_irq+0x2c/0x60) from
>> [<c04f26e4>] (__irq_svc+0x44/0x60)
>> [ 2.011718] Exception stack(0xdc851c48 to 0xdc851c90)
>> [ 2.017028] 1c40: c07a87a0 00000000 dc851c80
>> 00000000 c0743f80 dc851da8
>> [ 2.025604] 1c60: 00000001 00000001 dc851cd7 00000000 c02dbb6a
>> c02dbb6c ffffffff dc851c90
>> [ 2.034179] 1c80: c04f2440 c04f2444 60000113 ffffffff
>> [ 2.039489] [<c04f26e4>] (__irq_svc+0x44/0x60) from [<c04f2444>]
>> (_raw_spin_unlock_irq+0x28/0x2c)
>> [ 2.048797] [<c04f2444>] (_raw_spin_unlock_irq+0x28/0x2c) from
>> [<c001871c>] (die+0xe0/0x324)
>> [ 2.057647] [<c001871c>] (die+0xe0/0x324) from [<c0008444>]
>> (do_DataAbort+0x88/0x98)
>> [ 2.065765] [<c0008444>] (do_DataAbort+0x88/0x98) from [<c04f2664>]
>> (__dabt_svc+0x44/0x80)
>> [ 2.074462] Exception stack(0xdc851da8 to 0xdc851df0)
>> [ 2.079742] 1da0: 00000140 00000002 00000002
>> dc8ffc10 fa05b000 00000001
>> [ 2.088317] 1dc0: 00000000 20000093 dc8ffc10 00000000 00000081
>> 00000000 c07a8f00 dc851df0
>> [ 2.096893] 1de0: c02dc2dc c02dbb68 60000093 ffffffff
>> [ 2.102203] [<c04f2664>] (__dabt_svc+0x44/0x80) from [<c02dbb68>]
>> (_set_gpio_triggering+0x44/0x264)
>> [ 2.111724] [<c02dbb68>] (_set_gpio_triggering+0x44/0x264) from
>> [<c02dc224>] (gpio_irq_type+0x0/0x160)
>> [ 2.121490] [<c02dc224>] (gpio_irq_type+0x0/0x160) from [<00002002>] (0x2002)
>> [ 2.128967] ---[ end trace 8b4b88305e6b2aff ]---
>> [ 2.133911] ---[ end trace 8b4b88305e6b2b00 ]---
>> [ 2.138763] Kernel panic - not syncing: Attempted to kill init!
>> exitcode=0x0000000b
>> [ 2.138763]
>> [ 2.148376] CPU1: stopping
>> [ 2.151214] [<c001bd5c>] (unwind_backtrace+0x0/0xf4) from
>> [<c0019c74>] (handle_IPI+0x130/0x15c)
>> [ 2.160339] [<c0019c74>] (handle_IPI+0x130/0x15c) from [<c00084ac>]
>> (gic_handle_irq+0x58/0x60)
>> [ 2.169372] [<c00084ac>] (gic_handle_irq+0x58/0x60) from
>> [<c04f26e4>] (__irq_svc+0x44/0x60)
>> [ 2.178131] Exception stack(0xdc869e10 to 0xdc869e58)
>> [ 2.183410] 9e00: 00000000
>> 00000000 00000002 60000113
>> [ 2.191986] 9e20: 00000000 dc868000 c07970e4 00000000 00000000
>> 00000001 00000002 00000000
>> [ 2.200561] 9e40: 00000001 dc869e58 c00936f4 c0093704 60000113 ffffffff
>> [ 2.207489] [<c04f26e4>] (__irq_svc+0x44/0x60) from [<c0093704>]
>> (lock_acquire+0xac/0x104)
>> [ 2.216156] [<c0093704>] (lock_acquire+0xac/0x104) from
>> [<c007a8bc>] (rebalance_domains+0x5c/0x1a8)
>> [ 2.225646] [<c007a8bc>] (rebalance_domains+0x5c/0x1a8) from
>> [<c007ae84>] (run_rebalance_domains+0x44/0x14c)
>> [ 2.235961] [<c007ae84>] (run_rebalance_domains+0x44/0x14c) from
>> [<c004a864>] (__do_softirq+0xb4/0x214)
>> [ 2.245819] [<c004a864>] (__do_softirq+0xb4/0x214) from
>> [<c004aba0>] (irq_exit+0x90/0x98)
>> [ 2.254394] [<c004aba0>] (irq_exit+0x90/0x98) from [<c0019c48>]
>> (handle_IPI+0x104/0x15c)
>> [ 2.262878] [<c0019c48>] (handle_IPI+0x104/0x15c) from [<c00084ac>]
>> (gic_handle_irq+0x58/0x60)
>> [ 2.271911] [<c00084ac>] (gic_handle_irq+0x58/0x60) from
>> [<c04f26e4>] (__irq_svc+0x44/0x60)
>> [ 2.280670] Exception stack(0xdc869f88 to 0xdc869fd0)
>> [ 2.285949] 9f80: c07a87a0 c0022a40 00000000
>> 00000000 dc868000 c07c8d08
>> [ 2.294525] 9fa0: c04fe078 c0743d70 00000000 410fc0f0 c0743fa0
>> 00000000 00000000 dc869fd0
>>
>>
>> The device tree data for acquiring the above GPIO interrupt line looks
>> like this.
>>
>> +++ linux-omap-storage/arch/arm/boot/dts/omap5-evm.dts 2012-07-30
>> 14:11:08.931694001 +0530
>> @@ -42,7 +42,8 @@
>> tsl2771@39 {
>> compatible = "taos,tsl2771";
>> reg = <0x39>;
>> + interrupt-parent = <&gpio5>;
>> + interrupts = <21>; /* gpio line 149 */
>> };
>> };
>>
>> Note: using "gpio_request_one" in the driver solves the issue.
>> Is using this api in the driver required?
>> Any pointer on the above crash?
>
Hi Tarun,
> Any user/client driver of GPIO is supposed to go through
> gpio_request() API so that module clock
> is enabled correctly. Overriding of APIs would put the power
> management state machine in jeopardy.
> --
I tried putting "pm_runtime_get_sync" in gpio_irq_type api where the kernel
is crashing and the crash is no longer observed. So indeed, its about
enabling clocks.
One doubt: Can't we put runtime apis in "gpio_irq_type" and eliminate
the use of
"gpio_request_one"??
> Tarun
>
>>
>> ~Sourav
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists