Bug 1142586 – LTO: gstreamer-plugins-base build fails on aarch64

Bug 1142586 - LTO: gstreamer-plugins-base build fails on aarch64


Summary:	LTO: gstreamer-plugins-base build fails on aarch64

Status:	IN_PROGRESS

Classification:	openSUSE
Product:	openSUSE Tumbleweed
Classification:	openSUSE
Component:	Basesystem
Version:	Current
Hardware:	Other Other

Priority:	P5 - None Severity: Normal (vote)
Target Milestone:	---
Assigned To:	Martin Liška
QA Contact:	E-mail List

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:	1133084
	Show dependency tree / graph

Create test case

Clone This Bug

Reported:	2019-07-24 07:57 UTC by Martin Liška
Modified:	2019-07-24 08:13 UTC (History)
CC List:	1 user (show)

See Also:
Found By:	---
Services Priority:
Business Priority:
Blocker:	---
Marketing QA Status:	---
IT Deployment:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Martin Liška 2019-07-24 07:57:15 UTC

Fails here:
https://build.opensuse.org/package/live_build_log/openSUSE:Factory:ARM/gstreamer-plugins-base/standard/aarch64

Comment 1 Martin Liška 2019-07-24 07:58:06 UTC

Fails due to a wrong configure detection:

  checking for sys/types.h... (cached) yes
  checking for sys/wait.h... yes
  checking for sys/stat.h... (cached) yes
- checking for ARM NEON support in current arch/CFLAGS... no
+ checking for ARM NEON support in current arch/CFLAGS... yes
  checking for stdint types... stdint.h (shortcircuit)
  make use of stdint.h in _stdint.h (assuming C99 compatible system)
  checking for localtime_r... yes

Comment 2 Guillaume GARDET 2019-07-24 08:00:57 UTC

(In reply to Martin Liška from comment #1)
> Fails due to a wrong configure detection:
> 
>   checking for sys/types.h... (cached) yes
>   checking for sys/wait.h... yes
>   checking for sys/stat.h... (cached) yes
> - checking for ARM NEON support in current arch/CFLAGS... no
> + checking for ARM NEON support in current arch/CFLAGS... yes
>   checking for stdint types... stdint.h (shortcircuit)
>   make use of stdint.h in _stdint.h (assuming C99 compatible system)
>   checking for localtime_r... yes

aarch64 does support NEON. So, it should be ok to enable NEON on aarch64.

Comment 3 Martin Liška 2019-07-24 08:07:40 UTC

(In reply to Guillaume GARDET from comment #2)
> (In reply to Martin Liška from comment #1)
> > Fails due to a wrong configure detection:
> > 
> >   checking for sys/types.h... (cached) yes
> >   checking for sys/wait.h... yes
> >   checking for sys/stat.h... (cached) yes
> > - checking for ARM NEON support in current arch/CFLAGS... no
> > + checking for ARM NEON support in current arch/CFLAGS... yes
> >   checking for stdint types... stdint.h (shortcircuit)
> >   make use of stdint.h in _stdint.h (assuming C99 compatible system)
> >   checking for localtime_r... yes
> 
> aarch64 does support NEON. So, it should be ok to enable NEON on aarch64.

I've got a patch candidate..

Comment 4 Martin Liška 2019-07-24 08:13:55 UTC

I've just created an upstream merge request:
https://gitlab.freedesktop.org/gstreamer/gst-plugins-base/merge_requests/338