X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/1bd1bbd35034c2e93a030a676dc244ad2cf74c70..4ee3c75a375b31887676236fd5555c39e15a671a:/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 e4c7900eba..cfbc5dabef 100644 --- a/src/bindings/java/org/simgrid/msg/Process.java +++ b/src/bindings/java/org/simgrid/msg/Process.java @@ -1,28 +1,28 @@ -/* Copyright (c) 2006-2017. The SimGrid Team. All rights reserved. */ +/* Copyright (c) 2006-2019. 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. */ package org.simgrid.msg; -import java.util.Arrays; import java.util.ArrayList; +import java.util.Arrays; /** - * A process may be defined as a code, with some private data, executing + * A process may be defined as a code, with some private data, executing * in a location (host). All the process used by your simulation must be * declared in the deployment file (XML format). - * To create your own process you must inherit your own process from this - * class and override the method "main()". For example if you want to use + * To create your own process you must inherit your own process from this + * class and override the method "main()". For example if you want to use * a process named Slave proceed as it : * * (1) import the class Process of the package simgrid.msg * import simgrid.msg.Process; - * + * * public class Slave extends simgrid.msg.Process { * * (2) Override the method function - * + * * \verbatim * public void main(String[] args) { * System.out.println("Hello MSG"); @@ -33,10 +33,10 @@ import java.util.ArrayList; * For the example, for the previous process Slave this file must contains a line : * <process host="Maxims" function="Slave"/>, where Maxims is the host of the process * Slave. All the process of your simulation are automatically launched and managed by Msg. - * A process use tasks to simulate communications or computations with another process. - * For more information see Task. For more information on host concept + * A process use tasks to simulate communications or computations with another process. + * For more information see Task. For more information on host concept * see Host. - * + * */ public abstract class Process implements Runnable { @@ -70,7 +70,7 @@ public abstract class Process implements Runnable { * @param name The name of the process. * * @exception HostNotFoundException if no host with this name exists. - * + * * */ public Process(String hostname, String name) throws HostNotFoundException { @@ -86,12 +86,12 @@ public abstract class Process implements Runnable { * * @exception HostNotFoundException if no host with this name exists. * - */ + */ 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 + * Constructs a new process from a host and his name. The method function of the * process doesn't have argument. * * @param host Where to create the process. @@ -107,9 +107,9 @@ public abstract class Process implements Runnable { * * @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 argsParam The arguments of main method of the process. */ - public Process(Host host, String name, String[]args) + public Process(Host host, String name, String[]argsParam) { if (host == null) throw new IllegalArgumentException("Cannot create a process on the null host"); @@ -120,8 +120,8 @@ public abstract class Process implements Runnable { this.name = name; this.args = new ArrayList<>(); - if (null != args) - this.args.addAll(Arrays.asList(args)); + if (null != argsParam) + this.args.addAll(Arrays.asList(argsParam)); } /** * Constructs a new process from a host and his name, the arguments of here method function are @@ -147,15 +147,8 @@ public abstract class Process implements Runnable { /** * This method kills all running process of the simulation. - * - * @param resetPID Should we reset the PID numbers. A negative number means no reset - * and a positive number will be used to set the PID of the next newly - * created process. - * - * @return The function returns the PID of the next created process. - * - */ - public static native int killAll(int resetPID); + */ + public static native void killAll(); /** Simply kills the receiving process. * @@ -182,7 +175,7 @@ public abstract class Process implements Runnable { /** * Specify whether the process should restart when its host restarts after a failure - * + * * A process naturally stops when its host stops. It starts again only if autoRestart is set to true. * Otherwise, it just disappears when the host stops. */ @@ -198,7 +191,7 @@ public abstract class Process implements Runnable { /** * Returns the host of the process. * @return The host instance of the process. - */ + */ public Host getHost() { return this.host; } @@ -208,28 +201,33 @@ public abstract class Process implements Runnable { * @param pid The process identifier of the process to get. * * @return The process with the specified PID. - */ + */ public static native Process fromPID(int pid); /** * This method returns the PID of the process. * * @return The PID of the process. * - */ + */ 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. * * @return The PID of the parent of the process. * - */ + */ public int getPPID() { return ppid; } /** - * Returns the value of a given process property. + * Returns the value of a given process property. */ public native String getProperty(String name); @@ -244,7 +242,7 @@ public abstract class Process implements Runnable { * * @return The current process. * - */ + */ public static native Process getCurrentProcess(); /** * Migrates a process to another host. @@ -255,17 +253,17 @@ public abstract class Process implements Runnable { public native void migrate(Host host); /** * 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. + * You should note that unlike "waitFor" which takes seconds (as usual in SimGrid), this method takes milliseconds (as usual for sleep() in Java). + * * @param millis the length of time to sleep in milliseconds. */ public static void sleep(long millis) throws HostFailureException { sleep(millis,0); } /** - * Makes the current process sleep until millis milliseconds and nanos nanoseconds + * Makes the current process sleep until millis milliseconds and nanos nanoseconds * have elapsed. - * Unlike {@link #waitFor(double)} which takes seconds, this method takes + * Unlike {@link #waitFor(double)} which takes seconds, this method takes * milliseconds and nanoseconds. * Overloads Thread.sleep. * @param millis the length of time to sleep in milliseconds. @@ -275,8 +273,8 @@ public abstract class Process implements Runnable { /** * Makes the current process sleep until time seconds have elapsed. * @param seconds The time the current process must sleep. - */ - public native void waitFor(double seconds) throws HostFailureException; + */ + public native void waitFor(double seconds) throws HostFailureException; /** * This method actually creates and run the process. * It is a noop if the process is already launched. @@ -290,24 +288,18 @@ public abstract class Process implements Runnable { @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); - } + String[] argsArray = new String[this.args.size()]; + this.args.toArray(argsArray); - this.main(args); + this.main(argsArray); } 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. */ - } + /* Let the ProcessKilledError (that we'd get if the process is forcefully killed) flow back to the caller */ } /** @@ -334,7 +326,13 @@ public abstract class Process implements Runnable { * This static method returns the current amount of processes running * * @return The count of the running processes - */ + */ public static native int getCount(); + public static void debugAllThreads() { + // Search remaining threads that are not main nor daemon + for (Thread t : Thread.getAllStackTraces().keySet()) + if (! t.isDaemon() && !t.getName().equals("main")) + System.err.println("Thread "+t.getName()+" is still running! Please report that bug"); + } }