On Fri, Jan 13, 2012 at 05:07, David Miller <[email protected]> wrote: > From: Stephen Hemminger <[email protected]> > Date: Wed, 11 Jan 2012 21:30:38 -0800 > >> Subject: vhost-net: add module alias (v2.1) >> >> By adding some module aliases, programs (or users) 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. >> >> Also: >> - use C99 style initialization. >> - add missing entry in documentation for loop-control >> >> Signed-off-by: Stephen Hemminger <[email protected]> > > ACKs, NACKs? What is happening here?
In general, static minors are acceptable and very useful to make on-demand loading of kernel modules working. They should be used only for single-instance devices though, which usually means: One single static device name associated with a module. That looks all fine here, and for what it's worth: Acked-By: Kay Sievers <[email protected]> Kay _______________________________________________ Virtualization mailing list [email protected] https://lists.linuxfoundation.org/mailman/listinfo/virtualization
