Re: [PATCH] lib: update DEBUG_SHIRQ docs to match reality

2020-06-22 Thread Andy Shevchenko
On Mon, Jun 22, 2020 at 6:56 PM Wolfram Sang wrote: > > > Maybe your case was like Krzysztof's case where the issue turned out to > > > be the extra interrupt on deregistering after a deferred probe? He > > > thought it was the initial interrupt but it wasn't. > > > > Commit > > https://git.kerne

Re: [PATCH] lib: update DEBUG_SHIRQ docs to match reality

2020-06-22 Thread Wolfram Sang
Hi Andy, > > Maybe your case was like Krzysztof's case where the issue turned out to > > be the extra interrupt on deregistering after a deferred probe? He > > thought it was the initial interrupt but it wasn't. > > Commit > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commi

Re: [PATCH] lib: update DEBUG_SHIRQ docs to match reality

2020-06-13 Thread Andy Shevchenko
On Sat, Jun 13, 2020 at 2:39 PM Wolfram Sang wrote: > > > > > There is no extra interrupt when registering a shared interrupt handler > > > since 2011. Update the Kconfig text to make it clear and to avoid wrong > > > assumptions when debugging issues found by it. > > > > > > > I'm not sure. > > I

Re: [PATCH] lib: update DEBUG_SHIRQ docs to match reality

2020-06-13 Thread Wolfram Sang
> > There is no extra interrupt when registering a shared interrupt handler > > since 2011. Update the Kconfig text to make it clear and to avoid wrong > > assumptions when debugging issues found by it. > > > > I'm not sure. > I have recently fixed a bug in the IIO sensor during ->probe() due to

Re: [PATCH] lib: update DEBUG_SHIRQ docs to match reality

2020-06-13 Thread Andy Shevchenko
On Fri, Jun 12, 2020 at 3:54 PM Wolfram Sang wrote: > > There is no extra interrupt when registering a shared interrupt handler > since 2011. Update the Kconfig text to make it clear and to avoid wrong > assumptions when debugging issues found by it. > I'm not sure. I have recently fixed a bug in

Re: [PATCH] lib: update DEBUG_SHIRQ docs to match reality

2020-06-12 Thread Krzysztof Kozlowski
On Fri, Jun 12, 2020 at 02:48:44PM +0200, Wolfram Sang wrote: > There is no extra interrupt when registering a shared interrupt handler > since 2011. Update the Kconfig text to make it clear and to avoid wrong > assumptions when debugging issues found by it. > > Fixes: 6d83f94db95c ("genirq: Disab

[PATCH] lib: update DEBUG_SHIRQ docs to match reality

2020-06-12 Thread Wolfram Sang
There is no extra interrupt when registering a shared interrupt handler since 2011. Update the Kconfig text to make it clear and to avoid wrong assumptions when debugging issues found by it. Fixes: 6d83f94db95c ("genirq: Disable the SHIRQ_DEBUG call in request_threaded_irq for now") Link: https: