* [PATCH] genirq: move debugfs option to kernel hacking
@ 2019-09-01 3:55 Changbin Du
2019-09-01 6:23 ` Thomas Gleixner
0 siblings, 1 reply; 5+ messages in thread
From: Changbin Du @ 2019-09-01 3:55 UTC (permalink / raw)
To: Andrew Morton, Thomas Gleixner; +Cc: yamada.masahiro, linux-kernel, Changbin Du
Just like the other generic debug options, move the irq one to
'Kernel hacking' menu.
Signed-off-by: Changbin Du <changbin.du@gmail.com>
---
kernel/irq/Kconfig | 12 ------------
lib/Kconfig.debug | 11 +++++++++++
2 files changed, 11 insertions(+), 12 deletions(-)
diff --git a/kernel/irq/Kconfig b/kernel/irq/Kconfig
index f92d9a687372..4684c44e0390 100644
--- a/kernel/irq/Kconfig
+++ b/kernel/irq/Kconfig
@@ -123,18 +123,6 @@ config SPARSE_IRQ
out the interrupt descriptors in a more NUMA-friendly way. )
If you don't know what to do here, say N.
-
-config GENERIC_IRQ_DEBUGFS
- bool "Expose irq internals in debugfs"
- depends on DEBUG_FS
- default n
- ---help---
-
- Exposes internal state information through debugfs. Mostly for
- developers and debugging of hard to diagnose interrupt problems.
-
- If you don't know what to do here, say N.
-
endmenu
config GENERIC_IRQ_MULTI_HANDLER
diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
index 5960e2980a8a..c2fb63b4263b 100644
--- a/lib/Kconfig.debug
+++ b/lib/Kconfig.debug
@@ -2142,6 +2142,17 @@ config IO_STRICT_DEVMEM
If in doubt, say Y.
+config GENERIC_IRQ_DEBUGFS
+ bool "Expose irq internals to userspace via debugfs"
+ depends on DEBUG_FS
+ default n
+ ---help---
+
+ Exposes internal state information through debugfs. Mostly for
+ developers and debugging of hard to diagnose interrupt problems.
+
+ If you don't know what to do here, say N.
+
source "arch/$(SRCARCH)/Kconfig.debug"
endmenu # Kernel hacking
--
2.20.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] genirq: move debugfs option to kernel hacking
2019-09-01 3:55 [PATCH] genirq: move debugfs option to kernel hacking Changbin Du
@ 2019-09-01 6:23 ` Thomas Gleixner
2019-09-01 10:10 ` Changbin Du
0 siblings, 1 reply; 5+ messages in thread
From: Thomas Gleixner @ 2019-09-01 6:23 UTC (permalink / raw)
To: Changbin Du; +Cc: Andrew Morton, yamada.masahiro, LKML, Marc Zyngier
On Sun, 1 Sep 2019, Changbin Du wrote:
> Just like the other generic debug options, move the irq one to
> 'Kernel hacking' menu.
Why?
Kernel hacking is a inscrutable mess where you can waste a lot of time to
find what you are looking for.
If I want to debug interrupts then having the option right there where all
other interrupt related configuration is makes tons of sense.
I would be less opposed to this when the kernel hacking menu would be
halfways well structured, but you just chose another random place for that
option which is worse than what we have now.
Thanks,
tglx
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] genirq: move debugfs option to kernel hacking
2019-09-01 6:23 ` Thomas Gleixner
@ 2019-09-01 10:10 ` Changbin Du
2019-09-01 10:49 ` Marc Zyngier
0 siblings, 1 reply; 5+ messages in thread
From: Changbin Du @ 2019-09-01 10:10 UTC (permalink / raw)
To: Thomas Gleixner
Cc: Changbin Du, Andrew Morton, yamada.masahiro, LKML, Marc Zyngier
On Sun, Sep 01, 2019 at 08:23:02AM +0200, Thomas Gleixner wrote:
> On Sun, 1 Sep 2019, Changbin Du wrote:
>
> > Just like the other generic debug options, move the irq one to
> > 'Kernel hacking' menu.
>
> Why?
>
> Kernel hacking is a inscrutable mess where you can waste a lot of time to
> find what you are looking for.
>
yes, the 'kernel hacking' menu has many items now and are not well structured.
Let me see if it can be improved.
> If I want to debug interrupts then having the option right there where all
> other interrupt related configuration is makes tons of sense.
>
> I would be less opposed to this when the kernel hacking menu would be
> halfways well structured, but you just chose another random place for that
> option which is worse than what we have now.
>
We already have an irq debug option CONFIG_DEBUG_SHIRQ here. Maybe we can group
them into a submenu.
> Thanks,
>
> tglx
>
>
>
--
Cheers,
Changbin Du
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] genirq: move debugfs option to kernel hacking
2019-09-01 10:10 ` Changbin Du
@ 2019-09-01 10:49 ` Marc Zyngier
2019-09-04 0:50 ` Changbin Du
0 siblings, 1 reply; 5+ messages in thread
From: Marc Zyngier @ 2019-09-01 10:49 UTC (permalink / raw)
To: Changbin Du; +Cc: Thomas Gleixner, Andrew Morton, yamada.masahiro, LKML
On Sun, 1 Sep 2019 18:10:33 +0800
Changbin Du <changbin.du@gmail.com> wrote:
> On Sun, Sep 01, 2019 at 08:23:02AM +0200, Thomas Gleixner wrote:
> > On Sun, 1 Sep 2019, Changbin Du wrote:
> >
> > > Just like the other generic debug options, move the irq one to
> > > 'Kernel hacking' menu.
> >
> > Why?
> >
> > Kernel hacking is a inscrutable mess where you can waste a lot of time to
> > find what you are looking for.
> >
> yes, the 'kernel hacking' menu has many items now and are not well structured.
> Let me see if it can be improved.
>
> > If I want to debug interrupts then having the option right there where all
> > other interrupt related configuration is makes tons of sense.
> >
> > I would be less opposed to this when the kernel hacking menu would be
> > halfways well structured, but you just chose another random place for that
> > option which is worse than what we have now.
> >
> We already have an irq debug option CONFIG_DEBUG_SHIRQ here. Maybe we can group
> them into a submenu.
DEBUG_SHIRQ is extremely different from GENERIC_IRQ_DEBUGFS. The former
is a test option, verifying that endpoint drivers have a correct
behaviour. The latter is a dump of the kernel internals, which is
mostly for people dealing with the internals of the IRQ subsystem.
Preserving this distinction between the users of the IRQ API on one
side and the debugging of the IRQ subsystem on the other is important.
Moving these two things close together could make it even more confusing
for the users (who usually do not need to mess with the IRQ subsystem's
internals...).
Thanks,
M.
--
Without deviation from the norm, progress is not possible.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] genirq: move debugfs option to kernel hacking
2019-09-01 10:49 ` Marc Zyngier
@ 2019-09-04 0:50 ` Changbin Du
0 siblings, 0 replies; 5+ messages in thread
From: Changbin Du @ 2019-09-04 0:50 UTC (permalink / raw)
To: Marc Zyngier
Cc: Changbin Du, Thomas Gleixner, Andrew Morton, yamada.masahiro,
LKML
On Sun, Sep 01, 2019 at 11:49:36AM +0100, Marc Zyngier wrote:
> On Sun, 1 Sep 2019 18:10:33 +0800
> Changbin Du <changbin.du@gmail.com> wrote:
>
> > On Sun, Sep 01, 2019 at 08:23:02AM +0200, Thomas Gleixner wrote:
> > > On Sun, 1 Sep 2019, Changbin Du wrote:
> > >
> > > > Just like the other generic debug options, move the irq one to
> > > > 'Kernel hacking' menu.
> > >
> > > Why?
> > >
> > > Kernel hacking is a inscrutable mess where you can waste a lot of time to
> > > find what you are looking for.
> > >
> > yes, the 'kernel hacking' menu has many items now and are not well structured.
> > Let me see if it can be improved.
> >
> > > If I want to debug interrupts then having the option right there where all
> > > other interrupt related configuration is makes tons of sense.
> > >
> > > I would be less opposed to this when the kernel hacking menu would be
> > > halfways well structured, but you just chose another random place for that
> > > option which is worse than what we have now.
> > >
> > We already have an irq debug option CONFIG_DEBUG_SHIRQ here. Maybe we can group
> > them into a submenu.
>
> DEBUG_SHIRQ is extremely different from GENERIC_IRQ_DEBUGFS. The former
> is a test option, verifying that endpoint drivers have a correct
> behaviour. The latter is a dump of the kernel internals, which is
> mostly for people dealing with the internals of the IRQ subsystem.
>
> Preserving this distinction between the users of the IRQ API on one
> side and the debugging of the IRQ subsystem on the other is important.
> Moving these two things close together could make it even more confusing
> for the users (who usually do not need to mess with the IRQ subsystem's
> internals...).
>
IMHO, these are two distinct irq *debug* options. If we prefer
preserving current position, please skip this patch. Thanks.
> Thanks,
>
> M.
> --
> Without deviation from the norm, progress is not possible.
--
Cheers,
Changbin Du
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-09-04 0:50 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-09-01 3:55 [PATCH] genirq: move debugfs option to kernel hacking Changbin Du
2019-09-01 6:23 ` Thomas Gleixner
2019-09-01 10:10 ` Changbin Du
2019-09-01 10:49 ` Marc Zyngier
2019-09-04 0:50 ` Changbin Du
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox