Sorry, I forget to run the check_patch.pl script this time. Now I send the v5 update.
Regards Feng On Mon, Aug 22, 2016 at 10:43 PM, Guillaume Nault <g.na...@alphalink.fr> wrote: > On Mon, Aug 22, 2016 at 06:59:51PM +0800, f...@ikuai8.com wrote: >> From: Gao Feng <f...@ikuai8.com> >> >> Use PPP_ALLSTATIONS, PPP_UI, and SEND_SHUTDOWN instead of 0xff, >> 0x03, and 2 separately. >> > Apart from the checkpatch errors, > > Acked-by: Guillaume Nault <g.na...@alphalink.fr> > >> @@ -325,8 +324,8 @@ static int pppol2tp_sendmsg(struct socket *sock, struct >> msghdr *m, >> skb_reserve(skb, uhlen); >> >> /* Add PPP header */ >> - skb->data[0] = ppph[0]; >> - skb->data[1] = ppph[1]; >> + skb->data[0] = PPP_ALLSTATIONS; >> > Trailing whitespace. > >> @@ -398,14 +396,14 @@ static int pppol2tp_xmit(struct ppp_channel *chan, >> struct sk_buff *skb) >> sizeof(struct iphdr) + /* IP header */ >> uhlen + /* UDP header (if L2TP_ENCAPTYPE_UDP) >> */ >> session->hdr_len + /* L2TP header */ >> - sizeof(ppph); /* PPP header */ >> + 2; /* 2 bytes for PPP_ALLSTATIONS & >> PPP_UI */ >> if (skb_cow_head(skb, headroom)) >> goto abort_put_sess_tun; >> >> /* Setup PPP header */ >> - __skb_push(skb, sizeof(ppph)); >> - skb->data[0] = ppph[0]; >> - skb->data[1] = ppph[1]; >> + __skb_push(skb, 2); >> > Here too.