All activities have their own finish method \o/

[simgrid.git] / src / kernel / activity / CommImpl.hpp

diff --git a/src/kernel/activity/CommImpl.hpp b/src/kernel/activity/CommImpl.hpp
index 4f00dc0..715483a 100644 (file)
--- a/src/kernel/activity/CommImpl.hpp
+++ b/src/kernel/activity/CommImpl.hpp
@@ -1,4 +1,4 @@
-/* Copyright (c) 2007-2018. The SimGrid Team. All rights reserved.          */
+/* Copyright (c) 2007-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. */
@@ -7,9 +7,9 @@
 #define SIMGRID_KERNEL_ACTIVITY_COMM_HPP
 
 #include "src/kernel/activity/ActivityImpl.hpp"
+#include "src/simix/ActorImpl.hpp"
 #include "surf/surf.hpp"
 
-enum e_smx_comm_type_t { SIMIX_COMM_SEND, SIMIX_COMM_RECEIVE, SIMIX_COMM_READY, SIMIX_COMM_DONE };
 
 namespace simgrid {
 namespace kernel {
@@ -19,15 +19,20 @@ class XBT_PUBLIC CommImpl : public ActivityImpl {
   ~CommImpl() override;
 
 public:
-  explicit CommImpl(e_smx_comm_type_t type);
+  enum class Type { SEND = 0, RECEIVE, READY, DONE };
+
+  explicit CommImpl(Type type);
+  void start();
+  void copy_data();
   void suspend() override;
   void resume() override;
   void post() override;
+  void finish() override;
   void cancel();
   double remains();
   void cleanupSurf(); // FIXME: make me protected
 
-  e_smx_comm_type_t type;       /* Type of the communication (SIMIX_COMM_SEND or SIMIX_COMM_RECEIVE) */
+  CommImpl::Type type;          /* Type of the communication (SIMIX_COMM_SEND or SIMIX_COMM_RECEIVE) */
   smx_mailbox_t mbox = nullptr; /* Rendez-vous where the comm is queued */
 
 #if SIMGRID_HAVE_MC
@@ -45,23 +50,23 @@ expectations of the other side, too. See  */
   void (*copy_data_fun)(smx_activity_t, void*, size_t) = nullptr;
 
   /* Surf action data */
-  simgrid::kernel::resource::Action* surfAction_ = nullptr; /* The Surf communication action encapsulated */
-  simgrid::kernel::resource::Action* src_timeout = nullptr; /* Surf's actions to instrument the timeouts */
-  simgrid::kernel::resource::Action* dst_timeout = nullptr; /* Surf's actions to instrument the timeouts */
-  smx_actor_t src_proc      = nullptr;
-  smx_actor_t dst_proc      = nullptr;
-  double rate               = 0.0;
-  double task_size          = 0.0;
+  resource::Action* surf_action_ = nullptr; /* The Surf communication action encapsulated */
+  resource::Action* src_timeout_ = nullptr; /* Surf's actions to instrument the timeouts */
+  resource::Action* dst_timeout_ = nullptr; /* Surf's actions to instrument the timeouts */
+  actor::ActorImplPtr src_actor_ = nullptr;
+  actor::ActorImplPtr dst_actor_ = nullptr;
+  double rate_                   = 0.0;
+  double task_size_              = 0.0;
 
   /* Data to be transfered */
-  void* src_buff        = nullptr;
-  void* dst_buff        = nullptr;
-  size_t src_buff_size  = 0;
-  size_t* dst_buff_size = nullptr;
+  void* src_buff_        = nullptr;
+  void* dst_buff_        = nullptr;
+  size_t src_buff_size_  = 0;
+  size_t* dst_buff_size_ = nullptr;
   bool copied           = false; /* whether the data were already copied */
 
-  void* src_data = nullptr; /* User data associated to communication */
-  void* dst_data = nullptr;
+  void* src_data_ = nullptr; /* User data associated to the communication */
+  void* dst_data_ = nullptr;
 };
 }
 }