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 X-Spam-Level: X-Spam-Status: No, score=-12.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A,SIGNED_OFF_BY,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 36BAEC433DF for ; Fri, 21 Aug 2020 03:51:39 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id EC0C720855 for ; Fri, 21 Aug 2020 03:51:38 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=nvidia.com header.i=@nvidia.com header.b="fi9ERSL8" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org EC0C720855 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=nvidia.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 59EFE8D000F; Thu, 20 Aug 2020 23:51:38 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 550598D000A; Thu, 20 Aug 2020 23:51:38 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 418958D000F; Thu, 20 Aug 2020 23:51:38 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0170.hostedemail.com [216.40.44.170]) by kanga.kvack.org (Postfix) with ESMTP id 2C5DE8D000A for ; Thu, 20 Aug 2020 23:51:38 -0400 (EDT) Received: from smtpin29.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id E0AFC824556B for ; Fri, 21 Aug 2020 03:51:37 +0000 (UTC) X-FDA: 77173201434.29.shade62_32141a227036 Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin29.hostedemail.com (Postfix) with ESMTP id 4E02E18086E58 for ; Fri, 21 Aug 2020 03:51:37 +0000 (UTC) X-HE-Tag: shade62_32141a227036 X-Filterd-Recvd-Size: 4847 Received: from hqnvemgate26.nvidia.com (hqnvemgate26.nvidia.com [216.228.121.65]) by imf02.hostedemail.com (Postfix) with ESMTP for ; Fri, 21 Aug 2020 03:51:36 +0000 (UTC) Received: from hqpgpgate101.nvidia.com (Not Verified[216.228.121.13]) by hqnvemgate26.nvidia.com (using TLS: TLSv1.2, DES-CBC3-SHA) id ; Thu, 20 Aug 2020 20:51:21 -0700 Received: from hqmail.nvidia.com ([172.20.161.6]) by hqpgpgate101.nvidia.com (PGP Universal service); Thu, 20 Aug 2020 20:51:35 -0700 X-PGP-Universal: processed; by hqpgpgate101.nvidia.com on Thu, 20 Aug 2020 20:51:35 -0700 Received: from [10.2.56.96] (172.20.13.39) by HQMAIL107.nvidia.com (172.20.187.13) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Fri, 21 Aug 2020 03:51:28 +0000 Subject: Re: [PATCH] mm/gup_benchmark: update the documentation in Kconfig To: Barry Song , , , CC: , , Keith Busch , Ira Weiny , "Kirill A . Shutemov" References: <20200821032546.19992-1-song.bao.hua@hisilicon.com> From: John Hubbard Message-ID: <622e5ba7-41e9-0335-1630-cbcdaf9a162f@nvidia.com> Date: Thu, 20 Aug 2020 20:51:27 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 MIME-Version: 1.0 In-Reply-To: <20200821032546.19992-1-song.bao.hua@hisilicon.com> X-Originating-IP: [172.20.13.39] X-ClientProxiedBy: HQMAIL107.nvidia.com (172.20.187.13) To HQMAIL107.nvidia.com (172.20.187.13) Content-Type: text/plain; charset="utf-8"; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nvidia.com; s=n1; t=1597981881; bh=OFw05gqtm34jtOGr2u9s+eSTW+9vg7j82ohz9WyBrNw=; h=X-PGP-Universal:Subject:To:CC:References:From:Message-ID:Date: User-Agent:MIME-Version:In-Reply-To:X-Originating-IP: X-ClientProxiedBy:Content-Type:Content-Language: Content-Transfer-Encoding; b=fi9ERSL8dvklD8RB4vq2gFIHipBcnEQz3N2KkExotSl+ktAMRKdtVgaXpfJuBzP2C a0jmZIdu7s4rubJNRY5D7uFV3ltRU/MP5HZnU7+MXs+U8Gj7ehoCKptBatREdRtJNN J6RmRcJJI5OFOKp5NJVtwM5BG3yvskd0GgnP1Uc0rsR+FjaN42idBgodUZM197IcOI hr8zCFzai/IE0uHuUxBgQqxXVOcbcbQICUcwgCimpR7cIriI15YYjd/YNKopJyuqRG PTqWaBekanSLdviVIsoq346JFBHCt/GiXxRp0rGDQwpo9FgbMCUUxBFCZaniAEng33 KENOMf+DJCu8w== X-Rspamd-Queue-Id: 4E02E18086E58 X-Spamd-Result: default: False [0.00 / 100.00] X-Rspamd-Server: rspam01 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: On 8/20/20 8:25 PM, Barry Song wrote: > In the beginning, mm/gup_benchmark.c supported get_user_pages_fast() > only, but right now, it supports the benchmarking of a couple of > get_user_pages() related calls like: > * get_user_pages_fast() > * get_user_pages() > * pin_user_pages_fast() > * pin_user_pages() > The documentation is confusing and needs update. hmmm, it's not that confusing, given that pin_user_pages() and get_user_pages() use the same underlying get_user_pages() implementation. > > Cc: John Hubbard > Cc: Keith Busch > Cc: Ira Weiny > Cc: Kirill A. Shutemov > Signed-off-by: Barry Song > --- > mm/Kconfig | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/mm/Kconfig b/mm/Kconfig > index 6c974888f86f..f7c9374da7b3 100644 > --- a/mm/Kconfig > +++ b/mm/Kconfig > @@ -831,10 +831,10 @@ config PERCPU_STATS > be used to help understand percpu memory usage. > > config GUP_BENCHMARK > - bool "Enable infrastructure for get_user_pages_fast() benchmarking" > + bool "Enable infrastructure for get_user_pages() and related calls benchmarking" If we really want to go to the trouble of tweaking this, then I'd go with something more like: "Enable infrastructure for get_user_pages() and pin_user_pages benchmarking" ...but I don't think it really warrants a patch just yet. *However*, my judgment is skewed right now, because I'm planning a small patchset to split up gup_benchmark a little bit, and to add some more testing and take advantage of parts of it to do a dump_page() test. At which point "related calls" would make more sense, but then it would be different enough that this patch would still need changing. So I'm inclined to just recommend leaving this alone for a bit, but if others want to put it in, I'm OK with that too. > help > Provides /sys/kernel/debug/gup_benchmark that helps with testing > - performance of get_user_pages_fast(). > + performance of get_user_pages() and related calls. > > See tools/testing/selftests/vm/gup_benchmark.c > > thanks, -- John Hubbard NVIDIA