From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga09.intel.com ([134.134.136.24]:21104 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1764099AbcLSRTk (ORCPT ); Mon, 19 Dec 2016 12:19:40 -0500 Date: Mon, 19 Dec 2016 12:29:03 -0500 From: Keith Busch To: Logan Gunthorpe Cc: Myron Stowe , Bjorn Helgaas , Greg Kroah-Hartman , Geert Uytterhoeven , Jonathan Corbet , "David S. Miller" , Andrew Morton , Emil Velikov , Mauro Carvalho Chehab , Guenter Roeck , Kurt Schwemmer , Stephen Bates , linux-pci , linux-doc@vger.kernel.org, linux-nvme@lists.infradead.org, LKML Subject: Re: [RFC 0/1] New PCI Switch Management Driver Message-ID: <20161219172903.GB1043@localhost.localdomain> References: <1481994562-9283-1-git-send-email-logang@deltatee.com> <735068a7-7805-dad0-b5b0-5218a18f335c@deltatee.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <735068a7-7805-dad0-b5b0-5218a18f335c@deltatee.com> Sender: linux-pci-owner@vger.kernel.org List-ID: On Mon, Dec 19, 2016 at 10:06:56AM -0700, Logan Gunthorpe wrote: > As I noted, the hardware is compliant and works perfectly fine with the > in-kernel driver. However, the hardware has many additional custom > features that are not covered by the PCI specs. For example, it has an > interface to count packets that match a specific criteria. It also has > firmware that can be expanded to do completely custom things by the > user. Additionally, the switch is _very_ configurable and has a > configuration file that can be uploaded and downloaded. > > All these features and more are exposed through a special management > endpoint that is completely separate from the standard PCI switch > interface. This work is a driver for that endpoint and is not required > to use the switch. It only makes the advanced features available to the > user. Since the in-kernel driver binds to the device, won't this driver conflict with the initialization the in-kernel one already does? Bus master, MSI setup, etc? Could you also provide the reasoning against making the functionality this driver provides in user space?