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 9047BC433FE for ; Mon, 28 Nov 2022 08:17:15 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 96A056B0072; Mon, 28 Nov 2022 03:17:14 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 93FEB6B0073; Mon, 28 Nov 2022 03:17:14 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 807A66B0074; Mon, 28 Nov 2022 03:17:14 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id 725326B0072 for ; Mon, 28 Nov 2022 03:17:14 -0500 (EST) Received: from smtpin07.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id 42B07C0C0C for ; Mon, 28 Nov 2022 08:17:14 +0000 (UTC) X-FDA: 80182145988.07.9A0F50E Received: from ams.source.kernel.org (ams.source.kernel.org [145.40.68.75]) by imf21.hostedemail.com (Postfix) with ESMTP id AABA51C000C for ; Mon, 28 Nov 2022 08:17:12 +0000 (UTC) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 7EBFBB80C81; Mon, 28 Nov 2022 08:17:10 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id DF774C433D6; Mon, 28 Nov 2022 08:17:01 +0000 (UTC) Message-ID: <08b65ac6-6786-1080-18f8-d2be109c85fc@xs4all.nl> Date: Mon, 28 Nov 2022 09:17:00 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.5.0 Content-Language: en-US To: David Hildenbrand , linux-kernel@vger.kernel.org, Andrew Morton Cc: x86@kernel.org, linux-alpha@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-ia64@vger.kernel.org, linux-mips@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, sparclinux@vger.kernel.org, linux-um@lists.infradead.org, etnaviv@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-samsung-soc@vger.kernel.org, linux-rdma@vger.kernel.org, linux-media@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, linux-perf-users@vger.kernel.org, linux-security-module@vger.kernel.org, linux-kselftest@vger.kernel.org, Linus Torvalds , Jason Gunthorpe , John Hubbard , Peter Xu , Greg Kroah-Hartman , Andrea Arcangeli , Hugh Dickins , Nadav Amit , Vlastimil Babka , Matthew Wilcox , Mike Kravetz , Muchun Song , Shuah Khan , Lucas Stach , David Airlie , Oded Gabbay , Arnd Bergmann , Christoph Hellwig , Alex Williamson , Marek Szyprowski , Tomasz Figa , Mauro Carvalho Chehab References: <20221116102659.70287-1-david@redhat.com> <20221116102659.70287-17-david@redhat.com> <81fb0fa3-2e06-b765-56ac-a7d981194e59@redhat.com> From: Hans Verkuil Subject: Re: [PATCH mm-unstable v1 16/20] mm/frame-vector: remove FOLL_FORCE usage In-Reply-To: <81fb0fa3-2e06-b765-56ac-a7d981194e59@redhat.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1669623433; 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; bh=4M+eHSD3lZsFOpCRPlbO4LN4zRm33IaQkX0vPy7gm18=; b=Ssv86RSbEMgSF5iH0Ud0Bc7t8sOMX3W2j1Vw4DVJ5stbSPXbqawJR4kL3v+KFUzxit32lj HIDkXKxHqIKlUf7Uzro21YhC+8+he0c+reOMEnnojolMadV1YFaqY9EdZV83rpvydBuBhL G3A0yDEbc6ph+lC31Hq4qJeNHCEsAfE= ARC-Authentication-Results: i=1; imf21.hostedemail.com; dkim=none; spf=pass (imf21.hostedemail.com: domain of "SRS0=T0lg=34=xs4all.nl=hverkuil@kernel.org" designates 145.40.68.75 as permitted sender) smtp.mailfrom="SRS0=T0lg=34=xs4all.nl=hverkuil@kernel.org"; dmarc=fail reason="SPF not aligned (relaxed), No valid DKIM" header.from=xs4all.nl (policy=none) ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1669623433; a=rsa-sha256; cv=none; b=Rtxo8Tbl4VH2v+Isk3eGqCJu6DFSIQ0sl/xyHvRJVQv0VqEFu8GIuiBzKTtUCwwWM4GdF6 ZL62UrQf5LZLoPg4faHxPO2ZwxkqgrRmbPuH9t25fkmanruhIdBBRSwr058x7S8PMLx9/o wYQlko53JxDa7Vkmn0GQWC/r3P497Jo= X-Rspamd-Queue-Id: AABA51C000C Authentication-Results: imf21.hostedemail.com; dkim=none; spf=pass (imf21.hostedemail.com: domain of "SRS0=T0lg=34=xs4all.nl=hverkuil@kernel.org" designates 145.40.68.75 as permitted sender) smtp.mailfrom="SRS0=T0lg=34=xs4all.nl=hverkuil@kernel.org"; dmarc=fail reason="SPF not aligned (relaxed), No valid DKIM" header.from=xs4all.nl (policy=none) X-Rspam-User: X-Rspamd-Server: rspam11 X-Stat-Signature: cji4abp3mo7eezkinwexxm9hxz47iik9 X-HE-Tag: 1669623432-485648 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: Hi David, On 27/11/2022 11:35, David Hildenbrand wrote: > On 16.11.22 11:26, David Hildenbrand wrote: >> FOLL_FORCE is really only for ptrace access. According to commit >> 707947247e95 ("media: videobuf2-vmalloc: get_userptr: buffers are always >> writable"), get_vaddr_frames() currently pins all pages writable as a >> workaround for issues with read-only buffers. >> >> FOLL_FORCE, however, seems to be a legacy leftover as it predates >> commit 707947247e95 ("media: videobuf2-vmalloc: get_userptr: buffers are >> always writable"). Let's just remove it. >> >> Once the read-only buffer issue has been resolved, FOLL_WRITE could >> again be set depending on the DMA direction. >> >> Cc: Hans Verkuil >> Cc: Marek Szyprowski >> Cc: Tomasz Figa >> Cc: Marek Szyprowski >> Cc: Mauro Carvalho Chehab >> Signed-off-by: David Hildenbrand >> --- >>   drivers/media/common/videobuf2/frame_vector.c | 2 +- >>   1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/media/common/videobuf2/frame_vector.c b/drivers/media/common/videobuf2/frame_vector.c >> index 542dde9d2609..062e98148c53 100644 >> --- a/drivers/media/common/videobuf2/frame_vector.c >> +++ b/drivers/media/common/videobuf2/frame_vector.c >> @@ -50,7 +50,7 @@ int get_vaddr_frames(unsigned long start, unsigned int nr_frames, >>       start = untagged_addr(start); >>         ret = pin_user_pages_fast(start, nr_frames, >> -                  FOLL_FORCE | FOLL_WRITE | FOLL_LONGTERM, >> +                  FOLL_WRITE | FOLL_LONGTERM, >>                     (struct page **)(vec->ptrs)); >>       if (ret > 0) { >>           vec->got_ref = true; > > > Hi Andrew, > > see the discussion at [1] regarding a conflict and how to proceed with > upstreaming. The conflict would be easy to resolve, however, also > the patch description doesn't make sense anymore with [1]. Might it be easier and less confusing if you post a v2 of this series with my patch first? That way it is clear that 1) my patch has to come first, and 2) that it is part of a single series and should be merged by the mm subsystem. Less chances of things going wrong that way. Just mention in the v2 cover letter that the first patch was added to make it easy to backport that fix without being hampered by merge conflicts if it was added after your frame_vector.c patch. Regards, Hans > > > On top of mm-unstable, reverting this patch and applying [1] gives me > an updated patch: > > > From 1e66c25f1467c1f1e5f275312f2c6df29308d4df Mon Sep 17 00:00:00 2001 > From: David Hildenbrand > Date: Wed, 16 Nov 2022 11:26:55 +0100 > Subject: [PATCH] mm/frame-vector: remove FOLL_FORCE usage > > GUP now supports reliable R/O long-term pinning in COW mappings, such > that we break COW early. MAP_SHARED VMAs only use the shared zeropage so > far in one corner case (DAXFS file with holes), which can be ignored > because GUP does not support long-term pinning in fsdax (see > check_vma_flags()). > > Consequently, FOLL_FORCE | FOLL_WRITE | FOLL_LONGTERM is no longer required > for reliable R/O long-term pinning: FOLL_LONGTERM is sufficient. So stop > using FOLL_FORCE, which is really only for ptrace access. > > Reviewed-by: Daniel Vetter > Acked-by: Hans Verkuil > Cc: Hans Verkuil > Cc: Marek Szyprowski > Cc: Tomasz Figa > Cc: Marek Szyprowski > Cc: Mauro Carvalho Chehab > Signed-off-by: David Hildenbrand > --- >  drivers/media/common/videobuf2/frame_vector.c | 2 +- >  1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/common/videobuf2/frame_vector.c b/drivers/media/common/videobuf2/frame_vector.c > index aad72640f055..8606fdacf5b8 100644 > --- a/drivers/media/common/videobuf2/frame_vector.c > +++ b/drivers/media/common/videobuf2/frame_vector.c > @@ -41,7 +41,7 @@ int get_vaddr_frames(unsigned long start, unsigned int nr_frames, bool write, >      int ret_pin_user_pages_fast = 0; >      int ret = 0; >      int err; > -    unsigned int gup_flags = FOLL_FORCE | FOLL_LONGTERM; > +    unsigned int gup_flags = FOLL_LONGTERM; >   >      if (nr_frames == 0) >          return 0;