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 X-Spam-Level: X-Spam-Status: No, score=-5.3 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id B7C1CC433E0 for ; Thu, 18 Mar 2021 02:57:21 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 6E00664F20 for ; Thu, 18 Mar 2021 02:57:21 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231168AbhCRC4u (ORCPT ); Wed, 17 Mar 2021 22:56:50 -0400 Received: from szxga05-in.huawei.com ([45.249.212.191]:13562 "EHLO szxga05-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230157AbhCRC4h (ORCPT ); Wed, 17 Mar 2021 22:56:37 -0400 Received: from DGGEMS412-HUB.china.huawei.com (unknown [172.30.72.59]) by szxga05-in.huawei.com (SkyGuard) with ESMTP id 4F1BSL6lTqzNpPV; Thu, 18 Mar 2021 10:54:10 +0800 (CST) Received: from [10.67.110.136] (10.67.110.136) by DGGEMS412-HUB.china.huawei.com (10.3.19.212) with Microsoft SMTP Server id 14.3.498.0; Thu, 18 Mar 2021 10:56:26 +0800 Subject: Re: [PATCH v2] smp: kernel/panic.c - silence warnings To: Ingo Molnar , Peter Zijlstra CC: Christophe Leroy , , , , , , References: <20210316084150.75201-1-heying24@huawei.com> <20210317094908.GB1724119@gmail.com> <9691919b-d014-7433-3345-812c9b19a677@csgroup.eu> <20210317200922.GC3830960@gmail.com> From: "heying (H)" Message-ID: Date: Thu, 18 Mar 2021 10:56:26 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.8.1 MIME-Version: 1.0 In-Reply-To: <20210317200922.GC3830960@gmail.com> Content-Type: text/plain; charset="gbk"; format=flowed Content-Transfer-Encoding: 8bit X-Originating-IP: [10.67.110.136] X-CFilter-Loop: Reflected Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org ÔÚ 2021/3/18 4:09, Ingo Molnar дµÀ: > * Peter Zijlstra wrote: > >> Now, the C people figured that distinction was useless and allowed >> sloppiness. But I still think there's merrit to that. And as >> mentioned earlier, it is consistent with variable declarations. > Fully agreed, and my other point was that it's also consistent with > the other existing externs were used *in the same header file* > already. > > I.e. there's nothing more sloppy than mixing different styles within > the same header. Checkpatch needs to be fixed or ignored here. Thank you all for the reply! There are already mixing different styles within linux/smp.h. I mean 'extern' and non 'extern' func declarations both exist in this header. Since two of you three think that 'extern' is needed, I'll add it and resend my patch. Thanks again.