On Fri, Jun 16, 2017 at 12:02 PM, David Miller <da...@davemloft.net> wrote: > From: Wei Wang <tracyw...@gmail.com> > Date: Fri, 16 Jun 2017 10:47:25 -0700 > >> + if (dst) >> + /* set noref for now. >> + * any place which wants to hold dst has to call >> + * dst_hold_safe() >> + */ >> + skb_dst_set_noref(skb, dst); > > You must enclose the code in curly braces if you want to put a comment > in this one-line basic block of the 'if' statement. > > Otherwise it's hard to read. > > Likewise for the other similar change in this file. >
Got it. Will update in the next version.