Unity 7 GCC ICE on ARM64

Bug #1317276 reported by Stephen M. Webb on 2014-05-07
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Unity
Fix Released
High
Dimitri John Ledkov
7.2
Fix Released
High
Stephen M. Webb
gcc-4.8 (Ubuntu)
Undecided
Unassigned
Declined for Trusty by Matthias Klose
unity (Ubuntu)
Undecided
Unassigned
Declined for Trusty by Matthias Klose

Bug Description

Unity 7 failed to build from source on ARM64 on Utopic due to an internal compiler error in GCC.

Build log excerpt as follows.

[ 15%] Building CXX object UnityCore/CMakeFiles/unity-core-6.0.dir/Category.cpp.o
cd /build/buildd/unity-7.2.0+14.10.20140507/obj-aarch64-linux-gnu/UnityCore && /usr/bin/aarch64-linux-gnu-g++ -DUSE_X11 -Dunity_core_6_0_EXPORTS -g -O2 -fstack-protector --param=ssp-buffer-size=4 -Wformat -Werror=format-security -O2 -D_FORTIFY_SOURCE=2 -DGNOME_DESKTOP_USE_UNSTABLE_API -std=c++0x -fno-permissive -Werror -Wall -Wcast-align -Wempty-body -Wformat-security -Winit-self -Warray-bounds -Wno-error=unused-local-typedefs -fPIC -I/usr/include/gmock/include -I/usr/src/gmock/gtest/include -I/usr/include/Nux-4.0 -I/usr/include/sigc++-2.0 -I/usr/lib/aarch64-linux-gnu/sigc++-2.0/include -I/usr/include/unity/unity -I/usr/include/dee-1.0 -I/usr/include/gio-unix-2.0 -I/usr/include/glib-2.0 -I/usr/lib/aarch64-linux-gnu/glib-2.0/include -I/build/buildd/unity-7.2.0+14.10.20140507/obj-aarch64-linux-gnu -I/build/buildd/unity-7.2.0+14.10.20140507/obj-aarch64-linux-gnu/UnityCore -include unitycore_pch.hh -fpch-preprocess -Winvalid-pch -o CMakeFiles/unity-core-6.0.dir/Category.cpp.o -c /build/buildd/unity-7.2.0+14.10.20140507/UnityCore/Category.cpp

aarch64-linux-gnu-g++: internal compiler error: Segmentation fault (program cc1plus)
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-4.8/README.Bugs> for instructions.
UnityCore/CMakeFiles/unity-core-6.0.dir/build.make:126: recipe for target 'UnityCore/CMakeFiles/unity-core-6.0.dir/Category.cpp.o' failed
make[3]: *** [UnityCore/CMakeFiles/unity-core-6.0.dir/Category.cpp.o] Error 4

More information at https://launchpad.net/~ci-train-ppa-service/+archive/landing-008/+build/5987360

Related branches

Dimitri John Ledkov (xnox) wrote :

# cd /home/xnox/unity-7.2.0+14.10.20140507/obj-aarch64-linux-gnu/UnityCore && /usr/bin/aarch64-linux-gnu-g++ -DUSE_X11 -Dunity_core_6_0_EXPORTS -g -O2 -fstack-protector --param=ssp-buffer-size=4 -Wformat -Werror=format-security -O2 -D_FORTIFY_SOURCE=2 -DGNOME_DESKTOP_USE_UNSTABLE_API -std=c++0x -fno-permissive -Werror -Wall -Wcast-align -Wempty-body -Wformat-security -Winit-self -Warray-bounds -Wno-error=unused-local-typedefs -fPIC -I/usr/include/gmock/include -I/usr/src/gmock/gtest/include -I/usr/include/Nux-4.0 -I/usr/include/sigc++-2.0 -I/usr/lib/aarch64-linux-gnu/sigc++-2.0/include -I/usr/include/unity/unity -I/usr/include/dee-1.0 -I/usr/include/gio-unix-2.0 -I/usr/include/glib-2.0 -I/usr/lib/aarch64-linux-gnu/glib-2.0/include -I/home/xnox/unity-7.2.0+14.10.20140507/obj-aarch64-linux-gnu -I/home/xnox/unity-7.2.0+14.10.20140507/obj-aarch64-linux-gnu/UnityCore -include unitycore_pch.hh -fpch-preprocess -Winvalid-pch -o CMakeFiles/unity-core-6.0.dir/GnomeSessionManager.cpp.o -c /home/xnox/unity-7.2.0+14.10.20140507/UnityCore/GnomeSessionManager.cpp -save-temps

