Hi,

On Mon, Feb 17, 2014 at 02:26:54PM +0800, Clanlab (Taiwan) Linux Project wrote:
> @@ -1168,5 +1191,23 @@ void gether_disconnect(struct gether *link)
>  }
>  EXPORT_SYMBOL(gether_disconnect);
>  
> +static int __init gether_init(void)
> +{
> +     gether_wq  = create_singlethread_workqueue("gether");
> +     if (gether_wq == NULL) {
> +             pr_err("Cannot initialize work queue\n");
> +             return -ENOMEM;
> +     }
> +     return 0;
> +}
> +module_init(gether_init);
> +
> +static void __exit gether_exit(void)
> +{
> +     destroy_workqueue(gether_wq);
> +

trailing whitespace.

> +}
> +module_exit(gether_exit);

this is actually supposed to be a library which gets linked against the
actual function module. You need to find a better way to ensure
destroy_workqueue is called.

-- 
balbi

Attachment: signature.asc
Description: Digital signature

Reply via email to