Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Fri, 16 Mar 2018 08:59:12 +0100 (CET)
From:   Thomas Gleixner <tglx@linutronix.de>
To:     Dou Liyang <douly.fnst@cn.fujitsu.com>
cc:     linux-kernel@vger.kernel.org, sfr@canb.auug.org.au, mingo@elte.hu,
        peterz@infradead.org, jiangshanlai@gmail.com
Subject: Re: [PATCH] kernel/cpu: Move cpuhp_is_atomic_state() into #ifdef
 CONFIG_SMP
In-Reply-To: <20180316063852.22701-1-douly.fnst@cn.fujitsu.com>
Message-ID: <alpine.DEB.2.21.1803160858290.1598@nanos.tec.linutronix.de>
References: <20180316063852.22701-1-douly.fnst@cn.fujitsu.com>
User-Agent: Alpine 2.21 (DEB 202 2017-01-01)
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Fri, 16 Mar 2018, Dou Liyang wrote:

> Commit 17a2f1ced028 ("cpu/hotplug: Merge cpuhp_bp_states and cpuhp_ap_states")
> removed the last use of cpuhp_is_atomic_state() in common case, that caused
> Kernel produced a warning:
> 
>   'cpuhp_is_ap_state' defined but not used
> 
> So, Move it into #ifdef CONFIG_SMP

Thanks for the patch, but a fix is already queued in the smp/hotplug
branch. It just did not make it into the for-next branch yet,

Thanks,

	tglx