From: John Hubbard <jhubbard@nvidia.com> To: Andrew Morton <akpm@linux-foundation.org> Cc: Keith Busch <keith.busch@intel.com>, LKML <linux-kernel@vger.kernel.org>, <linux-mm@kvack.org>, <linux-kselftest@vger.kernel.org>, John Hubbard <jhubbard@nvidia.com> Subject: [PATCH 0/1] mm/gup_benchmark: fix MAP_HUGETLB case Date: Mon, 21 Oct 2019 14:24:34 -0700 [thread overview] Message-ID: <20191021212435.398153-1-jhubbard@nvidia.com> (raw) Hi, Here's another gup_benchmark.c fix, which I ran into while adding support for the upcoming FOLL_PIN work. Anyway, the problem is clearly described in the patch commit description, and the fix seems like the best way to me, but the fix is not *completely* black and white. This fix forces MAP_ANONYMOUS for the MAP_HUGETLB case. However, another way to do it might be to mmap() against a valid hugetlb page file, instead of /dev/zero. But that seems like a lot of trouble and if I'm reading the intent correctly, MAP_ANONYMOUS is what's desired anyway. John Hubbard (1): mm/gup_benchmark: fix MAP_HUGETLB case tools/testing/selftests/vm/gup_benchmark.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.23.0
next reply other threads:[~2019-10-21 21:24 UTC|newest] Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-10-21 21:24 John Hubbard [this message] 2019-10-21 21:24 ` [PATCH 1/1] " John Hubbard 2019-10-22 17:14 ` Jerome Glisse 2019-10-22 18:41 ` John Hubbard 2019-10-22 18:57 ` Jerome Glisse 2019-10-22 19:39 ` John Hubbard 2019-10-22 21:34 ` Andrew Morton
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20191021212435.398153-1-jhubbard@nvidia.com \ --to=jhubbard@nvidia.com \ --cc=akpm@linux-foundation.org \ --cc=keith.busch@intel.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-kselftest@vger.kernel.org \ --cc=linux-mm@kvack.org \ --subject='Re: [PATCH 0/1] mm/gup_benchmark: fix MAP_HUGETLB case' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).