Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Mon, 1 Feb 2021 17:04:56 +0100
From:   Greg KH <gregkh@linuxfoundation.org>
To:     Eli Billauer <eli.billauer@gmail.com>
Cc:     arnd@arndb.de, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2] char: xillybus: Add driver for XillyUSB (Xillybus
 variant for USB)
Message-ID: <YBgmqBi1PFDxWW1X@kroah.com>
References: <20210201141651.62853-1-eli.billauer@gmail.com>
 <YBgPcX0J5m0bkhuA@kroah.com>
 <601824B6.60803@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <601824B6.60803@gmail.com>
Precedence: bulk

On Mon, Feb 01, 2021 at 05:56:38PM +0200, Eli Billauer wrote:
> Hello, Greg.
> 
> On 01/02/21 16:25, Greg KH wrote:
> > As this has nothing to do with the existing XillyBus code, and you
> > aren't doing anything "fancy" with the character device nodes, why does
> > this need to be a kernel driver at all?  Why can't you just use libusb
> > and talk to the device directly from userspace with no kernel driver
> > needed?
> That's actually what I did in my first attempt.
> 
> However in order to provide the same API as the existing Xillybus driver,
> this driver must present some kind of device files.

But you are not providing the same API!  Or so you are saying in the
driver.

If you were, then you need to use the same device nodes that the
existing xillybus code uses and share them.  Which is what I asked about
before.

> Named pipes aren't sufficient, among others because closing a Xillybus
> device file is something that the hardware is informed about immediately.
> Also, when a device file is opened for write, the close() call should not
> return until all data has arrived to hardware (per existing API). This can't
> be guaranteed with named pipes.

What does a named pipe have to do with anything here?

> So a user-space driver needs to rely on CUSE. Which I wasted two months on
> before realizing it's not stable enough to rely upon. My whines on CUSE can
> be found in my blog:

Why is cuse an issue here?

Just use libusb and have your userspace programs link against it.

thanks,

greg k-h