Received: by 2002:a25:1985:0:0:0:0:0 with SMTP id 127csp517862ybz; Wed, 15 Apr 2020 13:10:23 -0700 (PDT) X-Google-Smtp-Source: APiQypLYVpUNkBsUDqaREiBtGpemZKFAjS8729joaU6F0Ges8UA/t0UuG24ieqapSz5elYz3IFlA X-Received: by 2002:a17:906:355b:: with SMTP id s27mr6897158eja.184.1586981423839; Wed, 15 Apr 2020 13:10:23 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1586981423; cv=none; d=google.com; s=arc-20160816; b=dW+UUv+d8kmmK+8A3nLw7bM35JogMXb3G+GM4LpLT6e0ufvKEc0kuJbTNVtztGv7Kc V6VN095Bgf7nJrefYA/1tiTdDMDhR3AlLcF/t4AubrV+z08VycSmD5neMnEwfzE5zfZJ sRbnNt83RyfnfEKiQ8zqUhDkdisnh+HYwphqBUftrjuSkONgXxZuOIEAHb14eoxJNXk/ zl3+Z/yTMda9QaOvDSE5tez4n3CgZaBYH/vYQ0U9hcHNGwG+dKL7dJ0/rplTtBA7ipS+ 48NKmWE2R9Tj8NDrWTEM0LrsrRciA4HOdV9sZ/r3ONN+/IyEPsIa7z2h19rIgZYzTIy3 npdg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:user-agent:references :message-id:in-reply-to:subject:cc:to:from:date; bh=FSMIBPynRElPwTRvqswEH/hpxFr1Z1Xt8rcSKTIAbvo=; b=jeyIot2dpqfIcreZhQ+2C7tVfOdfEDSYIO3vtJAHdJUfIbzjzEMjWRNtLOyuZa7gXo bdTcD5adrShTE/0soL3ZWE7KIhgtQanas13+P+g1x+IMB+diYD46K1QmFfAvep2C0SnZ Ip7CmEzmu4UEryjlpJaHrtOO6Cz9vnDWe0QNiGcBxfLj8SG0HKROSrOwMnBPznA2fO+q rfgg7n4InVdkPhNQhssOaU8xWDo1Prlc5x3g/9g93o66DRCGBlUIdlo5EaCHA2IiRiKW uXNYFhCS3xbtOBdozinhiyTt2owd6xC+eP/NNkM2bv3I5h1XveiDBHwMYVmchGkDQcvI Tn4A== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id z12si12530079edp.162.2020.04.15.13.09.59; Wed, 15 Apr 2020 13:10:23 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2437611AbgDNJXp (ORCPT + 99 others); Tue, 14 Apr 2020 05:23:45 -0400 Received: from mx2.suse.de ([195.135.220.15]:45804 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2437562AbgDNJXj (ORCPT ); Tue, 14 Apr 2020 05:23:39 -0400 X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (unknown [195.135.220.254]) by mx2.suse.de (Postfix) with ESMTP id 6AD76AEF5; Tue, 14 Apr 2020 09:23:37 +0000 (UTC) Date: Tue, 14 Apr 2020 11:23:37 +0200 (CEST) From: Miroslav Benes To: Muchun Song cc: tglx@linutronix.de, mingo@redhat.com, bp@alien8.de, hpa@zytor.com, x86@kernel.org, mingo@kernel.org, peterz@infradead.org, shile.zhang@linux.alibaba.com, linux-kernel@vger.kernel.org Subject: Re: [PATCH] x86/unwind: Make some variables static In-Reply-To: <20200413144727.51258-1-songmuchun@bytedance.com> Message-ID: References: <20200413144727.51258-1-songmuchun@bytedance.com> User-Agent: Alpine 2.21 (LSU 202 2017-01-01) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 13 Apr 2020, Muchun Song wrote: > The variables are local to the source and do not need to be > in global scope, so make them static. > > Signed-off-by: Muchun Song > --- > arch/x86/kernel/unwind_orc.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/arch/x86/kernel/unwind_orc.c b/arch/x86/kernel/unwind_orc.c > index e9cc182aa97ee..0dbabc469ce2d 100644 > --- a/arch/x86/kernel/unwind_orc.c > +++ b/arch/x86/kernel/unwind_orc.c > @@ -16,11 +16,11 @@ extern struct orc_entry __start_orc_unwind[]; > extern struct orc_entry __stop_orc_unwind[]; > > static DEFINE_MUTEX(sort_mutex); > -int *cur_orc_ip_table = __start_orc_unwind_ip; > -struct orc_entry *cur_orc_table = __start_orc_unwind; > +static int *cur_orc_ip_table = __start_orc_unwind_ip; > +static struct orc_entry *cur_orc_table = __start_orc_unwind; > > -unsigned int lookup_num_blocks; > -bool orc_init; > +static unsigned int lookup_num_blocks; > +static bool orc_init; > > static inline unsigned long orc_ip(const int *ip) > { > -- Already fixed in https://lore.kernel.org/lkml/63b5cab2e28b9c08854fc57f5b512a9ccf76ad95.1584033751.git.jpoimboe@redhat.com/ Thanks, Miroslav