Messages in this thread | | | From | Joe Stringer <> | Subject | Re: [PATCHv2 net 1/2] fm10k: Check tunnel header length in encap offload | Date | Thu, 13 Nov 2014 16:54:19 -0800 |
| |
On Thursday, November 13, 2014 16:41:03 Vick, Matthew wrote: > On 11/13/14, 3:36 PM, "Joe Stringer" <joestringer@nicira.com> wrote: > >fm10k supports up to 184 bytes of inner+outer headers. Add an initial > >check to fail encap offload if these are too large. > > > >Signed-off-by: Joe Stringer <joestringer@nicira.com> > >--- > >Matthew, I didn't see the equivalent patch on netdev so I went ahead and > >created it. If I've missed this somewhere, then please disregard. > > > >v2: First post. > > You didn't miss it Joe--it just hasn't made it up yet. :) It's currently > in Jeff's tree for testing. You're on the CC for the patch, so you'll get > a notification once it goes up. It's basically the same as what you have, > except the #define I use is 184 and I use inner_tcp_hdrlen() to account > for the inner TCP header length. > > Since your second patch should apply cleanly on top of mine, what do you > think about dropping the first patch in this series and Jeff can send our > two patches up together once they've passed testing?
Ah, that sounds great.
Cheers, Joe
| |