2014-10-17 05:23, Liu, Jijiang:
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> > 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

Reply via email to