Received: by 2002:a05:6a10:9848:0:0:0:0 with SMTP id x8csp1182595pxf; Fri, 12 Mar 2021 04:04:21 -0800 (PST) X-Google-Smtp-Source: ABdhPJxPMtEquXPrluvSatDDJLQadw2ZvmDqBVrvbPBNSihq9swgwwdByDYLMjFB9hgui0hZODkD X-Received: by 2002:a17:906:eb89:: with SMTP id mh9mr8370972ejb.122.1615550661771; Fri, 12 Mar 2021 04:04:21 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1615550661; cv=none; d=google.com; s=arc-20160816; b=dpE8ZH8dZLmM7TYsjG82IIKiicRINI9ycqgoT39IM7UavM5EAOdgIX2kfUCgNKelxF gcaIYzxa0tuAUC2QdVP+Y2+4UbYwpkr/yTrYgoLRaBt39jVfLagjLhSbNr+0yJezL+GE MOntBDaBDWDCkM8+Tj+t9BZOFVSpIIzwm1g9Gb35PlnC9iyNBFAVZHShc6hs61I9sFEQ b53swK+2yvZ/hA50BxAp7ZJvlWizdtfUoCnOwPc/z+YimS/geSB6x21zr3E9r1rfdnY+ Oc5lU5PPqeO9RWLybgbBSw9BtkS7boL0TAc4l7v1+wiwkJYslGkpST/W2/0OiF1eAK+d Bwrg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=qrEHj5I0bZYnneD3PLLPPSpfhRiDox4bq62WY+1YUCQ=; b=jBVHYMJwTibbc/abrTxETQ1VwxTxjqlV8HkE8naRtukcdW0rPP5mO21QwwT86G/3M2 G6ch+dSEqO2jIY62vHZ15wUHtTIaIJrGUqAj/SILpm6y8JFUxCV8ufYMN3gX3/CDlQUw eAQBIf2jMMuftjWXbtfryh1QidbY1pjHhbLah0Ug87ngHRal8a9vd8Md4PmBxkKz+PjO x6/tqNWBIb4COAAE0CR1evTOWrynSkuakZZbjjAvP7ccSocolNYaCzmrX1tpVwi02kvU Ja0r3RcQlPaCCFjBTHSVY6GsrCACTRNJwx50YA2HSfMCtpH/KzKSWiXxSYC7qp7tsCjc u4zA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=FpEcBJSX; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id pw11si3930725ejb.2.2021.03.12.04.03.58; Fri, 12 Mar 2021 04:04:21 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=FpEcBJSX; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230172AbhCLMCy (ORCPT + 99 others); Fri, 12 Mar 2021 07:02:54 -0500 Received: from us-smtp-delivery-124.mimecast.com ([63.128.21.124]:25648 "EHLO us-smtp-delivery-124.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231770AbhCLMCg (ORCPT ); Fri, 12 Mar 2021 07:02:36 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1615550556; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=qrEHj5I0bZYnneD3PLLPPSpfhRiDox4bq62WY+1YUCQ=; b=FpEcBJSXz94fi+p/3/f9SH/hV/U86pbI4xKAChu2t1jhQPpV22d4F3t8WnPC+z5uhE8IyV 4BVTnMjjdx0ZMMMdlmMLmH9tSOX+nhDfFb/nCwTK80hgR2UY99Ol2VlZbkfGw1iD1+FR4e jBHYgvzEE2AWnIJoRhP6QfqigMtXvZs= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-498-FBr7Ld_VMlW_trAbhxSQfg-1; Fri, 12 Mar 2021 07:02:34 -0500 X-MC-Unique: FBr7Ld_VMlW_trAbhxSQfg-1 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 6C5EC1940920; Fri, 12 Mar 2021 12:02:32 +0000 (UTC) Received: from kamzik.brq.redhat.com (unknown [10.40.192.104]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 89A6C60636; Fri, 12 Mar 2021 12:02:24 +0000 (UTC) Date: Fri, 12 Mar 2021 13:02:21 +0100 From: Andrew Jones To: Yanan Wang Cc: kvm@vger.kernel.org, linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org, Paolo Bonzini , Ben Gardon , Sean Christopherson , Vitaly Kuznetsov , Peter Xu , Marc Zyngier , Ingo Molnar , Adrian Hunter , Jiri Olsa , Arnaldo Carvalho de Melo , Arnd Bergmann , Michael Kerrisk , Thomas Gleixner , wanghaibin.wang@huawei.com, yezengruan@huawei.com, yuzenghui@huawei.com Subject: Re: [RFC PATCH v4 7/9] KVM: selftests: List all hugetlb src types specified with page sizes Message-ID: <20210312120221.urbbfl7o3vocblk7@kamzik.brq.redhat.com> References: <20210302125751.19080-1-wangyanan55@huawei.com> <20210302125751.19080-8-wangyanan55@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210302125751.19080-8-wangyanan55@huawei.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Mar 02, 2021 at 08:57:49PM +0800, Yanan Wang wrote: > With VM_MEM_SRC_ANONYMOUS_HUGETLB, we currently can only use system > default hugetlb pages to back the testing guest memory. In order to > add flexibility, now list all the known hugetlb backing src types with > different page sizes, so that we can specify use of hugetlb pages of the > exact granularity that we want. And as all the known hugetlb page sizes > are listed, it's appropriate for all architectures. > > Besides, the helper get_backing_src_pagesz() is added to get the > granularity of different backing src types(anonumous, thp, hugetlb). > > Suggested-by: Ben Gardon > Signed-off-by: Yanan Wang > --- > .../testing/selftests/kvm/include/test_util.h | 18 +++++- > tools/testing/selftests/kvm/lib/kvm_util.c | 2 +- > tools/testing/selftests/kvm/lib/test_util.c | 59 +++++++++++++++---- > 3 files changed, 66 insertions(+), 13 deletions(-) > > diff --git a/tools/testing/selftests/kvm/include/test_util.h b/tools/testing/selftests/kvm/include/test_util.h > index e087174eefe5..fade3130eb01 100644 > --- a/tools/testing/selftests/kvm/include/test_util.h > +++ b/tools/testing/selftests/kvm/include/test_util.h > @@ -71,16 +71,32 @@ enum vm_mem_backing_src_type { > VM_MEM_SRC_ANONYMOUS, > VM_MEM_SRC_ANONYMOUS_THP, > VM_MEM_SRC_ANONYMOUS_HUGETLB, > + VM_MEM_SRC_ANONYMOUS_HUGETLB_16KB, > + VM_MEM_SRC_ANONYMOUS_HUGETLB_64KB, > + VM_MEM_SRC_ANONYMOUS_HUGETLB_512KB, > + VM_MEM_SRC_ANONYMOUS_HUGETLB_1MB, > + VM_MEM_SRC_ANONYMOUS_HUGETLB_2MB, > + VM_MEM_SRC_ANONYMOUS_HUGETLB_8MB, > + VM_MEM_SRC_ANONYMOUS_HUGETLB_16MB, > + VM_MEM_SRC_ANONYMOUS_HUGETLB_32MB, > + VM_MEM_SRC_ANONYMOUS_HUGETLB_256MB, > + VM_MEM_SRC_ANONYMOUS_HUGETLB_512MB, > + VM_MEM_SRC_ANONYMOUS_HUGETLB_1GB, > + VM_MEM_SRC_ANONYMOUS_HUGETLB_2GB, > + VM_MEM_SRC_ANONYMOUS_HUGETLB_16GB, > + NUM_SRC_TYPES, > }; > > struct vm_mem_backing_src_alias { > const char *name; > - enum vm_mem_backing_src_type type; > + uint32_t flag; > }; > > bool thp_configured(void); > size_t get_trans_hugepagesz(void); > size_t get_def_hugetlb_pagesz(void); > +const struct vm_mem_backing_src_alias *vm_mem_backing_src_alias(uint32_t i); > +size_t get_backing_src_pagesz(uint32_t i); > void backing_src_help(void); > enum vm_mem_backing_src_type parse_backing_src_type(const char *type_name); > > diff --git a/tools/testing/selftests/kvm/lib/kvm_util.c b/tools/testing/selftests/kvm/lib/kvm_util.c > index cc22c4ab7d67..b91c8e3a7ee1 100644 > --- a/tools/testing/selftests/kvm/lib/kvm_util.c > +++ b/tools/testing/selftests/kvm/lib/kvm_util.c > @@ -757,7 +757,7 @@ void vm_userspace_mem_region_add(struct kvm_vm *vm, > region->mmap_start = mmap(NULL, region->mmap_size, > PROT_READ | PROT_WRITE, > MAP_PRIVATE | MAP_ANONYMOUS > - | (src_type == VM_MEM_SRC_ANONYMOUS_HUGETLB ? MAP_HUGETLB : 0), > + | vm_mem_backing_src_alias(src_type)->flag, > -1, 0); > TEST_ASSERT(region->mmap_start != MAP_FAILED, > "test_malloc failed, mmap_start: %p errno: %i", > diff --git a/tools/testing/selftests/kvm/lib/test_util.c b/tools/testing/selftests/kvm/lib/test_util.c > index 80d68dbd72d2..df8a42eff1f8 100644 > --- a/tools/testing/selftests/kvm/lib/test_util.c > +++ b/tools/testing/selftests/kvm/lib/test_util.c > @@ -11,6 +11,7 @@ > #include > #include > #include > +#include > #include "linux/kernel.h" > > #include "test_util.h" > @@ -112,12 +113,6 @@ void print_skip(const char *fmt, ...) > puts(", skipping test"); > } > > -const struct vm_mem_backing_src_alias backing_src_aliases[] = { > - {"anonymous", VM_MEM_SRC_ANONYMOUS,}, > - {"anonymous_thp", VM_MEM_SRC_ANONYMOUS_THP,}, > - {"anonymous_hugetlb", VM_MEM_SRC_ANONYMOUS_HUGETLB,}, > -}; > - > bool thp_configured(void) > { > int ret; > @@ -180,22 +175,64 @@ size_t get_def_hugetlb_pagesz(void) > return 0; > } > > +const struct vm_mem_backing_src_alias *vm_mem_backing_src_alias(uint32_t i) > +{ > + static const struct vm_mem_backing_src_alias aliases[] = { > + { "anonymous", 0 }, > + { "anonymous_thp", 0 }, > + { "anonymous_hugetlb", MAP_HUGETLB }, > + { "anonymous_hugetlb_16kb", MAP_HUGETLB | MAP_HUGE_16KB }, > + { "anonymous_hugetlb_64kb", MAP_HUGETLB | MAP_HUGE_64KB }, > + { "anonymous_hugetlb_512kb", MAP_HUGETLB | MAP_HUGE_512KB }, > + { "anonymous_hugetlb_1mb", MAP_HUGETLB | MAP_HUGE_1MB }, > + { "anonymous_hugetlb_2mb", MAP_HUGETLB | MAP_HUGE_2MB }, > + { "anonymous_hugetlb_8mb", MAP_HUGETLB | MAP_HUGE_8MB }, > + { "anonymous_hugetlb_16mb", MAP_HUGETLB | MAP_HUGE_16MB }, > + { "anonymous_hugetlb_32mb", MAP_HUGETLB | MAP_HUGE_32MB }, > + { "anonymous_hugetlb_256mb", MAP_HUGETLB | MAP_HUGE_256MB }, > + { "anonymous_hugetlb_512mb", MAP_HUGETLB | MAP_HUGE_512MB }, > + { "anonymous_hugetlb_1gb", MAP_HUGETLB | MAP_HUGE_1GB }, > + { "anonymous_hugetlb_2gb", MAP_HUGETLB | MAP_HUGE_2GB }, > + { "anonymous_hugetlb_16gb", MAP_HUGETLB | MAP_HUGE_16GB }, > + }; > + _Static_assert(ARRAY_SIZE(aliases) == NUM_SRC_TYPES, > + "Missing new backing src types?"); > + > + TEST_ASSERT(i < NUM_SRC_TYPES, "Backing src type ID %d too big", i); > + > + return &aliases[i]; > +} > + > +size_t get_backing_src_pagesz(uint32_t i) > +{ > + uint32_t flag = vm_mem_backing_src_alias(i)->flag; > + > + if (i == VM_MEM_SRC_ANONYMOUS) > + return getpagesize(); > + if (i == VM_MEM_SRC_ANONYMOUS_THP) > + return get_trans_hugepagesz(); > + if (i == VM_MEM_SRC_ANONYMOUS_HUGETLB) > + return get_def_hugetlb_pagesz(); > + > + return MAP_HUGE_PAGE_SIZE(flag); > +} > + > void backing_src_help(void) > { > int i; > > printf("Available backing src types:\n"); > - for (i = 0; i < ARRAY_SIZE(backing_src_aliases); i++) > - printf("\t%s\n", backing_src_aliases[i].name); > + for (i = 0; i < NUM_SRC_TYPES; i++) > + printf("\t%s\n", vm_mem_backing_src_alias(i)->name); > } > > enum vm_mem_backing_src_type parse_backing_src_type(const char *type_name) > { > int i; > > - for (i = 0; i < ARRAY_SIZE(backing_src_aliases); i++) > - if (!strcmp(type_name, backing_src_aliases[i].name)) > - return backing_src_aliases[i].type; > + for (i = 0; i < NUM_SRC_TYPES; i++) > + if (!strcmp(type_name, vm_mem_backing_src_alias(i)->name)) > + return i; This requires vm_mem_backing_src_alias.aliases[] to be in the same order as vm_mem_backing_src_type, so we should do the designated array initialization like in vm_guest_mode_string(). Thanks, drew > > backing_src_help(); > TEST_FAIL("Unknown backing src type: %s", type_name); > -- > 2.23.0 >