On Wed, Nov 20, 2019 at 05:20:35PM -0800, William Tu wrote:
> On Wed, Nov 20, 2019 at 11:25:12AM -0800, Ben Pfaff wrote:
> > On Wed, Nov 06, 2019 at 12:35:49PM -0800, William Tu wrote:
> > > On Tue, Nov 05, 2019 at 04:49:10PM -0800, Ben Pfaff wrote:
> > > > On Tue, Nov 05, 2019 at 02:57:46PM -0800, William Tu wrote:
> > > > > + /* ODP_SUPPORT_FIELDS */
> > > > > + str_value = xasprintf("%"PRIuSIZE, odp.max_vlan_headers);
> > > > > + smap_add(cap, "max_vlan_headers", str_value);
> > > > > + free(str_value);
> > > >
> > > > I think that you can shorten the above to:
> > > > smap_add_format(cap, "max_vlan_headers", "%"PRIuSIZE,
> > > > odp.max_vlan_headers);
> > > > and similarly for other cases.
Hi Ben,
I forgot to add above fixes.
I will send out another patch, sorry about that.
William
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
