* [Buildroot] [PATCH 1/1] package/pixman: fix powerpc64le build
@ 2022-07-08 14:21 Bernd Kuhls
2022-07-18 12:14 ` Thomas Petazzoni via buildroot
0 siblings, 1 reply; 6+ messages in thread
From: Bernd Kuhls @ 2022-07-08 14:21 UTC (permalink / raw)
To: buildroot
Disable vmx support to fix build error:
pixman-vmx.c:268:5: note: in expansion of macro 'LOAD_VECTOR'
268 | LOAD_VECTOR (src);
| ^~~~~~~~~~~
pixman-vmx.c:268:5: error: cannot convert a value of type 'int' to
vector type '__vector(4) unsigned int' which has different size
Fixes:
http://autobuild.buildroot.net/results/fda886768fce25ccd9b52b635ff5b13da7ba2d0c/
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
package/pixman/pixman.mk | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/package/pixman/pixman.mk b/package/pixman/pixman.mk
index 64302fa49a..0f00a20cd8 100644
--- a/package/pixman/pixman.mk
+++ b/package/pixman/pixman.mk
@@ -21,7 +21,8 @@ PIXMAN_AUTORECONF = YES
# don't build gtk based demos
PIXMAN_CONF_OPTS = \
--disable-gtk \
- --disable-loongson-mmi
+ --disable-loongson-mmi \
+ --disable-vmx
# The ARM SIMD code from pixman requires a recent enough ARM core, but
# there is a runtime CPU check that makes sure it doesn't get used if
--
2.30.2
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/pixman: fix powerpc64le build
2022-07-08 14:21 [Buildroot] [PATCH 1/1] package/pixman: fix powerpc64le build Bernd Kuhls
@ 2022-07-18 12:14 ` Thomas Petazzoni via buildroot
2022-07-19 7:46 ` Joel Stanley
0 siblings, 1 reply; 6+ messages in thread
From: Thomas Petazzoni via buildroot @ 2022-07-18 12:14 UTC (permalink / raw)
To: Joel Stanley, Cédric Le Goater; +Cc: Bernd Kuhls, buildroot
Hello,
Let's add some PowerPC folks into the loop. Joel, Cédric, could you
have a look at the below patch, and how to handle VMX support in the
pixman package?
It feels a bit unfortunate to unconditionally disable VMX support:
there is some logic in the pixman configure script to detect the
availability of VMX, but apparently this logic assumes it works on
powerpc64le, even though at build time it breaks.
Could you have a look and give us your advice?
Thanks a lot!
Thomas Petazzoni
On Fri, 8 Jul 2022 16:21:50 +0200
Bernd Kuhls <bernd.kuhls@t-online.de> wrote:
> Disable vmx support to fix build error:
>
> pixman-vmx.c:268:5: note: in expansion of macro 'LOAD_VECTOR'
> 268 | LOAD_VECTOR (src);
> | ^~~~~~~~~~~
> pixman-vmx.c:268:5: error: cannot convert a value of type 'int' to
> vector type '__vector(4) unsigned int' which has different size
>
> Fixes:
> http://autobuild.buildroot.net/results/fda886768fce25ccd9b52b635ff5b13da7ba2d0c/
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> package/pixman/pixman.mk | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/package/pixman/pixman.mk b/package/pixman/pixman.mk
> index 64302fa49a..0f00a20cd8 100644
> --- a/package/pixman/pixman.mk
> +++ b/package/pixman/pixman.mk
> @@ -21,7 +21,8 @@ PIXMAN_AUTORECONF = YES
> # don't build gtk based demos
> PIXMAN_CONF_OPTS = \
> --disable-gtk \
> - --disable-loongson-mmi
> + --disable-loongson-mmi \
> + --disable-vmx
>
> # The ARM SIMD code from pixman requires a recent enough ARM core, but
> # there is a runtime CPU check that makes sure it doesn't get used if
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/pixman: fix powerpc64le build
2022-07-18 12:14 ` Thomas Petazzoni via buildroot
@ 2022-07-19 7:46 ` Joel Stanley
2022-07-19 8:27 ` Thomas Petazzoni via buildroot
0 siblings, 1 reply; 6+ messages in thread
From: Joel Stanley @ 2022-07-19 7:46 UTC (permalink / raw)
To: Thomas Petazzoni
Cc: Bernd Kuhls, Cédric Le Goater, Buildroot Mailing List
On Mon, 18 Jul 2022 at 12:14, Thomas Petazzoni
<thomas.petazzoni@bootlin.com> wrote:
>
> Hello,
>
> Let's add some PowerPC folks into the loop. Joel, Cédric, could you
> have a look at the below patch, and how to handle VMX support in the
> pixman package?
>
> It feels a bit unfortunate to unconditionally disable VMX support:
> there is some logic in the pixman configure script to detect the
> availability of VMX, but apparently this logic assumes it works on
> powerpc64le, even though at build time it breaks.
Thanks!
There's a few build failures going around due to nonsense
configurations. In most cases the config has BR2_powerpc64le=y without
selecting a CPU. This defaults us to BR2_powerpc_970=y, which is a CPU
that isn't supported to run ppc64le.
The fix is to only allow CPUs that support ppc64le to be configured
for that platform. I've prepared some patches that I'm running though
CI now, and I'll send them out once they're tested.
If my changes don't fix this pixman issue then we can use the
BR2_POWERPC_CPU_HAS_ALTIVEC to force AltiVec/VMX off when we know the
CPU does not support it.
Cheers,
Joel
>
> Could you have a look and give us your advice?
>
> Thanks a lot!
>
> Thomas Petazzoni
>
> On Fri, 8 Jul 2022 16:21:50 +0200
> Bernd Kuhls <bernd.kuhls@t-online.de> wrote:
>
> > Disable vmx support to fix build error:
> >
> > pixman-vmx.c:268:5: note: in expansion of macro 'LOAD_VECTOR'
> > 268 | LOAD_VECTOR (src);
> > | ^~~~~~~~~~~
> > pixman-vmx.c:268:5: error: cannot convert a value of type 'int' to
> > vector type '__vector(4) unsigned int' which has different size
> >
> > Fixes:
> > http://autobuild.buildroot.net/results/fda886768fce25ccd9b52b635ff5b13da7ba2d0c/
> >
> > Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> > ---
> > package/pixman/pixman.mk | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/package/pixman/pixman.mk b/package/pixman/pixman.mk
> > index 64302fa49a..0f00a20cd8 100644
> > --- a/package/pixman/pixman.mk
> > +++ b/package/pixman/pixman.mk
> > @@ -21,7 +21,8 @@ PIXMAN_AUTORECONF = YES
> > # don't build gtk based demos
> > PIXMAN_CONF_OPTS = \
> > --disable-gtk \
> > - --disable-loongson-mmi
> > + --disable-loongson-mmi \
> > + --disable-vmx
> >
> > # The ARM SIMD code from pixman requires a recent enough ARM core, but
> > # there is a runtime CPU check that makes sure it doesn't get used if
>
>
>
> --
> Thomas Petazzoni, co-owner and CEO, Bootlin
> Embedded Linux and Kernel engineering and training
> https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/pixman: fix powerpc64le build
2022-07-19 7:46 ` Joel Stanley
@ 2022-07-19 8:27 ` Thomas Petazzoni via buildroot
2022-07-26 20:19 ` Thomas Petazzoni via buildroot
0 siblings, 1 reply; 6+ messages in thread
From: Thomas Petazzoni via buildroot @ 2022-07-19 8:27 UTC (permalink / raw)
To: Joel Stanley; +Cc: Bernd Kuhls, Cédric Le Goater, Buildroot Mailing List
Hello Joel,
On Tue, 19 Jul 2022 07:46:03 +0000
Joel Stanley <joel@jms.id.au> wrote:
> There's a few build failures going around due to nonsense
> configurations. In most cases the config has BR2_powerpc64le=y without
> selecting a CPU.
"Not selecting a CPU" is not possible. There is always a CPU within the:
choice
prompt "Target Architecture Variant"
default BR2_generic_powerpc
help
Specific CPU variant to use
...
endchoice
that will be selected. However...
> This defaults us to BR2_powerpc_970=y, which is a CPU
> that isn't supported to run ppc64le.
... it seems like some of them are selectable on ppc64le, even though
they do not make sense.
> The fix is to only allow CPUs that support ppc64le to be configured
> for that platform. I've prepared some patches that I'm running though
> CI now, and I'll send them out once they're tested.
Excellent! Looking forward to it!
> If my changes don't fix this pixman issue then we can use the
> BR2_POWERPC_CPU_HAS_ALTIVEC to force AltiVec/VMX off when we know the
> CPU does not support it.
But there's still a weird thing going on: pixman has a configure.ac
check that compiles a small test program to verify if VMX support is
available... and it concludes it is, but the actual pixman code then
fails to build.
Thanks for your feedback!
Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/pixman: fix powerpc64le build
2022-07-19 8:27 ` Thomas Petazzoni via buildroot
@ 2022-07-26 20:19 ` Thomas Petazzoni via buildroot
0 siblings, 0 replies; 6+ messages in thread
From: Thomas Petazzoni via buildroot @ 2022-07-26 20:19 UTC (permalink / raw)
To: Joel Stanley; +Cc: Bernd Kuhls, Cédric Le Goater, buildroot
Hello Joel,
Do you have some updates on this pixman issue?
Thanks a lot!
Thomas Petazzoni
On Tue, 19 Jul 2022 10:27:24 +0200
Thomas Petazzoni via buildroot <buildroot@buildroot.org> wrote:
> Hello Joel,
>
> On Tue, 19 Jul 2022 07:46:03 +0000
> Joel Stanley <joel@jms.id.au> wrote:
>
> > There's a few build failures going around due to nonsense
> > configurations. In most cases the config has BR2_powerpc64le=y without
> > selecting a CPU.
>
> "Not selecting a CPU" is not possible. There is always a CPU within the:
>
> choice
> prompt "Target Architecture Variant"
> default BR2_generic_powerpc
> help
> Specific CPU variant to use
>
> ...
> endchoice
>
> that will be selected. However...
>
> > This defaults us to BR2_powerpc_970=y, which is a CPU
> > that isn't supported to run ppc64le.
>
> ... it seems like some of them are selectable on ppc64le, even though
> they do not make sense.
>
> > The fix is to only allow CPUs that support ppc64le to be configured
> > for that platform. I've prepared some patches that I'm running though
> > CI now, and I'll send them out once they're tested.
>
> Excellent! Looking forward to it!
>
> > If my changes don't fix this pixman issue then we can use the
> > BR2_POWERPC_CPU_HAS_ALTIVEC to force AltiVec/VMX off when we know the
> > CPU does not support it.
>
> But there's still a weird thing going on: pixman has a configure.ac
> check that compiles a small test program to verify if VMX support is
> available... and it concludes it is, but the actual pixman code then
> fails to build.
>
> Thanks for your feedback!
>
> Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/pixman: fix powerpc64le build
[not found] <20220708142150.234611-1-bernd.kuhls__17925.5718061299$1657290142$gmane$org@t-online.de>
@ 2022-10-30 21:48 ` Bernd Kuhls
0 siblings, 0 replies; 6+ messages in thread
From: Bernd Kuhls @ 2022-10-30 21:48 UTC (permalink / raw)
To: buildroot
Am Fri, 8 Jul 2022 16:21:50 +0200 schrieb Bernd Kuhls:
> Disable vmx support to fix build error:
>
> pixman-vmx.c:268:5: note: in expansion of macro 'LOAD_VECTOR'
> 268 | LOAD_VECTOR (src);
> | ^~~~~~~~~~~
> pixman-vmx.c:268:5: error: cannot convert a value of type 'int' to
> vector type '__vector(4) unsigned int' which has different size
>
> Fixes:
> http://autobuild.buildroot.net/results/
fda886768fce25ccd9b52b635ff5b13da7ba2d0c/
Marked as "Handled Elsewhere" in patchworks due to
https://git.buildroot.net/buildroot/commit/?
id=fdb23693a4bff5e9e9472738a0278ae0b13411e0
Regards, Bernd
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-10-30 21:49 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-08 14:21 [Buildroot] [PATCH 1/1] package/pixman: fix powerpc64le build Bernd Kuhls
2022-07-18 12:14 ` Thomas Petazzoni via buildroot
2022-07-19 7:46 ` Joel Stanley
2022-07-19 8:27 ` Thomas Petazzoni via buildroot
2022-07-26 20:19 ` Thomas Petazzoni via buildroot
[not found] <20220708142150.234611-1-bernd.kuhls__17925.5718061299$1657290142$gmane$org@t-online.de>
2022-10-30 21:48 ` Bernd Kuhls
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.