On Tue, Oct 17, 2023 at 7:08 AM Ales Musil <amu...@redhat.com> wrote:

> 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.
>

On the other hand, thinking about it I'll post v2 which will check the
clang version, in case there is anyone using it with different combinations.


>
> >
> > 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
>

Thanks,
Ales
-- 

Ales Musil

Senior Software Engineer - OVN Core

Red Hat EMEA <https://www.redhat.com>

amu...@redhat.com
<https://red.ht/sig>
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to