Re: IRQ behaivour has been changed from v4.14 to v4.15-rc1

From: Andy Shevchenko
Date: Thu Dec 28 2017 - 12:23:19 EST


On Thu, 2017-12-28 at 19:17 +0200, Andy Shevchenko wrote:
> Hi!
>
> Experimenting with AtomISP (yes, code is ugly and MSI handling rather
> hackish, though...).
>
> So, with v4.14 base:

See additional note below.

>
> [ 33.639224] atomisp-isp2 0000:00:03.0: Start stream on pad 1 for
> asd0
> [ 33.652355] atomisp-isp2 0000:00:03.0: irq:0x20
> [ 33.662456] atomisp-isp2 0000:00:03.0: irq:0x20
> [ 33.698064] atomisp-isp2 0000:00:03.0: stream[0] started.
>
> Ctrl+C
>
> [ 48.185643] atomisp-isp2 0000:00:03.0: <atomisp_dqbuf: -512
> [ 48.204641] atomisp-isp2 0000:00:03.0: release device ATOMISP ISP
> CAPTURE output
> ...
>
> and machine still alive.
>
>
> With v4.15-rc1 base (basically your branch + some my hack patches)

Needs a bit of elaboration:
a) nothing had been changed WRT AtomISP driver or media stuff, under
"your branch" one reads Sakari's media_tree.git/atomisp branch;
b) my hack patches has nothing to do with anything except AtomISP
itself;
c) v4.14 base required media/v4.15-1 tag to be merged as well.

> the IRQ behaviour changed, i.e. I have got:
>
>
> [ 85.167061] spurious APIC interrupt through vector ff on CPU#0,
> should never happen.
> [ 85.199886] atomisp-isp2 0000:00:03.0: stream[0] started.
>
> and Ctrl+C does NOT work. Machine just hangs.
>
> It might be related to this:
> https://lkml.org/lkml/2017/12/22/697
>
> Any comments, Thomas?
>

--
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Intel Finland Oy