X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/40d3634e5448cd560a6989d07d10ec30e85cf533..5a4737df2315f02410f71bc65eb67fad0d87b351:/src/bindings/java/org/simgrid/msg/Process.java diff --git a/src/bindings/java/org/simgrid/msg/Process.java b/src/bindings/java/org/simgrid/msg/Process.java index d9b3b6e87e..fc4d205b55 100644 --- a/src/bindings/java/org/simgrid/msg/Process.java +++ b/src/bindings/java/org/simgrid/msg/Process.java @@ -1,5 +1,4 @@ -/* Copyright (c) 2006-2014. The SimGrid Team. - * All rights reserved. */ +/* Copyright (c) 2006-2018. The SimGrid Team. All rights reserved. */ /* This program is free software; you can redistribute it and/or modify it * under the terms of the license (GNU LGPL) which comes with this package. */ @@ -46,54 +45,28 @@ public abstract class Process implements Runnable { * a native process. Even if this attribute is public you must never * access to it. It is set automatically during the build of the object. */ - private long bind; + private long bind = 0; /** Indicates if the process is started */ - boolean started; - /** - * Even if this attribute is public you must never access to it. - * It is used to compute the id of an MSG process. - */ - private static long nextProcessId = 0; - - /** - * Even if this attribute is public you must never access to it. - * It is compute automatically during the creation of the object. - * The native functions use this identifier to synchronize the process. - */ - private long id; /** Time at which the process should be created */ protected double startTime = 0; - /** Time at which the process should be killed. - * - * Set at creation, and used internally by SimGrid - */ - private double killTime = -1; + /** Time at which the process should be killed */ + private double killTime = -1; // Used from the C world - private String name; + private String name = null; private int pid = -1; private int ppid = -1; private Host host = null; - /** The arguments of the method function of the process. */ - public ArrayList args; - - - /** Default constructor */ - protected Process() { - this.id = nextProcessId++; - this.name = null; - this.bind = 0; - this.args = new ArrayList<>(); - } - + /** The arguments of the method function of the process. */ + private ArrayList args_ = new ArrayList<>(); /** * Constructs a new process from the name of a host and his name. The method * function of the process doesn't have argument. * - * @param hostname The name of the host of the process to create. + * @param hostname Where to create the process. * @param name The name of the process. * * @exception HostNotFoundException if no host with this name exists. @@ -107,23 +80,21 @@ public abstract class Process implements Runnable { * Constructs a new process from the name of a host and his name. The arguments * of the method function of the process are specified by the parameter args. * - * @param hostname The name of the host of the process to create. + * @param hostname Where to create the process. * @param name The name of the process. * @param args The arguments of the main function of the process. * * @exception HostNotFoundException if no host with this name exists. - * NativeException - * @throws NativeException * */ - public Process(String hostname, String name, String[] args) throws HostNotFoundException, NativeException { + public Process(String hostname, String name, String[] args) throws HostNotFoundException { this(Host.getByName(hostname), name, args); } /** * Constructs a new process from a host and his name. The method function of the * process doesn't have argument. * - * @param host The host of the process to create. + * @param host Where to create the process. * @param name The name of the process. * */ @@ -134,28 +105,29 @@ public abstract class Process implements Runnable { * Constructs a new process from a host and his name, the arguments of here method function are * specified by the parameter args. * - * @param host The host of the process to create. + * @param host Where to create the process. * @param name The name of the process. * @param args The arguments of main method of the process. */ - public Process(Host host, String name, String[]args) { - this(); - this.host = host; + public Process(Host host, String name, String[]args) + { if (host == null) - throw new NullPointerException("Host cannot be NULL"); + throw new IllegalArgumentException("Cannot create a process on the null host"); if (name == null) - throw new NullPointerException("Process name cannot be NULL"); + throw new IllegalArgumentException("Process name cannot be null"); + + this.host = host; this.name = name; - this.args = new ArrayList<>(); + this.args_ = new ArrayList<>(); if (null != args) - this.args.addAll(Arrays.asList(args)); + this.args_.addAll(Arrays.asList(args)); } /** * Constructs a new process from a host and his name, the arguments of here method function are * specified by the parameter args. * - * @param host The host of the process to create. + * @param host Where to create the process. * @param name The name of the process. * @param args The arguments of main method of the process. * @param startTime Start time of the process @@ -163,26 +135,16 @@ public abstract class Process implements Runnable { * */ public Process(Host host, String name, String[]args, double startTime, double killTime) { - this(); - this.host = host; - if (host == null) - throw new NullPointerException("Process name cannot be NULL"); - if (name == null) - throw new NullPointerException("Process name cannot be NULL"); - this.name = name; - - this.args = new ArrayList<>(); - if (null != args) - this.args.addAll(Arrays.asList(args)); - + this(host, name, args); this.startTime = startTime; this.killTime = killTime; } /** - * The natively implemented method to create an MSG process. - * @param hostName A valid (bound) host where create the process. + * The native method to create an MSG process. + * @param host where to create the process. */ - protected native void create(String hostName) throws HostNotFoundException; + protected native void create(Host host); + /** * This method kills all running process of the simulation. * @@ -200,6 +162,9 @@ public abstract class Process implements Runnable { * SimGrid sometimes have issues when you kill processes that are currently communicating and such. We are working on it to fix the issues. */ public native void kill(); + public static void kill(Process p) { + p.kill(); + } /** Suspends the process. See {@link #resume()} to resume it afterward */ public native void suspend(); @@ -212,6 +177,9 @@ public abstract class Process implements Runnable { */ public native boolean isSuspended(); + /** Yield the current process. All other processes that are ready at the same timestamp will be executed first */ + public static native void yield(); + /** * Specify whether the process should restart when its host restarts after a failure * @@ -237,13 +205,11 @@ public abstract class Process implements Runnable { /** * This static method gets a process from a PID. * - * @param PID The process identifier of the process to get. + * @param pid The process identifier of the process to get. * * @return The process with the specified PID. - * - * @exception NativeException on error in the native SimGrid code */ - public static native Process fromPID(int PID) throws NativeException; + public static native Process fromPID(int pid); /** * This method returns the PID of the process. * @@ -251,8 +217,13 @@ public abstract class Process implements Runnable { * */ public int getPID() { + if (pid == -1) // Don't traverse the JNI barrier if you already have the answer + pid = nativeGetPID(); return pid; } + // This should not be used: the PID is supposed to be initialized from the C directly when the actor is created, + // but this sometimes fail, so let's play nasty but safe here. + private native int nativeGetPID(); /** * This method returns the PID of the parent of a process. * @@ -288,7 +259,7 @@ public abstract class Process implements Runnable { */ public native void migrate(Host host); /** - * Makes the current process sleep until millis millisecondes have elapsed. + * Makes the current process sleep until millis milliseconds have elapsed. * You should note that unlike "waitFor" which takes seconds, this method takes milliseconds. * FIXME: Not optimal, maybe we should have two native functions. * @param millis the length of time to sleep in milliseconds. @@ -303,7 +274,7 @@ public abstract class Process implements Runnable { * milliseconds and nanoseconds. * Overloads Thread.sleep. * @param millis the length of time to sleep in milliseconds. - * @param nanos additionnal nanoseconds to sleep. + * @param nanos additional nanoseconds to sleep. */ public static native void sleep(long millis, int nanos) throws HostFailureException; /** @@ -314,47 +285,48 @@ public abstract class Process implements Runnable { /** * This method actually creates and run the process. * It is a noop if the process is already launched. - * @throws HostNotFoundException */ - public final void start() throws HostNotFoundException { - if (!started) { - started = true; - create(host.getName()); - } + public final void start() { + if (bind == 0) + create(host); } - /** This method runs the process. Il calls the method function that you must overwrite. */ + /** This method runs the process. It calls the method function that you must overwrite. */ @Override public void run() { String[] args = null; /* do not fill it before the signal or this.args will be empty */ try { - args = new String[this.args.size()]; - if (!this.args.isEmpty()) { - this.args.toArray(args); + args = new String[this.args_.size()]; + if (!this.args_.isEmpty()) { + this.args_.toArray(args); } this.main(args); - } catch(MsgException e) { + } + catch(MsgException e) { e.printStackTrace(); Msg.info("Unexpected behavior. Stopping now"); System.exit(1); } catch(ProcessKilledError pk) { + /* The process was killed before its end. With a kill() or something. */ } - exit(); } /** - * The main function of the process (to implement). + * The main function of the process (to implement by the user). * * @param args * @throws MsgException */ public abstract void main(String[]args) throws MsgException; - public native void exit(); + /** Stops the execution of the current actor */ + public void exit() { + this.kill(); + } /** * Class initializer, to initialize various JNI stuff */ @@ -368,6 +340,6 @@ public abstract class Process implements Runnable { * * @return The count of the running processes */ - public native static int getCount(); + public static native int getCount(); }