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=-2.5 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_PASS,USER_AGENT_MUTT 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 0701CC282D4 for ; Wed, 30 Jan 2019 07:53:00 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id DA05821852 for ; Wed, 30 Jan 2019 07:52:59 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730141AbfA3Hw7 (ORCPT ); Wed, 30 Jan 2019 02:52:59 -0500 Received: from verein.lst.de ([213.95.11.211]:50704 "EHLO newverein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725830AbfA3Hw6 (ORCPT ); Wed, 30 Jan 2019 02:52:58 -0500 Received: by newverein.lst.de (Postfix, from userid 2407) id C07BF68CEC; Wed, 30 Jan 2019 08:52:56 +0100 (CET) Date: Wed, 30 Jan 2019 08:52:56 +0100 From: Christoph Hellwig To: Logan Gunthorpe Cc: Jerome Glisse , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Greg Kroah-Hartman , "Rafael J . Wysocki" , Bjorn Helgaas , Christian Koenig , Felix Kuehling , Jason Gunthorpe , linux-pci@vger.kernel.org, dri-devel@lists.freedesktop.org, Christoph Hellwig , Marek Szyprowski , Robin Murphy , Joerg Roedel , iommu@lists.linux-foundation.org Subject: Re: [RFC PATCH 3/5] mm/vma: add support for peer to peer to device vma Message-ID: <20190130075256.GA29665@lst.de> References: <20190129174728.6430-1-jglisse@redhat.com> <20190129174728.6430-4-jglisse@redhat.com> <20190129191120.GE3176@redhat.com> <20190129194418.GG3176@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.17 (2007-11-01) Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org On Tue, Jan 29, 2019 at 01:43:02PM -0700, Logan Gunthorpe wrote: > It's hard to reason about an interface when you can't see what all the > layers want to do with it. Most maintainers (I'd hope) would certainly > never merge code that has no callers, and for much the same reason, I'd > rather not review patches that don't have real use case examples. Yes, we should never review, nevermind merge code without users. We had one example recently where this was not followed, which was HMM and that turned out to be a desaster.