Bug 234989
Summary: | gcj seg faults on ia64 when aot compiling the jsch jar | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Ben Konrath <ben> | ||||||
Component: | gcc | Assignee: | Jakub Jelinek <jakub> | ||||||
Status: | CLOSED INSUFFICIENT_DATA | QA Contact: | |||||||
Severity: | medium | Docs Contact: | |||||||
Priority: | medium | ||||||||
Version: | rawhide | CC: | aoliva, aph, fitzsim, triage, tromey | ||||||
Target Milestone: | --- | ||||||||
Target Release: | --- | ||||||||
Hardware: | ia64 | ||||||||
OS: | Linux | ||||||||
Whiteboard: | bzcl34nup | ||||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||||
Doc Text: | Story Points: | --- | |||||||
Clone Of: | Environment: | ||||||||
Last Closed: | 2008-05-07 01:24:43 UTC | Type: | --- | ||||||
Regression: | --- | Mount Type: | --- | ||||||
Documentation: | --- | CRM: | |||||||
Verified Versions: | Category: | --- | |||||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||
Cloudforms Team: | --- | Target Upstream Version: | |||||||
Embargoed: | |||||||||
Attachments: |
|
Description
Ben Konrath
2007-04-03 05:13:42 UTC
Reproduceable with just KnownHosts.class: usr/libexec/gcc/ia64-redhat-linux/4.1.2/jc1 KnownHosts.class -fhash-synchronization -fuse-divide-subroutine -fuse-boehm-gc -fnon-call-exceptions -fkeep-inline-functions -quiet -dumpbase KnownHosts.class -auxbase-strip jsch-20070403.jar.o -g -O2 -Wall -version -fexceptions -fstack-protector -fPIC -findirect-dispatch -fjni -fbootclasspath=jsch-20070403.jar:usr/share/java/libgcj-4.1.2.jar --param ssp-buffer-size=4 -o x.s GNU Java version 4.1.2 20070327 (Red Hat 4.1.2-7) (ia64-redhat-linux) compiled by GNU C version 4.1.2 20070327 (Red Hat 4.1.2-7). GGC heuristics: --param ggc-min-expand=83 --param ggc-min-heapsize=100996 Class path starts here: ./ jsch-20070403.jar/ (system) (zip) usr/share/java/libgcj-4.1.2.jar/ (system) (zip) <unknown>: In class 'com.jcraft.jsch.KnownHosts': <unknown>: In method 'com.jcraft.jsch.KnownHosts.setKnownHosts(java.io.InputStream)': <unknown>:0: internal compiler error: Segmentation fault Please submit a full bug report, with preprocessed source if appropriate. See <URL:http://bugzilla.redhat.com/bugzilla> for instructions. I see what the problem is, but I still don't have a solution. Basically, config/ia64.c:ia64_add_bundle_selector_before() copies REG_EH_REGION notes from instructions that end basic blocks to nops inserted with the basic block number of other insns. Then, when sched-ebb.c:fix_basic_block_boundaries() scans the instruction stream looking for instructions that ended their blocks, it finds multiple instructions with the same block number for which control_flow_insn_p(insn) holds. This breaks add_missing_bbs(), that doesn't expect such duplicates. I suppose a reasonable solution would be to assign these nops with REG_EH_REGION different basic block numbers. Created attachment 154243 [details]
Patch that appears to fix the bug
This patch seems to fix the problem, by assigning the same basic block number
to nops as that of the previous insn, and arranging for only the last nop in a
sequence (consisting of a call and multiple nops, in the same basic block with
the same REG_EH_REGION note) to be regarded as a control_flow_insn_p().
Does this seem like a reasonable approach? I haven't run it through
bootstrap/regtesting yet.
Created attachment 154278 [details]
Fix a typo in the default definition of the new target hook
This patch passed bootstrap and regtesting on devel ia64.
Based on the date this bug was created, it appears to have been reported against rawhide during the development of a Fedora release that is no longer maintained. In order to refocus our efforts as a project we are flagging all of the open bugs for releases which are no longer maintained. If this bug remains in NEEDINFO thirty (30) days from now, we will automatically close it. If you can reproduce this bug in a maintained Fedora version (7, 8, or rawhide), please change this bug to the respective version and change the status to ASSIGNED. (If you're unable to change the bug's version or status, add a comment to the bug and someone will change it for you.) Thanks for your help, and we apologize again that we haven't handled these issues to this point. The process we're following is outlined here: http://fedoraproject.org/wiki/BugZappers/F9CleanUp We will be following the process here: http://fedoraproject.org/wiki/BugZappers/HouseKeeping to ensure this doesn't happen again. I have no ia64 box on which to test this, but would be happy to re-enable AOT compilation of jsch on ia64 if this is fixed. This bug has been in NEEDINFO for more than 30 days since feedback was first requested. As a result we are closing it. If you can reproduce this bug in the future against a maintained Fedora version please feel free to reopen it against that version. The process we're following is outlined here: http://fedoraproject.org/wiki/BugZappers/F9CleanUp |