tech-kern archive

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

Re: File sharing over virtio-9p



On Fri, Oct 25, 2019 at 3:30 AM Valery Ushakov <uwe%stderr.spb.ru@localhost> wrote:
>
> On Thu, Oct 24, 2019 at 13:32:59 +0900, Ryota Ozaki wrote:
>
> > I've prepared complete patches ready to commit:
> >   https://www.netbsd.org/~ozaki-r/tweak-MAKEDEV.diff
> >   https://www.netbsd.org/~ozaki-r/vio9p.diff
> >   https://www.netbsd.org/~ozaki-r/vio9p-configs.diff
> >   https://www.netbsd.org/~ozaki-r/mount_9p-cdev.diff
>
> Just a few minor nit-picks:
>
> @@ -60,6 +60,8 @@ usage(void)
> [...]
> +   fprintf(stderr, "       %s [-s] [-o mntopts] -cu cdevfile mountpoint\n",
>
> Why not [-su] as in the usage just above for the TCP case?

I wrote it as "-cu" because for qemu -u is always required.  But
as you said "-c [-su]" is more straightforward, I'll fix it (actually
I uses it in mount_9p.8).  Also there may be an implementation of
virtio-9p that doesn't require -u :)

>
>
> @@ -72,7 +74,7 @@ serverconnect(const char *addr, unsigned short port)
> [...]
> +           err(1, "setsockopt(SO_NOSIGPIPE)");
>
> I'd just trim it down to "SO_NOSIGPIPE".
>
> +        err(1, "open(%s)", path);
>
> Ditto.  Just make it "%s".

I prefer more informative messages.  Why do you want to trim them?

Thanks,
  ozaki-r


Home | Main Index | Thread Index | Old Index