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=-5.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS autolearn=no 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 E929FC433ED for ; Mon, 10 May 2021 07:31:03 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id B43AF60241 for ; Mon, 10 May 2021 07:31:03 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229863AbhEJHcG (ORCPT ); Mon, 10 May 2021 03:32:06 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54450 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229684AbhEJHcG (ORCPT ); Mon, 10 May 2021 03:32:06 -0400 Received: from casper.infradead.org (casper.infradead.org [IPv6:2001:8b0:10b:1236::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1A5D9C061573; Mon, 10 May 2021 00:31:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=casper.20170209; 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=RlPDTnrGP7VrkHHDZaeu5siASFXrL/7DV46d02KfOg8=; b=jVtYg1DpIhoMALo/8L1gTQX3ww 71nqSsTYByFChENtDDk7rl0AnbqCNYwo4kIyhxfdTGMVo2WSUaokSu6XHznVNC+zNG6w1+ZT7teAE U3xD//EkLHc4hnRNpZgrlDLhKr0c4uniH4OXlSCvuWDEqpbXAyJEuvBrmJwwjpntITYYjTHgWxP8+ TEhIsYdtK717BX+ucCD6EA1Ro67VHN9V61suNwsknLSPRw3z/gYot7uhl/qDZsHvzb9XvRvJ3WxY7 MYKLsyXSwII3NUgCBauC/Y3gs/9XnATB/PvFMAq+mU7jEsLMIynM0ji0AW3may0sqatghH7sRC3gx nZxczo9Q==; Received: from hch by casper.infradead.org with local (Exim 4.94 #2 (Red Hat Linux)) id 1lg0M8-005oIH-AU; Mon, 10 May 2021 07:29:37 +0000 Date: Mon, 10 May 2021 08:29:32 +0100 From: Christoph Hellwig To: Daniel Vetter Cc: Linus Torvalds , Tomasz Figa , Marek Szyprowski , Mauro Carvalho Chehab , DRI Development , LKML , Linux-MM , Linux ARM , Linux Media Mailing List , linux-samsung-soc Subject: Re: [PULL] topic/iomem-mmap-vs-gup Message-ID: References: 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 casper.infradead.org. See http://www.infradead.org/rpr.html Precedence: bulk List-ID: X-Mailing-List: linux-samsung-soc@vger.kernel.org On Mon, May 10, 2021 at 09:16:58AM +0200, Daniel Vetter wrote: > > End result: not pulling it, unless somebody can explain to me in small > > words why I'm wrong and have the mental capacity of a damaged rodent. > > No rodents I think, just more backstory of how this all fits. tldr; > pin_user_pages is the only safe use of this vb2 userptr thing. Yes, which is why I advocate for just ripping the follow_pfn path out entirely. It could have been used for crazy ad dangerous peer to peer transfers outside of any infrastructure making it safe, or for pre-CMA kernel memory carveouts for lage contiguous memory allocations (which are pretty broken by design as well). So IMHO the only sensible thing is to remove this cruft entirely, and if it breaks a currently working setup (which I think is unlikely) we'll have to make sure it can work the proper way.