Message ID | 20101209093646.1b5a6abb@nehalam |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
On Thu, Dec 09, 2010 at 09:36:46AM -0800, Stephen Hemminger (shemminger@vyatta.com) wrote: > Since connector can be built as a module and uses netlink socket > to communicate. The module should have an alias to autoload when socket > of NETLINK_CONNECTOR type is requested. Ack, thank you. David, please apply. > Signed-off-by: Stephen Hemminger <shemminger@vyatta.com> > > --- a/drivers/connector/connector.c 2010-12-09 09:19:05.549120343 -0800 > +++ b/drivers/connector/connector.c 2010-12-09 09:19:52.795051458 -0800 > @@ -36,6 +36,7 @@ > MODULE_LICENSE("GPL"); > MODULE_AUTHOR("Evgeniy Polyakov <zbr@ioremap.net>"); > MODULE_DESCRIPTION("Generic userspace <-> kernelspace connector."); > +MODULE_ALIAS_NET_PF_PROTO(PF_NETLINK, NETLINK_CONNECTOR); > > static struct cn_dev cdev; >
From: Evgeniy Polyakov <zbr@ioremap.net> Date: Thu, 9 Dec 2010 21:47:46 +0300 > On Thu, Dec 09, 2010 at 09:36:46AM -0800, Stephen Hemminger (shemminger@vyatta.com) wrote: >> Since connector can be built as a module and uses netlink socket >> to communicate. The module should have an alias to autoload when socket >> of NETLINK_CONNECTOR type is requested. > > Ack, thank you. Please use formalized "Acked-by: " tags so that the automated tools do all the work for me and I don't have to type it in by hand. > David, please apply. Done, thanks. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, Dec 10, 2010 at 12:28:13PM -0800, David Miller (davem@davemloft.net) wrote: > >> Since connector can be built as a module and uses netlink socket > >> to communicate. The module should have an alias to autoload when socket > >> of NETLINK_CONNECTOR type is requested. > > > > Ack, thank you. > > Please use formalized "Acked-by: " tags so that the automated tools do > all the work for me and I don't have to type it in by hand. Will do, thanks.
--- a/drivers/connector/connector.c 2010-12-09 09:19:05.549120343 -0800 +++ b/drivers/connector/connector.c 2010-12-09 09:19:52.795051458 -0800 @@ -36,6 +36,7 @@ MODULE_LICENSE("GPL"); MODULE_AUTHOR("Evgeniy Polyakov <zbr@ioremap.net>"); MODULE_DESCRIPTION("Generic userspace <-> kernelspace connector."); +MODULE_ALIAS_NET_PF_PROTO(PF_NETLINK, NETLINK_CONNECTOR); static struct cn_dev cdev;
Since connector can be built as a module and uses netlink socket to communicate. The module should have an alias to autoload when socket of NETLINK_CONNECTOR type is requested. Signed-off-by: Stephen Hemminger <shemminger@vyatta.com> -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html