failure on arm64
Successful travis build jobs report:
https://travis-ci.org/yzyuestc/ovs/builds/621037339
Reviewed-by: Yanqin Wei
Reviewed-by: Jieqiang Wang
Reviewed-by: Gavin Hu
Signed-off-by: Lance Yang
---
.travis.yml| 18 ++
.travis/linux-build.sh | 16
are under review, some codes or conflicts may need to be merged after
review.
Changes since v1:
- move the initialization code for pmd_perf_stats to an appropriate location
- remove cache split scripts
- merge two patches into a single one
- modify commit message to describe the issue more clearly
Lance
Reviewed-by: Ruifeng Wang
Signed-off-by: Lance Yang
---
.travis.yml | 2 --
.travis/linux-prepare.sh | 3 ++-
2 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index 482efd2..2dc4d43 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -14,7 +14,6 @@ addons
to
initialize the variable before using.
Reviewed-by: Yanqin Wei
Reviewed-by: Malvika Gupta
Signed-off-by: Lance Yang
---
lib/dpif-netdev-perf.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/dpif-netdev-perf.c b/lib/dpif-netdev-perf.c
index baf90b0..58728ab 100644
--- a/lib/dpif
are under review, some codes or conflicts may need to be merged after
review.
Lance Yang (4):
[ovs-dev] [PATCH 1/4] dpif-netdev: Fix the pmd_perf_stats uninitalized
issue
[ovs-dev] [PATCH 2/4] travis: Move x86-only addon packages to
linux-prepare.sh
[ovs-dev] [PATCH 3/4] travis: split
Reviewed-by: Ruifeng Wang
Signed-off-by: Lance Yang
---
.travis.yml | 2 --
.travis/linux-prepare.sh | 12
2 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index 482efd2..2dc4d43 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -14,7
To compile OvS with DPDK in some Travis jobs, it is necessary to set the
build target and split the dpdk cache directory according to different CPU
architectures.
Reviewed-by: Yanqin Wei
Reviewed-by: Malvika Gupta
Signed-off-by: Lance Yang
---
.travis/linux-build.sh | 32
When compiling Open vSwitch for aarch64, the compiler will warn about
an uninitailized variable in lib/dpif-netdev-perf.c. It is necessary to
initialize it to avoid build failure.
Reviewed-by: Yanqin Wei
Signed-off-by: Lance Yang
---
lib/dpif-netdev-perf.c | 1 +
1 file changed, 1 insertion
:
https://travis-ci.org/yzyuestc/ovs/builds/614299933
Reviewed-by: Yanqin Wei
Reviewed-by: Jieqiang Wang
Reviewed-by: Gavin Hu
Signed-off-by: Lance Yang
---
.travis.yml| 18 ++
.travis/linux-build.sh | 6 +-
2 files changed, 23 insertions(+), 1 deletion(-)
diff
> -Original Message-
> From: Ilya Maximets
> Sent: Saturday, December 7, 2019 12:39 AM
> To: Lance Yang (Arm Technology China) ; ovs-
> d...@openvswitch.org
> Cc: i.maxim...@ovn.org; b...@ovn.org; Yanqin Wei (Arm Technology China)
> ; dwil...@us.ibm.com; Gavin Hu
Hi William,
Thanks for your time to review and test the patches.
Best Regards,
Lance
> -Original Message-
> From: William Tu
> Sent: Wednesday, April 8, 2020 5:33 AM
> To: Lance Yang
> Cc: ovs-dev@openvswitch.org; nd ; i.maxim...@ovn.org
> Subject: Re: [ovs-dev]
In order to increase CI coverage for arm, the patches enabled clang and
testsuite jobs (skipped some UT cases on arm temporarily) by modifying
the existing arm CI jobs. This does not add any extra jobs.
Lance Yang (1):
[ovs-dev] [PATCH 3/3] Travis: Enable clang compiler and unit test
From: Malvika Gupta
From: Malvika Gupta
This patch adds a condition to check if the CPU architecture is aarch64. If the
condition evaluates to true, $IS_ARM64 variable is set to 'yes'. For all other
architectures, this variable is set to 'no'.
Reviewed-by: Yanqin Wei
Signed-off-by: Malvika
From: Lance Yang
Enable testsuite and clang compiler for arm CI. In order not to increase
the CI jobs, selectively enable them in the existing jobs instead of
adding extra jobs.
Successful travis job build report:
https://travis-ci.org/github/yzyuestc/ovs/builds/667539360
Reviewed-by: Yanqin
to leader - Python3 (leader
only)
2105: ovsdb-idl.at:1816 Check Python IDL reconnects to leader - Python3
(leader only)
Suggested-by: Yanqin Wei
Suggested-by: Lance Yang
Signed-off-by: Malvika Gupta
---
tests/bfd.at | 1 +
tests/ovsdb-idl.at | 1 +
2 files changed, 2 insertions(+)
diff
Reviewed-by: Yanqin Wei
Reviewed-by: Ruifeng Wang
Reviewed-by: JingZhao Ni
Reviewed-by: Gavin Hu
Signed-off-by: Lance Yang
---
v3:
- Remove some kernel jobs: 4.18, 4.17, 4.16, 4.15, 4.14, and 4.3.
- Remove one OvS-DPDK shared library job.
---
.travis.yml| 15
Reviewed-by: Yanqin Wei
Reviewed-by: Ruifeng Wang
Reviewed-by: JingZhao Ni
Reviewed-by: Gavin Hu
Signed-off-by: Lance Yang
---
v3:
- Remove some kernel jobs: 4.18, 4.17, 4.16, 4.15, 4.14, and 4.3.
- Remove one OvS-DPDK shared library job.
---
.travis.yml| 15
Ni
Reviewed-by: Gavin Hu
Signed-off-by: Lance Yang
---
v4:
- Move TARGET variable into install_dpdk function.
- Avoid to add SPARSE_FLAGS to CFLAGS_FOR_OVS when sparse is not
enabled.
---
v3:
- Remove some kernel jobs: 4.18, 4.17, 4.16, 4.15, 4.14, and 4.3.
- Remove one OvS-DPDK
Hi Ilya,
Thanks for your comments. We applied your suggestions and sent patch v4. Please
see the new patch.
Thanks for your time.
Best regards,
Lance Yang
> -Original Message-
> From: Ilya Maximets
> Sent: Tuesday, March 24, 2020 1:25 AM
> To: Lance Yang ; ovs-dev@ope
when they validate their patches.
Reviewed-by: Yanqin Wei
Signed-off-by: Lance Yang
---
.travis/linux-build.sh | 12 ++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/.travis/linux-build.sh b/.travis/linux-build.sh
index 02615a8ec..ec92c92d0 100755
--- a/.travis/linux
y hints on what is wrong here?
Thank you so much.
Best regards,
Lance Yang
IMPORTANT NOTICE: The contents of this email and any attachments are
confidential and may also be privileged. If you are not the intended recipient,
please notify the sender immediately and do not disclose the conte
Hi,
I encountered a unit test failure when I ran the Open vSwitch testsuite on
Travis CI aarch64 lxd container.
The environment can be found at line 7 : build system information section in
the report on https://travis-ci.org/yzyuestc/ovs/jobs/614941322 . The unit test
case name is "bfd decay" ,
or 1
make[1]: Leaving directory '/home/lance/ovs'
Makefile:5278: recipe for target 'check' failed
make: *** [check] Error 2
I attached the tests/testsuite.dir/testsuite.log to this email. I also
attached the logs for ovsdb. Any hints on what is wrong here?
Thank you so much.
Best regards,
Lance Yang
> -Original Message-
> From: Ilya Maximets
> Sent: Saturday, December 7, 2019 12:39 AM
> To: Lance Yang (Arm Technology China) ;
> ovs-dev@openvswitch.org
> Cc: i.maxim...@ovn.org; b...@ovn.org; Yanqin Wei (Arm Technology China)
> ; dwil...@us.ibm.com; Gavin Hu
> -Original Message-
> From: Ilya Maximets
> Sent: Saturday, December 14, 2019 1:55 AM
> To: dwilder ; Lance Yang (Arm Technology China)
>
> Cc: ovs-dev@openvswitch.org; i.maxim...@ovn.org; b...@ovn.org; Yanqin Wei (Arm
> Technology China) ; Gavin Hu (Arm Technolo
> -Original Message-
> From: Ilya Maximets
> Sent: Friday, November 22, 2019 6:36 PM
> To: Lance Yang (Arm Technology China) ; Ilya Maximets
> ; ovs-dev@openvswitch.org
> Cc: Jieqiang Wang (Arm Technology China) ; Gavin Hu
> (Arm
> Technology China) ; Jingzhao
> -Original Message-
> From: Ilya Maximets
> Sent: Friday, November 22, 2019 6:36 PM
> To: Lance Yang (Arm Technology China) ; Ilya Maximets
> ; ovs-dev@openvswitch.org
> Cc: Jieqiang Wang (Arm Technology China) ; Gavin Hu
> (Arm
> Technology China) ; Jingzhao
> -Original Message-
> From: Ilya Maximets
> Sent: Tuesday, November 26, 2019 8:48 PM
> To: Ilya Maximets ; Lance Yang (Arm Technology China)
> ; ovs-dev@openvswitch.org
> Cc: Jieqiang Wang (Arm Technology China) ; Gavin Hu
> (Arm
> Technology China) ; Jin
> -Original Message-
> From: Ilya Maximets
> Sent: Friday, November 29, 2019 3:06 AM
> To: Lance Yang (Arm Technology China) ; Ilya Maximets
> ; ovs-dev@openvswitch.org
> Cc: Jieqiang Wang (Arm Technology China) ; Gavin Hu
> (Arm
> Technology China) ; Jingzhao
Hi Ilya,
Thanks for your comments. Please take time to see the inline comments.
Best regards,
Lance
> -Original Message-
> From: Ilya Maximets
> Sent: Friday, November 22, 2019 3:23 AM
> To: Lance Yang (Arm Technology China) ; ovs-
> d...@openvswitch.org
> Cc:
Hi Ilya,
Thanks for your comments.
> -Original Message-
> From: Ilya Maximets
> Sent: Friday, November 22, 2019 3:39 AM
> To: Lance Yang (Arm Technology China) ; ovs-
> d...@openvswitch.org
> Cc: Jieqiang Wang (Arm Technology China) ; Gavin Hu
> (Arm
> Technol
Hi Ilya,
Thanks for your reply. Please see the inline reply.
Best regards,
Lance Yang
> -Original Message-
> From: Ilya Maximets
> Sent: Friday, November 22, 2019 3:47 AM
> To: Lance Yang (Arm Technology China) ; ovs-
> d...@openvswitch.org
> Cc: Jieqiang Wang (Ar
Hi,
For pip missing issue, we have a workaround. In .travis/linux-prepare.sh. You
can install pip or something else with package management commands.
For arm64 on travis, we check the environment variables in the container and
found a variable called "TRAVIS_ARCH". Thus we can use this
33 matches
Mail list logo