From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason Gunthorpe Subject: Re: [PATCH for-next 0/2] IB/opa_vnic: Add debugfs interface Date: Thu, 28 Sep 2017 14:02:58 -0600 Message-ID: <20170928200258.GA27343@obsidianresearch.com> References: <20170926140714.18110.74689.stgit@scvm10.sc.intel.com> <20170926174946.GA1218@mtr-leonro.local> <20170926175524.GA2297@mtr-leonro.local> <20170926181214.GA41364@knc-06.sc.intel.com> <1506525977.33755.3.camel@redhat.com> <20170928184727.GA33282@knc-06.sc.intel.com> <20170928185753.GG2297@mtr-leonro.local> <20170928190502.GA33289@knc-06.sc.intel.com> <20170928193740.GH2297@mtr-leonro.local> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20170928193740.GH2297-U/DQcQFIOTAAJjI8aNfphQ@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Leon Romanovsky Cc: "Vishwanathapura, Niranjana" , Doug Ledford , Dennis Dalessandro , linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Sudeep Dutt , Sadanand Warrier List-Id: linux-rdma@vger.kernel.org On Thu, Sep 28, 2017 at 10:37:40PM +0300, Leon Romanovsky wrote: > > We find it useful on an ongoing basis, more like some standard NICs > > providing debugfs interface to read/write from/to some of its registers. > > Usually if such interface is spotted during submission, it will be requested > to drop it. Ask Salil, who lately upstreamed new ETH driver and was asked > to remove it. To be fair Salil's was dropped more or less for security reasons.. I think debugfs to dump truely internal state that is not intended to be API stable is reasonable. Promise you will not be tooling on top of this, or use it as an interface in your manager or something crazy like that. You must be happy in all cases if your distro chooses to disable debugfs. The problem I have with reviewing all the vnic patches is that I don't have a spec for vnic or anything, and I don't really know what the data being dumped is.. eg I would have been much happier if ipoib dump'd its debugfs stuff through netlink connected to neighbor objects - I don't know if this is something similar... As Leon says, you should really think carefully if this needs to be something that admins are going to be told to look at when vnic isn't working properly, as the ipoib debufs is. In that case netlink is a stable API and more suitable. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html