Bug 12510 - darktable don't launch : "Instruction not on allowed" on shell
Summary: darktable don't launch : "Instruction not on allowed" on shell
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 4
Hardware: x86_64 Linux
Priority: Normal major
Target Milestone: ---
Assignee: QA Team
QA Contact:
URL:
Whiteboard: MGA4-32-OK MGA4-64-OK advisory
Keywords: Triaged, validated_update
Depends on:
Blocks:
 
Reported: 2014-02-02 07:39 CET by tony byet
Modified: 2014-07-04 20:55 CEST (History)
8 users (show)

See Also:
Source RPM: darktable
CVE:
Status comment:


Attachments

Description tony byet 2014-02-02 07:39:05 CET
Description of problem:don't launch 


Version-Release number of selected component (if applicable):


How reproducible:don't launch on GUI and shell


Steps to Reproduce:
1.
2.
3.


Reproducible: 

Steps to Reproduce:
Manuel Hiebel 2014-02-02 12:18:31 CET

Keywords: (none) => Triaged
Assignee: bugsquad => petos
Summary: don't launch : "Instruction non permise" on shell => darktable don't launch : "Instruction not on allowed" on shell

David Walser 2014-02-10 23:03:56 CET

CC: (none) => mageia

Comment 1 Florian Hubold 2014-02-11 22:29:56 CET
FWIW, here's the original error message:

[doktor5000@Mageia4 ~]$ export LC_ALL=C
[doktor5000@Mageia4 ~]$ darktable
Illegal instruction


This is normal when SSE2 support is not available, which is definitely NOT the case here (even SSE3 and 4_1 available) so this is just broken.

Please fix.

CC: (none) => doktor5000

Comment 2 Petos Safarik 2014-02-14 13:55:09 CET
Yes, I agree, this is an error in build obviously. But not able to find the parametr causing this...
After rebuild on fresh install from the SRPM it works as expected. Florian, is it possible it was not builded on our MGA build-system properly?

Petos

Status: NEW => ASSIGNED

Comment 3 Florian Hubold 2014-02-16 15:00:11 CET
(In reply to Petos Safarik from comment #2)
> After rebuild on fresh install from the SRPM it works as expected. Florian,
> is it possible it was not builded on our MGA build-system properly?

Well, not sure. You could try to push it after disabling parallel build via replacing %make with make, as %make will use as much threads as cores are found, so probably 16 on our buildsystem. 

FWIW, I got the same result, rebuild locally on Mageia 4 installation, and it works as intended. Here %make expands to two threads:

[doktor5000@Mageia4 SPECS]$ rpm -E %make
make -O -j2

Output when running darktable:

[doktor5000@Mageia4 SPECS]$ darktable
[defaults] found a 64-bit system with 4015072 kb ram and 1 cores (0 atom based)
[defaults] setting very conservative defaults
Comment 4 Barry Jackson 2014-02-16 15:46:53 CET
I notice a whole section of CMakeLists.txt related to SSE is commented out. It seems to have been for testing the SSE capability of the machine used for building, which is of no use for a package build as the package needs to run on all machines.
I would guess that the SSE capability detection has been moved to runtime (hence the commented section in CMakeLists.txt), but that it's implementation is incomplete/broken.

Just a guess, I may be totally wrong ;)

CC: (none) => zen25000

Comment 5 Magnus Rasche 2014-04-22 00:15:59 CEST
Some news on the prob?

CC: (none) => magnus.mud

Comment 6 Florian Hubold 2014-04-22 20:41:04 CEST
I've just submitted a update candidate to 4 core/updates_testing which disables parallel build, as there's no upstream report about a similar issue (or only from 3 years ago which have been fixed since).

Please test if that helps

darktable-1.2.3-4.1.mga4 should appear soon on your favorite mirror.
Comment 7 Magnus Rasche 2014-04-23 08:08:06 CEST
Thanks, will test tonight
Comment 8 Magnus Rasche 2014-04-23 17:51:48 CEST
No success, same error

I think sse2 i there on my machine:


