From nobody Mon Feb 9 05:29:26 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; dmarc=pass(p=reject dis=none) header.from=citrix.com ARC-Seal: i=1; a=rsa-sha256; t=1712777257; cv=none; d=zohomail.com; s=zohoarc; b=Kc6ckwoQOw2gQbjrEa9g9O6GKuykMwdFoB+pPVH39x1iRehR4rbHxzbXStcoL4dDfq/Dh0yF7FHCDjcolHn/0pCw5g09dvxOzPjelS5jYeIWbew890epMEAR7heQSklPvuxCWXLns2GTIAU42gLuNiFRnZkcwnd+OkEVaFa0vq4= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1712777257; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=P1XojucwcBoOj2YCVjGcUgrZ4dJuDGeY0F252K1EUEU=; b=CgTxRAfR+HC1kQB7Iybux1Ge1XvDM6MVP9FJ2j8xTfp9uIsQfP9F4Hy2FQtFECx0Ufr7hBLVpDrqOVXpzS9eyVDPv39CMaKgEY4eXztPvo0w4tPECi/TX8rYZNVnDaT9ZeJZx/qT92Z0VtFA/wStz8otEsAqWyXl6Os1hDdU6Vg= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; dmarc=pass header.from= (p=reject dis=none) Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1712777257043237.71091631202353; Wed, 10 Apr 2024 12:27:37 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.703572.1099260 (Exim 4.92) (envelope-from ) id 1rudb9-0003Xk-RK; Wed, 10 Apr 2024 19:27:07 +0000 Received: by outflank-mailman (output) from mailman id 703572.1099260; Wed, 10 Apr 2024 19:27:07 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1rudb9-0003Xd-Nh; Wed, 10 Apr 2024 19:27:07 +0000 Received: by outflank-mailman (input) for mailman id 703572; Wed, 10 Apr 2024 19:27:06 +0000 Received: from se1-gles-flk1-in.inumbo.com ([94.247.172.50] helo=se1-gles-flk1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1rudb8-0003XU-0T for xen-devel@lists.xenproject.org; Wed, 10 Apr 2024 19:27:06 +0000 Received: from mail-ed1-x529.google.com (mail-ed1-x529.google.com [2a00:1450:4864:20::529]) by se1-gles-flk1.inumbo.com (Halon) with ESMTPS id 4e7e1f93-f770-11ee-94a3-07e782e9044d; Wed, 10 Apr 2024 21:27:02 +0200 (CEST) Received: by mail-ed1-x529.google.com with SMTP id 4fb4d7f45d1cf-56e2b41187aso266512a12.1 for ; Wed, 10 Apr 2024 12:27:02 -0700 (PDT) Received: from andrewcoop.citrite.net (default-46-102-197-194.interdsl.co.uk. [46.102.197.194]) by smtp.gmail.com with ESMTPSA id x26-20020aa7d6da000000b0056e460eda12sm5449178edr.55.2024.04.10.12.27.00 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 10 Apr 2024 12:27:00 -0700 (PDT) X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 4e7e1f93-f770-11ee-94a3-07e782e9044d DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=citrix.com; s=google; t=1712777221; x=1713382021; darn=lists.xenproject.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=P1XojucwcBoOj2YCVjGcUgrZ4dJuDGeY0F252K1EUEU=; b=B5BBMatvWycRsAk25E7Z3CX8jsECCn6QegPF91ti281cz9WyWmjAwuaKx1IrCY3wGN IywLNGz5J/wm7M/qKemae3u4oUjYnRzynQ2aPL0bG97uvU1mICz1kAkXeZzZBin8h3WX 4lHuKiqWirZdeqiBGCTedgCmuPWmkdlPRVpNc= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1712777221; x=1713382021; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=P1XojucwcBoOj2YCVjGcUgrZ4dJuDGeY0F252K1EUEU=; b=XI+p3nbUmGc6HtSgHWt9XecYEtEVb+tCi/vflqPJE0lZ+XddjSi32DIomaYPlliwhC b81dr4d8QAU6DT+uqf7+6gBDNVX8B543iDd5X0cz/eTixkUd2AaUsZ7NkBqrSGcA+NXl NMkxoP75QRLVKmUyoQ0W7wcGMR7YjbE1QiJGRgc/0MhowRftNrOxLjiaulcZVTslG3N3 k3rWsFsMutjyY9KMnlD/mNrbxMeaoYIvE/Kpq8+5bpBHi1l1TBJjMzM7eGcrafEImrnm k4XJKDT7H67m4YcdoH04Qz4iNCHHjcWGF9TAJmMuBE++/VkmFeBgp8IF5UHYNNdufGoi P85w== X-Gm-Message-State: AOJu0Yzbj292RTqRESJBAcQS9yUfKOhLSns5iJ+AlI581dxQ5cT+Y1Lh vAK3xqx/25G5GzL/Rz8OqIrb+d6fdZ6vnzhjBXNkkbyIg3rD5VnVXwVnoNYwVI4auspxYY+ONaw lX6E= X-Google-Smtp-Source: AGHT+IFKJnRqQNCPURbLiHRZvUssBKJtjoArog7gHJPsbHES3mvs4XWzpcKgk9CxKHQcxftplTlYnQ== X-Received: by 2002:a50:a45a:0:b0:56e:1e47:c0b2 with SMTP id v26-20020a50a45a000000b0056e1e47c0b2mr480359edb.4.1712777221282; Wed, 10 Apr 2024 12:27:01 -0700 (PDT) From: Andrew Cooper To: Xen-devel Cc: Andrew Cooper , Jan Beulich , =?UTF-8?q?Roger=20Pau=20Monn=C3=A9?= , Stefano Stabellini , "consulting @ bugseng . com" , Roberto Bagnara , Federico Serafini , Nicola Vetrini Subject: [PATCH] xen/nospec: Remove unreachable code Date: Wed, 10 Apr 2024 20:26:59 +0100 Message-Id: <20240410192659.1490271-1-andrew.cooper3@citrix.com> X-Mailer: git-send-email 2.30.2 MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @citrix.com) X-ZM-MESSAGEID: 1712777257954100001 When CONFIG_SPECULATIVE_HARDEN_LOCK is active, this reads: static always_inline bool lock_evaluate_nospec(bool condition) { return arch_lock_evaluate_nospec(condition); return condition; } Insert an #else to take out the second return. Fixes: 7ef0084418e1 ("x86/spinlock: introduce support for blocking speculat= ion into critical regions") Signed-off-by: Andrew Cooper Reviewed-by: Luca Fancellu Reviewed-by: Stefano Stabellini --- CC: Jan Beulich CC: Roger Pau Monn=C3=A9 CC: Stefano Stabellini CC: consulting@bugseng.com CC: Roberto Bagnara CC: Federico Serafini CC: Nicola Vetrini --- xen/include/xen/nospec.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/xen/include/xen/nospec.h b/xen/include/xen/nospec.h index 9fb15aa26aa9..828dbd4e0ad8 100644 --- a/xen/include/xen/nospec.h +++ b/xen/include/xen/nospec.h @@ -82,8 +82,9 @@ static always_inline bool lock_evaluate_nospec(bool condi= tion) { #ifdef CONFIG_SPECULATIVE_HARDEN_LOCK return arch_lock_evaluate_nospec(condition); -#endif +#else return condition; +#endif } =20 #endif /* XEN_NOSPEC_H */ base-commit: 0e7ea8ca5fc9bce9248414f6aaf2dc861abd45d9 --=20 2.30.2