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 3661539FCB0 for ; Mon, 6 Apr 2026 23:27:36 +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=1775518057; cv=none; b=X7NmQlI6LwXK8Kc5zdPVGW4qD/y89ys0ArA3M1lp2EEP/g5mlsdEESOzBwGKyZqjFlWDtBdYlhoveZJeufhIalB2YP7JfxVB2uRp1hfz1C+t2drOV0mA8r8OFr476Uq0yynyVaqzvv/t+Xa9hrJPFyna0Q/TPpjJNiU+g8tJo4k= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775518057; c=relaxed/simple; bh=FRR5sT0zQ8QWo/7kVJ3OR2zaSQRcCfEM05VHXvhpgd0=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=NNBegP+kAi14/7Oi+D134a0XBAS/laT6O+r4fbxAlFrZ8LtUjLaMaOzCxrVxzC3dllMG4CH8WQydt0s/nePF//rWSrvwQxMvp8Z4skyax7xTSsQGWrmPuwgVo0DaXE4iL3Kk86CKdfv0F5oDxJTHmCG+4F1VUcTLmY0Fe6WF6jE= 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=JF1Fteto; 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="JF1Fteto" Received: by mail-pj1-f73.google.com with SMTP id 98e67ed59e1d1-35c0cbe0f64so10682555a91.0 for ; Mon, 06 Apr 2026 16:27:36 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20251104; t=1775518056; x=1776122856; 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=SJTcZf1UujVpWDgwH0mVsCXrQUf/rG8HnnvzMBPDQG0=; b=JF1FtetoaC+OMcNZ0zG0TBp7xLgLwlwG2Ndt+w/hH3lV4s1EYUNvGoBto4bEY6W3o9 R4TkUnCG+nV+ESGUrfxk02KIeD433mI8LEOaqj/sj+7urmH1gBRjaVsrR9zsGIj1Qwtl jEhOvhgX8FdFmka3DFGKY4XGfs42Uv+k3sb4kE+i/kUp5KEvdOJLyapvHUA1uQD0/W6+ Sc6TqAgDjQY1iL/rxywukMAP7FFRCpAIM/gzUury85jY2hwahom+4dJlW4Foiqtk4WFN CeKNyQrOGUaeeEO1tIIi9625+aAzx39TjEwXuhAWoerOXk5yJ3iZVHe3hFxOxM7X9R7q jYMg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1775518056; x=1776122856; 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=SJTcZf1UujVpWDgwH0mVsCXrQUf/rG8HnnvzMBPDQG0=; b=J6W7zCxchx9lf5KziOrdwsJljeUW2aSHqWqmeclnSX8EN0kdBmewYJF2H0OiKxjuIE ZtKQcV71fbqoI9EW9T/oreuBV447Nt+pQQOtB7kCja22OJOLyQkSkxfaM/uQGst3w6i0 pp/AQ71VLAi4d9LWvWVRd29aDtGFS0aykGrEGxObS3tAP0VDhiaRRiFqP6Y/n3YWJS6g xwUeSy5+xbLu1bb5ra40Z1+fWOpGD+gBd4eSQfgzYJ7GBA9BvW2aLYd3UYbC+2arer8D aPm9uO2HlBKiC4XlpuhPTK6h/LnFYU0Y5FMQGHFqVezYUkaBdKhLOFclMbEosMp1+2Vw v1UQ== X-Forwarded-Encrypted: i=1; AJvYcCU3sJfn7V2eYsh5jjsHkTlqoHrSfno7Xz7NyJHK6GHsmHUvmuVK/o1otBFrJ1SEMbMTjYjPONEVi1iBq1M=@vger.kernel.org X-Gm-Message-State: AOJu0Yxv04TtJTEee6R8bfW56qQ0tRDukN2MuTkXJ9HW57zc9vDeXpso tVhBZ9bLCqGjPZfuAMsnN8zfROaEijKHMcRKB2b+CQFpon+NwT2DeC9w00MqWpNrSPVc3CpLLVp JJt/o+w== X-Received: from pjre2.prod.google.com ([2002:a17:90a:b382:b0:35d:a63e:32e9]) (user=seanjc job=prod-delivery.src-stubby-dispatcher) by 2002:a17:90b:1d49:b0:35d:9927:e023 with SMTP id 98e67ed59e1d1-35de67df8c1mr13530992a91.8.1775518055475; Mon, 06 Apr 2026 16:27:35 -0700 (PDT) Date: Mon, 6 Apr 2026 16:27:34 -0700 In-Reply-To: <20260327234023.2659476-2-jmattson@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20260327234023.2659476-1-jmattson@google.com> <20260327234023.2659476-2-jmattson@google.com> Message-ID: Subject: Re: [PATCH v7 1/9] KVM: x86: Define KVM_X86_QUIRK_NESTED_SVM_SHARED_PAT From: Sean Christopherson To: Jim Mattson Cc: Paolo Bonzini , Jonathan Corbet , Shuah Khan , Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , x86@kernel.org, "H. Peter Anvin" , kvm@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, Yosry Ahmed Content-Type: text/plain; charset="us-ascii" On Fri, Mar 27, 2026, Jim Mattson wrote: > diff --git a/arch/x86/kvm/svm/svm.h b/arch/x86/kvm/svm/svm.h > index ff1e4b4dc998..74014110b550 100644 > --- a/arch/x86/kvm/svm/svm.h > +++ b/arch/x86/kvm/svm/svm.h > @@ -616,6 +616,17 @@ static inline bool nested_npt_enabled(struct vcpu_svm *svm) > return svm->nested.ctl.misc_ctl & SVM_MISC_ENABLE_NP; > } > > +static inline bool l2_has_separate_pat(struct vcpu_svm *svm) Take @vcpu instead of @svm. All of the callers have a "vcpu", but not all have a local "svm". That will shorten the quirk check far enough to let it poke out. > +{ > + /* > + * If KVM_X86_QUIRK_NESTED_SVM_SHARED_PAT is disabled while a vCPU > + * is running, the L2 IA32_PAT semantics for that vCPU are undefined. > + */ > + return nested_npt_enabled(svm) && > + !kvm_check_has_quirk(svm->vcpu.kvm, > + KVM_X86_QUIRK_NESTED_SVM_SHARED_PAT); Align indentation. With the @svm => @vcpu change, this becomes: return nested_npt_enabled(to_svm(vcpu)) && !kvm_check_has_quirk(vcpu->kvm, KVM_X86_QUIRK_NESTED_SVM_SHARED_PAT); > +} > + > static inline bool nested_vnmi_enabled(struct vcpu_svm *svm) > { > return guest_cpu_cap_has(&svm->vcpu, X86_FEATURE_VNMI) && > -- > 2.53.0.1018.g2bb0e51243-goog >