[root@edge magnus]# cat /proc/cpuinfo
processor       : 0
vendor_id       : AuthenticAMD
cpu family      : 16
model           : 6
model name      : AMD Turion(tm) II Neo K685 Dual-Core Processor
stepping        : 3
microcode       : 0x10000c8
cpu MHz         : 800.000
cache size      : 1024 KB
physical id     : 0
siblings        : 2
core id         : 0
cpu cores       : 2
apicid          : 0
initial apicid  : 0
fpu             : yes
fpu_exception   : yes
cpuid level     : 5
wp              : yes
flags           : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm 3dnowext 3dnow constant_tsc rep_good nopl nonstop_tsc extd_apicid pni monitor cx16 popcnt lahf_lm cmp_legacy svm extapic cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw ibs skinit wdt nodeid_msr hw_pstate npt lbrv svm_lock nrip_save
bogomips        : 3591.20
TLB size        : 1024 4K pages
clflush size    : 64
cache_alignment : 64
address sizes   : 48 bits physical, 48 bits virtual
power management: ts ttp tm stc 100mhzsteps hwpstate
Comment 9 Magnus Rasche 2014-04-23 18:30:08 CEST
Shouldn't it better to backport version 1.4.1?

On my cauldron it`s running
Comment 10 Magnus Rasche 2014-06-21 09:27:46 CEST
After solving some other probs with my Cauldron there is the same error with
version 1.4.2

Version: 4 => Cauldron

Comment 11 Magnus Rasche 2014-06-23 12:06:08 CEST
I`m very interested in solving the problem,
because I cannot update my MGA3 before.

Can I give any information to help?

I have a MGA4 and a Cauldron for testing
Comment 12 Florian Hubold 2014-06-23 21:14:07 CEST
I've pushed darktable-1.4.2-2.mga5 to cauldron, only change is not using parallel make as that fixed it fore me before, at least when rebuilding the package locally. 

