Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
mc: don't catch exceptions we cannot deal with
[simgrid.git] / src / mc / Session.cpp
index 63c0c74..8c0926e 100644 (file)
@@ -1,19 +1,20 @@
-/* Copyright (c) 2015-2016. The SimGrid Team.
- * All rights reserved.                                                     */
+/* Copyright (c) 2015-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. */
 
-#include <fcntl.h>
-#include <signal.h>
-
-#include <functional>
-
-#include <xbt/system_error.hpp>
-#include <simgrid/sg_config.h>
-#include <simgrid/modelchecker.h>
-
 #include "src/mc/Session.hpp"
+#include "src/mc/checker/Checker.hpp"
+#include "src/mc/mc_config.hpp"
+#include "src/mc/mc_private.hpp"
+#include "src/mc/mc_state.hpp"
+#include "xbt/log.h"
+#include "xbt/system_error.hpp"
+
+#include <fcntl.h>
+#ifdef __linux__
+#include <sys/prctl.h>
+#endif
 
 XBT_LOG_NEW_DEFAULT_SUBCATEGORY(mc_Session, mc, "Model-checker session");
 
@@ -26,18 +27,14 @@ static void setup_child_environment(int socket)
   // Make sure we do not outlive our parent:
   sigset_t mask;
   sigemptyset (&mask);
-  if (sigprocmask(SIG_SETMASK, &mask, nullptr) < 0)
-    throw simgrid::xbt::errno_error(errno, "Could not unblock signals");
-  if (prctl(PR_SET_PDEATHSIG, SIGHUP) != 0)
-    throw simgrid::xbt::errno_error(errno, "Could not PR_SET_PDEATHSIG");
+  xbt_assert(sigprocmask(SIG_SETMASK, &mask, nullptr) >= 0, "Could not unblock signals");
+  xbt_assert(prctl(PR_SET_PDEATHSIG, SIGHUP) == 0, "Could not PR_SET_PDEATHSIG");
 #endif
 
-  int res;
-
   // Remove CLOEXEC in order to pass the socket to the exec-ed program:
   int fdflags = fcntl(socket, F_GETFD, 0);
-  if (fdflags == -1 || fcntl(socket, F_SETFD, fdflags & ~FD_CLOEXEC) == -1)
-    throw simgrid::xbt::errno_error(errno, "Could not remove CLOEXEC for socket");
+  xbt_assert(fdflags != -1 && fcntl(socket, F_SETFD, fdflags & ~FD_CLOEXEC) != -1,
+             "Could not remove CLOEXEC for socket");
 
   // Set environment:
   setenv(MC_ENV_VARIABLE, "1", 1);
@@ -48,9 +45,8 @@ static void setup_child_environment(int socket)
   setenv("LC_BIND_NOW", "1", 1);
 
   char buffer[64];
-  res = std::snprintf(buffer, sizeof(buffer), "%i", socket);
-  if ((size_t) res >= sizeof(buffer) || res == -1)
-    std::abort();
+  int res = std::snprintf(buffer, sizeof(buffer), "%i", socket);
+  xbt_assert((size_t)res < sizeof(buffer) && res != -1);
   setenv(MC_ENV_SOCKET_FD, buffer, 1);
 }
 
@@ -60,53 +56,32 @@ static inline
 pid_t do_fork(F code)
 {
   pid_t pid = fork();
-  if (pid < 0)
-    throw simgrid::xbt::errno_error(errno, "Could not fork model-checked process");
+  xbt_assert(pid >= 0, "Could not fork model-checked process");
   if (pid != 0)
     return pid;
 
   // Child-process:
-  try {
-    code();
-    _exit(EXIT_SUCCESS);
-  }
-  catch(...) {
-    // The callback should catch exceptions:
-    std::terminate();
-  }
-}
-
-Session::Session(pid_t pid, int socket)
-{
-  std::unique_ptr<simgrid::mc::Process> process(new simgrid::mc::Process(pid, socket));
-  // TODO, automatic detection of the config from the process
-  process->privatized(
-    xbt_cfg_get_boolean("smpi/privatize_global_variables"));
-  modelChecker_ = std::unique_ptr<ModelChecker>(
-    new simgrid::mc::ModelChecker(std::move(process)));
-  xbt_assert(mc_model_checker == nullptr);
-  mc_model_checker = modelChecker_.get();
-  mc_model_checker->start();
+  code();
+  _exit(EXIT_SUCCESS);
 }
 
-Session::~Session()
+Session::Session(const std::function<void()>& code)
 {
-  this->close();
-}
+#if HAVE_SMPI
+  xbt_assert(smpi_privatize_global_variables != SmpiPrivStrategies::MMAP,
+             "Please use the dlopen privatization schema when model-checking SMPI code");
+#endif
 
-// static
-Session* Session::fork(std::function<void(void)> code)
-{
   // Create a AF_LOCAL socketpair used for exchanging messages
-  // bewteen the model-checker process (ourselves) and the model-checked
+  // between the model-checker process (ourselves) and the model-checked
   // process:
   int res;
   int sockets[2];
-  res = socketpair(AF_LOCAL, SOCK_DGRAM | SOCK_CLOEXEC, 0, sockets);
+  res = socketpair(AF_LOCAL, SOCK_SEQPACKET | SOCK_CLOEXEC, 0, sockets);
   if (res == -1)
-    throw simgrid::xbt::errno_error(errno, "Could not create socketpair");
+    throw simgrid::xbt::errno_error("Could not create socketpair");
 
-  pid_t pid = do_fork([&] {
+  pid_t pid = do_fork([sockets, &code] {
     ::close(sockets[1]);
     setup_child_environment(sockets[0]);
     code();
@@ -116,30 +91,57 @@ Session* Session::fork(std::function<void(void)> code)
   // Parent (model-checker):
   ::close(sockets[0]);
 
-  return new Session(pid, sockets[1]);
+  std::unique_ptr<simgrid::mc::RemoteClient> process(new simgrid::mc::RemoteClient(pid, sockets[1]));
+  model_checker_.reset(new simgrid::mc::ModelChecker(std::move(process)));
+
+  xbt_assert(mc_model_checker == nullptr);
+  mc_model_checker = model_checker_.get();
+  mc_model_checker->start();
 }
 
-// static
-Session* Session::spawnv(const char *path, char *const argv[])
+Session::~Session()
 {
-  return Session::fork([&] {
-    execv(path, argv);
-  });
+  this->close();
 }
 
-// static
-Session* Session::spawnvp(const char *path, char *const argv[])
+void Session::initialize()
 {
-  return Session::fork([&] {
-    execvp(path, argv);
-  });
+  xbt_assert(initial_snapshot_ == nullptr);
+  mc_model_checker->wait_for_requests();
+  initial_snapshot_ = std::make_shared<simgrid::mc::Snapshot>(0);
+}
+
+void Session::execute(Transition const& transition)
+{
+  model_checker_->handle_simcall(transition);
+  model_checker_->wait_for_requests();
+}
+
+void Session::restore_initial_state()
+{
+  this->initial_snapshot_->restore(&mc_model_checker->process());
+}
+
+void Session::log_state()
+{
+  mc_model_checker->getChecker()->log_state();
+
+  if (not _sg_mc_dot_output_file.get().empty()) {
+    fprintf(dot_output, "}\n");
+    fclose(dot_output);
+  }
+  if (getenv("SIMGRID_MC_SYSTEM_STATISTICS")){
+    int ret=system("free");
+    if(ret!=0)XBT_WARN("system call did not return 0, but %d",ret);
+  }
 }
 
 void Session::close()
 {
-  if (modelChecker_) {
-    modelChecker_->shutdown();
-    modelChecker_ = nullptr;
+  initial_snapshot_ = nullptr;
+  if (model_checker_) {
+    model_checker_->shutdown();
+    model_checker_   = nullptr;
     mc_model_checker = nullptr;
   }
 }