Re: [v4.5-rc3 REGRESSION] rcar_sound-related WARN_ON()

[Magnus Damm]

Hi Laurent,

On Fri, Feb 19, 2016 at 1:07 AM, Laurent Pinchart
 wrote:
> Hi Magnus,
>
> On Friday 19 February 2016 01:02:18 Magnus Damm wrote:
>> On Fri, Feb 19, 2016 at 12:33 AM, Laurent Pinchart wrote:
>> > On Thursday 18 February 2016 10:17:16 Geert Uytterhoeven wrote:
>> >> On Thu, Feb 18, 2016 at 10:01 AM, Kuninori Morimoto wrote:
>> >>  I tried this patch, but It didn't use this printk().
>> >>  Am I wrong ?
>> >> 
>> >>  -
>> >>  diff --git a/drivers/regulator/da9210-regulator.c
>> >>  b/drivers/regulator/da9210-regulator.c index 01c0e37..85c1166 100644
>> >>  --- a/drivers/regulator/da9210-regulator.c
>> >>  +++ b/drivers/regulator/da9210-regulator.c
>> >>  @@ -167,6 +167,8 @@ static irqreturn_t da9210_irq_handler(int irq,
>> >>  void *data)
>> >> 
>> >>  goto error_i2c;
>> >> 
>> >>  ret = IRQ_HANDLED;
>> >> 
>> >>  +   } else if (val != handled) {
>> >>  +   printk("---val %x : %x\n", val, handled);
>> >> 
>> >>  }
>> >> >>>
>> >> >>> That's what I meant.
>> >> >>> Do you still see the "irq 311: nobody cared"?
>> >> >>> Do you have the DA9210 driver enabled?
>> >> >
>> >> > You can reproduce it by
>> >> >
>> >> > > git checkout git checkout renesas-drivers-2016-02-09-v4.5-rc3
>> >> > > cp ${Laurent's .config} ${LINUX}
>> >>
>> >> I don't have Laurent's config, and only remote access to Lager.
>> >>
>> >> > and boot, and wait few second
>> >>
>> >> Da9210 and da9063 share the same interrupt. Perhaps da9063 is keeping
>> >> the interrupt line asserted?
>> >>
>> >> Do you have the DA9063 driver enabled?
>> >
>> > I had CONFIG_REGULATOR_DA9210 enabled but CONFIG_REGULATOR_DA9063
>> > disabled.
>> > After enabling CONFIG_REGULATOR_DA9063 the unhandled interrupt problem
>> > goes
>> > away.
>> >
>> > I however got the following different (and totally unrelated as far as I
>> > can see) warning once:
>> >
>> > [  310.252577] [ cut here ]
>> > [  310.257226] WARNING: CPU: 3 PID: 608 at
>> > /home/laurent/src/iob/renesas/linux/net/ipv4/af_inet.c:155
>> > inet_sock_destruct+0x188/0x1d8()
>> > [  310.269088] Modules linked in: mmc_block rcar_jpu v4l2_mem2mem
>> > sata_rcar
>> > libata rcar_vin scsi_mod sh_mobile_sdhi soc_camera sh_mmcif tmio_mmc_core
>> > soc_mediabus mmc_core videobuf_core soc_scale_crop(Pa
>> > [  310.294680] CPU: 3 PID: 608 Comm: kworker/3:1H Tainted: P
>> > 4.5.0-rc3-00463-gd57d2d31ebee #581
>> > [  310.304431] Hardware name: Generic R8A7790 (Flattened Device Tree)
>> > [  310.310625] Workqueue: rpciod xprt_autoclose
>> > [  310.314908] Backtrace:
>> > [  310.317380] [] (dump_backtrace) from []
>> > (show_stack+0x20/0x24)
>> > [  310.324959]  r6:c05be48c r5: r4:6013 r3:e9a62000
>> > [  310.330669] [] (show_stack) from []
>> > (dump_stack+0x8c/0xac)
>> > [  310.337906] [] (dump_stack) from []
>> > (warn_slowpath_common+0x88/0xc4)
>> > [  310.346006]  r5:009b r4:
>> > [  310.349606] [] (warn_slowpath_common) from []
>> > (warn_slowpath_null+0x2c/0x34)
>> > [  310.358400]  r8:e9a248d8 r7:ea375364 r6:e9a248c4 r5:e9a248d8
>> > r4:e9a247c0
>> > [  310.365156] [] (warn_slowpath_null) from []
>> > (inet_sock_destruct+0x188/0x1d8)
>> > [  310.373958] [] (inet_sock_destruct) from []
>> > (sk_destruct+0x28/0x118)
>> > [  310.382057]  r6:e9d593c0 r5:e9a248d8 r4:e9a247c0 r3:c0405b9c
>> > [  310.387763] [] (sk_destruct) from []
>> > (__sk_free+0x34/0xc0)
>> > [  310.394994]  r5:e9a248d8 r4:e9a247c0
>> > [  310.398595] [] (__sk_free) from []
>> > (sk_free+0x44/0x48) [  310.405474]  r4:e9a247c0 r3:e9a2486c
>> > [  310.409074] [] (sk_free) from []
>> > (sk_common_release+0xf0/0xfc)
>> > [  310.416662] [] (sk_common_release) from []
>> > (udp_lib_close+0x10/0x14)
>> > [  310.424761]  r5:e9d593c0 r4:e9a247c0
>> > [  310.428367] [] (udp_lib_close) from []
>> > (inet_release+0x54/0x80)
>> > [  310.436041] [] (inet_release) from []
>> > (sock_release+0x30/0xac)
>> > [  310.443620]  r5: r4:e9d593c0
>> > [  310.447224] [] (sock_release) from []
>> > (xs_reset_transport+0xc4/0x138)
>> > [  310.455411]  r5:e9a247c0 r4:ea375000
>> > [  310.459012] [] (xs_reset_transport) from []
>> > (xs_close+0x1c/0x30)
>> > [  310.466764]  r8:ff7c4300 r7:ea375228 r6:ea375000 r5:ea375278
>> > r4:ea375000
>> > r3:c04289d4
>> > [  310.474568] [] (xs_close) from []
>> > (xprt_autoclose+0x40/0x74)
>> > [  310.481972]  r4:ea375244 r3:c04289d4
>> > [  310.485579] [] (xprt_autoclose) from []
>> > (process_one_work+0x170/0x430)
>> > [  310.493853]  r7:c06a4878 r6:e876da00 r5:ea282880 r4:ea375244
>> > [  310.499558] [] (process_one_work) from []
>> > (worker_thread+0x3c/0x55c)
>> > [  310.507658]  r10:e876da00 r9:c0660100 r8:0008 r7:ea282898
>> > r6:e876da18 r5:e876da00
>> > [  310.515544]  r4:ea282880
>> > [  310.518096] [] (worker_thread) from []
>> > (kthread+0xf4/0x114)
>> > 

Re: [v4.5-rc3 REGRESSION] rcar_sound-related WARN_ON()

[Laurent Pinchart]

Hi Magnus,

On Friday 19 February 2016 01:02:18 Magnus Damm wrote:
> On Fri, Feb 19, 2016 at 12:33 AM, Laurent Pinchart wrote:
> > On Thursday 18 February 2016 10:17:16 Geert Uytterhoeven wrote:
> >> On Thu, Feb 18, 2016 at 10:01 AM, Kuninori Morimoto wrote:
> >>  I tried this patch, but It didn't use this printk().
> >>  Am I wrong ?
> >>  
> >>  -
> >>  diff --git a/drivers/regulator/da9210-regulator.c
> >>  b/drivers/regulator/da9210-regulator.c index 01c0e37..85c1166 100644
> >>  --- a/drivers/regulator/da9210-regulator.c
> >>  +++ b/drivers/regulator/da9210-regulator.c
> >>  @@ -167,6 +167,8 @@ static irqreturn_t da9210_irq_handler(int irq,
> >>  void *data)
> >>  
> >>  goto error_i2c;
> >>  
> >>  ret = IRQ_HANDLED;
> >>  
> >>  +   } else if (val != handled) {
> >>  +   printk("---val %x : %x\n", val, handled);
> >>  
> >>  }
> >> >>> 
> >> >>> That's what I meant.
> >> >>> Do you still see the "irq 311: nobody cared"?
> >> >>> Do you have the DA9210 driver enabled?
> >> > 
> >> > You can reproduce it by
> >> > 
> >> > > git checkout git checkout renesas-drivers-2016-02-09-v4.5-rc3
> >> > > cp ${Laurent's .config} ${LINUX}
> >> 
> >> I don't have Laurent's config, and only remote access to Lager.
> >> 
> >> > and boot, and wait few second
> >> 
> >> Da9210 and da9063 share the same interrupt. Perhaps da9063 is keeping
> >> the interrupt line asserted?
> >> 
> >> Do you have the DA9063 driver enabled?
> > 
> > I had CONFIG_REGULATOR_DA9210 enabled but CONFIG_REGULATOR_DA9063
> > disabled.
> > After enabling CONFIG_REGULATOR_DA9063 the unhandled interrupt problem
> > goes
> > away.
> > 
> > I however got the following different (and totally unrelated as far as I
> > can see) warning once:
> > 
> > [  310.252577] [ cut here ]
> > [  310.257226] WARNING: CPU: 3 PID: 608 at
> > /home/laurent/src/iob/renesas/linux/net/ipv4/af_inet.c:155
> > inet_sock_destruct+0x188/0x1d8()
> > [  310.269088] Modules linked in: mmc_block rcar_jpu v4l2_mem2mem
> > sata_rcar
> > libata rcar_vin scsi_mod sh_mobile_sdhi soc_camera sh_mmcif tmio_mmc_core
> > soc_mediabus mmc_core videobuf_core soc_scale_crop(Pa
> > [  310.294680] CPU: 3 PID: 608 Comm: kworker/3:1H Tainted: P
> > 4.5.0-rc3-00463-gd57d2d31ebee #581
> > [  310.304431] Hardware name: Generic R8A7790 (Flattened Device Tree)
> > [  310.310625] Workqueue: rpciod xprt_autoclose
> > [  310.314908] Backtrace:
> > [  310.317380] [] (dump_backtrace) from []
> > (show_stack+0x20/0x24)
> > [  310.324959]  r6:c05be48c r5: r4:6013 r3:e9a62000
> > [  310.330669] [] (show_stack) from []
> > (dump_stack+0x8c/0xac)
> > [  310.337906] [] (dump_stack) from []
> > (warn_slowpath_common+0x88/0xc4)
> > [  310.346006]  r5:009b r4:
> > [  310.349606] [] (warn_slowpath_common) from []
> > (warn_slowpath_null+0x2c/0x34)
> > [  310.358400]  r8:e9a248d8 r7:ea375364 r6:e9a248c4 r5:e9a248d8
> > r4:e9a247c0
> > [  310.365156] [] (warn_slowpath_null) from []
> > (inet_sock_destruct+0x188/0x1d8)
> > [  310.373958] [] (inet_sock_destruct) from []
> > (sk_destruct+0x28/0x118)
> > [  310.382057]  r6:e9d593c0 r5:e9a248d8 r4:e9a247c0 r3:c0405b9c
> > [  310.387763] [] (sk_destruct) from []
> > (__sk_free+0x34/0xc0)
> > [  310.394994]  r5:e9a248d8 r4:e9a247c0
> > [  310.398595] [] (__sk_free) from []
> > (sk_free+0x44/0x48) [  310.405474]  r4:e9a247c0 r3:e9a2486c
> > [  310.409074] [] (sk_free) from []
> > (sk_common_release+0xf0/0xfc)
> > [  310.416662] [] (sk_common_release) from []
> > (udp_lib_close+0x10/0x14)
> > [  310.424761]  r5:e9d593c0 r4:e9a247c0
> > [  310.428367] [] (udp_lib_close) from []
> > (inet_release+0x54/0x80)
> > [  310.436041] [] (inet_release) from []
> > (sock_release+0x30/0xac)
> > [  310.443620]  r5: r4:e9d593c0
> > [  310.447224] [] (sock_release) from []
> > (xs_reset_transport+0xc4/0x138)
> > [  310.455411]  r5:e9a247c0 r4:ea375000
> > [  310.459012] [] (xs_reset_transport) from []
> > (xs_close+0x1c/0x30)
> > [  310.466764]  r8:ff7c4300 r7:ea375228 r6:ea375000 r5:ea375278
> > r4:ea375000
> > r3:c04289d4
> > [  310.474568] [] (xs_close) from []
> > (xprt_autoclose+0x40/0x74)
> > [  310.481972]  r4:ea375244 r3:c04289d4
> > [  310.485579] [] (xprt_autoclose) from []
> > (process_one_work+0x170/0x430)
> > [  310.493853]  r7:c06a4878 r6:e876da00 r5:ea282880 r4:ea375244
> > [  310.499558] [] (process_one_work) from []
> > (worker_thread+0x3c/0x55c)
> > [  310.507658]  r10:e876da00 r9:c0660100 r8:0008 r7:ea282898
> > r6:e876da18 r5:e876da00
> > [  310.515544]  r4:ea282880
> > [  310.518096] [] (worker_thread) from []
> > (kthread+0xf4/0x114)
> > [  310.525414]  r10: r9: r8: r7:c0042874
> > r6:ea282880 r5:
> > [  310.533301]  r4:e9948d80
> > [  310.535850] [] (kthread) from []
> > (ret_from_fork+0x14/0x2c)
> 

Re: [v4.5-rc3 REGRESSION] rcar_sound-related WARN_ON()

[Laurent Pinchart]

Hi Geert,

On Thursday 18 February 2016 10:17:16 Geert Uytterhoeven wrote:
> On Thu, Feb 18, 2016 at 10:01 AM, Kuninori Morimoto wrote:
>  I tried this patch, but It didn't use this printk().
>  Am I wrong ?
>  
>  -
>  diff --git a/drivers/regulator/da9210-regulator.c
>  b/drivers/regulator/da9210-regulator.c index 01c0e37..85c1166 100644
>  --- a/drivers/regulator/da9210-regulator.c
>  +++ b/drivers/regulator/da9210-regulator.c
>  @@ -167,6 +167,8 @@ static irqreturn_t da9210_irq_handler(int irq,
>  void *data)
>  goto error_i2c;
>  
>  ret = IRQ_HANDLED;
>  +   } else if (val != handled) {
>  +   printk("---val %x : %x\n", val, handled);
>  }
> >>> 
> >>> That's what I meant.
> >>> Do you still see the "irq 311: nobody cared"?
> >>> Do you have the DA9210 driver enabled?
> > 
> > You can reproduce it by
> > 
> > > git checkout git checkout renesas-drivers-2016-02-09-v4.5-rc3
> > > cp ${Laurent's .config} ${LINUX}
> 
> I don't have Laurent's config, and only remote access to Lager.
> 
> > and boot, and wait few second
> 
> Da9210 and da9063 share the same interrupt. Perhaps da9063 is keeping
> the interrupt line asserted?
> 
> Do you have the DA9063 driver enabled?

I had CONFIG_REGULATOR_DA9210 enabled but CONFIG_REGULATOR_DA9063 disabled. 
After enabling CONFIG_REGULATOR_DA9063 the unhandled interrupt problem goes 
away.

I however got the following different (and totally unrelated as far as I can 
see) warning once:

[  310.252577] [ cut here ]
[  310.257226] WARNING: CPU: 3 PID: 608 at 
/home/laurent/src/iob/renesas/linux/net/ipv4/af_inet.c:155 
inet_sock_destruct+0x188/0x1d8()
[  310.269088] Modules linked in: mmc_block rcar_jpu v4l2_mem2mem sata_rcar 
libata rcar_vin scsi_mod sh_mobile_sdhi soc_camera sh_mmcif tmio_mmc_core 
soc_mediabus mmc_core videobuf_core soc_scale_crop(Pa
[  310.294680] CPU: 3 PID: 608 Comm: kworker/3:1H Tainted: P
4.5.0-rc3-00463-gd57d2d31ebee #581
[  310.304431] Hardware name: Generic R8A7790 (Flattened Device Tree)
[  310.310625] Workqueue: rpciod xprt_autoclose
[  310.314908] Backtrace: 
[  310.317380] [] (dump_backtrace) from [] 
(show_stack+0x20/0x24)
[  310.324959]  r6:c05be48c r5: r4:6013 r3:e9a62000
[  310.330669] [] (show_stack) from [] 
(dump_stack+0x8c/0xac)
[  310.337906] [] (dump_stack) from [] 
(warn_slowpath_common+0x88/0xc4)
[  310.346006]  r5:009b r4:
[  310.349606] [] (warn_slowpath_common) from [] 
(warn_slowpath_null+0x2c/0x34)
[  310.358400]  r8:e9a248d8 r7:ea375364 r6:e9a248c4 r5:e9a248d8 r4:e9a247c0
[  310.365156] [] (warn_slowpath_null) from [] 
(inet_sock_destruct+0x188/0x1d8)
[  310.373958] [] (inet_sock_destruct) from [] 
(sk_destruct+0x28/0x118)
[  310.382057]  r6:e9d593c0 r5:e9a248d8 r4:e9a247c0 r3:c0405b9c
[  310.387763] [] (sk_destruct) from [] 
(__sk_free+0x34/0xc0)
[  310.394994]  r5:e9a248d8 r4:e9a247c0
[  310.398595] [] (__sk_free) from [] (sk_free+0x44/0x48)
[  310.405474]  r4:e9a247c0 r3:e9a2486c
[  310.409074] [] (sk_free) from [] 
(sk_common_release+0xf0/0xfc)
[  310.416662] [] (sk_common_release) from [] 
(udp_lib_close+0x10/0x14)
[  310.424761]  r5:e9d593c0 r4:e9a247c0
[  310.428367] [] (udp_lib_close) from [] 
(inet_release+0x54/0x80)
[  310.436041] [] (inet_release) from [] 
(sock_release+0x30/0xac)
[  310.443620]  r5: r4:e9d593c0
[  310.447224] [] (sock_release) from [] 
(xs_reset_transport+0xc4/0x138)
[  310.455411]  r5:e9a247c0 r4:ea375000
[  310.459012] [] (xs_reset_transport) from [] 
(xs_close+0x1c/0x30)
[  310.466764]  r8:ff7c4300 r7:ea375228 r6:ea375000 r5:ea375278 r4:ea375000 
r3:c04289d4
[  310.474568] [] (xs_close) from [] 
(xprt_autoclose+0x40/0x74)
[  310.481972]  r4:ea375244 r3:c04289d4
[  310.485579] [] (xprt_autoclose) from [] 
(process_one_work+0x170/0x430)
[  310.493853]  r7:c06a4878 r6:e876da00 r5:ea282880 r4:ea375244
[  310.499558] [] (process_one_work) from [] 
(worker_thread+0x3c/0x55c)
[  310.507658]  r10:e876da00 r9:c0660100 r8:0008 r7:ea282898 r6:e876da18 
r5:e876da00
[  310.515544]  r4:ea282880
[  310.518096] [] (worker_thread) from [] 
(kthread+0xf4/0x114)
[  310.525414]  r10: r9: r8: r7:c0042874 r6:ea282880 
r5:
[  310.533301]  r4:e9948d80
[  310.535850] [] (kthread) from [] 
(ret_from_fork+0x14/0x2c)
[  310.543081]  r7: r6: r5:c0048848 r4:e9948d80
[  310.548803] ---[ end trace 02711a72b6e0a70c ]---

-- 
Regards,

Laurent Pinchart

Re: [v4.5-rc3 REGRESSION] rcar_sound-related WARN_ON()

[Geert Uytterhoeven]

Hi Morimoto-san,

On Thu, Feb 18, 2016 at 10:01 AM, Kuninori Morimoto
 wrote:
>> > > I tried this patch, but It didn't use this printk().
>> > > Am I wrong ?
>> > >
>> > > -
>> > > diff --git a/drivers/regulator/da9210-regulator.c 
>> > > b/drivers/regulator/da9210-regulator.c
>> > > index 01c0e37..85c1166 100644
>> > > --- a/drivers/regulator/da9210-regulator.c
>> > > +++ b/drivers/regulator/da9210-regulator.c
>> > > @@ -167,6 +167,8 @@ static irqreturn_t da9210_irq_handler(int irq, void 
>> > > *data)
>> > > goto error_i2c;
>> > >
>> > > ret = IRQ_HANDLED;
>> > > +   } else if (val != handled) {
>> > > +   printk("---val %x : %x\n", val, handled);
>> > > }
>> >
>> > That's what I meant.
>> > Do you still see the "irq 311: nobody cared"?
>> > Do you have the DA9210 driver enabled?
>
> You can reproduce it by
>
> > git checkout git checkout renesas-drivers-2016-02-09-v4.5-rc3
> > cp ${Laurent's .config} ${LINUX}

I don't have Laurent's config, and only remote access to Lager.

> and boot, and wait few second

Da9210 and da9063 share the same interrupt. Perhaps da9063 is keeping
the interrupt line asserted?

Do you have the DA9063 driver enabled?

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds

Re: [v4.5-rc3 REGRESSION] rcar_sound-related WARN_ON()

[Kuninori Morimoto]

Hi Geert, again

> > > I tried this patch, but It didn't use this printk().
> > > Am I wrong ?
> > >
> > > -
> > > diff --git a/drivers/regulator/da9210-regulator.c 
> > > b/drivers/regulator/da9210-regulator.c
> > > index 01c0e37..85c1166 100644
> > > --- a/drivers/regulator/da9210-regulator.c
> > > +++ b/drivers/regulator/da9210-regulator.c
> > > @@ -167,6 +167,8 @@ static irqreturn_t da9210_irq_handler(int irq, void 
> > > *data)
> > > goto error_i2c;
> > >
> > > ret = IRQ_HANDLED;
> > > +   } else if (val != handled) {
> > > +   printk("---val %x : %x\n", val, handled);
> > > }
> > 
> > That's what I meant.
> > Do you still see the "irq 311: nobody cared"?
> > Do you have the DA9210 driver enabled?

You can reproduce it by

> git checkout git checkout renesas-drivers-2016-02-09-v4.5-rc3
> cp ${Laurent's .config} ${LINUX}

and boot, and wait few second


- log ---
...
[2.625828] NET: Registered protocol family 17
[2.630356] Registering SWP/SWPB emulation handler
[2.635580] da9210 7-0068: I2C error : -6
[2.637960] [drm] Supports vblank timestamp caching Rev 2 (21.10.2013).
[2.637963] [drm] No driver support for vblank timestamp query.
[2.638840] [drm] Initialized rcar-du 1.0.0 20130110 on minor 0
[2.640235] input: keyboard as /devices/platform/keyboard/input/input0
[2.640559] hctosys: unable to open rtc device (rtc0)
[2.727737] Micrel KSZ8041RNLI ee70.etherne:01: attached PHY driver 
[Micrel KSZ8041RNLI] (mii_bus:phy_addr=ee70.etherne:01, irq=312)
[3.122720] random: nonblocking pool is initialized
[5.557550] sh-eth ee70.ethernet eth0: Link is Up - 100Mbps/Full - flow 
control rx/tx
[5.595876] Sending DHCP requests ., OK
[5.705886] IP-Config: Got DHCP answer from 192.168.10.77, my address is 
192.168.10.107
[5.713983] IP-Config: Complete:
[5.717234]  device=eth0, hwaddr=2e:09:0a:00:6e:3a, 
ipaddr=192.168.10.107, mask=255.255.255.0, gw=192.168.10.77
[5.727689]  host=192.168.10.107, domain=example.org, nis-domain=(none)
[5.734656]  bootserver=192.168.10.77, rootserver=192.168.10.77, 
rootpath=/tftpboot/rootfs_arm nameserver0=192.168.10.77
[5.746285] SDHI0 Vcc: disabling
[5.749518] SDHI2 Vcc: disabling
[5.752749] SDHI0 VccQ: disabling
[5.756079] SDHI2 VccQ: disabling
[5.759857] sh-sci e6e6.serial: dma_request_slave_channel_compat failed
[5.766852] sh-sci e6e6.serial: dma_request_slave_channel_compat failed
[5.779229] VFS: Mounted root (nfs filesystem) on device 0:15.
[5.785554] devtmpfs: mounted
[5.789837] Freeing unused kernel memory: 2048K (c0a0 - c0c0)
Starting logging: OK
Initializing random number generator... done.
Starting network...
ip: RTNETLINK answers: File exists
# 
# 
# 
# 
# [   59.014541] irq 311: nobody cared (try booting with the "irqpoll" option)
[   59.021353] CPU: 0 PID: 411 Comm: irq/311-da9210 Not tainted 
4.5.0-rc3-01980-ga34785b-dirty #620
[   59.030150] Hardware name: Generic R8A7790 (Flattened Device Tree)
[   59.036333] Backtrace: 
[   59.038808] [] (dump_backtrace) from [] 
(show_stack+0x20/0x24)
[   59.046386]  r7:0137 r6:6193 r5:c0c1d96c r4:
[   59.052094] [] (show_stack) from [] 
(dump_stack+0xa0/0xb4)
[   59.059333] [] (dump_stack) from [] 
(__report_bad_irq+0x38/0xd0)
[   59.067084]  r7:0137 r6:0137 r5: r4:ed00
[   59.072788] [] (__report_bad_irq) from [] 
(note_interrupt+0x280/0x2d0)
[   59.081060]  r7:0137 r6:0137 r5: r4:ed00
[   59.086763] [] (note_interrupt) from [] 
(handle_irq_event_percpu+0x1d0/0x238)
[   59.095644]  r10:2088 r9:0002 r8:e99f6000 r7:0002 r6:0137 
r5:ed60
[   59.103526]  r4: r3:
[   59.107126] [] (handle_irq_event_percpu) from [] 
(handle_irq_event+0x6c/0x90)
[   59.116005]  r10: r9: r8:f0852000 r7:ea1a9600 r6:ea1a9620 
r5:ed60
[   59.123887]  r4:ed00
[   59.126434] [] (handle_irq_event) from [] 
(handle_level_irq+0xb0/0x15c)
[   59.134793]  r7:ea1a9600 r6:ea1a9620 r5:ed10 r4:ed00
[   59.140492] [] (handle_level_irq) from [] 
(generic_handle_irq+0x34/0x44)
[   59.148938]  r5:c0c3ab60 r4:0004
[   59.152537] [] (generic_handle_irq) from [] 
(irqc_irq_handler+0x7c/0xac)
[   59.160986] [] (irqc_irq_handler) from [] 
(handle_irq_event_percpu+0x74/0x238)
[   59.169953]  r9: r8:e99f6000 r7: r6:0020 r5:ea268ca0 
r4:ea2699c0
[   59.177753] [] (handle_irq_event_percpu) from [] 
(handle_irq_event+0x6c/0x90)
[   59.186633]  r10:c026d32c r9:f0803000 r8:ea00a800 r7: r6: 
r5:ea268ca0
[   59.194516]  r4:ea268c40
[   59.197063] [] (handle_irq_event) from [] 
(handle_fasteoi_irq+0xc4/0x1bc)
[   59.205595]  r7: r6: r5:c0c02d10 r4:ea268c40
[   59.211296] [] (handle_fasteoi_irq) from [] 
(generic_handle_irq+0x34/0x44)
[   59.219915]  r7: 

Re: [v4.5-rc3 REGRESSION] rcar_sound-related WARN_ON()

[Kuninori Morimoto]

Hi Geert

> > I tried this patch, but It didn't use this printk().
> > Am I wrong ?
> >
> > -
> > diff --git a/drivers/regulator/da9210-regulator.c 
> > b/drivers/regulator/da9210-regulator.c
> > index 01c0e37..85c1166 100644
> > --- a/drivers/regulator/da9210-regulator.c
> > +++ b/drivers/regulator/da9210-regulator.c
> > @@ -167,6 +167,8 @@ static irqreturn_t da9210_irq_handler(int irq, void 
> > *data)
> > goto error_i2c;
> >
> > ret = IRQ_HANDLED;
> > +   } else if (val != handled) {
> > +   printk("---val %x : %x\n", val, handled);
> > }
> 
> That's what I meant.
> Do you still see the "irq 311: nobody cared"?
> Do you have the DA9210 driver enabled?

My .config

> grep DA9210 .config
CONFIG_REGULATOR_DA9210=y

My kernel log has this

   ...
   [2.624405] NET: Registered protocol family 17
   [2.628934] Registering SWP/SWPB emulation handler
=> [2.634141] da9210 7-0068: I2C error : -6
   [2.636604] [drm] Supports vblank timestamp caching Rev 2 
(21.10.2013).
   [2.636606] [drm] No driver support for vblan
   ...

And Yes, I still have "irq 311: nobody cared" panic after few second from boot

Re: [v4.5-rc3 REGRESSION] rcar_sound-related WARN_ON()

[Geert Uytterhoeven]

Hi Morimoto-san,

On Thu, Feb 18, 2016 at 9:39 AM, Kuninori Morimoto
 wrote:
>> > [   59.144598] irq 311: nobody cared (try booting with the "irqpoll" 
>> > option)
>> > [   59.151419] CPU: 0 PID: 410 Comm: irq/311-da9210 Not tainted 
>> > 4.5.0-rc3-02009-g83c516e #612
>>
>> Ugh, the da9210 interrupt.
>>
>> That probably means an event was set that isn't handled by
>> da9210_irq_handler().
>>
>> Is it reproducible? If yes, can you please print val in the !handled case?
>> Thanks!
>
> I tried this patch, but It didn't use this printk().
> Am I wrong ?
>
> -
> diff --git a/drivers/regulator/da9210-regulator.c 
> b/drivers/regulator/da9210-regulator.c
> index 01c0e37..85c1166 100644
> --- a/drivers/regulator/da9210-regulator.c
> +++ b/drivers/regulator/da9210-regulator.c
> @@ -167,6 +167,8 @@ static irqreturn_t da9210_irq_handler(int irq, void *data)
> goto error_i2c;
>
> ret = IRQ_HANDLED;
> +   } else if (val != handled) {
> +   printk("---val %x : %x\n", val, handled);
> }

That's what I meant.
Do you still see the "irq 311: nobody cared"?
Do you have the DA9210 driver enabled?

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds

Re: [v4.5-rc3 REGRESSION] rcar_sound-related WARN_ON()

[Kuninori Morimoto]

Hi Geert

> > [   59.144598] irq 311: nobody cared (try booting with the "irqpoll" option)
> > [   59.151419] CPU: 0 PID: 410 Comm: irq/311-da9210 Not tainted 
> > 4.5.0-rc3-02009-g83c516e #612
> 
> Ugh, the da9210 interrupt.
> 
> That probably means an event was set that isn't handled by
> da9210_irq_handler().
> 
> Is it reproducible? If yes, can you please print val in the !handled case?
> Thanks!

I tried this patch, but It didn't use this printk().
Am I wrong ?

-
diff --git a/drivers/regulator/da9210-regulator.c 
b/drivers/regulator/da9210-regulator.c
index 01c0e37..85c1166 100644
--- a/drivers/regulator/da9210-regulator.c
+++ b/drivers/regulator/da9210-regulator.c
@@ -167,6 +167,8 @@ static irqreturn_t da9210_irq_handler(int irq, void *data)
goto error_i2c;
 
ret = IRQ_HANDLED;
+   } else if (val != handled) {
+   printk("---val %x : %x\n", val, handled);
}
 
return ret;
-

Re: [v4.5-rc3 REGRESSION] rcar_sound-related WARN_ON()

[Kuninori Morimoto]

Hi Laurent

> I've ran into a WARN_ON related to the sound driver on v4.5-rc3 on Lager. The 
> problem doesn't seem to occur on v4.4. It also doesn't occur at every boot, 
> you sometimes have to boot the board several times before reproducing it (up 
> to 5 times in my tests, but usually less than that).
> 
> To make the problem worse, I've only been able to reproduce it when disabling 
> the vsp1 driver. I first blamed changes on the V4L2 side, but when reverting 
> the V4L2 subsystem to v4.4 on top of an otherwise vanilla v4.5-rc3 the 
> problem 
> still occurs.
> 
> Here's the WARN_ON log, I've also attached the full kernel log and my .config 
> to this e-mail. Is this a known issue ?

Thank you for your report, but I couldn't reproduce it.
Which tree/commit are you using ? does your "v4.5-rc3" means Linus/master ?
or renesas-driver ?
I used your .config, but my kernel log is not same as yours.

About sound, I'm always testing shmobile_defconfig, but it seems you
added many configs. I'm happy to solve this issue.

And, interesting things is that I got different panic

--
...
[5.747675]  host=192.168.10.107, domain=example.org, nis-domain=(none)
[5.754642]  bootserver=192.168.10.77, rootserver=192.168.10.77, 
rootpath=/tftpboot/rootfs_arm nameserver0=192.168.10.77
[5.766273] SDHI0 Vcc: disabling
[5.769507] SDHI2 Vcc: disabling
[5.772739] SDHI0 VccQ: disabling
[5.776067] SDHI2 VccQ: disabling
[5.779841] sh-sci e6e6.serial: dma_request_slave_channel_compat failed
[5.786837] sh-sci e6e6.serial: dma_request_slave_channel_compat failed
[5.799314] VFS: Mounted root (nfs filesystem) on device 0:15.
[5.805620] devtmpfs: mounted
[5.809890] Freeing unused kernel memory: 2048K (c0a0 - c0c0)
[   59.144598] irq 311: nobody cared (try booting with the "irqpoll" option)
[   59.151419] CPU: 0 PID: 410 Comm: irq/311-da9210 Not tainted 
4.5.0-rc3-02009-g83c516e #612
[   59.159695] Hardware name: Generic R8A7790 (Flattened Device Tree)
[   59.165879] Backtrace: 
[   59.168355] [] (dump_backtrace) from [] 
(show_stack+0x20/0x24)
[   59.175933]  r7:0137 r6:6193 r5:c0c1d96c r4:
[   59.181641] [] (show_stack) from [] 
(dump_stack+0xa0/0xb4)
[   59.15] [] (dump_stack) from [] 
(__report_bad_irq+0x38/0xd0)
[   59.196636]  r7:0137 r6:0137 r5: r4:e99c7b80
[   59.202340] [] (__report_bad_irq) from [] 
(note_interrupt+0x280/0x2d0)
[   59.210611]  r7:0137 r6:0137 r5: r4:e99c7b80
[   59.216315] [] (note_interrupt) from [] 
(handle_irq_event_percpu+0x1d0/0x238)
[   59.225195]  r10:2088 r9:0002 r8:e9a2a000 r7:0002 r6:0137 
r5:e99c7be0
[   59.233079]  r4: r3:
[   59.236679] [] (handle_irq_event_percpu) from [] 
(handle_irq_event+0x6c/0x90)
[   59.245559]  r10: r9: r8:f0852000 r7:ea22a600 r6:ea22a620 
r5:e99c7be0
[   59.253441]  r4:e99c7b80
[   59.255988] [] (handle_irq_event) from [] 
(handle_level_irq+0xb0/0x15c)
[   59.264347]  r7:ea22a600 r6:ea22a620 r5:e99c7b90 r4:e99c7b80
[   59.270048] [] (handle_level_irq) from [] 
(generic_handle_irq+0x34/0x44)
[   59.278493]  r5:c0c3ab60 r4:0004
[   59.282093] [] (generic_handle_irq) from [] 
(irqc_irq_handler+0x7c/0xac)
[   59.290542] [] (irqc_irq_handler) from [] 
(handle_irq_event_percpu+0x74/0x238)
[   59.299508]  r9: r8:e9a2a000 r7: r6:0020 r5:ea268ca0 
r4:ea227580
[   59.307310] [] (handle_irq_event_percpu) from [] 
(handle_irq_event+0x6c/0x90)
[   59.316190]  r10:c026d32c r9:f0803000 r8:ea00a800 r7: r6: 
r5:ea268ca0
[   59.324072]  r4:ea268c40
[   59.326619] [] (handle_irq_event) from [] 
(handle_fasteoi_irq+0xc4/0x1bc)
[   59.335151]  r7: r6: r5:c0c02d10 r4:ea268c40
[   59.340853] [] (handle_fasteoi_irq) from [] 
(generic_handle_irq+0x34/0x44)
[   59.349472]  r7: r6: r5:0020 r4:c0a61228
[   59.355174] [] (generic_handle_irq) from [] 
(__handle_domain_irq+0x6c/0xc4)
[   59.363884] [] (__handle_domain_irq) from [] 
(gic_handle_irq+0x58/0x98)
[   59.372243]  r9:f0803000 r8:f0802000 r7:e9a2be80 r6:f080200c r5:c0c02d10 
r4:c0c1dc58
[   59.380042] [] (gic_handle_irq) from [] 
(__irq_svc+0x40/0x54)
[   59.387533] Exception stack(0xe9a2be80 to 0xe9a2bec8)
[   59.392590] be80: e99c7bf8 6113 0001 8625 e99c7bf8 6113 
0003 0001
[   59.400778] bea0:  e9875f00 c026d32c e9a2bedc e9a2bee0 e9a2bed0 
c025fd90 c06439fc
[   59.408964] bec0: 6113 
[   59.412453]  r9:e9875f00 r8: r7:e9a2beb4 r6: r5:6113 
r4:c06439fc
[   59.420263] [] (_raw_spin_unlock_irqrestore) from [] 
(__wake_up+0x54/0x5c)
[   59.42] [] (__wake_up) from [] 
(wake_threads_waitq+0x50/0x54)
[   59.436726]  r8:e99c7b80 r7:c026d660 r6:e9875f24 r5: r4:e9a2a000
[   59.443479] [] (wake_threads_waitq) from [] 
(irq_thread+0x1a8/0x264)
[   59.451585] [] (irq_thread) from [] (kthread+0xf8/0x114)
[   59.458636]