[dpdk-dev] [PATCH 01/10] examples/tep_termination:initialize the VXLAN sample

2015-05-18 Thread Liu, Jijiang
> -Original Message- > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > Sent: Saturday, May 16, 2015 7:56 AM > To: Liu, Jijiang > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 01/10] examples/tep_termination:initialize the > VXLAN sample >

[dpdk-dev] [PATCH 01/10] examples/tep_termination:initialize the VXLAN sample

2015-05-18 Thread Liu, Jijiang
Hi Stephen, Thanks for reviewing the patch. > -Original Message- > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > Sent: Saturday, May 16, 2015 7:54 AM > To: Liu, Jijiang > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 01/10] examples/tep_term

[dpdk-dev] [PATCH 01/10] examples/tep_termination:initialize the VXLAN sample

2015-05-15 Thread Stephen Hemminger
On Fri, 15 May 2015 14:08:52 +0800 Jijiang Liu wrote: > + while (dev_ll != NULL) { > + /*get virtio device ID*/ Really minor style nit. Please put whitespace in comments. Do this instead. /* get virtio device ID */ Also, the name virtio is confusing since

[dpdk-dev] [PATCH 01/10] examples/tep_termination:initialize the VXLAN sample

2015-05-15 Thread Stephen Hemminger
I agree that this is great to see a real example of this On Fri, 15 May 2015 14:08:52 +0800 Jijiang Liu wrote: > +static unsigned > +check_ports_num(unsigned nb_ports) > +{ > + unsigned valid_nb_ports = nb_ports; > + unsigned portid; > + > + if (nb_ports > nb_ports) { > +

[dpdk-dev] [PATCH 01/10] examples/tep_termination:initialize the VXLAN sample

2015-05-15 Thread Jijiang Liu
This sample uses the basic virtio devices management function from the vHost example, which includes virtio device creation, destroying and maintenance. Signed-off-by: Jijiang Liu --- examples/Makefile |1 + examples/tep_termination/Makefile | 55 ++ examples/tep_terminati