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.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, 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 26EC2C433ED for ; Wed, 14 Apr 2021 12:38:25 +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 83E4961153 for ; Wed, 14 Apr 2021 12:38:24 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 83E4961153 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=age4v/3OfkusWjXtZzX3C34OMAD2OCmqayzyyaWLCOk=; b=IZaDpMUWlmbbcaMoqEi7+iuUo GzE+XI/hom+AwEB4syzNnvqj+tHEGiTUEc1OPQHVM3wIiKPEiANNpDPrIgpDAYywsPzRyM+bMCAaD 018U9u2YaFd5Vk1/OwDfSCW7xPmUrwI/s6zjpnx73WPRqi/rs9bO1mjtzIGr7nyd5QKESEsVVCGzX OFDzCz1smkWgu1wutkfdxrw1oeYI5M9JqUji+GZ8au2D1vkBk0lLCpugWgQwZolpamMh/xA+6ZVBe X76Ack2I5QFEqxmjLF8WiHfluGmaBnPta2xfVSlyrA+Pe1MLAFWQUvSrnumFOREqiUfq0Vzv7BTAr 8ocUecv+g==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1lWekm-00CbMh-1G; Wed, 14 Apr 2021 12:36:20 +0000 Received: from bombadil.infradead.org ([2607:7c80:54:e::133]) by desiato.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lWekf-00CbMK-Og for linux-arm-kernel@desiato.infradead.org; Wed, 14 Apr 2021 12:36:14 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=In-Reply-To:Content-Type:MIME-Version :References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=xc5DczXbeN8gdxrfG+aMTbwRmpOJMr2AC1MxqCx1dkg=; b=PnpmZGrktNI2SHBCy1j+LdiT4g Pj9Slwy9zcVLHTsNPdR/1aCbk5mICwajAND8KJXgUOXeudNMB/9lHFpmTKEwcYaVsgpX6NlpCNr72 3zLnU0ELasjTlYXgXx7zb9d3U01NkkrYloXC3kZWEqnvmT0GLHJPmtz1VDvuEpdDg36CxhH88+eWa KoQaav7uZ8dds/PPXsfnfr8BiYe1GKdMfgAiTuICFC72RHJB+76JicxuNao3Em1BObFCP5mi3s0cx BqYN+UsmwHaqnDuurz8d1Fw18hvYWYKiE0fNgOQE5GfCLb/rfUSvSn7BSfdHn0g7ACnXSEwFaabiS WSPNW88g==; Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lWekd-007l6j-53 for linux-arm-kernel@lists.infradead.org; Wed, 14 Apr 2021 12:36:12 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id 4D899613C7; Wed, 14 Apr 2021 12:36:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1618403770; bh=W8hZgGPrD8etDnaCxdxfCu2/xsJL8cgjruuafsxuso4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=mEL5EqktkPiD9D+IQ3bPF25l7TsKspbjBedHpBtjTwHaH0gK/4rNJMqXOtc8rQWt1 mxUi1M2Zhp1ocB3gL1l3AOPodPMbis89ZxE2b/HBrGg8FKLAgl1NW66kn42dYYpj5N HwnDMRrrAuQ8c3W8aSpbVp6XpU9Z6gi2ocZZ1zojXftgxZ6V0AV5XN7ppusjDOvRFA WZKvDJK+i060RwJ9E8Vu5kUryFPeXEn35OjJYqQaUUjZy24dfqO4/Vmq07XJNg4E5F M98PPRx5ufEg/EI8sz9nx9b+dvsjtOTtk27HIDlkuL5RubGzu9JZGDrVmfYlGM6xyO 9LYOPnY2jkE3A== Date: Wed, 14 Apr 2021 13:35:48 +0100 From: Mark Brown To: "Madhavan T. Venkataraman" Cc: Josh Poimboeuf , Mark Rutland , 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, Peter Zijlstra Subject: Re: [RFC PATCH v2 0/4] arm64: Implement stack trace reliability checks Message-ID: <20210414123548.GC4535@sirena.org.uk> References: <705993ccb34a611c75cdae0a8cb1b40f9b218ebd> <20210405204313.21346-1-madvenka@linux.microsoft.com> <20210409120859.GA51636@C02TD0UTHF1T.local> <20210409213741.kqmwyajoppuqrkge@treble> <20210412173617.GE5379@sirena.org.uk> <20210413110255.GB5586@sirena.org.uk> <714e748c-bb79-aa9a-abb5-cf5e677e847b@linux.microsoft.com> MIME-Version: 1.0 In-Reply-To: <714e748c-bb79-aa9a-abb5-cf5e677e847b@linux.microsoft.com> X-Cookie: George Orwell was an optimist. 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-20210414_053611_256661_CDCE04A1 X-CRM114-Status: GOOD ( 22.62 ) 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="===============3218926855280904500==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============3218926855280904500== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="TYecfFk8j8mZq+dy" Content-Disposition: inline --TYecfFk8j8mZq+dy Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Apr 14, 2021 at 05:23:38AM -0500, Madhavan T. Venkataraman wrote: > On 4/13/21 6:02 AM, Mark Brown wrote: > > On Mon, Apr 12, 2021 at 02:55:35PM -0500, Madhavan T. Venkataraman wrot= e: > >> 3. We are going to assume that the reliable unwinder is only for livep= atch purposes > >> and will only be invoked on a task that is not currently running. T= he task either > >=20 > > The reliable unwinder can also be invoked on itself. > I have not called out the self-directed case because I am assuming that t= he reliable unwinder > is only used for livepatch. So, AFAICT, this is applicable to the task th= at performs the > livepatch operation itself. In this case, there should be no unreliable f= unctions on the > self-directed stack trace (otherwise, livepatching would always fail). Someone might've added a probe of some kind which upsets things so there's a possibility things might fail. Like you say there's no way a system in such a state can succesfully apply a live patch but we might still run into that situation. > >> I suggest we do (3) first. Then, review the assembly functions to do (= 1). Then, review the > >> remaining ones to see which ones must be blacklisted, if any. > > I'm not clear what the concrete steps you're planning to do first are > > there - your 3 seems like a statement of assumptions. For flagging > > functions I do think it'd be safer to default to assuming that all > > SYM_CODE functions can't be unwound reliably rather than only explicitly > > listing ones that cause problems. > They are not assumptions. They are true statements. But I probably did no= t do a good > job of explaining. But Josh sent out a patch that updates the documentati= on that > explains what I said a lot better. You say true statements, I say assumptions :) --TYecfFk8j8mZq+dy Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmB24aQACgkQJNaLcl1U h9BYGAf9GM7SFhyqwsiwsjyKKhLQaHG+b/LiSgJ0iKN6/7a/tRaC5WBnJVrk5dLG WWC+hea0n3zdWT/kFu8AuBq0GHnVIZuMVav43Lu0VhBQxilRa1ySjLCgRbs+ePtj 4d31quaeFknkZVXXqsLE/42IozRJi9RFLWzPXUqUhkSELGi+Jl6zY8uYm1QonmMX I+iE5E0T8dg9FwmfRUPVs48jYlk3pYpmWxDF0P/vWmVJC3ffF6/Lbr4VIOHCyBSk rYBKmjZJZuNowGKoV8JHv4aLizk6Coq02br3534NMEUBIYOl9b9oFjkE0QvJOfor 8GiML6l1Qo5EHbfsZmRmIfWe0S6AZA== =XOVC -----END PGP SIGNATURE----- --TYecfFk8j8mZq+dy-- --===============3218926855280904500== 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 --===============3218926855280904500==--