Openwall GNU/*/Linux - a small security-enhanced Linux distro for servers
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Fri, 20 Oct 2017 17:26:14 -0400
From: Rich Felker <dalias@...c.org>
To: musl@...ts.openwall.com
Subject: Re: Build failure on ARM <5

On Fri, Oct 20, 2017 at 09:36:45PM +0200, Szabolcs Nagy wrote:
> * Reiner Herrmann <reiner@...ner-h.de> [2017-10-20 21:04:00 +0200]:
> > Hi,
> > 
> > musl 1.1.17 fails to build [1] on arm versions <5 because
> > of an unsupported clz instruction.
> > The attached patch by Adrian Bunk [2] fixes this issue.
> > 
> 
> this is my bug sorry,
> 
> > diff --git a/arch/arm/atomic_arch.h b/arch/arm/atomic_arch.h
> > index a121010f..c5c56f81 100644
> > --- a/arch/arm/atomic_arch.h
> > +++ b/arch/arm/atomic_arch.h
> > @@ -82,9 +82,13 @@ static inline void a_crash()
> >  		: : : "memory");
> >  }
> >  
> > +#if __ARM_ARCH >= 5
> > +
> 
> this is not enough: v5t and v6 only supports clz in arm mode
> v6t2 and v7 supports it in thumb mode too.

I don't think this matters yet because thumb1 is not supported at all.
If we support it in the future it will matter, though.

Rich

Powered by blists - more mailing lists

Your e-mail address:

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.