Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 157454 - [PATCH] For Linux x86_64 support org.eclipse.hyades.execution
Summary: [PATCH] For Linux x86_64 support org.eclipse.hyades.execution
Status: CLOSED WONTFIX
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: TPTP (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 major (vote)
Target Milestone: ---   Edit
Assignee: Bing Xu CLA
QA Contact:
URL:
Whiteboard: closed471
Keywords:
Depends on:
Blocks:
 
Reported: 2006-09-15 07:05 EDT by Darryl Miles CLA
Modified: 2016-05-05 11:02 EDT (History)
2 users (show)

See Also:


Attachments
org.eclipse.hyades.execution_20060915.diff (5.78 KB, patch)
2006-09-15 07:07 EDT, Darryl Miles CLA
no flags Details | Diff
platform/org.eclipse.hyades.execution/src-iac-native/makefile.linux.x86_64 (877 bytes, text/plain)
2006-09-15 07:08 EDT, Darryl Miles CLA
no flags Details
org.eclipse.hyades.execution.linux.x86_64.zip Project (3.40 KB, application/octet-stream)
2006-09-15 08:54 EDT, Darryl Miles CLA
no flags Details
org.eclipse.tptp.platform.agentcontroller.diff (8.94 KB, patch)
2006-09-15 09:21 EDT, Darryl Miles CLA
no flags Details | Diff
org.eclipse.hyades.execution.linux.x86_64.zip Project (3.41 KB, application/octet-stream)
2006-09-20 20:52 EDT, Darryl Miles CLA
no flags Details
org.eclipse.tptp.platform.agentcontroller.diff (9.83 KB, patch)
2006-09-20 21:10 EDT, Darryl Miles CLA
no flags Details | Diff
org.eclipse.tptp.platform.agentcontroller/src-native/collection/include.linux_em64t New File (1.98 KB, text/plain)
2006-09-20 21:18 EDT, Darryl Miles CLA
no flags Details
org.eclipse.hyades.execution.linux.x86_64.zip Project (3.41 KB, application/octet-stream)
2006-09-20 22:00 EDT, Darryl Miles CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Darryl Miles CLA 2006-09-15 07:05:18 EDT
platform/org.eclipse.hyades.execution/src-iac-native

This patch includes some generic fixes (missing system header file inclusion causing warnings) and also a new makefile for x86_64 support.  It is also advised that the "-fPIC" option be used with GCC when creating shared libraries.


I notice at the bottom of makefile.linux.x86 there is the line:

 zip iac.linux_ia32.zip libtptpCommon.so libtptpNP.so

I have changed this to:

 zip iac.linux_x86.zip libtptpCommon.so libtptpNP.so

There term "ia32" refers to the classical 8086/80386 archetecture cpus and is too generic in the context its used since both the 32bit (x86) and 64bit (x86_64) are both ia32 platform cpus.  The "ia64" is the Intel Itanium cpu which is a completely different instruction set.

The Native system and the OSGi use the terms "x86" and "x86_64" so I would propose that the zip filename be changed from "ia32".

Changing this name of the ZIP probably has knock on effect for the build process so other changes might be necessary for releng.


I have also done a test compile of the tptpShm module and applied the obvious fixes for Linux but this code has obvious coding errors in tptpShm/tptpShm_md.c and given that its not in any makefile it maybe obsolete or incomplete, but accept this patch as assitance in getting it working for Linux.


I intend to continue porting each of the DDL/DSO libraries listed in the MANIFEST for platform/org.eclipse.hyades.execution.linux.x86/ so that a new project platform/org.eclipse.hyades.execution.linux.x86_64/ can be created.
Comment 1 Darryl Miles CLA 2006-09-15 07:07:10 EDT
Created attachment 50242 [details]
org.eclipse.hyades.execution_20060915.diff

Linux compile improvements and x86_64 support.
Comment 2 Darryl Miles CLA 2006-09-15 07:08:36 EDT
Created attachment 50243 [details]
platform/org.eclipse.hyades.execution/src-iac-native/makefile.linux.x86_64

New file platform/org.eclipse.hyades.execution/src-iac-native/makefile.linux.x86_64
Comment 3 Darryl Miles CLA 2006-09-15 08:54:02 EDT
Created attachment 50249 [details]
org.eclipse.hyades.execution.linux.x86_64.zip Project

Basic project based on the existing 32bit Linux project.

As I have been able to build 64bit version of the native pre-requisites from parts of the agentcontroller and haydes.execution we can't be that far out from having a 64bit version.


I have removed the MANIFEST entry:

Bundle-NativeCode: libtptpShm.so;processor="x86_64";osname="Linux",


This is because is does not look like it is built or needed anymore.  I don't know what impact that has, I am looking from the point of view from CVS HEAD.   Maybe the current 32bit Linux packages wants investigation.
Comment 4 Darryl Miles CLA 2006-09-15 09:21:17 EDT
Created attachment 50252 [details]
org.eclipse.tptp.platform.agentcontroller.diff

THIS PATCH NEEDS MAJOR REVIEW, ESPECIALLY THE CHANGES IN StatelessHeapSnapshotManager_C.cpp.


StatelessHeapSnapshotManager_C.cpp: allow the component to compile but almost certainly create new bugs.

RADataTransfer.h missing system header file to reduce compile time warnings.

RAServer/messagePump.c when using GCC 4.0.2 the existing assignment would not compile but using the same assignment as _HPUX/MVS/OS400 was found to work.

RAServer/messagePump.c when using GCC 4.0.2 the compiler now generates an error for labels after blocks.  This is appearently a compiler extension, one workaround is to use break statements in loops.

collection/include.common changes to correcty detect 64bit linux.

collection/include.linux_x86_64 this is a new file to allow the 64bit build to take place.
Comment 5 Darryl Miles CLA 2006-09-20 20:52:18 EDT
Created attachment 50616 [details]
org.eclipse.hyades.execution.linux.x86_64.zip Project

Corrected initial project which includes:

Bundle-NativeCode: libtptpShm.so;processor="x86_64";osname="Linux"


I have sucessfully built this DSO from the project:

org.eclipse.tptp.platform.agentcontroller/collection/libtptpShm.so
Comment 6 Darryl Miles CLA 2006-09-20 21:10:55 EDT
Created attachment 50617 [details]
org.eclipse.tptp.platform.agentcontroller.diff

Updated version based on CVS checkout.  See Comment #4 for most notes.

collection/include.linux_em64t file not included, will be attached separatly.

StatelessHeapSnapshotManager_C.cpp: This still needs major review.

I have sucesfully built a set of ZIPs:

libtptpShm.linux_em64t.zip
tptpdc.linux_em64t.zip
tptpdc.linux_em64t.sdk.zip
pae.linux_em64t.zip

But not yet able to test as I have not found any documentation explaining how to recreate your releng build process to merge java files and other artifacts into a distributable zip.  The "make package" gives me have a partial distribution.
Comment 7 Darryl Miles CLA 2006-09-20 21:18:48 EDT
Created attachment 50618 [details]
org.eclipse.tptp.platform.agentcontroller/src-native/collection/include.linux_em64t New File

New file to go with attachment #50617 [details].
Comment 8 Darryl Miles CLA 2006-09-20 22:00:29 EDT
Created attachment 50619 [details]
org.eclipse.hyades.execution.linux.x86_64.zip Project

Duh! Third time lucky.

This one is tested by manually building the plugin, Bundle-NativeCode processor  works with amd64 (as per the Sun JVM processor label, tested with Sun JVM 1.5.0_08)

org.eclipse.hyades.execution.linux.x86_64 (4.2.0.qualifier) "TPTP Platform Execution Plug-In" [Resolved]
Comment 9 Guru Nagarajan CLA 2006-10-06 19:51:52 EDT
Samson - assigning it to you since this pertains to IAC.
Comment 10 Samson Wai CLA 2006-10-27 10:42:30 EDT
Hi Darryl. This bug is currently not PMC/PG approved for TPTP 4.3 due to its "normal" severity. By default all non-critical bugs will be retargeted to TPTP 4.4 or future releases. Do you have any issue with this not being fixed in TPTP 4.3? I would say due to the changes required for the build this would be better addressed in TPTP 4.4 instead.
Comment 11 Darryl Miles CLA 2006-10-27 10:54:22 EDT
I didn't expect it to make 4.3.  I'm happy for this to make an early weekly "towards 4.4" build.  I seek to help provide feedback and bug reports for 64bit linux unfortunely TPTP isn't quite mature enough to build on top on yet, so this is my gesture to help 64bit linux support along.  The sooner it gets committed into a weekly the sooner I look to see what else needs attention.
Comment 12 Samson Wai CLA 2006-10-27 10:59:11 EDT
Thanks Darryl. I will target this to 4.4 and increase the severity to "major" so that it can be picked up during the next release and catch attention.
Comment 13 Samson Wai CLA 2007-01-31 12:10:34 EST
Retarget to future due to resource limitation.
Comment 14 Samson Wai CLA 2007-11-27 09:33:12 EST
Hi Bing. I have transferred my bugs to you for triage. Thanks.
Comment 15 Darryl Miles CLA 2007-11-27 10:07:58 EST
It feels like musical chairs in here!
Open Source by Committee of Corporations.
Is there a list of committers on Eclipse projects who can actually get stuff done, the faster stuff gets done by the committers the faster contributions can flow into the product ?
Comment 16 Paul Slauenwhite CLA 2009-06-30 06:54:28 EDT
As of TPTP 4.6.0, TPTP is in maintenance mode and focusing on improving quality by resolving relevant defects and increasing test coverage through test creation, automation, Build Verification Tests (BVTs), and expanded run-time execution. Since this defect is more than 2 years old, it may be no longer relevant. As part of the TPTP Bugzilla housecleaning process (see http://wiki.eclipse.org/Bugzilla_Housecleaning_Processes), this defect is resolved as WONTFIX. If this defect is still relevant and reproducible in the latest TPTP release (http://www.eclipse.org/tptp/home/downloads/), please re-open.
Comment 17 Kathy Chan CLA 2010-11-18 18:56:11 EST
As of TPTP 4.6.0, TPTP is in maintenance mode and focusing on improving quality by resolving relevant enhancements/defects and increasing test coverage through test creation, automation, Build Verification Tests (BVTs), and expanded run-time execution. As part of the TPTP Bugzilla housecleaning process (see http://wiki.eclipse.org/Bugzilla_Housecleaning_Processes), this enhancement/defect is verified/closed by the Project Lead since this enhancement/defect has been resolved and unverified for more than 1 year and considered to be fixed. If this enhancement/defect is still unresolved and reproducible in the latest TPTP release (http://www.eclipse.org/tptp/home/downloads/), please re-open.