Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.

Bug 1096303

Summary: Enable ppc64le and gdb
Product: [Fedora] Fedora Reporter: Brent Baude <bbaude>
Component: gdbAssignee: Jan Kratochvil <jan.kratochvil>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: unspecified    
Version: rawhideCC: gbenson, jan.kratochvil, palves, pmuldoon, sergiodj, tromey
Target Milestone: ---   
Target Release: ---   
Hardware: ppc64le   
OS: Linux   
Whiteboard:
Fixed In Version: gdb-7.7.1-11.fc21 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-05-09 19:31:56 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1051573    
Attachments:
Description Flags
gdb-7.7-add-ppc64le.patch none

Description Brent Baude 2014-05-09 15:39:31 UTC
Created attachment 894073 [details]
gdb-7.7-add-ppc64le.patch

ppc64le support for gdb went into upstream version 7.8.  But for now, I have a patch and trivial spec changes to enable ppc64le in gdb-7.7.  Please incorporate these into rawhide and nvr bump then.  If you prefer to conditionally apply the patch for the ppc64le arch only, I am ok with that too.

The following spec changes are needed.  Change as you prefer.  Ill attach the gdb-7.7-add-ppc64le.patch file.



diff --git a/gdb.spec b/gdb.spec
index 86cfade..5067cd9 100644
--- a/gdb.spec
+++ b/gdb.spec
@@ -539,6 +539,8 @@ Patch852: gdb-gnat-dwarf-crash-3of3.patch
 Patch865: gdb-static-tls-1of2.patch
 Patch866: gdb-static-tls-2of2.patch
 
+Patch999: gdb-7.7-add-ppc64le.patch
+
 %if 0%{!?rhel:1} || 0%{?rhel} > 6
 # RL_STATE_FEDORA_GDB would not be found for:
 # Patch642: gdb-readline62-ask-more-rh.patch
@@ -616,7 +618,7 @@ BuildRequires: systemtap-sdt-devel
 # Copied from prelink-0.4.2-3.fc13.
 %ifarch %{ix86} alpha sparc sparcv9 sparc64 s390 s390x x86_64 ppc ppc64
 # Prelink is broken on sparcv9/sparc64.
-%ifnarch sparc sparcv9 sparc64
+%ifnarch sparc sparcv9 sparc64 ppc64le
 BuildRequires: prelink
 %endif
 %endif
@@ -649,7 +651,7 @@ BuildRequires: glibc-static%{bits_local}
 # for gcc-java linkage:
 BuildRequires: zlib-devel%{bits_local} zlib-devel%{bits_other}
 # Copied from valgrind-3.5.0-1.
-%ifarch %{ix86} x86_64 ppc ppc64
+%ifarch %{ix86} x86_64 ppc ppc64 ppc64le
 BuildRequires: valgrind%{bits_local} valgrind%{bits_other}
 %endif
 %if 0%{!?rhel:1} || 0%{?rhel} > 6
@@ -921,7 +923,7 @@ export LDFLAGS="%{?__global_ldflags} %{?_with_asan:-fsanitize=addr
        --with-system-gdbinit=%{_sysconfdir}/gdbinit            \
        --with-gdb-datadir=%{_datadir}/gdb                      \
        --enable-gdb-build-warnings=,-Wno-unused                \
-%ifnarch %{ix86} alpha ppc s390 s390x x86_64 ppc64 sparc sparcv9 sparc64
+%ifnarch %{ix86} alpha ppc s390 s390x x86_64 ppc64 sparc sparcv9 sparc64 ppc64le
        --disable-werror                                        \
 %else
        --enable-werror                                         \

Comment 1 Brent Baude 2014-05-09 15:40:31 UTC
fyi I built gdb on the ppcle koji in case you are interested in poking at it:

http://ppc-le.koji.fedoraproject.org/koji/taskinfo?taskID=14767

Comment 2 Jan Kratochvil 2014-05-09 15:50:20 UTC
It would be better to point at the upstream commits - I will look them up.
The patch file is not acceptable.

Comment 3 Jan Kratochvil 2014-05-09 19:31:56 UTC
It is building now in primary Koji.
http://pkgs.fedoraproject.org/cgit/gdb.git/commit/?id=6e0665a33c841cb72f25624655a40e62747390ed