All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michal Marek <mmarek@suse.cz>
To: Paul Thompson <set.mailinglist@gmail.com>
Cc: linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org,
	Thomas Gleixner <tglx@linutronix.de>
Subject: Re: IRQ subsystem menu
Date: Mon, 29 Oct 2012 15:03:08 +0100	[thread overview]
Message-ID: <20121029140308.GA19316@sepie.suse.cz> (raw)
In-Reply-To: <20121021074016.GA2604@squish.home.loc>

(Adding lkml and Thomas as IRQ subsystem maintainer to CC)

On Sun, Oct 21, 2012 at 03:40:16AM -0400, Paul Thompson wrote:
> 	Hi;
> 
> 	In menuconfig, General setup -> IRQ subsystem, there are two
> possible menu-items. One depends partly on IRQ_DOMAIN, and the other
> on MAY_HAVE_SPARSE_IRQ.
> 
> 	On the x86-64 arch, neither of these seems to be set, so the
> menu entry is blank if entered (displaying an odd ^@ if you try to
> navigate up or down. And sometimes some screen garbage on the sides.)
> 
> 	Previously (in the 3.2 kernel at least) this menu used to be
> populated by the "Support sparse IRQ numbering" menu-item, which
> at that time depended on HAVE_SPARSE_IRQ, and was forced to YES.
> 
> 	I do not know if the current behaviour is intended, but it
> is ugly and confusing to be confronted with a blank menu page.
> 
> 	Ideally, the "IRQ subsystem" menu would not even show up
> if it would contain no menu-items, but I do not know enough to
> formulate that constraint.
> 
> 	A simple alternative is a comment section to reasure babes in
> the wood of kernel configuration that something is not amiss. I include
> a sample patch.
> 
> Paul
> set.mailinglist@gmail.com

> --- linux-3.7-rc2/kernel/irq/Kconfig.orig	2012-10-20 15:11:32.000000000 -0400
> +++ linux-3.7-rc2/kernel/irq/Kconfig	2012-10-21 03:14:56.272952813 -0400
> @@ -4,6 +4,9 @@ config HAVE_GENERIC_HARDIRQS
>  
>  if HAVE_GENERIC_HARDIRQS
>  menu "IRQ subsystem"
> +
> +comment "Some configurations may have no available options here"
> +
>  #
>  # Interrupt subsystem related configuration options
>  #


  reply	other threads:[~2012-10-29 14:03 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-21  7:40 IRQ subsystem menu Paul Thompson
2012-10-29 14:03 ` Michal Marek [this message]
2012-11-29 20:32   ` Grant Likely
2012-12-06  2:16     ` Paul Thompson
2012-12-10 23:18       ` Grant Likely

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20121029140308.GA19316@sepie.suse.cz \
    --to=mmarek@suse.cz \
    --cc=linux-kbuild@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=set.mailinglist@gmail.com \
    --cc=tglx@linutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.