From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f201.google.com (mail-pl1-f201.google.com [209.85.214.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 2D211282EB for ; Fri, 21 Nov 2025 00:10:54 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763683855; cv=none; b=gc7co8nnGQkbjkOLMZJKbkHwRcQhHXGbhlnZw8/+IIYU3nkmAHFGASjK411quFv7kXXEg5g0crTSOnufYKZXuWqrJaMqQck2FyuG0X5KUu5SVElCwaANRssB0nEZURkMzzMtxNFE5jVk7W03H5IKhzjXr2nJGCMhTHJbvn8E0yk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763683855; c=relaxed/simple; bh=GTAeV0/U1Id0rOZySfsnYuhPR+q8AtT1fLfd5vOB2yU=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=Vz4SVaCcnBblPBgXj0Yv1QZzhglA4As4ioDswR0zW9ZQe9RUzt0rP5e4/8SogT7TaU/WSkRPu2kq1oD+Ogt6y2dshCYBJUoy6prrROCHYwoALEw0C6KUw/bagNAXk3gfKV9ieRfqMJolByLtuwI9R6xSc3pvaYig65VcMX1eqHA= 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=GvAxDDO6; arc=none smtp.client-ip=209.85.214.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="GvAxDDO6" Received: by mail-pl1-f201.google.com with SMTP id d9443c01a7336-297f587dc2eso36098415ad.2 for ; Thu, 20 Nov 2025 16:10:54 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1763683853; x=1764288653; 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=H2Jg5eGCJ4JWQv22m6t6f8HAltv6vrVf9pXQ9eNKNVk=; b=GvAxDDO6Y/OVZM6HzuaQ7LpBw++gdi/hV2OlmszbvJj2EW4RGWkE/bs8artLtMEXaq S4BqEMMTxg3AVTzNy+9wymUmhHvGAWF3x+HBdeP1ZVwbYdx5iceRcJXFXpvSDguKgyvm DNSHgv0Mt1gRHTh9/cJI/xUO5tqH+/7c9m5Eb8tj1QYi9WwypFLdQxQ/lxmKTnkVc/+I W0SDISbznlDntHdbBIoHcFnnvNfcWCIqN/lK1YbAlFQCFXy3ATHc0ysM+saqEYGBlO6q kmnGKbGowimsAyduyYl7NOUy+RFHj+/Wd8+3u7oG4zzVnCgI39X0KPIPpJUqZ0JLs+hQ xlnw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763683853; x=1764288653; 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=H2Jg5eGCJ4JWQv22m6t6f8HAltv6vrVf9pXQ9eNKNVk=; b=f22uCWwfkSGSfHseD5ep3rAaz3n8KYySKPY1Og+tPNfEcsDV+Pcrb16aJEqIcMsxrX fdmakj4GQcsVa4y2/IRRvxLeJD2If17eNclI0JKxfZgvI/zFG6AjgT3EN2k+rgNaZxyY egwbyjPxPBO3QviUDeNjLeaAdC0xJbrX+yFcX0eT93eUbo/USd78LvELDznDOpGUdEXn r/T1nBvyJ+jLcCgf17zjdwkVmABUF2W1pX46Hv8ZUJngt8gKZ68OK1YnMGXifY3AOHQ0 cr39sxRui7PxKJAOoU1Cd4y+vsRp1ODfafep2Yh74FiFFLwT8eLqzSoEf/KIKqRTQ12D Rx8w== X-Forwarded-Encrypted: i=1; AJvYcCWdwAgxUh7uvC4LgF13BhuY2uUehWoRgDuJAzE7Zan4REiztw6tAPlfTkS3QitjqpiN7hbQOR+KjEpKL+A=@vger.kernel.org X-Gm-Message-State: AOJu0YxkHm9aPzpXBFxIvXUxHriirdb+/R7apgt/LnFs3/dz10EJPO8M wfsK/yF+zSTLgdqBzXD0fKZ9nDL3U2/Epsj+2buZ9bsJeDnh5AFCVWl3oHT0fG5wlgQIv17PxOI DaxiadQ== X-Google-Smtp-Source: AGHT+IHQJw6u93572+5nYVgz+HVD5zQVNxdCoyv5uNHiLP4YNyDD01xIH8qaFDKeb3fG0eRan1ynFEUUq4c= X-Received: from plbki15.prod.google.com ([2002:a17:903:68f:b0:293:de:a528]) (user=seanjc job=prod-delivery.src-stubby-dispatcher) by 2002:a17:903:903:b0:295:9db1:ff4b with SMTP id d9443c01a7336-29b6be8b66dmr6127945ad.4.1763683853447; Thu, 20 Nov 2025 16:10:53 -0800 (PST) Date: Thu, 20 Nov 2025 16:10:51 -0800 In-Reply-To: <20251021074736.1324328-19-yosry.ahmed@linux.dev> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20251021074736.1324328-1-yosry.ahmed@linux.dev> <20251021074736.1324328-19-yosry.ahmed@linux.dev> Message-ID: Subject: Re: [PATCH v2 18/23] KVM: selftests: Generalize nested mapping functions From: Sean Christopherson To: Yosry Ahmed Cc: Paolo Bonzini , Jim Mattson , kvm@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset="us-ascii" On Tue, Oct 21, 2025, Yosry Ahmed wrote: > Instead of passing in a pointer to struct vmx_pages, pass in the GPA of > the root of the EPTs, as that's the only member being used. Furthermore, > only use ept_pte_masks for VMX, and use x86_pte_masks otherwise (which > is what NPT uses). > > This is in preparation of supporting NPTs as well. > > No functional change intended. > > Signed-off-by: Yosry Ahmed > --- > tools/testing/selftests/kvm/include/x86/vmx.h | 6 +++--- > .../testing/selftests/kvm/lib/x86/memstress.c | 4 ++-- > tools/testing/selftests/kvm/lib/x86/vmx.c | 20 ++++++++++--------- > .../selftests/kvm/x86/vmx_dirty_log_test.c | 6 +++--- > 4 files changed, 19 insertions(+), 17 deletions(-) > > diff --git a/tools/testing/selftests/kvm/include/x86/vmx.h b/tools/testing/selftests/kvm/include/x86/vmx.h > index 5aa14ceed050a..4429e83e1f52c 100644 > --- a/tools/testing/selftests/kvm/include/x86/vmx.h > +++ b/tools/testing/selftests/kvm/include/x86/vmx.h > @@ -561,11 +561,11 @@ bool load_vmcs(struct vmx_pages *vmx); > > bool ept_1g_pages_supported(void); > > -void nested_map(struct vmx_pages *vmx, struct kvm_vm *vm, > +void nested_map(struct kvm_vm *vm, vm_paddr_t root_gpa, > uint64_t nested_paddr, uint64_t paddr, uint64_t size); > -void nested_map_memslot(struct vmx_pages *vmx, struct kvm_vm *vm, > +void nested_map_memslot(struct kvm_vm *vm, vm_paddr_t root_gpa, > uint32_t memslot); > -void nested_identity_map_1g(struct vmx_pages *vmx, struct kvm_vm *vm, > +void nested_identity_map_1g(struct kvm_vm *vm, vm_paddr_t root_gpa, > uint64_t addr, uint64_t size); Ugh, "nested" is a bad namespace. Running L2 doesn't strictly require nested TDP, and the operations themselves are non-nested, in the sense that we're modifying stage-2 / TDP page tables. My vote would be to do a rename to either stage2_pg_map() or tdp_pg_map()