[dpdk-dev] [PATCH v4 01/10] examples/tep_termination:initialize the VXLAN sample
Liu, Jijiang
jijiang.liu at intel.com
Mon Jun 22 15:55:27 CEST 2015
> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Sent: Monday, June 22, 2015 3:49 PM
> To: Liu, Jijiang
> Cc: dev at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v4 01/10] examples/tep_termination:initialize
> the VXLAN sample
>
> 2015-06-18 09:03, Jijiang Liu:
> > This sample uses the basic virtio devices management function from the vHost
> example, which includes virtio device creation, destroying and maintenance.
>
> Please wrap commit message to about 72 characters.
> https://github.com/erlang/otp/wiki/Writing-good-commit-messages
> https://git-scm.com/book/ch5-2.html
Ok, will follow this rule.
> typo in Makefile:
> overriden -> overridden
>
> There are 2 functions where ret is incremented but never initialized:
> examples/tep_termination/main.c:473:6: error: ‘ret’ may be used uninitialized in
> this function
> examples/tep_termination/main.c:536:8: error: ‘ret’ may be used uninitialized in
> this function
>
> Making a patch buildable with recent GCC and clang is the minimum before
> submitting it.
There is no a such error when I used GCC version 4.6.2.
I will check the codes and use latest gcc version and clang to try.
Anyway, thanks.
More information about the dev
mailing list