* [PATCH] x264: SPE is not compatible with Altivec
@ 2014-02-21 13:56 David Nyström
2014-02-21 18:28 ` Saul Wold
2014-02-21 21:59 ` [PATCHv2] x246: Disable Altivec asm when SPE is enabled David Nyström
0 siblings, 2 replies; 6+ messages in thread
From: David Nyström @ 2014-02-21 13:56 UTC (permalink / raw)
To: openembedded-core
Signed-off-by: David Nyström <david.c.nystrom@gmail.com>
---
meta/recipes-multimedia/x264/x264_git.bb | 1 +
1 file changed, 1 insertion(+)
diff --git a/meta/recipes-multimedia/x264/x264_git.bb b/meta/recipes-multimedia/x264/x264_git.bb
index 5ebf885..782c26e 100644
--- a/meta/recipes-multimedia/x264/x264_git.bb
+++ b/meta/recipes-multimedia/x264/x264_git.bb
@@ -23,6 +23,7 @@ inherit lib_package pkgconfig
X264_DISABLE_ASM = ""
X264_DISABLE_ASM_armv4 = "--disable-asm"
X264_DISABLE_ASM_armv5 = "--disable-asm"
+X264_DISABLE_ASM_e500v2 = "--disable-asm"
EXTRA_OECONF = '--prefix=${prefix} \
--host=${HOST_SYS} \
--
1.8.3.2
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] x264: SPE is not compatible with Altivec
2014-02-21 13:56 [PATCH] x264: SPE is not compatible with Altivec David Nyström
@ 2014-02-21 18:28 ` Saul Wold
2014-02-21 19:40 ` David Nystrom
2014-02-21 21:59 ` [PATCHv2] x246: Disable Altivec asm when SPE is enabled David Nyström
1 sibling, 1 reply; 6+ messages in thread
From: Saul Wold @ 2014-02-21 18:28 UTC (permalink / raw)
To: David Nyström, openembedded-core
On 02/21/2014 05:56 AM, David Nyström wrote:
> Signed-off-by: David Nyström <david.c.nystrom@gmail.com>
> ---
> meta/recipes-multimedia/x264/x264_git.bb | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/meta/recipes-multimedia/x264/x264_git.bb b/meta/recipes-multimedia/x264/x264_git.bb
> index 5ebf885..782c26e 100644
> --- a/meta/recipes-multimedia/x264/x264_git.bb
> +++ b/meta/recipes-multimedia/x264/x264_git.bb
> @@ -23,6 +23,7 @@ inherit lib_package pkgconfig
> X264_DISABLE_ASM = ""
> X264_DISABLE_ASM_armv4 = "--disable-asm"
> X264_DISABLE_ASM_armv5 = "--disable-asm"
> +X264_DISABLE_ASM_e500v2 = "--disable-asm"
>
I think that this change really belongs in the e500v2 BSP, it's not
generic enough to include in core.
Thanks
Sau!
> EXTRA_OECONF = '--prefix=${prefix} \
> --host=${HOST_SYS} \
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] x264: SPE is not compatible with Altivec
2014-02-21 18:28 ` Saul Wold
@ 2014-02-21 19:40 ` David Nystrom
2014-02-21 21:20 ` Phil Blundell
0 siblings, 1 reply; 6+ messages in thread
From: David Nystrom @ 2014-02-21 19:40 UTC (permalink / raw)
To: Saul Wold; +Cc: Patches and discussions about the oe-core layer
[-- Attachment #1: Type: text/plain, Size: 1176 bytes --]
On Feb 21, 2014 7:28 PM, "Saul Wold" <sgw@linux.intel.com> wrote:
>
> On 02/21/2014 05:56 AM, David Nyström wrote:
>>
>> Signed-off-by: David Nyström <david.c.nystrom@gmail.com>
>> ---
>> meta/recipes-multimedia/x264/x264_git.bb | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/meta/recipes-multimedia/x264/x264_git.bbb/meta/recipes-multimedia/x264/
x264_git.bb
>> index 5ebf885..782c26e 100644
>> --- a/meta/recipes-multimedia/x264/x264_git.bb
>> +++ b/meta/recipes-multimedia/x264/x264_git.bb
>> @@ -23,6 +23,7 @@ inherit lib_package pkgconfig
>> X264_DISABLE_ASM = ""
>> X264_DISABLE_ASM_armv4 = "--disable-asm"
>> X264_DISABLE_ASM_armv5 = "--disable-asm"
>> +X264_DISABLE_ASM_e500v2 = "--disable-asm"
>>
> I think that this change really belongs in the e500v2 BSP, it's not
generic enough to include in core.
Perhaps you know this already, but e500v2 is a cpu arch, same as armv5.
For future reference, should all ppc related CPU-arch specific stuff be
directed to the FSL bsp ?
Br,
David
>
> Thanks
> Sau!
>
>
>
>> EXTRA_OECONF = '--prefix=${prefix} \
>> --host=${HOST_SYS} \
>>
[-- Attachment #2: Type: text/html, Size: 1839 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] x264: SPE is not compatible with Altivec
2014-02-21 19:40 ` David Nystrom
@ 2014-02-21 21:20 ` Phil Blundell
2014-02-21 21:45 ` David Nyström
0 siblings, 1 reply; 6+ messages in thread
From: Phil Blundell @ 2014-02-21 21:20 UTC (permalink / raw)
To: David Nystrom; +Cc: Patches and discussions about the oe-core layer
On Fri, 2014-02-21 at 20:40 +0100, David Nystrom wrote:
> Perhaps you know this already, but e500v2 is a cpu arch, same as
> armv5.
Is that really true? I thought e500v2 was an actual core and the
equivalent architecture was Power v2.03 or something.
>For future reference, should all ppc related CPU-arch specific stuff
>be directed to the FSL bsp ?
Since we do ship tune-ppce500.inc in oe-core, I think it makes sense to
also include the overrides necessary to make compilation work with SPE
enabled. It doesn't seem unreasonable that users would expect that the
recipes in oe-core would work with a configuration that comes as
standard.
However, adding overrides for e500v2 specifically does seem like a bad
plan since we will then end up having to duplicate the thing for every
other non-altivec configuration. Can you come up with some override
that will apply to all SPE configurations, or failing that a patch for
the configure script that will disable the altivec assembler if it
notices that ${CC} is configured for SPE?
p.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] x264: SPE is not compatible with Altivec
2014-02-21 21:20 ` Phil Blundell
@ 2014-02-21 21:45 ` David Nyström
0 siblings, 0 replies; 6+ messages in thread
From: David Nyström @ 2014-02-21 21:45 UTC (permalink / raw)
To: Phil Blundell; +Cc: Patches and discussions about the oe-core layer
On 2014-02-21 22:20, Phil Blundell wrote:
> On Fri, 2014-02-21 at 20:40 +0100, David Nystrom wrote:
>> Perhaps you know this already, but e500v2 is a cpu arch, same as
>> armv5.
> Is that really true? I thought e500v2 was an actual core and the
> equivalent architecture was Power v2.03 or something.
Yes, good point.
I understand Sauls concern.
>
>> For future reference, should all ppc related CPU-arch specific stuff
>> be directed to the FSL bsp ?
> Since we do ship tune-ppce500.inc in oe-core, I think it makes sense to
> also include the overrides necessary to make compilation work with SPE
> enabled. It doesn't seem unreasonable that users would expect that the
> recipes in oe-core would work with a configuration that comes as
> standard.
>
> However, adding overrides for e500v2 specifically does seem like a bad
> plan since we will then end up having to duplicate the thing for every
> other non-altivec configuration. Can you come up with some override
> that will apply to all SPE configurations,
Yep, will return with V2.
> or failing that a patch for
> the configure script that will disable the altivec assembler if it
> notices that ${CC} is configured for SPE?
>
> p.
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCHv2] x246: Disable Altivec asm when SPE is enabled.
2014-02-21 13:56 [PATCH] x264: SPE is not compatible with Altivec David Nyström
2014-02-21 18:28 ` Saul Wold
@ 2014-02-21 21:59 ` David Nyström
1 sibling, 0 replies; 6+ messages in thread
From: David Nyström @ 2014-02-21 21:59 UTC (permalink / raw)
To: openembedded-core
asm code is written for Altivec, and should be disabled
for powerpc SPE enabled targets.
Signed-off-by: David Nyström <david.c.nystrom@gmail.com>
---
meta/recipes-multimedia/x264/x264_git.bb | 1 +
1 file changed, 1 insertion(+)
diff --git a/meta/recipes-multimedia/x264/x264_git.bb b/meta/recipes-multimedia/x264/x264_git.bb
index 5ebf885..0e305ea 100644
--- a/meta/recipes-multimedia/x264/x264_git.bb
+++ b/meta/recipes-multimedia/x264/x264_git.bb
@@ -23,6 +23,7 @@ inherit lib_package pkgconfig
X264_DISABLE_ASM = ""
X264_DISABLE_ASM_armv4 = "--disable-asm"
X264_DISABLE_ASM_armv5 = "--disable-asm"
+X264_DISABLE_ASM_powerpc = "${@bb.utils.contains("TUNE_FEATURES", "spe", "--disable-asm", "", d)}"
EXTRA_OECONF = '--prefix=${prefix} \
--host=${HOST_SYS} \
--
1.8.3.2
^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2014-02-21 21:59 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-02-21 13:56 [PATCH] x264: SPE is not compatible with Altivec David Nyström
2014-02-21 18:28 ` Saul Wold
2014-02-21 19:40 ` David Nystrom
2014-02-21 21:20 ` Phil Blundell
2014-02-21 21:45 ` David Nyström
2014-02-21 21:59 ` [PATCHv2] x246: Disable Altivec asm when SPE is enabled David Nyström
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.