Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
convert maestro-set to S4U
authorFrederic Suter <frederic.suter@cc.in2p3.fr>
Fri, 22 Feb 2019 18:13:15 +0000 (19:13 +0100)
committerFrederic Suter <frederic.suter@cc.in2p3.fr>
Fri, 22 Feb 2019 18:15:27 +0000 (19:15 +0100)
.gitignore
examples/deprecated/msg/CMakeLists.txt
examples/s4u/CMakeLists.txt
examples/s4u/maestro-set/s4u-maestro-set.cpp [moved from examples/deprecated/msg/maestro-set/maestro-set.cpp with 62% similarity]
examples/s4u/maestro-set/s4u-maestro-set.tesh [moved from examples/deprecated/msg/maestro-set/maestro-set.tesh with 76% similarity]

index 3009272..59e3d9a 100644 (file)
@@ -123,7 +123,6 @@ examples/deprecated/msg/cloud-masterworker/cloud-masterworker
 examples/deprecated/msg/dht-kademlia/dht-kademlia
 examples/deprecated/msg/dht-pastry/dht-pastry
 examples/deprecated/msg/energy-vm/energy-vm
-examples/deprecated/msg/maestro-set/maestro-set
 examples/deprecated/msg/mc/bugged1
 examples/deprecated/msg/mc/bugged1_liveness
 examples/deprecated/msg/mc/bugged2
@@ -184,6 +183,7 @@ examples/s4u/io-async/s4u-io-async
 examples/s4u/io-file-remote/s4u-io-file-remote
 examples/s4u/io-file-system/s4u-io-file-system
 examples/s4u/io-storage-raw/s4u-io-storage-raw
+examples/s4u/maestro-set/s4u-maestro-set
 examples/s4u/platform-failures/s4u-platform-failures
 examples/s4u/platform-properties/s4u-platform-properties
 examples/s4u/plugin-hostload/s4u-plugin-hostload
index b8b6bef..0e62567 100644 (file)
@@ -13,16 +13,6 @@ foreach(x app-masterworker cloud-masterworker
   set(tesh_files   ${tesh_files}   ${CMAKE_CURRENT_SOURCE_DIR}/${x}/${x}.tesh)
 endforeach()
 
-# CPP examples
-foreach(x maestro-set)
-  add_executable       (${x}  EXCLUDE_FROM_ALL ${x}/${x}.cpp)
-  target_link_libraries(${x}     simgrid)
-  set_target_properties(${x}  PROPERTIES RUNTIME_OUTPUT_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/${x})
-  add_dependencies(tests ${x})
-  set(examples_src ${examples_src} ${CMAKE_CURRENT_SOURCE_DIR}/${x}/${x}.cpp)
-  set(tesh_files   ${tesh_files}   ${CMAKE_CURRENT_SOURCE_DIR}/${x}/${x}.tesh)
-endforeach()
-
 if(SIMGRID_HAVE_NS3)
   add_executable       (network-ns3 EXCLUDE_FROM_ALL network-ns3/network-ns3.c)
   target_link_libraries(network-ns3 simgrid)
@@ -87,12 +77,6 @@ ADD_TESH_FACTORIES(msg-dht-kademlia-parallel "thread;ucontext;raw;boost" --cfg c
                                              --setenv platfdir=${CMAKE_HOME_DIRECTORY}/examples/platforms
                                             ${CMAKE_HOME_DIRECTORY}/examples/deprecated/msg/dht-kademlia/dht-kademlia.tesh)
 
-if(NOT WIN32)
-  ADD_TESH_FACTORIES(msg-maestro-set   "thread" --setenv srcdir=${CMAKE_HOME_DIRECTORY}/examples/deprecated/msg --cd ${CMAKE_BINARY_DIR}/examples/deprecated/msg ${CMAKE_HOME_DIRECTORY}/examples/deprecated/msg/maestro-set/maestro-set.tesh)
-else()
-  message("Test maestro-set disabled on windows: this feature is not working")
-endif()
-
 ADD_TESH_FACTORIES(app-masterworker-multicore   "thread;ucontext;raw;boost" 
                                                 --setenv srcdir=${CMAKE_HOME_DIRECTORY}/examples/deprecated/msg/app-masterworker 
                                                 --setenv bindir=${CMAKE_BINARY_DIR}/examples/deprecated/msg/app-masterworker
index 6327f60..3942d21 100644 (file)
@@ -110,6 +110,26 @@ if (NOT enable_memcheck AND NOT WIN32)
                             ${CMAKE_CURRENT_SOURCE_DIR}/app-pingpong/simix-breakpoint.tesh)
 endif()
 
+# Examples accepting only thread factories
+##################################
+foreach (example maestro-set)
+  add_executable       (s4u-${example} EXCLUDE_FROM_ALL ${example}/s4u-${example}.cpp)
+  target_link_libraries(s4u-${example} simgrid)
+  set_target_properties(s4u-${example} PROPERTIES RUNTIME_OUTPUT_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/${example})
+  add_dependencies(tests s4u-${example})
+
+  set(tesh_files    ${tesh_files}    ${CMAKE_CURRENT_SOURCE_DIR}/${example}/s4u-${example}.tesh)
+  set(examples_src  ${examples_src}  ${CMAKE_CURRENT_SOURCE_DIR}/${example}/s4u-${example}.cpp)
+
+  if(NOT WIN32)
+    ADD_TESH_FACTORIES(s4u-${example} "thread" --setenv bindir=${CMAKE_CURRENT_BINARY_DIR}/${example}
+                                               --setenv platfdir=${CMAKE_HOME_DIRECTORY}/examples/platforms 
+                                               ${CMAKE_HOME_DIRECTORY}/examples/s4u/${example}/s4u-${example}.tesh)
+  else()
+    message("Test maestro-set disabled on windows: this feature is not working")
+  endif()
+endforeach()
+
 # Add all extra files to the archive
 ####################################
 
@@ -3,7 +3,7 @@
 /* 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. */
 
-/** @addtogroup MSG_examples
+/** @addtogroup S4U_examples
  *
  *  - <b>maestro-set/maestro-set.cpp: Switch the system thread hosting our maestro</b>.
  *    That's a very advanced example in which we move the maestro thread to another process.
  *    expected. You can still use that feature from a C code.
  */
 
-#include "simgrid/msg.h"
+#include "simgrid/Exception.hpp"
+#include "simgrid/actor.h"
+#include "simgrid/s4u.hpp"
 
+#include <string>
 #include <thread>
 
-XBT_LOG_NEW_DEFAULT_CATEGORY(msg_test, "Messages specific for this msg example");
+XBT_LOG_NEW_DEFAULT_CATEGORY(s4u_test, "Messages specific for this s4u example");
 
 std::thread::id root_id;
 
-static void ensure_root_tid() {
+static void ensure_root_tid()
+{
   std::thread::id this_id = std::this_thread::get_id();
   xbt_assert(root_id == this_id, "I was supposed to be the main thread");
   XBT_INFO("I am the main thread, as expected");
 }
-static void ensure_other_tid() {
+static void ensure_other_tid()
+{
   std::thread::id this_id = std::this_thread::get_id();
   xbt_assert(this_id != root_id, "I was NOT supposed to be the main thread");
   XBT_INFO("I am not the main thread, as expected");
 }
 
-
-
-static int sender(int argc, char *argv[])
+static void sender()
 {
   ensure_root_tid();
-
-  msg_task_t task_la = MSG_task_create("Some task", 0.0, 10e8, NULL);
-  MSG_task_send(task_la, "some mailbox");
-
-  return 0;
+  std::string* payload = new std::string("some message");
+  simgrid::s4u::Mailbox::by_name("some mailbox")->put((void*)payload, 10e8);
 }
 
-static int receiver(int argc, char *argv[])
+static void receiver()
 {
   ensure_other_tid();
 
-  msg_task_t task_la = NULL;
-  xbt_assert(MSG_task_receive(&task_la,"some mailbox") == MSG_OK);
+  std::string* payload = static_cast<std::string*>(simgrid::s4u::Mailbox::by_name("some mailbox")->get());
   XBT_INFO("Task received");
-  MSG_task_destroy(task_la);
-
-  return 0;
+  delete payload;
 }
 
 static void maestro(void* data)
 {
   ensure_other_tid();
-  MSG_process_create("receiver",&receiver,NULL,MSG_host_by_name("Jupiter"));
-  MSG_main();
+  simgrid::s4u::Actor::create("receiver", simgrid::s4u::Host::by_name("Jupiter"), receiver);
+  simgrid::s4u::Engine::get_instance()->run();
 }
 
 /** Main function */
-int main(int argc, char *argv[])
+int main(int argc, charargv[])
 {
   root_id = std::this_thread::get_id();
 
   SIMIX_set_maestro(maestro, NULL);
-  MSG_init(&argc, argv);
+  simgrid::s4u::Engine e(&argc, argv);
 
   if (argc != 2) {
     XBT_CRITICAL("Usage: %s platform_file\n", argv[0]);
-    xbt_die("example: %s msg_platform.xml\n",argv[0]);
+    xbt_die("example: %s msg_platform.xml\n", argv[0]);
   }
 
-  MSG_create_environment(argv[1]);
+  e.load_platform(argv[1]);
 
   /* Become one of the simulated process.
    *
    * This must be done after the creation of the platform because we are depending attaching to a host.*/
-  MSG_process_attach("sender", NULL, MSG_host_by_name("Tremblay"), NULL);
+  sg_actor_attach("sender", nullptr, simgrid::s4u::Host::by_name("Tremblay"), nullptr);
   ensure_root_tid();
 
   // Execute the sender code:
-  const char* subargv[3] = { "sender", "Jupiter", NULL };
-  sender(2, (char**) subargv);
+  sender();
 
-  MSG_process_detach(); // Become root thread again
+  sg_actor_detach(); // Become root thread again
   XBT_INFO("Detached");
   ensure_root_tid();
 
@@ -1,6 +1,6 @@
 #!/usr/bin/env tesh
 
-$ $SG_TEST_EXENV maestro-set/maestro-set$EXEEXT ${srcdir:=.}/../../platforms/small_platform.xml "--log=root.fmt:[%10.6r]%e(%i:%P@%h)%e%m%n"
+$ $SG_TEST_EXENV ${bindir:=.}/s4u-maestro-set$EXEEXT ${platfdir:=.}/small_platform.xml "--log=root.fmt:[%10.6r]%e(%i:%P@%h)%e%m%n"
 > [  0.000000] (0:maestro@) I am not the main thread, as expected
 > [  0.000000] (2:receiver@Jupiter) I am not the main thread, as expected
 > [  0.000000] (1:sender@Tremblay) I am the main thread, as expected