Re: [ovs-dev] [PATCH v10 06/10] odp-execute: Add ISA implementation of actions.

2022-07-14 Thread Eelco Chaudron
On 13 Jul 2022, at 20:28, Harry van Haaren wrote: > From: Emma Finn > > This commit adds the AVX512 implementation of the action functionality. > > Usage: > $ ovs-appctl odp-execute/action-impl-set avx512 > > Signed-off-by: Emma Finn > Signed-off-by: Harry van Haaren > > --- > diff --git

Re: [ovs-dev] [PATCH v10 06/10] odp-execute: Add ISA implementation of actions.

2022-07-14 Thread Pai G, Sunil
> Like patch 1 and 4, GHA couldn't complete its run for this patch because > of infra reasons, so I manually triggered one on my forked repo to be more > confident, and it succeeds: > https://github.com/Sunil-Pai-G/ovs/actions/runs/2668424540 Seems GHA passed on the first run on patchwork, I

Re: [ovs-dev] [PATCH v10 06/10] odp-execute: Add ISA implementation of actions.

2022-07-14 Thread Pai G, Sunil
Hi Harry/Emma, > -Original Message- > From: Van Haaren, Harry > Sent: Wednesday, July 13, 2022 11:58 PM > To: d...@openvswitch.org > Cc: i.maxim...@ovn.org; echau...@redhat.com; Amber, Kumar > ; Pai G, Sunil ; Finn, Emma > ; Stokes, Ian ; Van Haaren, > Harry > Subject: [PATCH v10

Re: [ovs-dev] [PATCH v10 06/10] odp-execute: Add ISA implementation of actions.

2022-07-14 Thread Amber, Kumar
Hey all, Have tested the all the patches in the series. Can be fixed while applying: checkpatch: WARNING: Unexpected sign-offs from developers who are not authors or co-authors or committers: Harry van Haaren Lines checked: 255, Warnings: 1, Errors: 0 Tested-by: Kumar Amber BR Amber

Re: [ovs-dev] [PATCH v10 06/10] odp-execute: Add ISA implementation of actions.

2022-07-13 Thread 0-day Robot
Bleep bloop. Greetings Harry van Haaren, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Unexpected sign-offs from developers who are not authors or co-authors or

[ovs-dev] [PATCH v10 06/10] odp-execute: Add ISA implementation of actions.

2022-07-13 Thread Harry van Haaren
From: Emma Finn This commit adds the AVX512 implementation of the action functionality. Usage: $ ovs-appctl odp-execute/action-impl-set avx512 Signed-off-by: Emma Finn Signed-off-by: Harry van Haaren --- v10: - Move location and rework documentation (Ilya) - Improve ISA checks to fix CI