> From: Varka Bhadram [mailto:varkabhad...@gmail.com]
> Sent: Friday, July 18, 2014 6:13 PM
> 
> On 07/18/2014 04:25 PM, Yue Zhang wrote:
> > @@ -816,8 +816,21 @@ static void netvsc_link_change(struct work_struct
> *w)
> >
> >     rtnl_unlock();
> >
> > -   if (refresh)
> > -           call_usermodehelper(argv[0], argv, envp, UMH_WAIT_EXEC);
> > +   if (refresh) {
> > +           /*
> > +            * Keep the carrier offline for 10 seconds
> > +            * to notify ifplugd daemon network change
> > +            */
> 
> This should be networking comment style..
> 
>       /* bla bla..
>        * bla
>        */
> 
> --
> Regards,
> Varka Bhadram.

I will fix this.

Thanks
Yue
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to