diff mbox

connector: add module alias

Message ID 20101209093646.1b5a6abb@nehalam
State Accepted, archived
Delegated to: David Miller
Headers show

Commit Message

stephen hemminger Dec. 9, 2010, 5:36 p.m. UTC
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

Comments

Evgeniy Polyakov Dec. 9, 2010, 6:47 p.m. UTC | #1
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;
>
David Miller Dec. 10, 2010, 8:28 p.m. UTC | #2
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
Evgeniy Polyakov Dec. 10, 2010, 8:44 p.m. UTC | #3
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.
diff mbox

Patch

--- 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;