On Mon, Oct 16, 2023 at 10:20 PM Mark Michelson <mmich...@redhat.com> wrote:
>
> Hi Ales,
>
> The patch makes sense to me. Is there something that guarantees that
> clang >= 16 is installed when running CI?

Yeah, clang 16 is default on Fedora 38 which we use for ARM64 runs.

>
> Thanks,
> Mark Michelson
>
> On 10/16/23 04:08, Ales Musil wrote:
> > The clang from version 16 and further fixes
> > the issue which was causing the slowness.
> > Remove the workaround which allows
> > the leak sanitizers to run on ARM64 as well.
> >
> > Signed-off-by: Ales Musil <amu...@redhat.com>
> > ---
> >   .ci/ci.sh | 5 -----
> >   1 file changed, 5 deletions(-)
> >
> > diff --git a/.ci/ci.sh b/.ci/ci.sh
> > index 6bb211f2c..282d30b84 100755
> > --- a/.ci/ci.sh
> > +++ b/.ci/ci.sh
> > @@ -148,11 +148,6 @@ while true; do
> >       shift
> >   done
> >
> > -# Workaround for https://bugzilla.redhat.com/2153359
> > -if [ "$ARCH" = "aarch64" ]; then
> > -    ASAN_OPTIONS="detect_leaks=0"
> > -fi
> > -
> >   if [ -z "$DPDK" ]; then
> >      mkdir -p "$DPDK_PATH"
> >   fi
>

Thanks,
Ales

-- 

Ales Musil

Senior Software Engineer - OVN Core

Red Hat EMEA

amu...@redhat.com

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to