From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 85759C433EF for ; Mon, 1 Nov 2021 09:36:02 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 4C38F60F70 for ; Mon, 1 Nov 2021 09:36:02 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 4C38F60F70 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:Cc:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=VXJjc/Qymhn4GXvRr488RnlRc49CS2u4xEQhvyKxvmI=; b=hLBIQKMF+8aGB5VwX9I6sZ2+pY F9lOtNov8gIowIxRQwXZZFmW7s5CPt4hplImwa9n4mmBNWd0t88uTxShUWVf+KFf4u14OTpNppRva VPAFJ7GJ+2Quex2cFMY44ropBnuSfl9b6fY7lxbReTbIUpqcjMoldt2qQZdfi5pBSajDAuVrfctFb 1X28Xtx0JJn3MFuFzh+MUjEhQ2p2RN2Pejut38+mY8aqf6aEsg+i70RqAZ41/HtiglXWPdoFBlMOH hxfSwN7lVMDsQw13+KBFiyFtpC1c85pTay49rKbrsADlI5mF2tSVQNkVlbj7MjrQz8OsXTwkPY5tD kE9DsYmA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mhThs-00Folf-IK; Mon, 01 Nov 2021 09:34:20 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mhTho-00Fojr-5S for linux-arm-kernel@lists.infradead.org; Mon, 01 Nov 2021 09:34:17 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 3B83D101E; Mon, 1 Nov 2021 02:34:12 -0700 (PDT) Received: from [192.168.4.86] (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 42D553F719; Mon, 1 Nov 2021 02:34:10 -0700 (PDT) Subject: Re: [PATCH] [RFC] arm64: export this_cpu_has_cap To: Will Deacon , Catalin Marinas Cc: Arnd Bergmann , Mathieu Poirier , Anshuman Khandual , Arnd Bergmann , Marc Zyngier , Mark Rutland , David Brazdil , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Greg Kroah-Hartman References: <20211029113023.760421-1-arnd@kernel.org> <9d4e09b4-47dc-ed3c-2b6d-e0d1a081e592@arm.com> <20211101090143.GA27181@willie-the-truck> From: Suzuki K Poulose Message-ID: Date: Mon, 1 Nov 2021 09:34:08 +0000 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:78.0) Gecko/20100101 Thunderbird/78.14.0 MIME-Version: 1.0 In-Reply-To: <20211101090143.GA27181@willie-the-truck> Content-Language: en-GB X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211101_023416_359407_90A77B9D X-CRM114-Status: GOOD ( 25.20 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Will On 01/11/2021 09:01, Will Deacon wrote: > On Fri, Oct 29, 2021 at 07:06:23PM +0100, Catalin Marinas wrote: >> On Fri, Oct 29, 2021 at 02:31:23PM +0100, Suzuki K Poulose wrote: >>> On 29/10/2021 12:30, Arnd Bergmann wrote: >>>> From: Arnd Bergmann >>>> >>>> It's now used in a coresight driver that can be a loadable module: >>>> >>>> ERROR: modpost: "this_cpu_has_cap" [drivers/hwtracing/coresight/coresight-trbe.ko] undefined! >>>> >>>> Fixes: 8a1065127d95 ("coresight: trbe: Add infrastructure for Errata handling") >>> >>> Reviewed-by: Suzuki K Poulose >>> Tested-by: Suzuki K Poulose >>> >>> Will, Catalin, Mathieu, >>> >>> Do you have a preference on how this fix can be pulled in ? This may >>> be safe to go via coresight tree if it is not too late. Otherwise, >>> it could go via the arm64 tree. >> >> I think Will already closed/tagged the arm64 tree for the upcoming >> merging window, though he could take it as a fix afterwards. >> >> If it doesn't conflict with the arm64 for-next/core, it's fine by me to >> go through the coresight tree. >> >>>> Signed-off-by: Arnd Bergmann >>>> --- >>>> Not sure if we actually want this to be exported, this is my local >>>> workaround for the randconfig build bot. >>>> --- >>>> arch/arm64/kernel/cpufeature.c | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>>> diff --git a/arch/arm64/kernel/cpufeature.c b/arch/arm64/kernel/cpufeature.c >>>> index ecbdff795f5e..beccbcfa7391 100644 >>>> --- a/arch/arm64/kernel/cpufeature.c >>>> +++ b/arch/arm64/kernel/cpufeature.c >>>> @@ -2864,6 +2864,7 @@ bool this_cpu_has_cap(unsigned int n) >>>> return false; >>>> } >>>> +EXPORT_SYMBOL(this_cpu_has_cap); >> >> EXPORT_SYMBOL_GPL? I think this_cpu_has_cap() is a bit more more >> specialised than cpus_have_const_cap(). >> >> With that: >> >> Acked-by: Catalin Marinas > > Yes, at this stage I think it's best for this to go via the Coresight tree. > So with the _GPL export: > > Acked-by: Will Deacon > > If that doesn't work for some reason, I can take it next week after the > initial arm64 queue has been merged. Please just let me know. As I understand correctly, this will now need to go via arm64 tree. The CoreSight tree changes are pulled into Greg's tree and the next it will happen is for the next release. Usually the fixes don't end up there during the -rc cycles. So, I believe it is better if this goes via arm64. Suzuki _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7D021C433EF for ; Mon, 1 Nov 2021 09:36:47 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 6474F610F7 for ; Mon, 1 Nov 2021 09:36:47 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232905AbhKAJjR (ORCPT ); Mon, 1 Nov 2021 05:39:17 -0400 Received: from foss.arm.com ([217.140.110.172]:36112 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232976AbhKAJgq (ORCPT ); Mon, 1 Nov 2021 05:36:46 -0400 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 3B83D101E; Mon, 1 Nov 2021 02:34:12 -0700 (PDT) Received: from [192.168.4.86] (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 42D553F719; Mon, 1 Nov 2021 02:34:10 -0700 (PDT) Subject: Re: [PATCH] [RFC] arm64: export this_cpu_has_cap To: Will Deacon , Catalin Marinas Cc: Arnd Bergmann , Mathieu Poirier , Anshuman Khandual , Arnd Bergmann , Marc Zyngier , Mark Rutland , David Brazdil , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Greg Kroah-Hartman References: <20211029113023.760421-1-arnd@kernel.org> <9d4e09b4-47dc-ed3c-2b6d-e0d1a081e592@arm.com> <20211101090143.GA27181@willie-the-truck> From: Suzuki K Poulose Message-ID: Date: Mon, 1 Nov 2021 09:34:08 +0000 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:78.0) Gecko/20100101 Thunderbird/78.14.0 MIME-Version: 1.0 In-Reply-To: <20211101090143.GA27181@willie-the-truck> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-GB Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Will On 01/11/2021 09:01, Will Deacon wrote: > On Fri, Oct 29, 2021 at 07:06:23PM +0100, Catalin Marinas wrote: >> On Fri, Oct 29, 2021 at 02:31:23PM +0100, Suzuki K Poulose wrote: >>> On 29/10/2021 12:30, Arnd Bergmann wrote: >>>> From: Arnd Bergmann >>>> >>>> It's now used in a coresight driver that can be a loadable module: >>>> >>>> ERROR: modpost: "this_cpu_has_cap" [drivers/hwtracing/coresight/coresight-trbe.ko] undefined! >>>> >>>> Fixes: 8a1065127d95 ("coresight: trbe: Add infrastructure for Errata handling") >>> >>> Reviewed-by: Suzuki K Poulose >>> Tested-by: Suzuki K Poulose >>> >>> Will, Catalin, Mathieu, >>> >>> Do you have a preference on how this fix can be pulled in ? This may >>> be safe to go via coresight tree if it is not too late. Otherwise, >>> it could go via the arm64 tree. >> >> I think Will already closed/tagged the arm64 tree for the upcoming >> merging window, though he could take it as a fix afterwards. >> >> If it doesn't conflict with the arm64 for-next/core, it's fine by me to >> go through the coresight tree. >> >>>> Signed-off-by: Arnd Bergmann >>>> --- >>>> Not sure if we actually want this to be exported, this is my local >>>> workaround for the randconfig build bot. >>>> --- >>>> arch/arm64/kernel/cpufeature.c | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>>> diff --git a/arch/arm64/kernel/cpufeature.c b/arch/arm64/kernel/cpufeature.c >>>> index ecbdff795f5e..beccbcfa7391 100644 >>>> --- a/arch/arm64/kernel/cpufeature.c >>>> +++ b/arch/arm64/kernel/cpufeature.c >>>> @@ -2864,6 +2864,7 @@ bool this_cpu_has_cap(unsigned int n) >>>> return false; >>>> } >>>> +EXPORT_SYMBOL(this_cpu_has_cap); >> >> EXPORT_SYMBOL_GPL? I think this_cpu_has_cap() is a bit more more >> specialised than cpus_have_const_cap(). >> >> With that: >> >> Acked-by: Catalin Marinas > > Yes, at this stage I think it's best for this to go via the Coresight tree. > So with the _GPL export: > > Acked-by: Will Deacon > > If that doesn't work for some reason, I can take it next week after the > initial arm64 queue has been merged. Please just let me know. As I understand correctly, this will now need to go via arm64 tree. The CoreSight tree changes are pulled into Greg's tree and the next it will happen is for the next release. Usually the fixes don't end up there during the -rc cycles. So, I believe it is better if this goes via arm64. Suzuki