From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 885DDC54E5D for ; Sat, 16 Mar 2024 06:24:47 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id ACAAB8015A; Sat, 16 Mar 2024 02:24:46 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id A7B70800B4; Sat, 16 Mar 2024 02:24:46 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 942D38015A; Sat, 16 Mar 2024 02:24:46 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by kanga.kvack.org (Postfix) with ESMTP id 86B24800B4 for ; Sat, 16 Mar 2024 02:24:46 -0400 (EDT) Received: from smtpin02.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay07.hostedemail.com (Postfix) with ESMTP id 1DAA9160BAF for ; Sat, 16 Mar 2024 06:24:46 +0000 (UTC) X-FDA: 81901913772.02.DE95AEF Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.11]) by imf18.hostedemail.com (Postfix) with ESMTP id 5963D1C0005 for ; Sat, 16 Mar 2024 06:24:43 +0000 (UTC) Authentication-Results: imf18.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b="e+1G/RDo"; dmarc=pass (policy=none) header.from=intel.com; spf=pass (imf18.hostedemail.com: domain of zide.chen@intel.com designates 192.198.163.11 as permitted sender) smtp.mailfrom=zide.chen@intel.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1710570284; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=MGzqIk/YjGbUxGMJEu2EpoOJkdOgm4CQy7a9ZznLDqM=; b=1+ceEVuphlvSebpIdmSY/FCsSKg4DvCGDIUnMxOArSqgNP2u3CzZaYfWbrBmWhYDqHy5NQ Z8PKcw5WlQ9jGB+klwczQlz8xPuIma6SFxz2rsek1uljQXUnQ3pI1pmGqloUSqWoh40fMc W02UZO3XP8KqJGUjdcuQpj3QFw9C8j0= ARC-Authentication-Results: i=1; imf18.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b="e+1G/RDo"; dmarc=pass (policy=none) header.from=intel.com; spf=pass (imf18.hostedemail.com: domain of zide.chen@intel.com designates 192.198.163.11 as permitted sender) smtp.mailfrom=zide.chen@intel.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1710570284; a=rsa-sha256; cv=none; b=qHNcqWyk4AU3769yZluCnUrVCxafML+uaTyj2ycNeIODRotFasnbVl26JOSkGZ2cu4dm8y ltD+HX/XpD3Z/28xnAQbmfm9CxG26nQSjNTm3ebjLQf22AWVjo5cwFPI62M8xmVe98UoaH zONPUjcmm4ipwiQM+xs9vZhzSeTWDmM= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1710570283; x=1742106283; h=message-id:date:mime-version:from:subject:to:references: cc:in-reply-to:content-transfer-encoding; bh=Z9CPgMx4k6xbyqdhStgFoDUX8JdMY2zitvwklGw/CEw=; b=e+1G/RDo3SjImkvbEUD95ZrNUAEa6Q3S0h4QdjhItgr/X7xe0FlO4hqp mPJ0DYQIqW3uuR9psyR/Z79hNMsp/qkLDNeTpKD2IfxiLcUSnGH+K1p/6 iRZZAZ/U4xJPeO1zL3lxt8ml4nshT5yYzYT8oGsuUDaCeKLZm4TSqISbA 0DBgmdAYSxw9IawOsBAZ62r2I1ej5+jMbx3vrbEUz0+AZfEEHPQ0eV4Bp 2ZDJAONosW2I0b72SLADkZGA3lHwQXT+K4OmScqWYco6tBaeMq+Gl2p97 oDkSe/z2hUB0RuFRRvHjWpL97bQX7WiVjqu80t77J+e5ALZRGlu1c8d5K Q==; X-IronPort-AV: E=McAfee;i="6600,9927,11014"; a="16099327" X-IronPort-AV: E=Sophos;i="6.07,129,1708416000"; d="scan'208";a="16099327" Received: from fmviesa001.fm.intel.com ([10.60.135.141]) by fmvoesa105.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Mar 2024 23:24:41 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.07,129,1708416000"; d="scan'208";a="43953437" Received: from jhanbaba-mobl.amr.corp.intel.com (HELO [10.124.36.86]) ([10.124.36.86]) by smtpauth.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 15 Mar 2024 23:24:40 -0700 Message-ID: <98b54b29-6f1d-4a3b-97d1-1e08b0ed0465@intel.com> Date: Fri, 15 Mar 2024 23:24:41 -0700 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird From: "Chen, Zide" Subject: Re: [RFC PATCH v5 22/29] KVM: selftests: Add functions to allow mapping as shared To: linux-kselftest@vger.kernel.org, Ackerley Tng , "Afranji, Ryan" , "Aktas, Erdem" , Sagi Shahar , "Yamahata, Isaku" References: <20231212204647.2170650-1-sagis@google.com> <20231212204647.2170650-23-sagis@google.com> Content-Language: en-US Cc: Sean Christopherson , Paolo Bonzini , Shuah Khan , Peter Gonda , "Xu, Haibo1" , Chao Peng , "Annapurve, Vishal" , Roger Wang , Vipin Sharma , jmattson@google.com, dmatlack@google.com, linux-kernel@vger.kernel.org, kvm@vger.kernel.org, linux-mm@kvack.org In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Rspamd-Queue-Id: 5963D1C0005 X-Rspam-User: X-Rspamd-Server: rspam04 X-Stat-Signature: krogcsxgb79a7kukuzkkseaqk5wnhab6 X-HE-Tag: 1710570283-123946 X-HE-Meta: U2FsdGVkX1/u1O2wwGZRuADbHb4H6PntG7s28g3znHEJ4XcaBH1E2SiC0REssXjq7nf3xzWvITuZ0VQQds9RBVPqrhZDShL2D59Jj3vIKiuNQughairWYsCBZu3K2EH9gXt5GIMyNLhd91PK0nUsnbrhE9p6+HpQZSHsI/bMOEJDn9w09XaCujXgFL7pJFvad3ADnUuyvRJ5H+BtCh0/aU3+OgZeDRelNOWinao3PSF3eJkhbJZ+/LPhYEddo70hGA1JdnVGGY4zseLOJpabqpYdJ+qx/8hbphYgbg9QoBYWzVHNgTXkJKk7xucKoq9nFHk67yRd/BY0szxRg894UeLJUtBtayiHm/Y67WPIeoTAHqmAOH7rsQhn7ueG9XcT6O41CiWhymhf8XB5jUwT27moN/dH/RcKnqw/FSLPstV4f22bCmQIpgKOLuj2R2uIE3M6NLTZIbUnsYWNo3c5123qFOUW+V47qSn2RJ3iNmnOdsWGQe7/lQtFMz5Z+T+oC5iizaEK2M7ULiD0B2plbF0Liq1WTjXpx/OZ6vifFRfFHdFpZ+sq/zBLDK/Fh+bqD+//eANIsrhhQsk1KBcgLiMW+VwNEwPL2DiHx3cwwHKsyE8fnh7EPFGMcsupEMEg6VjXlln3b5XVeqrsMo4QTmKT5aw9PFt79pBPquAwMsKt95GBl606bDFT8lgC86kr3CDICGqKxFUVcRdyRfFdu5Gzgi4h/uxMRE5rDJhdt/Odu8B1BLsJC5f2pmc2RmpOgPpwgsKHd3BfUS+T5Ks9kVuxtCvkXgeRE6Ma5lyOMNzxFQ66Da/LrVyLPm2jDmTv939Onl7XWGY+ofq2zkvRDQtA4DI7ZLp0nNdd8WNjmogaH/5P3vLZoQVIPuu1kfZh X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On 12/12/2023 12:47 PM, Sagi Shahar wrote: > > > -----Original Message----- > From: Sagi Shahar > Sent: Tuesday, December 12, 2023 12:47 PM > To: linux-kselftest@vger.kernel.org; Ackerley Tng ; Afranji, Ryan ; Aktas, Erdem ; Sagi Shahar ; Yamahata, Isaku > Cc: Sean Christopherson ; Paolo Bonzini ; Shuah Khan ; Peter Gonda ; Xu, Haibo1 ; Chao Peng ; Annapurve, Vishal ; Roger Wang ; Vipin Sharma ; jmattson@google.com; dmatlack@google.com; linux-kernel@vger.kernel.org; kvm@vger.kernel.org; linux-mm@kvack.org > Subject: [RFC PATCH v5 22/29] KVM: selftests: Add functions to allow mapping as shared Since protected_phy_pages is introduced to keep track of the guest memory's private/shared property, it's better to keep it consistent with the guest mappings. Instead of having a set of new APIs to force it to map shared guest pages, how about to update protected_phy_pages sparsebits right before the mapping, and just call the existing virt_pg_map() to do the mapping? > From: Ackerley Tng > > Signed-off-by: Ackerley Tng > Signed-off-by: Ryan Afranji > Signed-off-by: Sagi Shahar > --- > .../selftests/kvm/include/kvm_util_base.h | 24 ++++++++++++++ > tools/testing/selftests/kvm/lib/kvm_util.c | 32 +++++++++++++++++++ > .../selftests/kvm/lib/x86_64/processor.c | 15 +++++++-- > 3 files changed, 69 insertions(+), 2 deletions(-) > > diff --git a/tools/testing/selftests/kvm/include/kvm_util_base.h b/tools/testing/selftests/kvm/include/kvm_util_base.h > index b353617fcdd1..efd7ae8abb20 100644 > --- a/tools/testing/selftests/kvm/include/kvm_util_base.h > +++ b/tools/testing/selftests/kvm/include/kvm_util_base.h > @@ -574,6 +574,8 @@ vm_vaddr_t vm_vaddr_alloc_page(struct kvm_vm *vm); > > void virt_map(struct kvm_vm *vm, uint64_t vaddr, uint64_t paddr, > unsigned int npages); > +void virt_map_shared(struct kvm_vm *vm, uint64_t vaddr, uint64_t paddr, > + unsigned int npages); > void *addr_gpa2hva(struct kvm_vm *vm, vm_paddr_t gpa); void *addr_gva2hva(struct kvm_vm *vm, vm_vaddr_t gva); vm_paddr_t addr_hva2gpa(struct kvm_vm *vm, void *hva); @@ -1034,6 +1036,28 @@ static inline void virt_pg_map(struct kvm_vm *vm, uint64_t vaddr, uint64_t paddr > virt_arch_pg_map(vm, vaddr, paddr); > } > > +/* > + * VM Virtual Page Map as Shared > + * > + * Input Args: > + * vm - Virtual Machine > + * vaddr - VM Virtual Address > + * paddr - VM Physical Address > + * memslot - Memory region slot for new virtual translation tables > + * > + * Output Args: None > + * > + * Return: None > + * > + * Within @vm, creates a virtual translation for the page starting > + * at @vaddr to the page starting at @paddr. > + */ > +void virt_arch_pg_map_shared(struct kvm_vm *vm, uint64_t vaddr, > +uint64_t paddr); > + > +static inline void virt_pg_map_shared(struct kvm_vm *vm, uint64_t > +vaddr, uint64_t paddr) { > + virt_arch_pg_map_shared(vm, vaddr, paddr); } > > /* > * Address Guest Virtual to Guest Physical diff --git a/tools/testing/selftests/kvm/lib/kvm_util.c b/tools/testing/selftests/kvm/lib/kvm_util.c > index 4f1ae0f1eef0..28780fa1f0f2 100644 > --- a/tools/testing/selftests/kvm/lib/kvm_util.c > +++ b/tools/testing/selftests/kvm/lib/kvm_util.c > @@ -1573,6 +1573,38 @@ void virt_map(struct kvm_vm *vm, uint64_t vaddr, uint64_t paddr, > } > } > > +/* > + * Map a range of VM virtual address to the VM's physical address as > +shared > + * > + * Input Args: > + * vm - Virtual Machine > + * vaddr - Virtuall address to map > + * paddr - VM Physical Address > + * npages - The number of pages to map > + * > + * Output Args: None > + * > + * Return: None > + * > + * Within the VM given by @vm, creates a virtual translation for > + * @npages starting at @vaddr to the page range starting at @paddr. > + */ > +void virt_map_shared(struct kvm_vm *vm, uint64_t vaddr, uint64_t paddr, > + unsigned int npages) > +{ > + size_t page_size = vm->page_size; > + size_t size = npages * page_size; > + > + TEST_ASSERT(vaddr + size > vaddr, "Vaddr overflow"); > + TEST_ASSERT(paddr + size > paddr, "Paddr overflow"); > + > + while (npages--) { > + virt_pg_map_shared(vm, vaddr, paddr); > + vaddr += page_size; > + paddr += page_size; > + } > +} > + > /* > * Address VM Physical to Host Virtual > * > diff --git a/tools/testing/selftests/kvm/lib/x86_64/processor.c b/tools/testing/selftests/kvm/lib/x86_64/processor.c > index 566d82829da4..aa2a57ddb8d3 100644 > --- a/tools/testing/selftests/kvm/lib/x86_64/processor.c > +++ b/tools/testing/selftests/kvm/lib/x86_64/processor.c > @@ -190,7 +190,8 @@ static uint64_t *virt_create_upper_pte(struct kvm_vm *vm, > return pte; > } > > -void __virt_pg_map(struct kvm_vm *vm, uint64_t vaddr, uint64_t paddr, int level) > +static void ___virt_pg_map(struct kvm_vm *vm, uint64_t vaddr, uint64_t paddr, > + int level, bool protected) > { > const uint64_t pg_size = PG_LEVEL_SIZE(level); > uint64_t *pml4e, *pdpe, *pde; > @@ -235,17 +236,27 @@ void __virt_pg_map(struct kvm_vm *vm, uint64_t vaddr, uint64_t paddr, int level) > "PTE already present for 4k page at vaddr: 0x%lx\n", vaddr); > *pte = PTE_PRESENT_MASK | PTE_WRITABLE_MASK | (paddr & PHYSICAL_PAGE_MASK); > > - if (vm_is_gpa_protected(vm, paddr)) > + if (protected)> *pte |= vm->arch.c_bit; > else > *pte |= vm->arch.s_bit; > } > > +void __virt_pg_map(struct kvm_vm *vm, uint64_t vaddr, uint64_t paddr, > +int level) { > + ___virt_pg_map(vm, vaddr, paddr, level, vm_is_gpa_protected(vm, > +paddr)); } > + > void virt_arch_pg_map(struct kvm_vm *vm, uint64_t vaddr, uint64_t paddr) { > __virt_pg_map(vm, vaddr, paddr, PG_LEVEL_4K); } > > +void virt_arch_pg_map_shared(struct kvm_vm *vm, uint64_t vaddr, > +uint64_t paddr) { > + ___virt_pg_map(vm, vaddr, paddr, PG_LEVEL_4K, false); } Here, it tries to create the guest mappings regardless of what the value of protected_phy_pages is, which could create confusion. > + > void virt_map_level(struct kvm_vm *vm, uint64_t vaddr, uint64_t paddr, > uint64_t nr_bytes, int level) > { > -- > 2.43.0.472.g3155946c3a-goog >