All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
To: James Hogan <james.hogan@imgtec.com>
Cc: linux-kernel@vger.kernel.org,
	Steven Rostedt <rostedt@goodmis.org>,
	Dipankar Sarma <dipankar@in.ibm.com>
Subject: Re: [PATCH 1/1] rcu: select IRQ_WORK from TREE_PREEMPT_RCU
Date: Thu, 25 Jul 2013 08:26:36 -0700	[thread overview]
Message-ID: <20130725152636.GH3889@linux.vnet.ibm.com> (raw)
In-Reply-To: <1374762865-13635-1-git-send-email-james.hogan@imgtec.com>

On Thu, Jul 25, 2013 at 03:34:25PM +0100, James Hogan wrote:
> TREE_RCU and TREE_PREEMPT_RCU both cause kernel/rcutree.c to be built,
> but only TREE_RCU selects IRQ_WORK, which can result in an undefined
> reference to irq_work_queue for some (random) configs:
> 
> kernel/built-in.o In function `rcu_start_gp_advanced':
> kernel/rcutree.c:1564: undefined reference to `irq_work_queue'
> 
> Select IRQ_WORK from TREE_PREEMPT_RCU too to fix this.

Queued for 3.12, good catch, James!

							Thanx, Paul

> Signed-off-by: James Hogan <james.hogan@imgtec.com>
> Cc: Steven Rostedt <rostedt@goodmis.org>
> Cc: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
> Cc: Dipankar Sarma <dipankar@in.ibm.com>
> ---
>  init/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/init/Kconfig b/init/Kconfig
> index 247084b..c08a549 100644
> --- a/init/Kconfig
> +++ b/init/Kconfig
> @@ -470,6 +470,7 @@ config TREE_RCU
>  config TREE_PREEMPT_RCU
>  	bool "Preemptible tree-based hierarchical RCU"
>  	depends on PREEMPT
> +	select IRQ_WORK
>  	help
>  	  This option selects the RCU implementation that is
>  	  designed for very large SMP systems with hundreds or
> -- 
> 1.8.1.2
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
> 


      reply	other threads:[~2013-07-25 15:26 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-25 14:34 [PATCH 1/1] rcu: select IRQ_WORK from TREE_PREEMPT_RCU James Hogan
2013-07-25 15:26 ` Paul E. McKenney [this message]

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=20130725152636.GH3889@linux.vnet.ibm.com \
    --to=paulmck@linux.vnet.ibm.com \
    --cc=dipankar@in.ibm.com \
    --cc=james.hogan@imgtec.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rostedt@goodmis.org \
    /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.