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.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,MAILING_LIST_MULTI,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 5EB73C433DB for ; Thu, 18 Mar 2021 18:28:08 +0000 (UTC) Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) (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 DB8BC64E22 for ; Thu, 18 Mar 2021 18:28:07 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org DB8BC64E22 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=desiato.20200630; h=Sender:Content-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date: Reply-To:Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=5S4A1lNye+gLjcsKpGQ7T7IEY6cJbAbkz3UMHfThaEA=; b=hfRLGspfI7N1ZHQM5TUGbV8j+ TWQQrEtlHUpXkJ2B9nb8aPQMqCu0BzxjRMiZkoXcjqg09TReg82sqlD0ViMOUThjS24u+9rBw/x/4 aMb5DuubiYyCfk99WR5XPxTN5Xbnk81slyi5POvHbBKTAnZiboG6+t1WrLWK//azLyVjUa4+bkKpb 7uYTjH3h1CD9J98eLMie8r2kAZ+HJxF7Qq7O+7L9UcG8X3AnWRKOfDU9IVmH1pXGb4CtDin8HC5od 1R7b3WLCuqtIBhCDcQ/BKf1g07Ze0KCVDHFhuWM4aUAIdt4lf+VCB7vY2tK1vuYJKKaUygtcHQgc0 +o4aQVLVg==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1lMxLd-005qWO-5z; Thu, 18 Mar 2021 18:26:17 +0000 Received: from mail.kernel.org ([198.145.29.99]) by desiato.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lMxLY-005qVD-T8 for linux-arm-kernel@lists.infradead.org; Thu, 18 Mar 2021 18:26:14 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id E01F364F1F; Thu, 18 Mar 2021 18:26:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1616091971; bh=LA+r6D7m6AvZU8wuhj/IzzDyDVbkFpktbFnAV3fhL28=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=AzB/AcnyIbEKFHztNHVvwDayVAqwTDd2mSG/o6sbtKIyBZHKlff8PIf5SIWr2iKw5 mIhJ4+JMTZXoUTvBot/OXKopUL2AvoxNkdOjKM1iTs90ghNW+Vm2f5NdXZh3qv/w+L xXR2hLGjovCMgLODZbytYaIdXUE4iqXfLsqM/HBfHs3+SIuadV1ZMsTnjdWUsz0Bzj VoK73kMvzFF1dyK0VJnPxulq539B0aJoLOHIuauc/xYi2LDIrOJcdkf4cwCQE7wjfN hLZBRmTqbXtQED9JX5m79Bs2reMnTIo9thxv9rtx8zWFAsvezlZD4QabM0wca4Oq5Z uow2Ffn7qEeHQ== Date: Thu, 18 Mar 2021 18:26:07 +0000 From: Mark Brown To: madvenka@linux.microsoft.com Cc: mark.rutland@arm.com, jpoimboe@redhat.com, jthierry@redhat.com, catalin.marinas@arm.com, will@kernel.org, linux-arm-kernel@lists.infradead.org, live-patching@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [RFC PATCH v2 3/8] arm64: Terminate the stack trace at TASK_FRAME and EL0_FRAME Message-ID: <20210318182607.GO5469@sirena.org.uk> References: <5997dfe8d261a3a543667b83c902883c1e4bd270> <20210315165800.5948-1-madvenka@linux.microsoft.com> <20210315165800.5948-4-madvenka@linux.microsoft.com> MIME-Version: 1.0 In-Reply-To: <20210315165800.5948-4-madvenka@linux.microsoft.com> X-Cookie: You are false data. User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210318_182613_176108_61DE2D7C X-CRM114-Status: GOOD ( 11.78 ) 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-Type: multipart/mixed; boundary="===============3375742045124523797==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============3375742045124523797== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="z87VqPJ/HsYrR2WM" Content-Disposition: inline --z87VqPJ/HsYrR2WM Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Mar 15, 2021 at 11:57:55AM -0500, madvenka@linux.microsoft.com wrot= e: > + /* Terminal record, nothing to unwind */ > + if (fp =3D=3D (unsigned long) regs->stackframe) { > + if (regs->frame_type =3D=3D TASK_FRAME || > + regs->frame_type =3D=3D EL0_FRAME) > + return -ENOENT; > return -EINVAL; > + } This is conflating the reliable stacktrace checks (which your series will later flag up with frame->reliable) with verifying that we found the bottom of the stack by looking for this terminal stack frame record. For the purposes of determining if the unwinder got to the bottom of the stack we don't care what stack type we're looking at, we just care if it managed to walk to this defined final record. =20 At the minute nothing except reliable stack trace has any intention of checking the specific return code but it's clearer to be consistent. --z87VqPJ/HsYrR2WM Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmBTmz8ACgkQJNaLcl1U h9Cf4wf9GiUVSakCRf5uwTSCNkO20M1jY8gGOIf0HAWMgakyvTKAsskA/ejAOckA rAFeoOQAiW+U82Xxi3cvoM7Q9/nPp4D0nUEbSVN+LEoNM1BD3PXuYwrv8BLr2mh8 Ow3HG0jZ87pBCvQa0tKD+l7sGJEDplfsx/GeM/diTBn8YTquajA5HQdDlJEJeWzz vhJiQlkNLmsJVAk3rJ+IuzteoOb6L7CfkdVT+aBtfa1TLqRK3XmLJU5GLe+nbHBP 0/MdK5UMb7jvvZnfn3jDyFpYzhk8RgUu9VYrLYHeDAr1OdpYdY/KB/zrcSb58brt 3jGrgdNQgiIMaAToV9BX4H/4nslg3A== =jp9g -----END PGP SIGNATURE----- --z87VqPJ/HsYrR2WM-- --===============3375742045124523797== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============3375742045124523797==--