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=-2.5 required=3.0 tests=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 49473C2BA83 for ; Fri, 14 Feb 2020 15:04:41 +0000 (UTC) Received: from mm01.cs.columbia.edu (mm01.cs.columbia.edu [128.59.11.253]) by mail.kernel.org (Postfix) with ESMTP id D0A9224649 for ; Fri, 14 Feb 2020 15:04:40 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org D0A9224649 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=kvmarm-bounces@lists.cs.columbia.edu Received: from localhost (localhost [127.0.0.1]) by mm01.cs.columbia.edu (Postfix) with ESMTP id 247624AEA5; Fri, 14 Feb 2020 10:04:40 -0500 (EST) X-Virus-Scanned: at lists.cs.columbia.edu Received: from mm01.cs.columbia.edu ([127.0.0.1]) by localhost (mm01.cs.columbia.edu [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id CFe8iQzLM-MQ; Fri, 14 Feb 2020 10:04:39 -0500 (EST) Received: from mm01.cs.columbia.edu (localhost [127.0.0.1]) by mm01.cs.columbia.edu (Postfix) with ESMTP id 15E4A4AE95; Fri, 14 Feb 2020 10:04:39 -0500 (EST) Received: from localhost (localhost [127.0.0.1]) by mm01.cs.columbia.edu (Postfix) with ESMTP id 36BFD4AE95 for ; Fri, 14 Feb 2020 10:04:38 -0500 (EST) X-Virus-Scanned: at lists.cs.columbia.edu Received: from mm01.cs.columbia.edu ([127.0.0.1]) by localhost (mm01.cs.columbia.edu [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id I7+ugZglqY5H for ; Fri, 14 Feb 2020 10:04:37 -0500 (EST) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mm01.cs.columbia.edu (Postfix) with ESMTP id 24ED14A5A3 for ; Fri, 14 Feb 2020 10:04:37 -0500 (EST) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id B4156328; Fri, 14 Feb 2020 07:04:36 -0800 (PST) Received: from localhost (unknown [10.37.6.21]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 363BC3F6CF; Fri, 14 Feb 2020 07:04:36 -0800 (PST) Date: Fri, 14 Feb 2020 15:04:34 +0000 From: Mark Brown To: Marc Zyngier Subject: Re: [PATCH] arm64: kvm: Annotate assembly using modern annoations Message-ID: <20200214150434.GO4827@sirena.org.uk> References: <20200213153820.32049-1-broonie@kernel.org> <20200214114027.GA4827@sirena.org.uk> MIME-Version: 1.0 In-Reply-To: X-Cookie: Shipping not included. User-Agent: Mutt/1.10.1 (2018-07-13) Cc: Catalin Marinas , Will Deacon , kvmarm@lists.cs.columbia.edu, linux-arm-kernel@lists.infradead.org X-BeenThere: kvmarm@lists.cs.columbia.edu X-Mailman-Version: 2.1.14 Precedence: list List-Id: Where KVM/ARM decisions are made List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: multipart/mixed; boundary="===============4245105576788195669==" Errors-To: kvmarm-bounces@lists.cs.columbia.edu Sender: kvmarm-bounces@lists.cs.columbia.edu --===============4245105576788195669== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="LwbuP8dfxhLLLUfV" Content-Disposition: inline --LwbuP8dfxhLLLUfV Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Fri, Feb 14, 2020 at 02:19:18PM +0000, Marc Zyngier wrote: > > > > .align 11 > > > > -ENTRY(__bp_harden_hyp_vecs_start) > > > > +SYM_CODE_START_NOALIGN(__bp_harden_hyp_vecs) > > > > +SYM_INNER_LABEL(__bp_harden_hyp_vecs_start, SYM_L_GLOBAL) > > > Why isn't SYM_CODE_START_NOALIGN enough? And what is the rational for > > The _start and _end labels that were there before are explicitly > > referenced by code, removing them would break the build. > But if we're going to clean things up, I'd rather we actually do that. > The only time __bp_harden_hyp_vecs_end is used is when computing the > size of the vectors, and that'd better be BP_HARDEN_EL2_SLOTS * 2kB > (which can be statically asserted at compile time). I just realized that the same structure is also being used for __smccc_workaround_1_smc which doesn't have such an obviously fixed size - am I missing something with that one? I'll take a look at it anyway. --LwbuP8dfxhLLLUfV Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAl5GtwEACgkQJNaLcl1U h9C6/gf/dZ6CtQcwHYEN6oFASyB51OXGzJdcZkv/VIX98G/ByskCFG7dgYE4gsG6 hyT6EQu0CcXoTfCAz4FqNsryk1KYMDx71IAc5qrJlXu9iRyYDYB9awTGRsuU2On5 v8Xvha0NBJlHNC17Dk/NfbM1j/urZON4L5qUExVNdxSENwn+oPulMswqC5embVAh nDaRYqmNJlqAL6RAJ14TLc6+jMOUU3QHcl77ZVsh24zxBNLrSw9wunUmSGG3utGo 3yfMzcOFRWZTOrL31/kWHd1CsKgglwwMf+kt/sKHfdAFvMcNfHToSZVh2k+sobE8 mjF9n7OPM3TwxJAnhonkX1OX6lYC6g== =e5Z9 -----END PGP SIGNATURE----- --LwbuP8dfxhLLLUfV-- --===============4245105576788195669== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ kvmarm mailing list kvmarm@lists.cs.columbia.edu https://lists.cs.columbia.edu/mailman/listinfo/kvmarm --===============4245105576788195669==-- 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=-2.5 required=3.0 tests=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 9FF75C2BA83 for ; Fri, 14 Feb 2020 15:04:48 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 72F8B24649 for ; Fri, 14 Feb 2020 15:04:48 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="RnTPMcr9" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 72F8B24649 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+infradead-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=bombadil.20170209; h=Sender:Content-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject: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=25bQ/JlSopfaOlpZeMuFobnajLZFnimd8FvywyIJuTI=; b=RnTPMcr9FQv3ntmhQTbdKyHVN EzwgMwGhDKoFKY8x9Hx2YnJPaf4ZZcu+eEimEnntAybHzTd7bR67tYlCvyr75UisErYOIrdLn1uOM e4Hn3SX29Lfiefzh9bUhNZjcNABUZ4diwYPvVT3iWsqwspxM+ytKRzBC69Spu8RSGSemBSOBAvpEM p7Pt6+LQa1B55/oSiuLy7CQPp6VZTAjKFqt185zS0WS31MI1rNPMworQ2fz+RmcX0KK5o2rCKgyuf PMk++8Q+YMN03Cy48+d4vWXYyIPWScf8KrJBN+k4tdiUNqWPsoekTkK4tD6ixtCyqjtQPJCCxhybM 0DrHhlW7A==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1j2cWG-0008Jc-T3; Fri, 14 Feb 2020 15:04:40 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1j2cWD-0008J6-RG for linux-arm-kernel@lists.infradead.org; Fri, 14 Feb 2020 15:04:39 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id B4156328; Fri, 14 Feb 2020 07:04:36 -0800 (PST) Received: from localhost (unknown [10.37.6.21]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 363BC3F6CF; Fri, 14 Feb 2020 07:04:36 -0800 (PST) Date: Fri, 14 Feb 2020 15:04:34 +0000 From: Mark Brown To: Marc Zyngier Subject: Re: [PATCH] arm64: kvm: Annotate assembly using modern annoations Message-ID: <20200214150434.GO4827@sirena.org.uk> References: <20200213153820.32049-1-broonie@kernel.org> <20200214114027.GA4827@sirena.org.uk> MIME-Version: 1.0 In-Reply-To: X-Cookie: Shipping not included. 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-20200214_070437_922668_28744427 X-CRM114-Status: GOOD ( 13.35 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Suzuki K Poulose , Catalin Marinas , James Morse , Julien Thierry , Will Deacon , kvmarm@lists.cs.columbia.edu, linux-arm-kernel@lists.infradead.org Content-Type: multipart/mixed; boundary="===============7411450995719932313==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============7411450995719932313== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="LwbuP8dfxhLLLUfV" Content-Disposition: inline --LwbuP8dfxhLLLUfV Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Fri, Feb 14, 2020 at 02:19:18PM +0000, Marc Zyngier wrote: > > > > .align 11 > > > > -ENTRY(__bp_harden_hyp_vecs_start) > > > > +SYM_CODE_START_NOALIGN(__bp_harden_hyp_vecs) > > > > +SYM_INNER_LABEL(__bp_harden_hyp_vecs_start, SYM_L_GLOBAL) > > > Why isn't SYM_CODE_START_NOALIGN enough? And what is the rational for > > The _start and _end labels that were there before are explicitly > > referenced by code, removing them would break the build. > But if we're going to clean things up, I'd rather we actually do that. > The only time __bp_harden_hyp_vecs_end is used is when computing the > size of the vectors, and that'd better be BP_HARDEN_EL2_SLOTS * 2kB > (which can be statically asserted at compile time). I just realized that the same structure is also being used for __smccc_workaround_1_smc which doesn't have such an obviously fixed size - am I missing something with that one? I'll take a look at it anyway. --LwbuP8dfxhLLLUfV Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAl5GtwEACgkQJNaLcl1U h9C6/gf/dZ6CtQcwHYEN6oFASyB51OXGzJdcZkv/VIX98G/ByskCFG7dgYE4gsG6 hyT6EQu0CcXoTfCAz4FqNsryk1KYMDx71IAc5qrJlXu9iRyYDYB9awTGRsuU2On5 v8Xvha0NBJlHNC17Dk/NfbM1j/urZON4L5qUExVNdxSENwn+oPulMswqC5embVAh nDaRYqmNJlqAL6RAJ14TLc6+jMOUU3QHcl77ZVsh24zxBNLrSw9wunUmSGG3utGo 3yfMzcOFRWZTOrL31/kWHd1CsKgglwwMf+kt/sKHfdAFvMcNfHToSZVh2k+sobE8 mjF9n7OPM3TwxJAnhonkX1OX6lYC6g== =e5Z9 -----END PGP SIGNATURE----- --LwbuP8dfxhLLLUfV-- --===============7411450995719932313== 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 --===============7411450995719932313==--