* [dpdk-stable] " app/testpmd: fix NVGRE encap configuration" incompatible with 19.11.x @ 2021-06-02 13:25 Christian Ehrhardt 2021-06-03 14:54 ` Jiawei(Jonny) Wang 0 siblings, 1 reply; 6+ messages in thread From: Christian Ehrhardt @ 2021-06-02 13:25 UTC (permalink / raw) To: Jiawei Wang; +Cc: dpdk stable, Ori Kam Hi, this change was targeted for the stable releases commit 9b0da816bdec98c0d6c5d17ccdd337bdbafc3f75 Author: Jiawei Wang <jiaweiw@nvidia.com> Date: Tue Mar 16 06:18:27 2021 +0200 app/testpmd: fix NVGRE encap configuration But when applied it causes: [ 263s] /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/app/test-pmd/cmdline_flow.c: In function 'parse_vc_action_nvgre_encap': [ 263s] /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/app/test-pmd/cmdline_flow.c:4254:3: error: missing initializer for field 'protocol' of 'struct rte_flow_item_nvgre' [-Werror=missing-field-initializers] [ 263s] .item_nvgre.protocol = RTE_BE16(RTE_ETHER_TYPE_TEB), [ 263s] ^ [ 263s] In file included from /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/x86_64-default-linux-gcc/include/rte_eth_ctrl.h:11:0, [ 263s] from /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/x86_64-default-linux-gcc/include/rte_ethdev.h:978, [ 263s] from /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/app/test-pmd/cmdline_flow.c:18: [ 263s] /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/x86_64-default-linux-gcc/include/rte_flow.h:917:13: note: 'protocol' declared here [ 263s] rte_be16_t protocol; /**< Protocol type (0x6558). */ [ 263s] ^ This is failing not everywhere but in: - SLE_12_SP4/x86_64 - RHEL_7/x86_64 - openSUSE_Leap_42.3/x86_64 The type and the init code looks the same here and in the recent main branch. Neither do I see recent changes that would cause this. And the init just looks right to me, so I'm sure I'm overlooking something. So I wondered (if you consider this patch important for 19.11.x) if you could have a look and consider backporting it so that it works well (or as an alternative let me know that it won't be needed for 19.11.x) References/Logs: 19.11.8 working: https://paste.ubuntu.com/p/vvm4qCcmJT/ 19.11.9 fail RH7: https://paste.ubuntu.com/p/2wph5nfbCg/ 19.11.9 fail SLE12SP4: https://paste.ubuntu.com/p/nGHhkPJbR3/ 19.11.9 fail openSuse42.3: https://paste.ubuntu.com/p/43mMx3sY2n/ 19.11.9 witzh the patch dropped works: https://paste.ubuntu.com/p/JGZ44BhkC6/ -- Christian Ehrhardt Staff Engineer, Ubuntu Server Canonical Ltd ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-stable] " app/testpmd: fix NVGRE encap configuration" incompatible with 19.11.x 2021-06-02 13:25 [dpdk-stable] " app/testpmd: fix NVGRE encap configuration" incompatible with 19.11.x Christian Ehrhardt @ 2021-06-03 14:54 ` Jiawei(Jonny) Wang 2021-06-04 5:07 ` Christian Ehrhardt 0 siblings, 1 reply; 6+ messages in thread From: Jiawei(Jonny) Wang @ 2021-06-03 14:54 UTC (permalink / raw) To: Christian Ehrhardt; +Cc: dpdk stable, Ori Kam Hello Christian, I applied this patch on the 19.11.9-rc1, but not found this issue during compilation. Could you send me your code branch then I can try on my side? Code base: commit 15d8f64d18a891a70d664d18eb918eee0705993e Author: Christian Ehrhardt <christian.ehrhardt@canonical.com> Date: Wed Jun 2 16:33:26 2021 +0200 version: 19.11.9-rc1 Signed-off-by: Christian Ehrhardt <christian.ehrhardt@canonical.com> #make O=x86_64-default-linux-gcc T=x86_64-native-linux-gcc config #sed -ri 's,(RTE_MACHINE=).*,\1default,' x86_64-default-linux-gcc/.config #sed -ri 's,(RTE_APP_TEST=).*,\1n,' x86_64-default-linux-gcc/.config #sed -ri 's,(RTE_BUILD_SHARED_LIB=).*,\1y,' x86_64-default-linux-gcc/.config #sed -ri 's,(RTE_NEXT_ABI=).*,\1n,' x86_64-default-linux-gcc/.config #sed -ri 's,(LIBRTE_VHOST=).*,\1y,' x86_64-default-linux-gcc/.config #sed -ri 's,(LIBRTE_VHOST=).*,\1y,' x86_64-default-linux-gcc/.config #sed -ri 's,(RTE_KNI_KMOD=).*,\1y,' x86_64-default-linux-gcc/.config #sed -ri 's,(RTE_EAL_IGB_UIO=).*,\1y,' x86_64-default-linux-gcc/.config # #make O=x86_64-default-linux-gcc -j3 EXTRA_CFLAGS=-Werror ...... INSTALL-APP dpdk-test-crypto-perf INSTALL-MAP dpdk-test-crypto-perf.map CC test_perf_queue.o CC test_perf_atq.o CC test_pipeline_common.o CC test_pipeline_queue.o CC test_pipeline_atq.o LD dpdk-test-eventdev INSTALL-APP dpdk-test-eventdev INSTALL-MAP dpdk-test-eventdev.map Build complete [x86_64-default-linuxapp-gcc] Thanks. B.R. Jonny > -----Original Message----- > From: Christian Ehrhardt <christian.ehrhardt@canonical.com> > Sent: Wednesday, June 2, 2021 9:25 PM > To: Jiawei(Jonny) Wang <jiaweiw@nvidia.com> > Cc: dpdk stable <stable@dpdk.org>; Ori Kam <orika@nvidia.com> > Subject: " app/testpmd: fix NVGRE encap configuration" incompatible with > 19.11.x > > Hi, > this change was targeted for the stable releases > > commit 9b0da816bdec98c0d6c5d17ccdd337bdbafc3f75 > Author: Jiawei Wang <jiaweiw@nvidia.com> > Date: Tue Mar 16 06:18:27 2021 +0200 > > app/testpmd: fix NVGRE encap configuration > > But when applied it causes: > > [ 263s] /home/abuild/rpmbuild/BUILD/dpdk- > 1622611368.078834f40/app/test-pmd/cmdline_flow.c: > In function 'parse_vc_action_nvgre_encap': > [ 263s] /home/abuild/rpmbuild/BUILD/dpdk- > 1622611368.078834f40/app/test-pmd/cmdline_flow.c:4254:3: > error: missing initializer for field 'protocol' of 'struct rte_flow_item_nvgre' [- > Werror=missing-field-initializers] > [ 263s] .item_nvgre.protocol = RTE_BE16(RTE_ETHER_TYPE_TEB), > [ 263s] ^ > [ 263s] In file included from > /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/x86_64- > default-linux-gcc/include/rte_eth_ctrl.h:11:0, > [ 263s] from > /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/x86_64- > default-linux-gcc/include/rte_ethdev.h:978, > [ 263s] from > /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/app/test- > pmd/cmdline_flow.c:18: > [ 263s] /home/abuild/rpmbuild/BUILD/dpdk- > 1622611368.078834f40/x86_64-default-linux-gcc/include/rte_flow.h:917:13: > note: 'protocol' declared here > [ 263s] rte_be16_t protocol; /**< Protocol type (0x6558). */ > [ 263s] ^ > > This is failing not everywhere but in: > - SLE_12_SP4/x86_64 > - RHEL_7/x86_64 > - openSUSE_Leap_42.3/x86_64 > > The type and the init code looks the same here and in the recent main branch. > Neither do I see recent changes that would cause this. > And the init just looks right to me, so I'm sure I'm overlooking something. > > So I wondered (if you consider this patch important for 19.11.x) if you could > have a look and consider backporting it so that it works well (or as an > alternative let me know that it won't be needed for 19.11.x) > > References/Logs: > 19.11.8 working: https://paste.ubuntu.com/p/vvm4qCcmJT/ > 19.11.9 fail RH7: https://paste.ubuntu.com/p/2wph5nfbCg/ > 19.11.9 fail SLE12SP4: https://paste.ubuntu.com/p/nGHhkPJbR3/ > 19.11.9 fail openSuse42.3: https://paste.ubuntu.com/p/43mMx3sY2n/ > 19.11.9 witzh the patch dropped works: > https://paste.ubuntu.com/p/JGZ44BhkC6/ > > > -- > Christian Ehrhardt > Staff Engineer, Ubuntu Server > Canonical Ltd ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-stable] " app/testpmd: fix NVGRE encap configuration" incompatible with 19.11.x 2021-06-03 14:54 ` Jiawei(Jonny) Wang @ 2021-06-04 5:07 ` Christian Ehrhardt 2021-06-04 13:44 ` Jiawei(Jonny) Wang 0 siblings, 1 reply; 6+ messages in thread From: Christian Ehrhardt @ 2021-06-04 5:07 UTC (permalink / raw) To: Jiawei(Jonny) Wang; +Cc: dpdk stable, Ori Kam On Thu, Jun 3, 2021 at 4:54 PM Jiawei(Jonny) Wang <jiaweiw@nvidia.com> wrote: > > Hello Christian, > > I applied this patch on the 19.11.9-rc1, but not found this issue during compilation. > > Could you send me your code branch then I can try on my side? Hi, For me it fails when applied on top of the 19.11.x WIP branch. You can use that from https://github.com/cpaelzer/dpdk-stable-queue/commits/19.11 But as I said it only fails in some releases/configurations. That is why I have linked the failing error logs. For example on my Ubuntu 20.04 it builds just fine. > Code base: > commit 15d8f64d18a891a70d664d18eb918eee0705993e > Author: Christian Ehrhardt <christian.ehrhardt@canonical.com> > Date: Wed Jun 2 16:33:26 2021 +0200 > > version: 19.11.9-rc1 > > Signed-off-by: Christian Ehrhardt <christian.ehrhardt@canonical.com> > > > #make O=x86_64-default-linux-gcc T=x86_64-native-linux-gcc config > #sed -ri 's,(RTE_MACHINE=).*,\1default,' x86_64-default-linux-gcc/.config > #sed -ri 's,(RTE_APP_TEST=).*,\1n,' x86_64-default-linux-gcc/.config > #sed -ri 's,(RTE_BUILD_SHARED_LIB=).*,\1y,' x86_64-default-linux-gcc/.config > #sed -ri 's,(RTE_NEXT_ABI=).*,\1n,' x86_64-default-linux-gcc/.config > #sed -ri 's,(LIBRTE_VHOST=).*,\1y,' x86_64-default-linux-gcc/.config > #sed -ri 's,(LIBRTE_VHOST=).*,\1y,' x86_64-default-linux-gcc/.config > #sed -ri 's,(RTE_KNI_KMOD=).*,\1y,' x86_64-default-linux-gcc/.config > #sed -ri 's,(RTE_EAL_IGB_UIO=).*,\1y,' x86_64-default-linux-gcc/.config > # > #make O=x86_64-default-linux-gcc -j3 EXTRA_CFLAGS=-Werror Yeah as I said it seems to only affect some releases/configs - I hope the logs help to find what is different between good/bad cases. > ...... > > INSTALL-APP dpdk-test-crypto-perf > INSTALL-MAP dpdk-test-crypto-perf.map > CC test_perf_queue.o > CC test_perf_atq.o > CC test_pipeline_common.o > CC test_pipeline_queue.o > CC test_pipeline_atq.o > LD dpdk-test-eventdev > INSTALL-APP dpdk-test-eventdev > INSTALL-MAP dpdk-test-eventdev.map > Build complete [x86_64-default-linuxapp-gcc] > > Thanks. > B.R. > > Jonny > > > > -----Original Message----- > > From: Christian Ehrhardt <christian.ehrhardt@canonical.com> > > Sent: Wednesday, June 2, 2021 9:25 PM > > To: Jiawei(Jonny) Wang <jiaweiw@nvidia.com> > > Cc: dpdk stable <stable@dpdk.org>; Ori Kam <orika@nvidia.com> > > Subject: " app/testpmd: fix NVGRE encap configuration" incompatible with > > 19.11.x > > > > Hi, > > this change was targeted for the stable releases > > > > commit 9b0da816bdec98c0d6c5d17ccdd337bdbafc3f75 > > Author: Jiawei Wang <jiaweiw@nvidia.com> > > Date: Tue Mar 16 06:18:27 2021 +0200 > > > > app/testpmd: fix NVGRE encap configuration > > > > But when applied it causes: > > > > [ 263s] /home/abuild/rpmbuild/BUILD/dpdk- > > 1622611368.078834f40/app/test-pmd/cmdline_flow.c: > > In function 'parse_vc_action_nvgre_encap': > > [ 263s] /home/abuild/rpmbuild/BUILD/dpdk- > > 1622611368.078834f40/app/test-pmd/cmdline_flow.c:4254:3: > > error: missing initializer for field 'protocol' of 'struct rte_flow_item_nvgre' [- > > Werror=missing-field-initializers] > > [ 263s] .item_nvgre.protocol = RTE_BE16(RTE_ETHER_TYPE_TEB), > > [ 263s] ^ > > [ 263s] In file included from > > /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/x86_64- > > default-linux-gcc/include/rte_eth_ctrl.h:11:0, > > [ 263s] from > > /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/x86_64- > > default-linux-gcc/include/rte_ethdev.h:978, > > [ 263s] from > > /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/app/test- > > pmd/cmdline_flow.c:18: > > [ 263s] /home/abuild/rpmbuild/BUILD/dpdk- > > 1622611368.078834f40/x86_64-default-linux-gcc/include/rte_flow.h:917:13: > > note: 'protocol' declared here > > [ 263s] rte_be16_t protocol; /**< Protocol type (0x6558). */ > > [ 263s] ^ > > > > This is failing not everywhere but in: > > - SLE_12_SP4/x86_64 > > - RHEL_7/x86_64 > > - openSUSE_Leap_42.3/x86_64 > > > > The type and the init code looks the same here and in the recent main branch. > > Neither do I see recent changes that would cause this. > > And the init just looks right to me, so I'm sure I'm overlooking something. > > > > So I wondered (if you consider this patch important for 19.11.x) if you could > > have a look and consider backporting it so that it works well (or as an > > alternative let me know that it won't be needed for 19.11.x) > > > > References/Logs: > > 19.11.8 working: https://paste.ubuntu.com/p/vvm4qCcmJT/ > > 19.11.9 fail RH7: https://paste.ubuntu.com/p/2wph5nfbCg/ > > 19.11.9 fail SLE12SP4: https://paste.ubuntu.com/p/nGHhkPJbR3/ > > 19.11.9 fail openSuse42.3: https://paste.ubuntu.com/p/43mMx3sY2n/ > > 19.11.9 witzh the patch dropped works: > > https://paste.ubuntu.com/p/JGZ44BhkC6/ > > > > > > -- > > Christian Ehrhardt > > Staff Engineer, Ubuntu Server > > Canonical Ltd -- Christian Ehrhardt Staff Engineer, Ubuntu Server Canonical Ltd ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-stable] " app/testpmd: fix NVGRE encap configuration" incompatible with 19.11.x 2021-06-04 5:07 ` Christian Ehrhardt @ 2021-06-04 13:44 ` Jiawei(Jonny) Wang 2021-06-08 4:50 ` Christian Ehrhardt 0 siblings, 1 reply; 6+ messages in thread From: Jiawei(Jonny) Wang @ 2021-06-04 13:44 UTC (permalink / raw) To: Christian Ehrhardt; +Cc: dpdk stable, Ori Kam Hello Christian, > -----Original Message----- > From: Christian Ehrhardt <christian.ehrhardt@canonical.com> > Sent: Friday, June 4, 2021 1:07 PM > To: Jiawei(Jonny) Wang <jiaweiw@nvidia.com> > Cc: dpdk stable <stable@dpdk.org>; Ori Kam <orika@nvidia.com> > Subject: Re: " app/testpmd: fix NVGRE encap configuration" incompatible > with 19.11.x > > On Thu, Jun 3, 2021 at 4:54 PM Jiawei(Jonny) Wang <jiaweiw@nvidia.com> > wrote: > > > > Hello Christian, > > > > I applied this patch on the 19.11.9-rc1, but not found this issue during > compilation. > > > > Could you send me your code branch then I can try on my side? > > Hi, > For me it fails when applied on top of the 19.11.x WIP branch. > You can use that from > https://github.com/cpaelzer/dpdk-stable-queue/commits/19.11 > > But as I said it only fails in some releases/configurations. > That is why I have linked the failing error logs. > For example on my Ubuntu 20.04 it builds just fine. > Yes, I compared and found it's related with older gcc version. I had sent the code patch. > > Code base: > > commit 15d8f64d18a891a70d664d18eb918eee0705993e > > Author: Christian Ehrhardt <christian.ehrhardt@canonical.com> > > Date: Wed Jun 2 16:33:26 2021 +0200 > > > > version: 19.11.9-rc1 > > > > Signed-off-by: Christian Ehrhardt > > <christian.ehrhardt@canonical.com> > > > > > > #make O=x86_64-default-linux-gcc T=x86_64-native-linux-gcc config #sed > > -ri 's,(RTE_MACHINE=).*,\1default,' x86_64-default-linux-gcc/.config > > #sed -ri 's,(RTE_APP_TEST=).*,\1n,' x86_64-default-linux-gcc/.config > > #sed -ri 's,(RTE_BUILD_SHARED_LIB=).*,\1y,' > > x86_64-default-linux-gcc/.config #sed -ri 's,(RTE_NEXT_ABI=).*,\1n,' > > x86_64-default-linux-gcc/.config #sed -ri 's,(LIBRTE_VHOST=).*,\1y,' > > x86_64-default-linux-gcc/.config #sed -ri 's,(LIBRTE_VHOST=).*,\1y,' > > x86_64-default-linux-gcc/.config #sed -ri 's,(RTE_KNI_KMOD=).*,\1y,' > > x86_64-default-linux-gcc/.config #sed -ri > > 's,(RTE_EAL_IGB_UIO=).*,\1y,' x86_64-default-linux-gcc/.config # #make > > O=x86_64-default-linux-gcc -j3 EXTRA_CFLAGS=-Werror > > Yeah as I said it seems to only affect some releases/configs - I hope the logs > help to find what is different between good/bad cases. > Yes, it's helpful, I had updated the code to make sure it works with older gcc version. Thanks. > > ...... > > > > INSTALL-APP dpdk-test-crypto-perf > > INSTALL-MAP dpdk-test-crypto-perf.map > > CC test_perf_queue.o > > CC test_perf_atq.o > > CC test_pipeline_common.o > > CC test_pipeline_queue.o > > CC test_pipeline_atq.o > > LD dpdk-test-eventdev > > INSTALL-APP dpdk-test-eventdev > > INSTALL-MAP dpdk-test-eventdev.map > > Build complete [x86_64-default-linuxapp-gcc] > > > > Thanks. > > B.R. > > > > Jonny > > > > > > > -----Original Message----- > > > From: Christian Ehrhardt <christian.ehrhardt@canonical.com> > > > Sent: Wednesday, June 2, 2021 9:25 PM > > > To: Jiawei(Jonny) Wang <jiaweiw@nvidia.com> > > > Cc: dpdk stable <stable@dpdk.org>; Ori Kam <orika@nvidia.com> > > > Subject: " app/testpmd: fix NVGRE encap configuration" incompatible > > > with 19.11.x > > > > > > Hi, > > > this change was targeted for the stable releases > > > > > > commit 9b0da816bdec98c0d6c5d17ccdd337bdbafc3f75 > > > Author: Jiawei Wang <jiaweiw@nvidia.com> > > > Date: Tue Mar 16 06:18:27 2021 +0200 > > > > > > app/testpmd: fix NVGRE encap configuration > > > > > > But when applied it causes: > > > > > > [ 263s] /home/abuild/rpmbuild/BUILD/dpdk- > > > 1622611368.078834f40/app/test-pmd/cmdline_flow.c: > > > In function 'parse_vc_action_nvgre_encap': > > > [ 263s] /home/abuild/rpmbuild/BUILD/dpdk- > > > 1622611368.078834f40/app/test-pmd/cmdline_flow.c:4254:3: > > > error: missing initializer for field 'protocol' of 'struct > > > rte_flow_item_nvgre' [- Werror=missing-field-initializers] > > > [ 263s] .item_nvgre.protocol = RTE_BE16(RTE_ETHER_TYPE_TEB), > > > [ 263s] ^ > > > [ 263s] In file included from > > > /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/x86_64- > > > default-linux-gcc/include/rte_eth_ctrl.h:11:0, > > > [ 263s] from > > > /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/x86_64- > > > default-linux-gcc/include/rte_ethdev.h:978, > > > [ 263s] from > > > /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/app/test- > > > pmd/cmdline_flow.c:18: > > > [ 263s] /home/abuild/rpmbuild/BUILD/dpdk- > > > 1622611368.078834f40/x86_64-default-linux- > gcc/include/rte_flow.h:917:13: > > > note: 'protocol' declared here > > > [ 263s] rte_be16_t protocol; /**< Protocol type (0x6558). */ > > > [ 263s] ^ > > > > > > This is failing not everywhere but in: > > > - SLE_12_SP4/x86_64 > > > - RHEL_7/x86_64 > > > - openSUSE_Leap_42.3/x86_64 > > > > > > The type and the init code looks the same here and in the recent main > branch. > > > Neither do I see recent changes that would cause this. > > > And the init just looks right to me, so I'm sure I'm overlooking something. > > > > > > So I wondered (if you consider this patch important for 19.11.x) if > > > you could have a look and consider backporting it so that it works > > > well (or as an alternative let me know that it won't be needed for > > > 19.11.x) > > > > > > References/Logs: > > > 19.11.8 working: https://paste.ubuntu.com/p/vvm4qCcmJT/ > > > 19.11.9 fail RH7: https://paste.ubuntu.com/p/2wph5nfbCg/ > > > 19.11.9 fail SLE12SP4: https://paste.ubuntu.com/p/nGHhkPJbR3/ > > > 19.11.9 fail openSuse42.3: https://paste.ubuntu.com/p/43mMx3sY2n/ > > > 19.11.9 witzh the patch dropped works: > > > https://paste.ubuntu.com/p/JGZ44BhkC6/ > > > > > > > > > -- > > > Christian Ehrhardt > > > Staff Engineer, Ubuntu Server > > > Canonical Ltd > > > > -- > Christian Ehrhardt > Staff Engineer, Ubuntu Server > Canonical Ltd ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-stable] " app/testpmd: fix NVGRE encap configuration" incompatible with 19.11.x 2021-06-04 13:44 ` Jiawei(Jonny) Wang @ 2021-06-08 4:50 ` Christian Ehrhardt 2021-06-08 5:59 ` Jiawei(Jonny) Wang 0 siblings, 1 reply; 6+ messages in thread From: Christian Ehrhardt @ 2021-06-08 4:50 UTC (permalink / raw) To: Jiawei(Jonny) Wang; +Cc: dpdk stable, Ori Kam On Fri, Jun 4, 2021 at 3:44 PM Jiawei(Jonny) Wang <jiaweiw@nvidia.com> wrote: > > Hello Christian, > > > -----Original Message----- > > From: Christian Ehrhardt <christian.ehrhardt@canonical.com> > > Sent: Friday, June 4, 2021 1:07 PM > > To: Jiawei(Jonny) Wang <jiaweiw@nvidia.com> > > Cc: dpdk stable <stable@dpdk.org>; Ori Kam <orika@nvidia.com> > > Subject: Re: " app/testpmd: fix NVGRE encap configuration" incompatible > > with 19.11.x > > > > On Thu, Jun 3, 2021 at 4:54 PM Jiawei(Jonny) Wang <jiaweiw@nvidia.com> > > wrote: > > > > > > Hello Christian, > > > > > > I applied this patch on the 19.11.9-rc1, but not found this issue during > > compilation. > > > > > > Could you send me your code branch then I can try on my side? > > > > Hi, > > For me it fails when applied on top of the 19.11.x WIP branch. > > You can use that from > > https://github.com/cpaelzer/dpdk-stable-queue/commits/19.11 > > > > But as I said it only fails in some releases/configurations. > > That is why I have linked the failing error logs. > > For example on my Ubuntu 20.04 it builds just fine. > > > > Yes, I compared and found it's related with older gcc version. > I had sent the code patch. I've got the patch it applies and seems to work - thank you! It isn't critical enough to respin the release candidates though. If we hit a functional issue in testing that will force us to respin it will be in 19.11.9-rc3, otherwise it will be the first patch that gets queued for 19.11.10 > > > Code base: > > > commit 15d8f64d18a891a70d664d18eb918eee0705993e > > > Author: Christian Ehrhardt <christian.ehrhardt@canonical.com> > > > Date: Wed Jun 2 16:33:26 2021 +0200 > > > > > > version: 19.11.9-rc1 > > > > > > Signed-off-by: Christian Ehrhardt > > > <christian.ehrhardt@canonical.com> > > > > > > > > > #make O=x86_64-default-linux-gcc T=x86_64-native-linux-gcc config #sed > > > -ri 's,(RTE_MACHINE=).*,\1default,' x86_64-default-linux-gcc/.config > > > #sed -ri 's,(RTE_APP_TEST=).*,\1n,' x86_64-default-linux-gcc/.config > > > #sed -ri 's,(RTE_BUILD_SHARED_LIB=).*,\1y,' > > > x86_64-default-linux-gcc/.config #sed -ri 's,(RTE_NEXT_ABI=).*,\1n,' > > > x86_64-default-linux-gcc/.config #sed -ri 's,(LIBRTE_VHOST=).*,\1y,' > > > x86_64-default-linux-gcc/.config #sed -ri 's,(LIBRTE_VHOST=).*,\1y,' > > > x86_64-default-linux-gcc/.config #sed -ri 's,(RTE_KNI_KMOD=).*,\1y,' > > > x86_64-default-linux-gcc/.config #sed -ri > > > 's,(RTE_EAL_IGB_UIO=).*,\1y,' x86_64-default-linux-gcc/.config # #make > > > O=x86_64-default-linux-gcc -j3 EXTRA_CFLAGS=-Werror > > > > Yeah as I said it seems to only affect some releases/configs - I hope the logs > > help to find what is different between good/bad cases. > > > Yes, it's helpful, I had updated the code to make sure it works with older gcc version. > Thanks. > > > > ...... > > > > > > INSTALL-APP dpdk-test-crypto-perf > > > INSTALL-MAP dpdk-test-crypto-perf.map > > > CC test_perf_queue.o > > > CC test_perf_atq.o > > > CC test_pipeline_common.o > > > CC test_pipeline_queue.o > > > CC test_pipeline_atq.o > > > LD dpdk-test-eventdev > > > INSTALL-APP dpdk-test-eventdev > > > INSTALL-MAP dpdk-test-eventdev.map > > > Build complete [x86_64-default-linuxapp-gcc] > > > > > > Thanks. > > > B.R. > > > > > > Jonny > > > > > > > > > > -----Original Message----- > > > > From: Christian Ehrhardt <christian.ehrhardt@canonical.com> > > > > Sent: Wednesday, June 2, 2021 9:25 PM > > > > To: Jiawei(Jonny) Wang <jiaweiw@nvidia.com> > > > > Cc: dpdk stable <stable@dpdk.org>; Ori Kam <orika@nvidia.com> > > > > Subject: " app/testpmd: fix NVGRE encap configuration" incompatible > > > > with 19.11.x > > > > > > > > Hi, > > > > this change was targeted for the stable releases > > > > > > > > commit 9b0da816bdec98c0d6c5d17ccdd337bdbafc3f75 > > > > Author: Jiawei Wang <jiaweiw@nvidia.com> > > > > Date: Tue Mar 16 06:18:27 2021 +0200 > > > > > > > > app/testpmd: fix NVGRE encap configuration > > > > > > > > But when applied it causes: > > > > > > > > [ 263s] /home/abuild/rpmbuild/BUILD/dpdk- > > > > 1622611368.078834f40/app/test-pmd/cmdline_flow.c: > > > > In function 'parse_vc_action_nvgre_encap': > > > > [ 263s] /home/abuild/rpmbuild/BUILD/dpdk- > > > > 1622611368.078834f40/app/test-pmd/cmdline_flow.c:4254:3: > > > > error: missing initializer for field 'protocol' of 'struct > > > > rte_flow_item_nvgre' [- Werror=missing-field-initializers] > > > > [ 263s] .item_nvgre.protocol = RTE_BE16(RTE_ETHER_TYPE_TEB), > > > > [ 263s] ^ > > > > [ 263s] In file included from > > > > /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/x86_64- > > > > default-linux-gcc/include/rte_eth_ctrl.h:11:0, > > > > [ 263s] from > > > > /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/x86_64- > > > > default-linux-gcc/include/rte_ethdev.h:978, > > > > [ 263s] from > > > > /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/app/test- > > > > pmd/cmdline_flow.c:18: > > > > [ 263s] /home/abuild/rpmbuild/BUILD/dpdk- > > > > 1622611368.078834f40/x86_64-default-linux- > > gcc/include/rte_flow.h:917:13: > > > > note: 'protocol' declared here > > > > [ 263s] rte_be16_t protocol; /**< Protocol type (0x6558). */ > > > > [ 263s] ^ > > > > > > > > This is failing not everywhere but in: > > > > - SLE_12_SP4/x86_64 > > > > - RHEL_7/x86_64 > > > > - openSUSE_Leap_42.3/x86_64 > > > > > > > > The type and the init code looks the same here and in the recent main > > branch. > > > > Neither do I see recent changes that would cause this. > > > > And the init just looks right to me, so I'm sure I'm overlooking something. > > > > > > > > So I wondered (if you consider this patch important for 19.11.x) if > > > > you could have a look and consider backporting it so that it works > > > > well (or as an alternative let me know that it won't be needed for > > > > 19.11.x) > > > > > > > > References/Logs: > > > > 19.11.8 working: https://paste.ubuntu.com/p/vvm4qCcmJT/ > > > > 19.11.9 fail RH7: https://paste.ubuntu.com/p/2wph5nfbCg/ > > > > 19.11.9 fail SLE12SP4: https://paste.ubuntu.com/p/nGHhkPJbR3/ > > > > 19.11.9 fail openSuse42.3: https://paste.ubuntu.com/p/43mMx3sY2n/ > > > > 19.11.9 witzh the patch dropped works: > > > > https://paste.ubuntu.com/p/JGZ44BhkC6/ > > > > > > > > > > > > -- > > > > Christian Ehrhardt > > > > Staff Engineer, Ubuntu Server > > > > Canonical Ltd > > > > > > > > -- > > Christian Ehrhardt > > Staff Engineer, Ubuntu Server > > Canonical Ltd -- Christian Ehrhardt Staff Engineer, Ubuntu Server Canonical Ltd ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-stable] " app/testpmd: fix NVGRE encap configuration" incompatible with 19.11.x 2021-06-08 4:50 ` Christian Ehrhardt @ 2021-06-08 5:59 ` Jiawei(Jonny) Wang 0 siblings, 0 replies; 6+ messages in thread From: Jiawei(Jonny) Wang @ 2021-06-08 5:59 UTC (permalink / raw) To: Christian Ehrhardt; +Cc: dpdk stable, Ori Kam > -----Original Message----- > From: Christian Ehrhardt <christian.ehrhardt@canonical.com> > Sent: Tuesday, June 8, 2021 12:51 PM > To: Jiawei(Jonny) Wang <jiaweiw@nvidia.com> > Cc: dpdk stable <stable@dpdk.org>; Ori Kam <orika@nvidia.com> > Subject: Re: " app/testpmd: fix NVGRE encap configuration" incompatible > with 19.11.x > > On Fri, Jun 4, 2021 at 3:44 PM Jiawei(Jonny) Wang <jiaweiw@nvidia.com> > wrote: > > > > Hello Christian, > > > > > -----Original Message----- > > > From: Christian Ehrhardt <christian.ehrhardt@canonical.com> > > > Sent: Friday, June 4, 2021 1:07 PM > > > To: Jiawei(Jonny) Wang <jiaweiw@nvidia.com> > > > Cc: dpdk stable <stable@dpdk.org>; Ori Kam <orika@nvidia.com> > > > Subject: Re: " app/testpmd: fix NVGRE encap configuration" > > > incompatible with 19.11.x > > > > > > On Thu, Jun 3, 2021 at 4:54 PM Jiawei(Jonny) Wang > > > <jiaweiw@nvidia.com> > > > wrote: > > > > > > > > Hello Christian, > > > > > > > > I applied this patch on the 19.11.9-rc1, but not found this issue > > > > during > > > compilation. > > > > > > > > Could you send me your code branch then I can try on my side? > > > > > > Hi, > > > For me it fails when applied on top of the 19.11.x WIP branch. > > > You can use that from > > > https://github.com/cpaelzer/dpdk-stable-queue/commits/19.11 > > > > > > But as I said it only fails in some releases/configurations. > > > That is why I have linked the failing error logs. > > > For example on my Ubuntu 20.04 it builds just fine. > > > > > > > Yes, I compared and found it's related with older gcc version. > > I had sent the code patch. > > I've got the patch it applies and seems to work - thank you! > It isn't critical enough to respin the release candidates though. > If we hit a functional issue in testing that will force us to respin it will be in > 19.11.9-rc3, otherwise it will be the first patch that gets queued for 19.11.10 > Thanks Christian! > > > > Code base: > > > > commit 15d8f64d18a891a70d664d18eb918eee0705993e > > > > Author: Christian Ehrhardt <christian.ehrhardt@canonical.com> > > > > Date: Wed Jun 2 16:33:26 2021 +0200 > > > > > > > > version: 19.11.9-rc1 > > > > > > > > Signed-off-by: Christian Ehrhardt > > > > <christian.ehrhardt@canonical.com> > > > > > > > > > > > > #make O=x86_64-default-linux-gcc T=x86_64-native-linux-gcc config > > > > #sed -ri 's,(RTE_MACHINE=).*,\1default,' > > > > x86_64-default-linux-gcc/.config #sed -ri > > > > 's,(RTE_APP_TEST=).*,\1n,' x86_64-default-linux-gcc/.config #sed -ri > 's,(RTE_BUILD_SHARED_LIB=).*,\1y,' > > > > x86_64-default-linux-gcc/.config #sed -ri 's,(RTE_NEXT_ABI=).*,\1n,' > > > > x86_64-default-linux-gcc/.config #sed -ri 's,(LIBRTE_VHOST=).*,\1y,' > > > > x86_64-default-linux-gcc/.config #sed -ri 's,(LIBRTE_VHOST=).*,\1y,' > > > > x86_64-default-linux-gcc/.config #sed -ri 's,(RTE_KNI_KMOD=).*,\1y,' > > > > x86_64-default-linux-gcc/.config #sed -ri > > > > 's,(RTE_EAL_IGB_UIO=).*,\1y,' x86_64-default-linux-gcc/.config # > > > > #make O=x86_64-default-linux-gcc -j3 EXTRA_CFLAGS=-Werror > > > > > > Yeah as I said it seems to only affect some releases/configs - I > > > hope the logs help to find what is different between good/bad cases. > > > > > Yes, it's helpful, I had updated the code to make sure it works with older > gcc version. > > Thanks. > > > > > > ...... > > > > > > > > INSTALL-APP dpdk-test-crypto-perf > > > > INSTALL-MAP dpdk-test-crypto-perf.map > > > > CC test_perf_queue.o > > > > CC test_perf_atq.o > > > > CC test_pipeline_common.o > > > > CC test_pipeline_queue.o > > > > CC test_pipeline_atq.o > > > > LD dpdk-test-eventdev > > > > INSTALL-APP dpdk-test-eventdev > > > > INSTALL-MAP dpdk-test-eventdev.map Build complete > > > > [x86_64-default-linuxapp-gcc] > > > > > > > > Thanks. > > > > B.R. > > > > > > > > Jonny > > > > > > > > > > > > > -----Original Message----- > > > > > From: Christian Ehrhardt <christian.ehrhardt@canonical.com> > > > > > Sent: Wednesday, June 2, 2021 9:25 PM > > > > > To: Jiawei(Jonny) Wang <jiaweiw@nvidia.com> > > > > > Cc: dpdk stable <stable@dpdk.org>; Ori Kam <orika@nvidia.com> > > > > > Subject: " app/testpmd: fix NVGRE encap configuration" > > > > > incompatible with 19.11.x > > > > > > > > > > Hi, > > > > > this change was targeted for the stable releases > > > > > > > > > > commit 9b0da816bdec98c0d6c5d17ccdd337bdbafc3f75 > > > > > Author: Jiawei Wang <jiaweiw@nvidia.com> > > > > > Date: Tue Mar 16 06:18:27 2021 +0200 > > > > > > > > > > app/testpmd: fix NVGRE encap configuration > > > > > > > > > > But when applied it causes: > > > > > > > > > > [ 263s] /home/abuild/rpmbuild/BUILD/dpdk- > > > > > 1622611368.078834f40/app/test-pmd/cmdline_flow.c: > > > > > In function 'parse_vc_action_nvgre_encap': > > > > > [ 263s] /home/abuild/rpmbuild/BUILD/dpdk- > > > > > 1622611368.078834f40/app/test-pmd/cmdline_flow.c:4254:3: > > > > > error: missing initializer for field 'protocol' of 'struct > > > > > rte_flow_item_nvgre' [- Werror=missing-field-initializers] > > > > > [ 263s] .item_nvgre.protocol = RTE_BE16(RTE_ETHER_TYPE_TEB), > > > > > [ 263s] ^ > > > > > [ 263s] In file included from > > > > > /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/x86_64- > > > > > default-linux-gcc/include/rte_eth_ctrl.h:11:0, > > > > > [ 263s] from > > > > > /home/abuild/rpmbuild/BUILD/dpdk-1622611368.078834f40/x86_64- > > > > > default-linux-gcc/include/rte_ethdev.h:978, > > > > > [ 263s] from > > > > > /home/abuild/rpmbuild/BUILD/dpdk- > 1622611368.078834f40/app/test- > > > > > pmd/cmdline_flow.c:18: > > > > > [ 263s] /home/abuild/rpmbuild/BUILD/dpdk- > > > > > 1622611368.078834f40/x86_64-default-linux- > > > gcc/include/rte_flow.h:917:13: > > > > > note: 'protocol' declared here > > > > > [ 263s] rte_be16_t protocol; /**< Protocol type (0x6558). */ > > > > > [ 263s] ^ > > > > > > > > > > This is failing not everywhere but in: > > > > > - SLE_12_SP4/x86_64 > > > > > - RHEL_7/x86_64 > > > > > - openSUSE_Leap_42.3/x86_64 > > > > > > > > > > The type and the init code looks the same here and in the recent > > > > > main > > > branch. > > > > > Neither do I see recent changes that would cause this. > > > > > And the init just looks right to me, so I'm sure I'm overlooking > something. > > > > > > > > > > So I wondered (if you consider this patch important for 19.11.x) > > > > > if you could have a look and consider backporting it so that it > > > > > works well (or as an alternative let me know that it won't be > > > > > needed for > > > > > 19.11.x) > > > > > > > > > > References/Logs: > > > > > 19.11.8 working: https://paste.ubuntu.com/p/vvm4qCcmJT/ > > > > > 19.11.9 fail RH7: https://paste.ubuntu.com/p/2wph5nfbCg/ > > > > > 19.11.9 fail SLE12SP4: https://paste.ubuntu.com/p/nGHhkPJbR3/ > > > > > 19.11.9 fail openSuse42.3: > > > > > https://paste.ubuntu.com/p/43mMx3sY2n/ > > > > > 19.11.9 witzh the patch dropped works: > > > > > https://paste.ubuntu.com/p/JGZ44BhkC6/ > > > > > > > > > > > > > > > -- > > > > > Christian Ehrhardt > > > > > Staff Engineer, Ubuntu Server > > > > > Canonical Ltd > > > > > > > > > > > > -- > > > Christian Ehrhardt > > > Staff Engineer, Ubuntu Server > > > Canonical Ltd > > > > -- > Christian Ehrhardt > Staff Engineer, Ubuntu Server > Canonical Ltd ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-06-08 5:59 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-06-02 13:25 [dpdk-stable] " app/testpmd: fix NVGRE encap configuration" incompatible with 19.11.x Christian Ehrhardt 2021-06-03 14:54 ` Jiawei(Jonny) Wang 2021-06-04 5:07 ` Christian Ehrhardt 2021-06-04 13:44 ` Jiawei(Jonny) Wang 2021-06-08 4:50 ` Christian Ehrhardt 2021-06-08 5:59 ` Jiawei(Jonny) Wang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).