Dimitri John Ledkov (xnox) wrote :
Dimitri John Ledkov (xnox) wrote :
Matthias Klose (doko) wrote :

please provide preprocessed source without any pch file

Changed in gcc-4.8 (Ubuntu):
status: New → Incomplete
Matthias Klose (doko) wrote :

could be a duplicate of #1318635.

Stephen M. Webb (bregma) wrote :

The ICE occurs on a different source file for each run, and in many cases the source file itself has not been modified in years. Going on the assumption that this bug is a duplicate of #1318635 I will disable PCH for the arm64 arch and retry the builds.

Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity - 7.2.0+14.10.20140512.4-0ubuntu1

---------------
unity (7.2.0+14.10.20140512.4-0ubuntu1) utopic; urgency=low

  [ Stephen M. Webb ]
  * lockscreen/LockScreenAccelerators.h: added required header for
    std:vector definition
  * disabled precompiled headers on arm64 Ubuntu package builds (LP:
    #1317276)

  [ Brandon Schaefer ]
  * Set the max width, so our static text wraps. So we don't end up
    rendering text out of the view area. (LP: #1312749)

  [ Marco Trevisan (Treviño) ]
  * UnityScreen: toggle activation of all the unity compiz actions when
    locking the screen Also, make sure that the lockscreen views are
    treated as "always-on-front" windows by nux (and this applies to
    both visibility and events).
  * PanelService: emit an invalid EntryActivated signal if the menu
    hasn't actually been shown In addition, if the menu is not visible
    after the first popup call, try to show a menu without keyboard
    focus. This will make possible to see menus also when there's a key-
    grab (although they will be without keyboard support). Finally,
    reset the menu state, if that failed.
  * UnityScreen: always paint the lockscreen above, just add menu and
    onboard as exceptions Some code cleanup, factorizing similar code.
    (LP: #1313280)
  * UnityScreen: don't try to show Dash/Hud if the screen is grabbed
    Also move the dash opening out from LauncherController, and get rid
    of UBus as first initialization source, as it can only slow things
    down here. (LP: #741869)
  * PluginAdapter: make sure we don't try to call an invalid
    initiate/terminate callback function (LP: #1221673)
  * DebugDBusInterface: match properties if they are in the AP array
    form [<type>, <value>] (LP: #1307748)

  [ William Hua ]
  * Add proper support for modifier-only shortcuts on the lock screen.
    (LP: #1291461)

  [ Jinkyu Yi ]
  * Dash: Fix problem at checking blacklisted unicode range (LP:
    #1251193)

  [ Andrea Azzarone ]
  * Add a warning in the session dialog if other sessions are still
    open. (LP: #1281058)
 -- Ubuntu daily release <email address hidden> Mon, 12 May 2014 22:03:27 +0000

Changed in unity (Ubuntu):
status: New → Fix Released
Matthias Klose (doko) wrote :

if anyone can get a stack trace, that would be helpful. I was unable to reproduce this on a local machine.

Changed in unity:
status: New → In Progress
milestone: none → 7.2.1
milestone: 7.2.1 → 7.3.0
status: In Progress → Fix Committed
Stephen M. Webb (bregma) wrote :

I have only seen this problem happen in the build farm (arm64 hardware is pretty hard to come by) and it always happens on a different source file. Our current workaround to unblock Unity releases is to disabled PCH for the arm64 arch.

Hello Stephen, or anyone else affected,

Accepted unity into trusty-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/unity/7.2.1+14.04.20140513-0ubuntu1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

tags: added: verification-needed
Brian Murray (brian-murray) wrote :

Hello Stephen, or anyone else affected,

Accepted unity into trusty-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/unity/7.2.1+14.04.20140513-0ubuntu2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Seeing as this built in the ci-train silo, I'd considered this fixed. Otherwise, it wouldn't build and we would be blocked:)

tags: added: verification-done
removed: verification-needed

The verification of the Stable Release Update for unity has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regresssions.

Stephen M. Webb (bregma) on 2014-06-20
Changed in unity:
status: Fix Committed → Fix Released
Matthias Klose (doko) wrote :

please check if pch can be turned on again with GCC 4.9 in 14.10.

Changed in gcc-4.8 (Ubuntu):
status: Incomplete → Won't Fix
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers