[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [Nbd] [PATCH 3/9] nbd: Remove kernel internal header



On 2015-02-15 22:56, Markus Pargmann wrote:
On Sat, Feb 14, 2015 at 11:30:24AM +0100, Wouter Verhelst wrote:
On Thu, Feb 12, 2015 at 09:57:31PM +0100, Markus Pargmann wrote:
> The header is not included anywhere. Remove it and include the private
> nbd_device struct in nbd.c.

It exists mostly for the benefit of userspace trying to speak the NBD protocol. I've stopped trying to depend on it (since nbd-server needs to run on !Linux, too), but there are other implementations that might want
to use it.

nbd.h is part of a public API. Let's not drop it.

This is just about the kernel internal header include/linux/nbd.h. It is not about the uapi header. I don't want to remove the protocol header.

The header this patch is about defines only 'struct nbd_device' which is
as far as I can tell only used by nbd.c.

Whoops; I hadn't noticed that had been split up. Yeah, no objection then.

--
It is easy to love a country that is famous for chocolate and beer

  -- Barack Obama, speaking in Brussels, Belgium, 2014-03-26




Reply to: