Re: [PATCH v5 3/8]app/test-pmd:test VxLAN packet identification

[Thomas Monjalon <thomas.monjalon-pdR9zngts4EAvxtiuMwx3w@public.gmane.org>]

2014-10-17 05:23, Liu, Jijiang:
> From: Thomas Monjalon [mailto:thomas.monjalon-pdR9zngts4EAvxtiuMwx3w@public.gmane.org]
> > 2014-10-11 13:55, Jijiang Liu:
> > > -			"tx_checksum set mask (port_id)\n"
> > > +			"tx_checksum set (mask) (port_id)\n"
> > >  			"    Enable hardware insertion of checksum offload
> > with"
> > > -			" the 4-bit mask, 0~0xf, in packets sent on a port.\n"
> > > +			" the 8-bit mask, 0~0xff, in packets sent on a port.\n"
> > >  			"        bit 0 - insert ip   checksum offload if set\n"
> > >  			"        bit 1 - insert udp  checksum offload if set\n"
> > >  			"        bit 2 - insert tcp  checksum offload if set\n"
> > >  			"        bit 3 - insert sctp checksum offload if set\n"
> > > +			"        bit 4 - insert inner ip  checksum offload if
> > set\n"
> > > +			"        bit 5 - insert inner udp checksum offload if
> > set\n"
> > > +			"        bit 6 - insert inner tcp checksum offload if
> > set\n"
> > > +			"        bit 7 - insert inner sctp checksum offload if
> > set\n"
> > >  			"    Please check the NIC datasheet for HW
> > limits.\n\n"
> > [...]
> > >  	.help_str = "enable hardware insertion of L3/L4checksum with a
> > given "
> > > -	"mask in packets sent on a port, the bit mapping is given as, Bit 0 for
> > ip"
> > > -	"Bit 1 for UDP, Bit 2 for TCP, Bit 3 for SCTP",
> > > +	"mask in packets sent on a port, the bit mapping is given as, Bit 0 for
> > ip "
> > > +	"Bit 1 for UDP, Bit 2 for TCP, Bit 3 for SCTP, Bit 4 for inner ip "
> > > +	"Bit 5 for inner UDP, Bit 6 for inner TCP, Bit 7 for inner SCTP",
> > >  	.tokens = {
> > >  		(void *)&cmd_tx_cksum_set_tx_cksum,
> > >  		(void *)&cmd_tx_cksum_set_set,
> > 
> > How is it related to VXLAN?
> > I may have missed something. But if not, I note the name of the reviewers ;)
> 
> Yes, it is related to VxLAN Tx Checksum test, I will put the changes to test VxLAN Tx checksum patch.

Yes it must be moved in the patch related to checksum test.

-- 
Thomas