Re: [Nbd] [PATCH 4/4] nbd: add a nbd-control interface
- To: Alex Gartrell <agartrell@...2204...>
- Cc: "nbd-general@lists.sourceforge.net" <nbd-general@lists.sourceforge.net>, "linux-block@...25..." <linux-block@...25...>, Arnd Bergmann <arnd@...2332...>, Josef Bacik <jbacik@...2204...>, Greg KH <gregkh@...1299...>, Jens Axboe <axboe@...2204...>, Kernel Team <Kernel-team@...2204...>
- Subject: Re: [Nbd] [PATCH 4/4] nbd: add a nbd-control interface
- From: Alex Bligh <alex@...872...>
- Date: Wed, 25 Jan 2017 18:25:11 +0000
- Message-id: <5F1F69A4-CDE1-4519-9FB4-C6B1DED5BBAE@...872...>
- In-reply-to: <F7E2FE6F-B7A8-4A36-854D-AE1620D88762@...2204...>
- References: <1484949412-6903-1-git-send-email-jbacik@...2204...> <1484949412-6903-4-git-send-email-jbacik@...2204...> <20170121090531.GB27048@...926...> <1485182528.9861.22@...2791...> <20170123145212.GA19582@...926...> <1485183472.21123.0@...2791...> <20170123150325.GB26884@...926...> <1485186762.21123.1@...2791...> <20170124071152.GB13251@...926...> <1485352053.5902.0@...2791...> <CAK8P3a10K-8GwwcZLd4HwYHzxcB8cwww3y73np_Th65af7m6gQ@...18...> <F7E2FE6F-B7A8-4A36-854D-AE1620D88762@...2204...>
> On 25 Jan 2017, at 16:48, Alex Gartrell <agartrell@...2204...> wrote:
>
>
> If nbd were *all* netlink I think that that'd be fine, but you'd have
> problems implementing the NBD_DOIT function in that fashion. So I'd
> rather stick to the char device ioctl thing because it's more
> consistent with the old NBD stuff as well as the loop device stuff.
I spend most of my time looking at the userspace side of NBD so
apologies if this is off base.
Given (because of NBD_DO_IT) we need an ioctl anyway, and we have
an ioctl that isn't going to go away, it would seem better if possible
to stick with ioctls, and not introduce either a dependency
on netlink (which would presumably bloat static binaries that
are used early in the boot process). Personally I'd have thought
adding a new NBD ioctl (or extending an existing one) would be
less entropy than adding a new char device.
--
Alex Bligh
Reply to: