[ovs-dev] [PATCHv3] userspace: Add GTP-U support.

William Tu u9012063 at gmail.com
Thu Dec 5 18:17:46 UTC 2019


On Thu, Dec 05, 2019 at 09:26:06AM -0800, Ben Pfaff wrote:
> On Thu, Dec 05, 2019 at 05:41:45AM -0800, William Tu wrote:
> > One question I have in this patch is whether to increase FLOW_WC_SEQ.
> > And what's the purpose of this number?
> > 
> >  /* This sequence number should be incremented whenever anything involving flows
> >    * or the wildcarding of flows changes.  This will cause build assertion
> >    * failures in places which likely need to be updated. */
> >  #define FLOW_WC_SEQ 41
> > 
> > In this patch, the struct flow_tnl is modified but size does not increase,
> > so build assertion is passed.
> 
> The purpose is to trigger build assertions in all the places that are
> likely to require updates when a new field is added.  The change in size
> is just a way to trigger a reminder to update FLOW_WC_SEQ; not all new
> fields change the size, but many new fields do.
> 
> I recommend incrementing it, because the updates to the build assertions
> throughout the tree also help with review.
OK will do it.
Thank you
William


More information about the dev mailing list