Should be available on your favorite mirror in a few hours.
Comment 13 Magnus Rasche 2014-06-24 06:58:29 CEST
Same error :(

Will try it with my second Cauldron in the evening.
In the moment there is a prob with aria.
Comment 14 Florian Hubold 2014-06-24 07:36:47 CEST
barja mentioned he could also reproduce it with parallel and non-parallel build.

Somebody should report this upstream at http://darktable.org/redmine/projects/darktable/issues

I've only found two issues that look a bit similar on a quick search:
http://darktable.org/redmine/issues/9994
And a much older one:
http://darktable.org/redmine/issues/8427
Comment 15 Magnus Rasche 2014-06-24 09:23:55 CEST
(In reply to Florian Hubold from comment #14)

Somebody should report this upstream at
> http://darktable.org/redmine/projects/darktable/issues

This is a good idea, but I`m not able to give the darktable-team effetive technical informations (of the Mageia build options etc).

If some other will do it I will look on this issue and will confirm this error. If possible i will give also informations of my machines and tests.
Comment 16 Barry Jackson 2014-06-24 14:25:10 CEST
I had a chat with upstream and have just pushed a new build to Cauldron with this change:

Index: SPECS/darktable.spec
===================================================================
--- SPECS/darktable.spec	(revision 639063)
+++ SPECS/darktable.spec	(working copy)
@@ -74,8 +74,8 @@
 rm -rf src/external/osm-gps-map/
 
 %build
-%cmake
-make
+%cmake -DBINARY_PACKAGE_BUILD=On
+%make
 
 %install
 %makeinstall_std -C build

Please test.
Comment 17 Magnus Rasche 2014-06-24 14:51:35 CEST
Thanks :)
I will test tonight.
Comment 18 Barry Jackson 2014-06-24 14:57:08 CEST
@ Petos 
If the above change works, and an update to Mga4 is done then please apply the clean-up that I did to the summary and description etc. in Cauldron at the same time :)
http://svnweb.mageia.org/packages?view=revision&revision=639063
Comment 19 Barry Jackson 2014-06-24 17:30:32 CEST
(In reply to Magnus Rasche from comment #17)
> Thanks :)
> I will test tonight.

Good - the minimum requirement for darktable-1.4.2 is sse2 so you should be OK.
darktable-5.x.x when released will require a minimum of sse3 - so I suppose we should also mention this in the description of each package.
Comment 20 Barry Jackson 2014-06-24 17:31:50 CEST
I meant 1.5.x of course not 5.x.x :\
Comment 21 Magnus Rasche 2014-06-24 22:46:59 CEST
Well done :)
no more error!
Comment 22 Barry Jackson 2014-06-25 01:59:52 CEST
(In reply to Magnus Rasche from comment #21)
> Well done :)
> no more error!

\o/

I have pushed darktable-1.2.3-4.2.mga4 in Mageia 4 core/updates_testing.

Please test this too if you can.
Comment 23 Magnus Rasche 2014-06-25 06:29:12 CEST
Thanks
I will test tonight
Comment 24 Magnus Rasche 2014-06-25 21:10:29 CEST
Same procedure as yesterday :)

Well done, no more error
Comment 25 Barry Jackson 2014-06-25 22:04:46 CEST
###################
Update Advisory:

darktable-1.2.3-4.2.mga4 is  now in Mageia 4 core/updates_testing

This update adds a build directive to cmake (suggested by upstream) to allow the program to run on various processors with sse2 and above available.

Affected packages:
darktable-1.2.3-4.2.mga4.src.rpm
darktable-1.2.3-4.2.mga4.<arch>.rpm
darktable-debuginfo-1.2.3-4.2.mga4.<arch>.rpm

Testing:
This bug only affects older cpus with nothing higher than sse2 available, so it's not easy to test unless you have this hardware. Note sse2 is the minimum requirement for this software - it won't run with less.

On affected machines the program fails to run and the error in a terminal is "Illegal instruction"

This update has fixed the issue for the reporter in Mga4 and in Cauldron where the same change has been applied to the current release.

Version: Cauldron => 4
Assignee: petos => qa-bugs

Comment 26 claire robinson 2014-06-25 22:28:39 CEST
You can check with ..

$ cat /proc/cpuinfo | grep sse
flags           : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx lm constant_tsc arch_perfmon pebs bts rep_good nopl aperfmperf pni dtes64 monitor ds_cpl vmx est tm2 ssse3 cx16 xtpr pdcm lahf_lm dtherm tpr_shadow vnmi flexpriority
Comment 27 Rémi Verschelde 2014-06-25 22:33:56 CEST
For the reference I was able (a week ago or so) to reproduce the bug in a Mageia 4 VM (even though my CPU has sse2 and more). Maybe because it was in a VM?

CC: (none) => remi

Comment 28 Barry Jackson 2014-06-25 23:45:33 CEST
(In reply to Rémi Verschelde from comment #27)
> For the reference I was able (a week ago or so) to reproduce the bug in a
> Mageia 4 VM (even though my CPU has sse2 and more). Maybe because it was in
> a VM?

Interesting,
I had not tried this in a VM, but yes in Mga4 in VBox the current version does exhibit the bug and the update fixes it :)
Comment 29 Rémi Verschelde 2014-06-26 00:15:24 CEST
Testing Mageia 4 x86_64 in VBox.

I could reproduce the bug from comment 1. The update candidate fixes it, and basic functionalities work as expected:
- importing .NEF and .JPG photos
- apply various effects
- browse the modification history
- click randomly on some menus :-)
- export modified image

Whiteboard: (none) => MGA4-64-OK

Comment 30 Rémi Verschelde 2014-06-26 00:18:41 CEST
Note: I tested Mageia 4 i586 in VBox and could not reproduce the bug. The update candidate also seems to work fine there but I did not test thoroughly. A Mageia 4 i586 test on an older CPU would be nice.
Comment 31 Barry Jackson 2014-06-26 00:55:58 CEST
(In reply to Rémi Verschelde from comment #30)
> Note: I tested Mageia 4 i586 in VBox and could not reproduce the bug. The
> update candidate also seems to work fine there but I did not test
> thoroughly. A Mageia 4 i586 test on an older CPU would be nice.

FWIW
A comment about 32bit and darktable from upstream:
"32bit machines are tricky in the first place, so using a very old one is probably not a good idea"
Comment 32 Otto Leipälä 2014-06-28 00:55:20 CEST
I can start to test this,now i'm part of qa team.

CC: (none) => ozkyster

Comment 33 Otto Leipälä 2014-06-29 21:52:13 CEST
Mageia 4 32bit testing ok no issues found.
Comment 34 Rémi Verschelde 2014-06-29 22:17:53 CEST
Adding the corresponding tag on the whiteboard then, thanks Otto!

Whiteboard: MGA4-64-OK => MGA4-32-OK MGA4-64-OK

Comment 35 Rémi Verschelde 2014-06-29 22:45:26 CEST
Validating the update. The advisory has been uploaded.

Please push darktable to Mageia 4 core/updates.

Keywords: (none) => validated_update
Whiteboard: MGA4-32-OK MGA4-64-OK => MGA4-32-OK MGA4-64-OK advisory
CC: (none) => sysadmin-bugs

Comment 36 Thomas Backlund 2014-07-04 20:55:33 CEST
Update pushed:
http://advisories.mageia.org/MGAA-2014-0140.html

Status: ASSIGNED => RESOLVED
CC: (none) => tmb
Resolution: (none) => FIXED


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