Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
work around issues by passing the hostname instead of the host to processCreate()
authorMartin Quinson <martin.quinson@loria.fr>
Mon, 30 Apr 2012 22:06:22 +0000 (00:06 +0200)
committerMartin Quinson <martin.quinson@loria.fr>
Mon, 30 Apr 2012 22:06:22 +0000 (00:06 +0200)
org/simgrid/msg/ApplicationHandler.java
org/simgrid/msg/MsgNative.java
org/simgrid/msg/Process.java
src/jmsg.c

index 4f0951b..ae57e0f 100644 (file)
@@ -102,9 +102,8 @@ public final class ApplicationHandler {
                                 Process process = cls.newInstance();
                                 process.name = function;
                                 process.id = org.simgrid.msg.Process.nextProcessId++;
-                                Host host = Host.getByName(hostName);
 
-                                MsgNative.processCreate(process, host);
+                                MsgNative.processCreate(process, hostName);
                                 Vector<String> args_ = args;
                                 int size = args_.size();
 
index 3f3df21..3c11df8 100644 (file)
@@ -32,7 +32,7 @@ final class MsgNative {
         * @see  Process constructors.
         */
        final static native
-       void processCreate(Process process, Host host);
+       void processCreate(Process process, String hostName) throws HostNotFoundException;
 
        /**
         * The natively implemented method to kill all the process of the simulation.
index 7289d96..10591d3 100644 (file)
@@ -185,7 +185,11 @@ public abstract class Process extends Thread {
                if (null != args)
                        this.args.addAll(Arrays.asList(args));
 
-               MsgNative.processCreate(this, host);
+               try {
+                       MsgNative.processCreate(this, host.getName());
+               } catch (HostNotFoundException e) {
+                       throw new RuntimeException("The impossible happend (yet again): the host that I have were not found",e);
+               }
                
        }
 
index 0c1bbc2..31ae4d6 100644 (file)
@@ -61,18 +61,21 @@ static jobject native_to_java_process(m_process_t process)
 JNIEXPORT void JNICALL
 Java_org_simgrid_msg_MsgNative_processCreate(JNIEnv * env, jclass cls,
                                          jobject jprocess_arg,
-                                         jobject jhost)
+                                         jobject jhostname)
 {
      
    
   jobject jprocess;             /* the global reference to the java process instance    */
   jstring jname;                /* the name of the java process instance                */
   const char *name;             /* the C name of the process                            */
+  const char *hostname;
   m_process_t process;          /* the native process to create                         */
   m_host_t host;                /* Where that process lives */
    
-  XBT_DEBUG("Java_org_simgrid_msg_MsgNative_processCreate(env=%p,cls=%p,jproc=%p,jhost=%p)",
-        env, cls, jprocess_arg, jhost);
+  hostname = (*env)->GetStringUTFChars(env, jhostname, 0);
+
+  XBT_DEBUG("Java_org_simgrid_msg_MsgNative_processCreate(env=%p,cls=%p,jproc=%p,host=%s)",
+        env, cls, jprocess_arg, hostname);
    
    
   /* get the name of the java process */
@@ -84,10 +87,10 @@ Java_org_simgrid_msg_MsgNative_processCreate(JNIEnv * env, jclass cls,
   }
 
   /* bind/retrieve the msg host */
-  host = jhost_get_native(env, jhost);
+  host = MSG_get_host_by_name(hostname);
 
   if (!(host)) {    /* not binded */
-    jxbt_throw_notbound(env, "host", jhost);
+    jxbt_throw_host_not_found(env, hostname);
     return;
   }
 
@@ -116,6 +119,7 @@ Java_org_simgrid_msg_MsgNative_processCreate(JNIEnv * env, jclass cls,
   /* release our reference to the process name (variable name becomes invalid) */
   //FIXME : This line should be uncommented but with mac it doesn't work. BIG WARNING
   //(*env)->ReleaseStringUTFChars(env, jname, name);
+  (*env)->ReleaseStringUTFChars(env, jhostname, hostname);
    
   /* bind the java process instance to the native process */
   jprocess_bind(jprocess, process, env);