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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id C9D56C00140 for ; Thu, 18 Aug 2022 13:52:10 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S245323AbiHRNwI (ORCPT ); Thu, 18 Aug 2022 09:52:08 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47344 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S245321AbiHRNwH (ORCPT ); Thu, 18 Aug 2022 09:52:07 -0400 Received: from mail-qt1-x82c.google.com (mail-qt1-x82c.google.com [IPv6:2607:f8b0:4864:20::82c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 777686172B for ; Thu, 18 Aug 2022 06:52:06 -0700 (PDT) Received: by mail-qt1-x82c.google.com with SMTP id a4so1111541qto.10 for ; Thu, 18 Aug 2022 06:52:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kudzu-us.20210112.gappssmtp.com; s=20210112; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc; bh=a7uF+75/Wt7aRoPYiTPJg2bxJUKJNIPU7dSBbvja52Q=; b=Jp2T7PgF+OjA7rDoP9sdAq+Y4UvHBUu+SEvm9qEUBLvmKBYelmQR4HyHPC+tI8uvSa 5gaN9drDrn5mr7s9EQIF5+D1Inn/9cBVy5BFgQ+cBA843tVibjAxL6AfcEN6wYqNfsdr kst5COZwNTE9QroSfP94YA1E6soPjlDlXOhcc8+KzuHc2n3U501Lb667PnorZ2hximm2 MNyCRive4iFTWtg1VfGarxCJrXAuDHdkzS2gsUH4sHoSchT8kjopaPd3QUPa0dfFgP3f 7I1So/3l+68lI5JQ2+PP4tzKkhTCV7hWxKeC/G1xECgTT2XSvAJD13v42VRZhgqA+aRi cg8Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc; bh=a7uF+75/Wt7aRoPYiTPJg2bxJUKJNIPU7dSBbvja52Q=; b=0Kv68R3k+l2ckqTSXRON0ZmrZGZx4JUzmKByUB5AD7G+7pGxfLW3A8Qij1HLRBNCMT qqhaVea7R3LpQb/eFTqEg1oA7ssKDGHgMwcAkV12xaGccfo5YS4uX2KT0D+zebWWQUfP ZHCvCBxNzwLzHpuldUwI/M//xRIaJv2k4zJygoLrbhTKPF3K+XssIKDWO4lH1oUiaPLE +Y1+RKLkaShl+ASGA9dMg+nrAmWeiy7sexEzgywgijEs90vRJpdMv4vEq7I/OtY50dtt ssRGN85Oxbund0or9MK8xX1BL6ePffq/ERc6hAu1JrGjVv+Bx1p6sEMolhoW8C+CUtn0 o+vg== X-Gm-Message-State: ACgBeo1sFAblR1IWQxAn6FoIq+6v2mSYmQLk8oHYDWN7jaBBX9CiPb7k 3NnHcm5MhEM3f/mA40IGf8gMxgs7L/mCvJxGHEA= X-Google-Smtp-Source: AA6agR5dqBya97ZDwddbePvoc+pRWag8YyuXbA+dXO45tDdm7P7a5zRrAhhuzvoNCePELcz+ttXUaA== X-Received: by 2002:ac8:5a01:0:b0:344:6aef:9a8d with SMTP id n1-20020ac85a01000000b003446aef9a8dmr2655532qta.131.1660830725566; Thu, 18 Aug 2022 06:52:05 -0700 (PDT) Received: from kudzu.us ([2605:a601:a608:5600::59]) by smtp.gmail.com with ESMTPSA id y7-20020a05620a09c700b006b93b61bc74sm1503793qky.9.2022.08.18.06.52.04 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 18 Aug 2022 06:52:05 -0700 (PDT) Date: Thu, 18 Aug 2022 09:52:03 -0400 From: Jon Mason To: Manivannan Sadhasivam Cc: ntb@lists.linux.dev, Bjorn Helgaas , Frank.Li@nxp.com, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, kishon@ti.com Subject: Re: [PATCH] MAINTAINERS: add PCI Endpoint NTB drivers to NTB files Message-ID: References: <20220812194205.388967-1-jdmason@kudzu.us> <20220818060230.GA12008@thinkpad> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20220818060230.GA12008@thinkpad> Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org On Thu, Aug 18, 2022 at 11:32:30AM +0530, Manivannan Sadhasivam wrote: > + Kishon (PCI EP Maintainer) > > On Fri, Aug 12, 2022 at 03:42:05PM -0400, Jon Mason wrote: > > The PCI Endpoint NTB drivers are under the NTB umbrella. Add an entry > > there to allow for notification of changes for it. > > > > Signed-off-by: Jon Mason > > Hi Jason, I assume you mean me. Odd that you got my name wrong 2 lines below it being properly written out. > I know that this patch is already in Linus's tree but I think this PCI Endpoint > VNTB driver is not going in a correct path. First, Kishon is not convinced with > the way the PCI Endpoint VNTB function driver is written currently. He prefers > the VirtIO approach over the current one [1]. To your point, this is already in Linus' tree. If it is not the way people want it, patches accepted. Kishon (in the thread) recommended doing it one way, and Frank responded he liked doing it another. Kishon didn't respond to that last email. To me, this is an acceptable technical disagreement that can be addressed in the future and no need to prevent working patches from being accepted. > But while the conversation was still going on, the series got merged via NTB > tree without any ACKs from the PCI/PCI_EP maintainers. Also, note that there > was a patch touching the PCI Controller driver as well and that was also not > ACKed [2]. I put the series in my ntb-next branch, which was pulled into linux-next for roughly 3 months, and he did not object then (though likely he did not notice). Multiple patches were submitted to the relevant mailing lists to address minor issues in the series (from being in linux-next) and most/all of those hit the PCI mailing list. Bjorn responded to all of them saying they needed to go through the ntb tree (because of the dependency on Frank Li's original series). So while not an explicit ack, it was implicit to me in that he was aware of the series. Given the length of time and the public work on the series, how much longer should I have waited for a nack? > If this trend is going to continue in the coming days, then I'm afraid that NTB > might end up being a backdoor for PCI/PCI_EP patches :( Completely unfounded, per Bjorn's comment on https://lore.kernel.org/all/20220815183920.GA1960006@bhelgaas/ Thanks, Jon > > Thanks, > Mani > > [1] https://lore.kernel.org/all/20220222162355.32369-4-Frank.Li@nxp.com > [2] https://lore.kernel.org/all/20220222162355.32369-2-Frank.Li@nxp.com > > > --- > > MAINTAINERS | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > index 64379c699903..47e9f86bd712 100644 > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -14254,6 +14254,7 @@ W: https://github.com/jonmason/ntb/wiki > > T: git git://github.com/jonmason/ntb.git > > F: drivers/net/ntb_netdev.c > > F: drivers/ntb/ > > +F: drivers/pci/endpoint/functions/pci-epf-*ntb.c > > F: include/linux/ntb.h > > F: include/linux/ntb_transport.h > > F: tools/testing/selftests/ntb/ > > -- > > 2.30.2 > > > > -- > மணிவண்ணன் சதாசிவம்