From nobody Tue Dec 16 14:38:33 2025 Received: from mail-pg1-f201.google.com (mail-pg1-f201.google.com [209.85.215.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6B3DD393DE7 for ; Thu, 30 Oct 2025 20:10:48 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.215.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1761855050; cv=none; b=KJmzxiPNwBwX/AI/NGO8BDCM64+voMe3QMAfue9zXGe4SvuZaIjBB7dcDgeaB+74jDb0/bKxJEdjDMEV42w2jkJIUAAdizpyENKRRpIofLl3zYkkxkFB8TxcIbFZ0HPMNhfP30ZgntwFv344KvzRtcdU1FF+Huj8cdkIks9iGSM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1761855050; c=relaxed/simple; bh=4WxHGoE93VOocFQ5wN0AUJ/80sFNRiS2hdmgOFi4y/A=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=ftg/wYif5ZkjWZ9vAQuHjy/4rNV4681LeCXef3eZJgn9efDiZ9i9FYR5H/0r5Um5s9cNc5xiVdAXNCkt9IZtOhuRfkE1I8rwq2rGJ59sMRvWmChH/o0ig275sx6PNfPafleiT07bAGKZgVj0/AsWWfaN65/xleiRzqh55RqBl9M= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--seanjc.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=VmRawTbN; arc=none smtp.client-ip=209.85.215.201 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--seanjc.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="VmRawTbN" Received: by mail-pg1-f201.google.com with SMTP id 41be03b00d2f7-b55283ff3fcso965942a12.3 for ; Thu, 30 Oct 2025 13:10:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1761855048; x=1762459848; darn=vger.kernel.org; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:reply-to:from:to:cc:subject:date:message-id:reply-to; bh=q3vUMZT0yYe1EXEiRgC8c4U03xvX09k6brWY3B4yx5I=; b=VmRawTbNAViiJiQpj/T71gWATAeSlhmr7dzX5KPAUeTJpZ1tsTsuufgapB047yQAmP J8yRP2yThpvFvFlSDMqdfD0g4SKcP7WpHKZQciWcQn/SkCPXVumuPE9P0dPM00sqE7xM fke0EbpU7W5Ri39dFAj1nL7srVlmwNY7oj4Go39PIUqs54a1+J9hEZ6bi/09u/jIrDLa VSl44p1XarJ51KQK7Zuv8dJPYoCDJince6wGoV7Sz06tB0rHw2ntJh0kgCXkfCfmJUHH Fq/p1+EMBMiFqSWiV4HuOol/v6nxdgVqVKn5uPzIBkGwdWKCanB+pgkYeZ7Al8XEN9mA AK4g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1761855048; x=1762459848; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:reply-to:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=q3vUMZT0yYe1EXEiRgC8c4U03xvX09k6brWY3B4yx5I=; b=B3eSzw2ZP07vDJOEp5Zix37dhNT7WpyxrP0XLNISq1v2RN9uES/+eG5fLAs6vT6jwz 3FlLpbWFey1lAdFY8PUghYERa77S9JHR3U6OnSTKvDihq3eBLGNK4CfsH/XNMRrYW/JK IZbDYNzUZjECepMgx8URz6CObLsGGxyOsuB8sVhNf7nARif8i7ho3k+KZHsv5O5MoCaY MOprb1sHdYWZ9j1TVUTpIiTxUfQvo1UMUxpt4JhT0rnIGChT/U7J+jpImkzgkkvXOyE0 IVrN/a3ONE5l3DpiyoABzrW3k+uuV7OkI6v9rOBMxIIcSc73JAo8ftVrlcHs41CSYU8l mZPw== X-Forwarded-Encrypted: i=1; AJvYcCUogw96ZguVAUUayR5VzbhbkAsqQVEoq1GCEESUk2XPd2AkRPoIp5wAyR739TSze6jyT2JVGEkgnJ+mpDw=@vger.kernel.org X-Gm-Message-State: AOJu0YzDHb6G2RiT9MmMF0PSiaMROs4577mb7mn5OXNXq87oz1kHnSRW J111cJT81fZCAwLYqv96yTIktq7/CeQ1jDqrVOnqe5yw6T/mVhzVSe8g4P7fTgJzz8LrZM0KXad SFPpffg== X-Google-Smtp-Source: AGHT+IGMIUiQfnFXRwKgU4xakuBoSsZwl0KV10BIrj34ojpFyNKXcMvJLG+bxMBFJqWMl0yCEf0bmf3P3I0= X-Received: from pltj3.prod.google.com ([2002:a17:902:76c3:b0:294:8e58:7348]) (user=seanjc job=prod-delivery.src-stubby-dispatcher) by 2002:a17:903:2348:b0:295:1a63:57b0 with SMTP id d9443c01a7336-2951a635c07mr12598385ad.23.1761855047670; Thu, 30 Oct 2025 13:10:47 -0700 (PDT) Reply-To: Sean Christopherson Date: Thu, 30 Oct 2025 13:09:42 -0700 In-Reply-To: <20251030200951.3402865-1-seanjc@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20251030200951.3402865-1-seanjc@google.com> X-Mailer: git-send-email 2.51.1.930.gacf6e81ea2-goog Message-ID: <20251030200951.3402865-20-seanjc@google.com> Subject: [PATCH v4 19/28] KVM: TDX: Derive error argument names from the local variable names From: Sean Christopherson To: Marc Zyngier , Oliver Upton , Tianrui Zhao , Bibo Mao , Huacai Chen , Madhavan Srinivasan , Anup Patel , Paul Walmsley , Palmer Dabbelt , Albert Ou , Christian Borntraeger , Janosch Frank , Claudio Imbrenda , Sean Christopherson , Paolo Bonzini , "Kirill A. Shutemov" Cc: linux-arm-kernel@lists.infradead.org, kvmarm@lists.linux.dev, kvm@vger.kernel.org, loongarch@lists.linux.dev, linux-mips@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, kvm-riscv@lists.infradead.org, linux-riscv@lists.infradead.org, x86@kernel.org, linux-coco@lists.linux.dev, linux-kernel@vger.kernel.org, Ira Weiny , Kai Huang , Binbin Wu , Michael Roth , Yan Zhao , Vishal Annapurve , Rick Edgecombe , Ackerley Tng Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" When printing SEAMCALL errors, use the name of the variable holding an error parameter instead of the register from whence it came, so that flows which use descriptive variable names will similarly print descriptive error messages. Suggested-by: Rick Edgecombe Signed-off-by: Sean Christopherson Reviewed-by: Binbin Wu --- arch/x86/kvm/vmx/tdx.c | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/arch/x86/kvm/vmx/tdx.c b/arch/x86/kvm/vmx/tdx.c index 5e6f2d8b6014..63d4609cc3bc 100644 --- a/arch/x86/kvm/vmx/tdx.c +++ b/arch/x86/kvm/vmx/tdx.c @@ -41,14 +41,15 @@ #define TDX_BUG_ON(__err, __fn, __kvm) \ __TDX_BUG_ON(__err, #__fn, __kvm, "%s", "") =20 -#define TDX_BUG_ON_1(__err, __fn, __rcx, __kvm) \ - __TDX_BUG_ON(__err, #__fn, __kvm, ", rcx 0x%llx", __rcx) +#define TDX_BUG_ON_1(__err, __fn, a1, __kvm) \ + __TDX_BUG_ON(__err, #__fn, __kvm, ", " #a1 " 0x%llx", a1) =20 -#define TDX_BUG_ON_2(__err, __fn, __rcx, __rdx, __kvm) \ - __TDX_BUG_ON(__err, #__fn, __kvm, ", rcx 0x%llx, rdx 0x%llx", __rcx, __rd= x) +#define TDX_BUG_ON_2(__err, __fn, a1, a2, __kvm) \ + __TDX_BUG_ON(__err, #__fn, __kvm, ", " #a1 " 0x%llx, " #a2 " 0x%llx", a1,= a2) =20 -#define TDX_BUG_ON_3(__err, __fn, __rcx, __rdx, __r8, __kvm) \ - __TDX_BUG_ON(__err, #__fn, __kvm, ", rcx 0x%llx, rdx 0x%llx, r8 0x%llx", = __rcx, __rdx, __r8) +#define TDX_BUG_ON_3(__err, __fn, a1, a2, a3, __kvm) \ + __TDX_BUG_ON(__err, #__fn, __kvm, ", " #a1 " 0x%llx, " #a2 ", 0x%llx, " #= a3 " 0x%llx", \ + a1, a2, a3) =20 =20 bool enable_tdx __ro_after_init; --=20 2.51.1.930.gacf6e81ea2-goog