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 60C3CC25B4E for ; Tue, 24 Jan 2023 05:58:25 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id DCC216B0083; Tue, 24 Jan 2023 00:58:24 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id D55F46B0085; Tue, 24 Jan 2023 00:58:24 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id BCEF96B0089; Tue, 24 Jan 2023 00:58:24 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0010.hostedemail.com [216.40.44.10]) by kanga.kvack.org (Postfix) with ESMTP id AB6BB6B0083 for ; Tue, 24 Jan 2023 00:58:24 -0500 (EST) Received: from smtpin30.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id 8F937403F2 for ; Tue, 24 Jan 2023 05:58:24 +0000 (UTC) X-FDA: 80388637728.30.511A691 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) by imf14.hostedemail.com (Postfix) with ESMTP id 4859E100009 for ; Tue, 24 Jan 2023 05:58:21 +0000 (UTC) Authentication-Results: imf14.hostedemail.com; dkim=pass header.d=infradead.org header.s=bombadil.20210309 header.b=KOkZfWHE; spf=none (imf14.hostedemail.com: domain of BATV+3f260f0a54c0d014680c+7093+infradead.org+hch@bombadil.srs.infradead.org has no SPF policy when checking 198.137.202.133) smtp.mailfrom=BATV+3f260f0a54c0d014680c+7093+infradead.org+hch@bombadil.srs.infradead.org; dmarc=none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1674539903; 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: in-reply-to:in-reply-to:references:references:dkim-signature; bh=MWNNRHw2sWxkb7fRYPZPKkDs8yAj4vB10hhJP/CIuZk=; b=rwYs5fXFHiJ3CoSOdA9H4sN5R7k59GbMVKHY0rtmqaU+DNBWtIjESBs7NL7lukaONvqBZZ w5YvgrdSlabD3/+Od1HUk0tUaHVcZSmEkIhl8q2lLfkXB8cQ1oVNk8LvdtPgE8yg13rr23 ScRjnfZ7aC5ntXqNu+ZjOFrymFRlYYU= ARC-Authentication-Results: i=1; imf14.hostedemail.com; dkim=pass header.d=infradead.org header.s=bombadil.20210309 header.b=KOkZfWHE; spf=none (imf14.hostedemail.com: domain of BATV+3f260f0a54c0d014680c+7093+infradead.org+hch@bombadil.srs.infradead.org has no SPF policy when checking 198.137.202.133) smtp.mailfrom=BATV+3f260f0a54c0d014680c+7093+infradead.org+hch@bombadil.srs.infradead.org; dmarc=none ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1674539903; a=rsa-sha256; cv=none; b=PGRfft/nkO/hNkFrIEh3Js9kfj98ZkKpagNTBLDakRUHqbsRfM/Vjo0mOF73R7toA1PhPZ lw5kcHdGxNOXl7FdKJs/bjKqqhmOjugW5XOyIUp8zMAqXKgsVrymBz3Kee7FiSY3Bm+/Vk 0HFkEU0CEJ4DZG80Tp3CYt6sIV8dcKw= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=In-Reply-To:Content-Type:MIME-Version :References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=MWNNRHw2sWxkb7fRYPZPKkDs8yAj4vB10hhJP/CIuZk=; b=KOkZfWHE68/PeSOfME2tee5dDa JkGrAYpWiQ1HobLr3Xq2I7iD3zq0otuz8oGO2tXGm4dJ1IKX2rNc+K7JD28EhK8j2IXdFoNuHyAiP CbvzZ1ajVll/vWzV6Ybsqo4o3C38aGXBqOhWAQY+rmSuV2oKTj0gIiAmP6A4el9dv8wGAEicPjDit SoWBYSIhq/qXQXz2Tn0yGQsh0ijeYSkiy+iZE4/r+MLTq1wnbn/eP2I6mul4Xu2Ri4EEEq18IXh/O TxDDMSA1V8zwm2iTXu11GRC/YevpRlxnxJ7plHMSaizi5/nBVfKGJBKgHCzSoQVSnKtpIUmNVhUM8 QmIqEn0g==; Received: from hch by bombadil.infradead.org with local (Exim 4.94.2 #2 (Red Hat Linux)) id 1pKCJi-002T1k-5W; Tue, 24 Jan 2023 05:57:58 +0000 Date: Mon, 23 Jan 2023 21:57:58 -0800 From: Christoph Hellwig To: John Hubbard Cc: Jan Kara , David Howells , David Hildenbrand , Al Viro , Christoph Hellwig , Matthew Wilcox , Jens Axboe , Jeff Layton , Logan Gunthorpe , linux-fsdevel@vger.kernel.org, linux-block@vger.kernel.org, linux-kernel@vger.kernel.org, Christoph Hellwig , linux-mm@kvack.org Subject: Re: [PATCH v7 2/8] iov_iter: Add a function to extract a page list from an iterator Message-ID: References: <7bbcccc9-6ebf-ffab-7425-2a12f217ba15@redhat.com> <246ba813-698b-8696-7f4d-400034a3380b@redhat.com> <20230120175556.3556978-1-dhowells@redhat.com> <20230120175556.3556978-3-dhowells@redhat.com> <3814749.1674474663@warthog.procyon.org.uk> <3903251.1674479992@warthog.procyon.org.uk> <3911637.1674481111@warthog.procyon.org.uk> <20230123161114.4jv6hnnbckqyrurs@quack3> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-SRS-Rewrite: SMTP reverse-path rewritten from by bombadil.infradead.org. See http://www.infradead.org/rpr.html X-Rspamd-Server: rspam07 X-Rspamd-Queue-Id: 4859E100009 X-Rspam-User: X-Stat-Signature: gx7fuo9aibbq7iegjic61q3r8g7cjp1h X-HE-Tag: 1674539901-302241 X-HE-Meta: U2FsdGVkX19OwGMdk3ri17Hewb8yqY5vznOtZ90epd5AlDiI9pIejf/Cee0rMkvhUvJ9J+Az1Z07P+RpyC2mAaUPrZUvQsN3bbrY5/ztBManqACJSBt0kqFGdtF/+xw6336Dq6hk2cnqYBV+g6rBS1hCfVsCVR+cFFAwkMX2uRtTO3/6ZwF9cLyFq/l54YBsVsgPzCQ3S2AfsSIQI5VLcCyz7wLEdnx4KjXmMVVONcQpModLwNGwph01Ju04EwGFsBbldIZSFI/VRKIJfZIo5lhP1QZjANvIOVII9EJQ0ZILvVe/4QZSfUs0eWTnX+gZ5+VQ/PFSybhBMo0qPcOMTUP11aLraAE0RMviGlXpro8SCGtgEIokO4Bzi7kCaVh5NPDH+S6rN6a9AqJxnuUjyHSNx1rd6FX2BDND8vp0UVnExkciNuoU8TLjjhf6F+N1UURKlWZobVaWShSYLN9HToW+q7WIX1IkIxXTALJWJ68HCe4tQ40yT/rRpglVtFTT/Ai4dIuqXLb7GHYqjFeS3saSY/Zsvrs9Szrd0uHWfeHfQMI2ohhLga6WNQAHZSZrp03yRRu7SzYABC+n8iqaep2tC7X4APmsLkOnzuxRwzPo5AjxpmEsh3LHGNiQmVei9Zsp9gt6IVP8K3vE/M1MZFqdjSGiKtSyCcAUqZxkrVDC+djxhflUGRT9yFsVXEyJgtioNXsCeR9iJ5g5FP1Mvvqf1XcxLPD74gpK97th4HvJpBUICQ/2reVN/KsvkXtuswlXSM2sABXFMqcKI/GrQLD3KoK3roFbOz3lVHC6ZG4c7JGSFjy9mdQ/EwYxQWzmkDUwiv4BEXETIA8+cXwiSH5545M2LwPy6pOGavBbdiEAXAEG4XIor8mxUcgtahNuIin6cJJMkiBsskmrduBCWrna7SyDZZB1TDc90wZX61eyt0tyvlUHBsbZYOKr8jGHAlL75K7GKtIiuIQXGiq u/6Pc43P VMTbasTOQTZXJRqCUbS8teQjj90NViEG8tvCejzpQM8SgprHVMOtXGBhHBC15I6IrEZFYMKjrT/GNLJW1axLUqWIkvQQiSGpvxXx3xc2lml3qu8a8RcCiv6Q0Qf1jwKcttSgQlqrfnEQ/MEAK9fg8ZLYtHoZfrbwgeXY7j6GKgIYeygrAGEuduHKqXvtezDOkusKbMl/+Ty0rlTVv3Fdboh9cw4B6U5vRvIcy/lYP6xRAauo/MdrpOIq22X+V3uXgI5/51y0p34ctuHmBK8lTjx3jVxXPCcf1TZb3smSB+d+76ekVU3zvvRT/x203t+RHgcm0Eq5Orw975UTLcERDZvvRivhAX5wQSK7e47HRMsCP5+qeu/oT6vLh/vExMbZsikzrG6PSV3Z/DUl1mSOjndeknI3ZSn/Fb4Mfo3xgYXO6ofcx4BwgdL9kA7Oqo3tVMSYVyef2UdqdT/rDuPL7sUpHI7ujpbR3IPKDOc5kt7sfzdtMr+freW3JX2uVCRnBeZN9Gt+/frtVdPg= 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 Mon, Jan 23, 2023 at 03:07:48PM -0800, John Hubbard wrote: > On 1/23/23 08:11, Jan Kara wrote: > > > For cifs RDMA, do I need to make it pass in FOLL_LONGTERM? And does that need > > > a special cleanup? > > > > FOLL_LONGTERM doesn't need a special cleanup AFAIK. It should be used > > whenever there isn't reasonably bound time after which the page is > > unpinned. So in case CIFS sets up RDMA and then it is up to userspace how > > long the RDMA is going to be running it should be using FOLL_LONGTERM. The > > Yes, we have been pretty consistently deciding that RDMA generally > implies FOLL_LONGTERM. (And furthermore, FOLL_LONGTERM implies > FOLL_PIN--that one is actually enforced by the gup/pup APIs.) That's weird. For storage or file systems, pages are pinnen just as long when using RDMA as when using local DMA, in fact if you do RDMA to really fast remote media vs slow local media (e.g. SSD vs disk) you might pin it shorter when using RDMA. I think FOLL_LONGTERM makes sense for non-ODP user space memory registrations for RDMA, which will last basically forever. It does not really make much sense at all for in-kernel memory registration for RDMA that are very short term.