From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f73.google.com (mail-pj1-f73.google.com [209.85.216.73]) (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 43D03327797 for ; Wed, 20 Aug 2025 16:18:32 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.73 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1755706713; cv=none; b=U0qseDDRSwivizYv382Xr8D5TYJTk0tgJ7sRB0eXWxUArS3q/ILBRYEg2zw72rCS/h9Fw/a7OydYMMFy0tP8NwzoqV7SyO9T7bT6ha4mtbu5Qm/9MkJj0eUwywbmVtOHX5oD/BvdRpJWC5Bpbdwl1Zaz0MG3eQb21AhSGkkqQ/Y= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1755706713; c=relaxed/simple; bh=aryoRv5tAqznCZnbWBlEmRqJE3caNfx+OkacUf1bQXM=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=dZQwLLh5xIy6TH4QC35TmrqHRovdTGoE+fPPFqHbaq6MLQSVuI4lRvMHNzrOn3Gr2lRe4H7K/Ghcdukt8vf57HHKWP5gzXSRXudXl7jxqj04kRuevl6zn+ES56ZttcCcWa3M1fNW9Lu1tkZaj7d5cjxMzs+IZSOKEc3vIn6mc54= 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=a7t3TJml; arc=none smtp.client-ip=209.85.216.73 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="a7t3TJml" Received: by mail-pj1-f73.google.com with SMTP id 98e67ed59e1d1-3232669f95eso100687a91.0 for ; Wed, 20 Aug 2025 09:18:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1755706711; x=1756311511; darn=vger.kernel.org; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:from:to:cc:subject:date:message-id:reply-to; bh=LbIBXkjzpro1lSKQx3DixkUfhZO5cnnCemwVjT931F8=; b=a7t3TJml2qZzGlZGJDqItorgUi5EnVdzKSjfSyRddahQsq67bi0tXTP/CcIlApUTME 2qbaqRj6sxS5++0GRewdz3WdcATITz6w5slbdAub0m6eeVkz0c65wwwFvse2dBa6BcJJ YakpNQFFDm/NT9yyy51YI9GbO+7DzPo4/XCbjcDm/r8cvKwu4c8oxGOeaamCfHKPSRON fDnw1JWh+UAkgFVUAg+26VrtNnE7IicH8WKh4bJJF0FDEvoUsPrd8r3mOJrmEB6dT7Pw 4kkw7IdFW99HhdRVXEi3NY4/pVzflOOQ/5B2KFMf4iCJIUsJCgR/MAKsdODtwV1Xtp5c qs/A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1755706711; x=1756311511; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=LbIBXkjzpro1lSKQx3DixkUfhZO5cnnCemwVjT931F8=; b=ZLZA1CmvdSxjdYvp8GjHuiAxmN41bk1noA6TB8ZtpN0N0DgM6gmW4NCjyIH22J5M+n WQYd69uwkTRjc5Az6WIu9g1VNtyl1nID6MDHlW1chaCXSJ46PA6bLPAKlKaKCBLzwTGd Wx0JJpiXvT4C/PnFBktLWE495vYCAi13r5L9hu11w89N+/tcDCVdqiLkx/8iv1twTSHK Gyco9SGj0nDIVqsRHVPsGTomtnXrsPzLkWwK01UFllyIH8UMI43zmAlfFpY2tsPccne3 is+jKO89oBQXwS0aefsdDwoTDqjlN8L9mRHLoywDzK5DzcifLztAsECzg33y6ZGLycKH KVhg== X-Forwarded-Encrypted: i=1; AJvYcCXmtT/v/eIYNr+Yw18GS+LkYYJJ3VFKKdI4VYx+myz/4SOleEaMXRdcDW+GFmWh8YTtfxBolB3axgg=@vger.kernel.org X-Gm-Message-State: AOJu0YyITPGBDAsI1Amhn+wYmYafAErlYvTu99KwGXL4kdloQ5lfMFQG NEnmXNX9pVgV9OLeKmG7HbIw89dDhPgBW9+sdJlrVUSGd8lrOGvW/9DmaBObRuO9RWUHmLiGHKI 6V5mrEA== X-Google-Smtp-Source: AGHT+IFf7xM6eNozrIlEIQuQlsTpvkezZkqf70mjUn0hYGfwmHZjVUeiPiaXieLgH9P/BZPxZ7N9paBfqZs= X-Received: from pjbof13.prod.google.com ([2002:a17:90b:39cd:b0:31f:6a10:6ea6]) (user=seanjc job=prod-delivery.src-stubby-dispatcher) by 2002:a17:90b:4a:b0:321:abd4:b108 with SMTP id 98e67ed59e1d1-324e1334fe2mr5783918a91.12.1755706711464; Wed, 20 Aug 2025 09:18:31 -0700 (PDT) Date: Wed, 20 Aug 2025 09:18:29 -0700 In-Reply-To: Precedence: bulk X-Mailing-List: linux-doc@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20250515152621.50648-1-amit@kernel.org> <20250515152621.50648-2-amit@kernel.org> <43bbb306-782b-401d-ac96-cc8ca550af7d@amd.com> Message-ID: Subject: Re: [PATCH v5 1/1] x86: kvm: svm: set up ERAPS support for guests From: Sean Christopherson To: Amit Shah Cc: Tom Lendacky , linux-kernel@vger.kernel.org, kvm@vger.kernel.org, x86@kernel.org, linux-doc@vger.kernel.org, bp@alien8.de, tglx@linutronix.de, peterz@infradead.org, jpoimboe@kernel.org, pawan.kumar.gupta@linux.intel.com, corbet@lwn.net, mingo@redhat.com, dave.hansen@linux.intel.com, hpa@zytor.com, pbonzini@redhat.com, daniel.sneddon@linux.intel.com, kai.huang@intel.com, sandipan.das@amd.com, boris.ostrovsky@oracle.com, Babu.Moger@amd.com, david.kaplan@amd.com, dwmw@amazon.co.uk, andrew.cooper3@citrix.com, amit.shah@amd.com Content-Type: text/plain; charset="us-ascii" On Wed, May 28, 2025, Amit Shah wrote: > On Mon, 2025-05-19 at 16:22 -0500, Tom Lendacky wrote: > > > +static inline void vmcb_set_flush_guest_rap(struct vmcb *vmcb) > > > +{ > > > + vmcb->control.erap_ctl |= ERAP_CONTROL_FLUSH_RAP; > > > +} > > > + > > > +static inline void vmcb_clr_flush_guest_rap(struct vmcb *vmcb) > > > +{ > > > + vmcb->control.erap_ctl &= ~ERAP_CONTROL_FLUSH_RAP; > > > +} > > > + > > > +static inline void vmcb_enable_extended_rap(struct vmcb *vmcb) > > > > s/extended/larger/ to match the bit name ? > > I also prefer it with the "larger" name, but that is a confusing bit > name -- so after the last round of review, I renamed the accessor > functions to be "better", while leaving the bit defines match what the > CPU has. > > I don't mind switching this back - anyone else have any other opinions? They both suck equally? :-) My dislike of "larger" is that it's a relative and intermediate term. What is the "smaller" size? Is there an "even larger" or "largest size"? "extended" doesn't help in any way, because that simply "solves" the problem of size ambiguity by saying absolutely nothing about the size. I also dislike "allow", because in virtualization context, "allow" usually refers to what the _guest_ can do, but in this case "allow" refers to what the CPU can do. If we want to diverge from the APM, my vote would be for something like ERAP_CONTROL_FULL_SIZE_RAP