Received: by 2002:a05:6358:d09b:b0:dc:cd0c:909e with SMTP id jc27csp6748857rwb; Tue, 22 Nov 2022 18:54:07 -0800 (PST) X-Google-Smtp-Source: AA0mqf6eOhWb6f54/WrgODplsRFHnHJsWGR8ZCd6nFd5bDrfl8ZQYfrcxJyGfgX60dHFMBYQJtYo X-Received: by 2002:a65:644b:0:b0:477:6a44:ae9b with SMTP id s11-20020a65644b000000b004776a44ae9bmr9009889pgv.591.1669172047045; Tue, 22 Nov 2022 18:54:07 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1669172047; cv=none; d=google.com; s=arc-20160816; b=uU4+HIL6tZxTai0gwPPcIVqchKkL01PIChSfjeMIDR50bxYmxcHY7l0yjppa2p4Dvf CANO/hmq52q3ObMIHlsn4gaKVwWE0uaznt6FaoT1bVtOpPYQzOHQf3of3QDK+BJdX62v 8wa1T4oV/mU4p5SO37oT0uUdkD4i47YPMrryipVtQPnoTTXQAPuJy2QKUKibYITJQHny fcZxCYms/JBCyNaj9OOvtw4xvz9loHl9OQSCEGmdxPys42rIslpQAtEGyhJ+7VAl3w5i b69ych92mr07y5L4W+d444+2Hpjdwq6UHNBZS7mztME8wiq8uNyqZ/u/nmZ58wno304b Y1zw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:cc:to:subject:user-agent:mime-version:date:message-id; bh=7Ggnxp+KAEN8OHYWxuDlMzH99uHMNKYJeSqr0K0+/Dw=; b=Qp9LtH7DHdQgH7sjO7rMrpk0AMoldGqgtpzgSMuMl2GxvuU5RE+n3yjKgNEaeJcQdZ SpOuZewC73lCTR1S1ipG/WNKCe/69hndnTDa4KJm7ySJdkpbk9JdjUatAGTKiL/BeIXF wHUmFnr2ifNCC+Vn4DI695CxP1izc3b7D1r7TekAihntvZR3hNRzuWi5jQKCwB0dm7A8 JOHflG2en0dCKMuvZokOezZg0wrfn1Oopfk28J/2MIV1pL9LOayAwdIe+d5wGhR7fOiu +x18JqyRDuXLpYd/Y9DoOa/19f+/fEFmRaVnfS1ofaX9T/sBotp3jr3qtypPjw/QfSvr Y9tQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=huawei.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id 29-20020a63135d000000b0043908386679si14951060pgt.373.2022.11.22.18.53.56; Tue, 22 Nov 2022 18:54:07 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=huawei.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235376AbiKWCGI (ORCPT + 89 others); Tue, 22 Nov 2022 21:06:08 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57170 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235324AbiKWCGH (ORCPT ); Tue, 22 Nov 2022 21:06:07 -0500 Received: from szxga08-in.huawei.com (szxga08-in.huawei.com [45.249.212.255]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4787C8DA4E for ; Tue, 22 Nov 2022 18:06:06 -0800 (PST) Received: from kwepemi500022.china.huawei.com (unknown [172.30.72.57]) by szxga08-in.huawei.com (SkyGuard) with ESMTP id 4NH4GQ1Bf0z15Mpd; Wed, 23 Nov 2022 10:05:34 +0800 (CST) Received: from [10.67.111.83] (10.67.111.83) by kwepemi500022.china.huawei.com (7.221.188.64) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.31; Wed, 23 Nov 2022 10:06:03 +0800 Message-ID: <28a90ac0-dc8f-f0dd-74d8-d2884e2cbd30@huawei.com> Date: Wed, 23 Nov 2022 10:06:03 +0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.7.0 Subject: Re: [PATCH] arm64: armv8_deprecated: fix unused-function error To: Mark Rutland CC: , , , , References: <20221122032010.202956-1-renzhijie2@huawei.com> From: Ren Zhijie In-Reply-To: Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 8bit X-Originating-IP: [10.67.111.83] X-ClientProxiedBy: dggems706-chm.china.huawei.com (10.3.19.183) To kwepemi500022.china.huawei.com (7.221.188.64) X-CFilter-Loop: Reflected X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00,NICE_REPLY_A, RCVD_IN_DNSWL_MED,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 在 2022/11/23 0:48, Mark Rutland 写道: > On Tue, Nov 22, 2022 at 03:20:10AM +0000, Ren Zhijie wrote: >> If CONFIG_SWP_EMULATION is not set and >> CONFIG_CP15_BARRIER_EMULATION is not set, >> aarch64-linux-gnu complained about unused-function : >> >> arch/arm64/kernel/armv8_deprecated.c:67:21: error: ‘aarch32_check_condition’ defined but not used [-Werror=unused-function] >> static unsigned int aarch32_check_condition(u32 opcode, u32 psr) >> ^~~~~~~~~~~~~~~~~~~~~~~ >> cc1: all warnings being treated as errors >> >> To fix this error, warp the definition of >> aarch32_check_condition() by defined(CONFIG_SWP_EMULATION) || >> defined(CONFIG_CP15_BARRIER_EMULATION) >> >> Fixes: 0c5f416219da ("arm64: armv8_deprecated: move aarch32 helper earlier") > This also depends on building with additional options to turn warnings into > errors, no? No,i just run the normal command as follow: make ARCH="arm64" CROSS_COMPILE="aarch64-linux-gnu-" >> Signed-off-by: Ren Zhijie >> --- >> arch/arm64/kernel/armv8_deprecated.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/arch/arm64/kernel/armv8_deprecated.c b/arch/arm64/kernel/armv8_deprecated.c >> index ed0788cf6bbb..3f29ceb6653a 100644 >> --- a/arch/arm64/kernel/armv8_deprecated.c >> +++ b/arch/arm64/kernel/armv8_deprecated.c >> @@ -64,6 +64,7 @@ struct insn_emulation { >> >> #define ARM_OPCODE_CONDITION_UNCOND 0xf >> >> +#if defined(CONFIG_SWP_EMULATION) || defined(CONFIG_CP15_BARRIER_EMULATION) >> static unsigned int aarch32_check_condition(u32 opcode, u32 psr) >> { >> u32 cc_bits = opcode >> 28; >> @@ -76,6 +77,7 @@ static unsigned int aarch32_check_condition(u32 opcode, u32 psr) >> } >> return ARM_OPCODE_CONDTEST_UNCOND; >> } >> +#endif > Could we mark this as '__maybe_unused' or 'inline' instead? I think that's > preferable to the ifdeferry. sure, i will use __maybe_unused in v2. Thanks, Ren. > Thanks, > Mark. > >> >> #ifdef CONFIG_SWP_EMULATION >> /* >> -- >> 2.17.1 >> > .