Received: by 2002:a05:7208:13ce:b0:7f:395a:35b6 with SMTP id r14csp177676rbe; Wed, 28 Feb 2024 16:43:03 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCW1yijjAST3hAnhfxrDms+KTSjyDfkIcnLx6NuGFstcoNdcEWoRCUmbiE/TcVraLoeOWPX9XQGrqpt9m863dviWWo2YibK4wW82DN1UPw== X-Google-Smtp-Source: AGHT+IHANHVZXVT/PY8qo+IV/DucnMo3T/GCTIOrG9Irh+LcTTITilK5FH5e/Qsa0cOKtqPBktbA X-Received: by 2002:a17:906:48ca:b0:a44:2f23:1bdc with SMTP id d10-20020a17090648ca00b00a442f231bdcmr315323ejt.26.1709167382887; Wed, 28 Feb 2024 16:43:02 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1709167382; cv=pass; d=google.com; s=arc-20160816; b=AveXVLP2eJjFvvWEmj6t7La0tygzGU1cokSiVeVWnMWlYdTwo3JKvo4u8/4/gPLPrn Ua/c9hGNUE3z7ALInpgvLCALQH9ucx0BZSfmPM9TV60EadLmrN1sJFTGtOaG+M6VjUom zaqe3gmE1oxuDU5xyZH756isRs2JRA4DoAGg11swRm63iUHKbyy5QETLmUvVlEOfUAUA yH+HDKl3ygOWWMg4uEAF1LTFDaiZf/8LSEFASedFXD9n4drezFprI1IDLT0Znw+WZMok iew0FNA3YWS+OullwmeDYSTTeAnCqCp/qxEU5U1sVtLjoGK9NyXDPnrjz8nhlGKlJq7b rhQA== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:content-disposition:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:message-id:subject:cc :to:from:date:dkim-signature; bh=cpJ1hjVa5fe5rsuyfyf22y6ayNFINTRMjAqgj+Imkvs=; fh=AN0Hx2DXvn1ddSHpStG4fs5CXFQiL88aV+NJA+ykG7w=; b=q7znhJ1ckZFmGy8mQ2WmtDoiSPH6ew1Ox+VLnTbV47tfZItanKipylq7wRAvNS8kz6 DLCFu+k9R+yn5OO9ltj7xcQkp5xLn9EQXJi9bcJ28IrwUfPzHfMXEZ4G5iTH4Av6abNT 30ubmPGSmbSP0WeF7lN41Qn8llHO5rYjvW76WTwc/EWyDjOurtWJ6rNUJhuOl0XHmtFd K/75RbtpU3xZK37qrPqjkymX/tKuzVR1VPvTL4OVaZA8OhH93aO/YPLW1u50cS43rVEy DiF+1DSYQcEtInLHJvRUNS9KEW0/pLQS++vlRbgTMhuEH48tLUYmktuHoBq/gjfGlTjv XBkA==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=q6oqXrPC; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-85933-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-kernel+bounces-85933-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [2604:1380:4601:e00::3]) by mx.google.com with ESMTPS id sh30-20020a1709076e9e00b00a442ff387b1si68754ejc.498.2024.02.28.16.43.02 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 28 Feb 2024 16:43:02 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-85933-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=q6oqXrPC; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-85933-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) smtp.mailfrom="linux-kernel+bounces-85933-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id A13A61F248D3 for ; Thu, 29 Feb 2024 00:43:02 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 6E347168AB; Thu, 29 Feb 2024 00:42:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="q6oqXrPC" Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 98CD8184F for ; Thu, 29 Feb 2024 00:42:54 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709167374; cv=none; b=DrixLPDE/EJ7pg8tiySFl1T+jyWo/YoxDLqMxhhj7l3KGJggBr7d2QCDrwGrcnrdHfSrsMEV+kF28lMiRohy758WyY56AM2Q0r+bc3myBjACKK3x8r2Ry4VKmNHnYlGkcOj8iUdZJMGuFiuoZ3Rjfa+ZyR6KejgPcj6mtLGTbk8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709167374; c=relaxed/simple; bh=QCsl9pKp5x7LKsoIXjVP5LHrz8qCzYiE3exFfnnqcQc=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=NWFirOEyY4evO3KjfEiEIicWde01ufVX8nmh6FAg7w+hrp93FmEL6lgbU0W/qL1cIb+sGdr3/FW5U1c+Kt4160NfxvjHLV48JqC3dZArz6gGU62x0lQxLtRrZPjGItW+d6v4rizD7+t2h2q6A/LMfftd2mz/PYe6b/MTN17bhfA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=q6oqXrPC; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9359EC433C7; Thu, 29 Feb 2024 00:42:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1709167374; bh=QCsl9pKp5x7LKsoIXjVP5LHrz8qCzYiE3exFfnnqcQc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=q6oqXrPCa6UtjC0C7ukWFqBHWpF59rbCjF2fm75Y7annFJmeakMabKgpaney3chYB W07AWi694HDbvkHD8krVbY68fIt0ht21naZWLAUhzv4oC4mrVijLr1VLijgMYVlJdy WQwIKuBdn+EajpYjaw/T0NRPhVUV85k5u9/wJuRb/1EBhmINWmETJ203joHFE0SgWh KCu0WlMk7wrbNZZXB4Awn7ELYzmHxfCwoaDiSGJqQ4fYOiBY5JXWc/58ctniL2CXLT 9Aqu08Qgzvg8OgFxd9Y8ADGharVwXRheXmhm73avK/fRDiJ4qKv+tqB3b+7uc8SBxf CJ8+T/6gwUgeA== Date: Wed, 28 Feb 2024 16:42:52 -0800 From: Josh Poimboeuf To: Steven Rostedt Cc: Josh Poimboeuf , x86@kernel.org, linux-kernel@vger.kernel.org, Peter Zijlstra , Vince Weaver , Dave Jones , Jann Horn , Miroslav Benes , Andy Lutomirski , Thomas Gleixner , Masami Hiramatsu , Nilay Vaish Subject: Re: [PATCH v2 06/11] x86/unwind/orc: Convert global variables to static Message-ID: <20240229004252.awmammhp63f5udyy@treble> References: <43ae310bf7822b9862e571f36ae3474cfde8f301.1587808742.git.jpoimboe@redhat.com> <20240228183507.78c5f130@gandalf.local.home> <20240229000309.ibtuk7gr27d7cbxz@treble> <20240228191406.21e894b0@gandalf.local.home> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20240228191406.21e894b0@gandalf.local.home> On Wed, Feb 28, 2024 at 07:14:06PM -0500, Steven Rostedt wrote: > On Wed, 28 Feb 2024 16:03:09 -0800 > Josh Poimboeuf wrote: > > > On Wed, Feb 28, 2024 at 06:35:07PM -0500, Steven Rostedt wrote: > > > On Sat, 25 Apr 2020 05:03:05 -0500 > > > Josh Poimboeuf wrote: > > > > > > > These variables aren't used outside of unwind_orc.c, make them static. > > > > > > > > Also annotate some of them with '__ro_after_init', as applicable. > > > > > > So it appears that crash uses "lookup_num_blocks" to be able to do > > > back-traces with the ORC unwinder. But because it's now static, crash can no > > > longer do that. > > > > Hm, but why? Even a static variable has a known address. > > > > I'm guessing because we don't have the full dwarf info? DWARF isn't needed for that. Even the symbol table has it (as does System.map). For both globals and statics. -- Josh