diff mbox series

[ovs-dev,v2,2/2] tests: Skip memory error in DPDK tests.

Message ID 20240627083815.720370-2-amusil@redhat.com
State Accepted
Delegated to: Dumitru Ceara
Headers show
Series [ovs-dev,v2,1/2] ci: Save some DPDK compilation time. | expand

Checks

Context Check Description
ovsrobot/apply-robot success apply and check: success
ovsrobot/github-robot-_Build_and_Test success github build: passed
ovsrobot/github-robot-_ovn-kubernetes success github build: passed

Commit Message

Ales Musil June 27, 2024, 8:38 a.m. UTC
Add skip for memory error produced by DPDK on ARM with 2MB hugepages
which isn't harmful [0].

[0] http://mails.dpdk.org/archives/dev/2024-June/296764.html

Suggested-by: David Marchand <david.marchand@redhat.com>
Signed-off-by: Ales Musil <amusil@redhat.com>
---
 tests/system-dpdk-macros.at | 1 +
 1 file changed, 1 insertion(+)

Comments

David Marchand June 28, 2024, 9:16 a.m. UTC | #1
On Thu, Jun 27, 2024 at 10:38 AM Ales Musil <amusil@redhat.com> wrote:
>
> Add skip for memory error produced by DPDK on ARM with 2MB hugepages
> which isn't harmful [0].
>
> [0] http://mails.dpdk.org/archives/dev/2024-June/296764.html
>
> Suggested-by: David Marchand <david.marchand@redhat.com>
> Signed-off-by: Ales Musil <amusil@redhat.com>

I would mention ARM in the commit title (maybe it can be tweaked when
applying), but otherwise the patch lgtm.
Reviewed-by: David Marchand <david.marchand@redhat.com>
Dumitru Ceara June 28, 2024, 10:20 a.m. UTC | #2
On 6/28/24 11:16, David Marchand wrote:
> On Thu, Jun 27, 2024 at 10:38 AM Ales Musil <amusil@redhat.com> wrote:
>>
>> Add skip for memory error produced by DPDK on ARM with 2MB hugepages
>> which isn't harmful [0].
>>
>> [0] http://mails.dpdk.org/archives/dev/2024-June/296764.html
>>
>> Suggested-by: David Marchand <david.marchand@redhat.com>
>> Signed-off-by: Ales Musil <amusil@redhat.com>
> 
> I would mention ARM in the commit title (maybe it can be tweaked when
> applying), but otherwise the patch lgtm.

I changed it to:

tests: Skip memory error triggered on ARM in DPDK tests.

> Reviewed-by: David Marchand <david.marchand@redhat.com>
> 

Thanks, Ales and David!  Applied to main.

Regards,
Dumitru
diff mbox series

Patch

diff --git a/tests/system-dpdk-macros.at b/tests/system-dpdk-macros.at
index 440908af7..0d0a19130 100644
--- a/tests/system-dpdk-macros.at
+++ b/tests/system-dpdk-macros.at
@@ -68,6 +68,7 @@  m4_define([OVS_TRAFFIC_VSWITCHD_START],
 m4_define([OVS_TRAFFIC_VSWITCHD_STOP],
   [OVS_VSWITCHD_STOP([dnl
 $1";/EAL: No \(available\|free\) .*hugepages reported/d
+/EAL: eal_memalloc_alloc_seg_bulk(): couldn't find suitable memseg_list/d
 /TELEMETRY: No legacy callbacks, legacy socket not created/d
 /dpif(revalidator.*)|WARN|netdev@ovs-netdev: failed to.*proto=2.*/d
 /dpif_netdev(revalidator.*)|ERR|internal error parsing flow key.*proto=2.*/d