netdev - Re: [PATCH] vhost-net: add module alias (v2)

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1326315617.2705.10.camel@bwh-desktop>
Date:	Wed, 11 Jan 2012 21:00:17 +0000
From:	Ben Hutchings <bhutchings@...arflare.com>
To:	Stephen Hemminger <shemminger@...tta.com>
CC:	Alan Cox <device@...ana.org>,
	"Michael S. Tsirkin" <mst@...hat.com>, <netdev@...r.kernel.org>,
	<kvm@...r.kernel.org>, <virtualization@...ts.linux-foundation.org>,
	Kay Sievers <kay.sievers@...y.org>
Subject: Re: [PATCH] vhost-net: add module alias (v2)

On Wed, 2012-01-11 at 09:16 -0800, Stephen Hemminger wrote:
> By adding the correct module alias, programs won't have to explicitly
> call modprobe. Vhost-net will always be available if built into the kernel.
> It does require assigning a permanent minor number for depmod to work.
> Choose one next to TUN since this driver is related to it.
> 
> Also, use C99 style initialization.
> 
> Signed-off-by: Stephen Hemminger <shemminger@...tta.com>
> 
> ---
> v2 - document minor number and make sure to not overlap
[...]
> --- a/include/linux/miscdevice.h	2012-01-10 10:56:59.779189436 -0800
> +++ b/include/linux/miscdevice.h	2012-01-11 09:13:20.803694316 -0800
> @@ -42,6 +42,7 @@
>  #define AUTOFS_MINOR		235
>  #define MAPPER_CTRL_MINOR	236
>  #define LOOP_CTRL_MINOR		237
> +#define VHOST_NET_MINOR		238
>  #define MISC_DYNAMIC_MINOR	255
>  
>  struct device;
> --- a/Documentation/devices.txt	2012-01-10 10:56:53.399116518 -0800
> +++ b/Documentation/devices.txt	2012-01-11 09:12:49.251197653 -0800
> @@ -447,6 +447,8 @@ Your cooperation is appreciated.
>  		234 = /dev/btrfs-control	Btrfs control device
>  		235 = /dev/autofs	Autofs control device
>  		236 = /dev/mapper/control	Device-Mapper control device
> +		237 = /dev/vhost-net	Host kernel accelerator for virtio net
[...]

238 != 237.  It looks like someone forgot to add loopctrl here.

Ben.

-- 
Ben Hutchings, Staff Engineer, Solarflare
Not speaking for my employer; that's the marketing department's job.
They asked us to note that Solarflare product names are trademarked.

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html