From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754053AbbIHXjX (ORCPT ); Tue, 8 Sep 2015 19:39:23 -0400 Received: from mail.tpi.com ([74.45.170.26]:38065 "EHLO mail.tpi.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751942AbbIHXjS (ORCPT ); Tue, 8 Sep 2015 19:39:18 -0400 X-Greylist: delayed 1164 seconds by postgrey-1.27 at vger.kernel.org; Tue, 08 Sep 2015 19:39:18 EDT Subject: Re: [PATCH] powerpc: define empty enable_kernel_vsx() when CONFIG_VSX=n To: Paul Mackerras References: <1441735991-29199-1-git-send-email-tim.gardner@canonical.com> <20150908224735.GA12791@fergus.ozlabs.ibm.com> Cc: linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org, Benjamin Herrenschmidt , Michael Ellerman , Leonidas Da Silva Barbosa , Herbert Xu From: Tim Gardner Message-ID: <55EF6D17.6050809@canonical.com> Date: Tue, 8 Sep 2015 17:19:51 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-Version: 1.0 In-Reply-To: <20150908224735.GA12791@fergus.ozlabs.ibm.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 09/08/2015 04:47 PM, Paul Mackerras wrote: > On Tue, Sep 08, 2015 at 12:13:11PM -0600, tim.gardner@canonical.com wrote: >> From: Tim Gardner >> >> commit 72cd7b44bc99 ("powerpc: Uncomment and make enable_kernel_vsx() >> routine available") neglected to define an empty inline replacement for >> enable_kernel_vsx() when CONFIG_VSX=n. > > If code that wants to call enable_kernel_vsx() is getting compiled in > when CONFIG_VSX=n, that's a worry. Is this patch motivated by an > actual compile failure? If so what was the failure? > > Paul. > I was having link failures after backporting 'crypto: nx' patches to a 4.2 based kernel. You may have a point in that the upstream Kconfig will not allow those files to be compiled if CONFIG_VSX=n. I will check in my morning if to see if I can reproduce the same link error in mainline. rtg -- Tim Gardner tim.gardner@canonical.com