* [PATCH] scripts/build-many-glibcs.py: Switch one Arm target to thumb mode
@ 2021-07-07 14:01 Florian Weimer
2021-07-07 14:07 ` Szabolcs Nagy
2021-07-07 15:07 ` Joseph Myers
0 siblings, 2 replies; 7+ messages in thread
From: Florian Weimer @ 2021-07-07 14:01 UTC (permalink / raw)
To: libc-alpha
This commit changes arm-linux-gnueabihf-v7a-disable-multi-arch
to use thumb mode. The choice is somewhat arbitrary, but there
should be at least one thumb mode target.
---
scripts/build-many-glibcs.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/scripts/build-many-glibcs.py b/scripts/build-many-glibcs.py
index 243323b1ce..6454f67c4a 100755
--- a/scripts/build-many-glibcs.py
+++ b/scripts/build-many-glibcs.py
@@ -189,7 +189,8 @@ class Context(object):
extra_glibcs=[{'variant': 'v7a',
'ccopts': '-march=armv7-a -mfpu=vfpv3'},
{'variant': 'v7a-disable-multi-arch',
- 'ccopts': '-march=armv7-a -mfpu=vfpv3',
+ 'ccopts':
+ '-mthumb -march=armv7-a -mfpu=vfpv3',
'cfg': ['--disable-multi-arch']}])
self.add_config(arch='armeb',
os_name='linux-gnueabihf',
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] scripts/build-many-glibcs.py: Switch one Arm target to thumb mode
2021-07-07 14:01 [PATCH] scripts/build-many-glibcs.py: Switch one Arm target to thumb mode Florian Weimer
@ 2021-07-07 14:07 ` Szabolcs Nagy
2021-07-07 15:07 ` Joseph Myers
1 sibling, 0 replies; 7+ messages in thread
From: Szabolcs Nagy @ 2021-07-07 14:07 UTC (permalink / raw)
To: Florian Weimer; +Cc: libc-alpha
The 07/07/2021 16:01, Florian Weimer via Libc-alpha wrote:
> This commit changes arm-linux-gnueabihf-v7a-disable-multi-arch
> to use thumb mode. The choice is somewhat arbitrary, but there
> should be at least one thumb mode target.
>
thanks.
Reviewed-by: Szabolcs Nagy <szabolcs.nagy@arm.com>
> ---
> scripts/build-many-glibcs.py | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/scripts/build-many-glibcs.py b/scripts/build-many-glibcs.py
> index 243323b1ce..6454f67c4a 100755
> --- a/scripts/build-many-glibcs.py
> +++ b/scripts/build-many-glibcs.py
> @@ -189,7 +189,8 @@ class Context(object):
> extra_glibcs=[{'variant': 'v7a',
> 'ccopts': '-march=armv7-a -mfpu=vfpv3'},
> {'variant': 'v7a-disable-multi-arch',
> - 'ccopts': '-march=armv7-a -mfpu=vfpv3',
> + 'ccopts':
> + '-mthumb -march=armv7-a -mfpu=vfpv3',
> 'cfg': ['--disable-multi-arch']}])
> self.add_config(arch='armeb',
> os_name='linux-gnueabihf',
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] scripts/build-many-glibcs.py: Switch one Arm target to thumb mode
2021-07-07 14:01 [PATCH] scripts/build-many-glibcs.py: Switch one Arm target to thumb mode Florian Weimer
2021-07-07 14:07 ` Szabolcs Nagy
@ 2021-07-07 15:07 ` Joseph Myers
2021-07-07 15:15 ` Florian Weimer
1 sibling, 1 reply; 7+ messages in thread
From: Joseph Myers @ 2021-07-07 15:07 UTC (permalink / raw)
To: Florian Weimer; +Cc: libc-alpha
On Wed, 7 Jul 2021, Florian Weimer via Libc-alpha wrote:
> This commit changes arm-linux-gnueabihf-v7a-disable-multi-arch
> to use thumb mode. The choice is somewhat arbitrary, but there
> should be at least one thumb mode target.
My inclination would be to add a new variant for this rather than changing
an existing one.
--
Joseph S. Myers
joseph@codesourcery.com
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] scripts/build-many-glibcs.py: Switch one Arm target to thumb mode
2021-07-07 15:07 ` Joseph Myers
@ 2021-07-07 15:15 ` Florian Weimer
2021-07-07 15:22 ` Joseph Myers
0 siblings, 1 reply; 7+ messages in thread
From: Florian Weimer @ 2021-07-07 15:15 UTC (permalink / raw)
To: Joseph Myers; +Cc: libc-alpha
* Joseph Myers:
> On Wed, 7 Jul 2021, Florian Weimer via Libc-alpha wrote:
>
>> This commit changes arm-linux-gnueabihf-v7a-disable-multi-arch
>> to use thumb mode. The choice is somewhat arbitrary, but there
>> should be at least one thumb mode target.
>
> My inclination would be to add a new variant for this rather than changing
> an existing one.
Running build-many-glibcs.py isn't free. We can't test all possible
combinations, and -mthumb is a real gap.
Can we remove another Arm target instead?
If we absolutely have to add a new target, should it be based on
arm-linux-gnueabi?
Thanks,
Florian
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] scripts/build-many-glibcs.py: Switch one Arm target to thumb mode
2021-07-07 15:15 ` Florian Weimer
@ 2021-07-07 15:22 ` Joseph Myers
2021-07-07 15:48 ` Florian Weimer
0 siblings, 1 reply; 7+ messages in thread
From: Joseph Myers @ 2021-07-07 15:22 UTC (permalink / raw)
To: Florian Weimer; +Cc: libc-alpha
On Wed, 7 Jul 2021, Florian Weimer via Libc-alpha wrote:
> * Joseph Myers:
>
> > On Wed, 7 Jul 2021, Florian Weimer via Libc-alpha wrote:
> >
> >> This commit changes arm-linux-gnueabihf-v7a-disable-multi-arch
> >> to use thumb mode. The choice is somewhat arbitrary, but there
> >> should be at least one thumb mode target.
> >
> > My inclination would be to add a new variant for this rather than changing
> > an existing one.
>
> Running build-many-glibcs.py isn't free. We can't test all possible
> combinations, and -mthumb is a real gap.
It also makes sense for the -mthumb variant to have that as the only
difference from another variant (rather than the only -mthumb Arm variant
also being the only --disable-multi-arch Arm variant).
> If we absolutely have to add a new target, should it be based on
> arm-linux-gnueabi?
Well, it needs to build with at least -march=armv7-a (technically, at
least v6t2) - building glibc as Thumb-1 rather than Thumb-2 isn't
supported - so that suggests making it like the arm-linux-gnueabihf-v7a
variant except for -mthumb added.
--
Joseph S. Myers
joseph@codesourcery.com
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] scripts/build-many-glibcs.py: Switch one Arm target to thumb mode
2021-07-07 15:22 ` Joseph Myers
@ 2021-07-07 15:48 ` Florian Weimer
2021-07-07 15:52 ` Joseph Myers
0 siblings, 1 reply; 7+ messages in thread
From: Florian Weimer @ 2021-07-07 15:48 UTC (permalink / raw)
To: Joseph Myers; +Cc: libc-alpha
* Joseph Myers:
> On Wed, 7 Jul 2021, Florian Weimer via Libc-alpha wrote:
>
>> * Joseph Myers:
>>
>> > On Wed, 7 Jul 2021, Florian Weimer via Libc-alpha wrote:
>> >
>> >> This commit changes arm-linux-gnueabihf-v7a-disable-multi-arch
>> >> to use thumb mode. The choice is somewhat arbitrary, but there
>> >> should be at least one thumb mode target.
>> >
>> > My inclination would be to add a new variant for this rather than changing
>> > an existing one.
>>
>> Running build-many-glibcs.py isn't free. We can't test all possible
>> combinations, and -mthumb is a real gap.
>
> It also makes sense for the -mthumb variant to have that as the only
> difference from another variant (rather than the only -mthumb Arm variant
> also being the only --disable-multi-arch Arm variant).
>
>> If we absolutely have to add a new target, should it be based on
>> arm-linux-gnueabi?
>
> Well, it needs to build with at least -march=armv7-a (technically, at
> least v6t2) - building glibc as Thumb-1 rather than Thumb-2 isn't
> supported - so that suggests making it like the arm-linux-gnueabihf-v7a
> variant except for -mthumb added.
Like this? The new target builds with the out-of-line futex wake patch
applied.
Thanks,
Florian
Subject: build-many-glibcs.py: Add glibcs-arm-linux-gnueabihf-thumb
Previously, there was now thumb variant, despite that building
glibc in Thumb-2 mode is supported.
---
scripts/build-many-glibcs.py | 3 +++
1 file changed, 3 insertions(+)
diff --git a/scripts/build-many-glibcs.py b/scripts/build-many-glibcs.py
index 243323b1ce..6643bb6e92 100755
--- a/scripts/build-many-glibcs.py
+++ b/scripts/build-many-glibcs.py
@@ -188,6 +188,9 @@ class Context(object):
gcc_cfg=['--with-float=hard', '--with-cpu=arm926ej-s'],
extra_glibcs=[{'variant': 'v7a',
'ccopts': '-march=armv7-a -mfpu=vfpv3'},
+ {'variant': 'thumb',
+ 'ccopts':
+ '-mthumb -march=armv7-a -mfpu=vfpv3'},
{'variant': 'v7a-disable-multi-arch',
'ccopts': '-march=armv7-a -mfpu=vfpv3',
'cfg': ['--disable-multi-arch']}])
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] scripts/build-many-glibcs.py: Switch one Arm target to thumb mode
2021-07-07 15:48 ` Florian Weimer
@ 2021-07-07 15:52 ` Joseph Myers
0 siblings, 0 replies; 7+ messages in thread
From: Joseph Myers @ 2021-07-07 15:52 UTC (permalink / raw)
To: Florian Weimer; +Cc: libc-alpha
On Wed, 7 Jul 2021, Florian Weimer via Libc-alpha wrote:
> Like this? The new target builds with the out-of-line futex wake patch
> applied.
Yes.
> Previously, there was now thumb variant, despite that building
s/now thumb/no thumb/
--
Joseph S. Myers
joseph@codesourcery.com
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2021-07-07 15:52 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-07 14:01 [PATCH] scripts/build-many-glibcs.py: Switch one Arm target to thumb mode Florian Weimer
2021-07-07 14:07 ` Szabolcs Nagy
2021-07-07 15:07 ` Joseph Myers
2021-07-07 15:15 ` Florian Weimer
2021-07-07 15:22 ` Joseph Myers
2021-07-07 15:48 ` Florian Weimer
2021-07-07 15:52 ` Joseph Myers
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).