Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
Storage-kill: last crumbs
authorFrederic Suter <frederic.suter@cc.in2p3.fr>
Tue, 2 Feb 2021 22:45:17 +0000 (23:45 +0100)
committerFrederic Suter <frederic.suter@cc.in2p3.fr>
Wed, 3 Feb 2021 09:01:46 +0000 (10:01 +0100)
include/simgrid/host.h
include/simgrid/kernel/resource/Resource.hpp
include/simgrid/plugins/file_system.h
include/simgrid/simdag.h
include/xbt/PropertyHolder.hpp
src/bindings/java/jmsg_host.h
src/kernel/resource/DiskImpl.hpp
src/surf/HostImpl.hpp
src/surf/xml/platf_private.hpp

index bdd6e08..ced19be 100644 (file)
@@ -60,17 +60,6 @@ XBT_ATTRIB_DEPRECATED_v328("Please use sg_host_set_data(h, NULL)") XBT_PUBLIC vo
 #endif
 
 // ========= storage related functions ============
-/** @brief Return the list of mount point names on a host.
- * @param host a host
- * @return a dict containing all mount point on the host (mount_name => sg_storage_t)
- */
-XBT_PUBLIC xbt_dict_t sg_host_get_mounted_storage_list(sg_host_t host);
-
-/** @brief Return the list of storages attached to a host.
- * @param host a host
- * @return a dynar containing all storages (name) attached to the host
- */
-XBT_PUBLIC xbt_dynar_t sg_host_get_attached_storage_list(const_sg_host_t host);
 XBT_PUBLIC void sg_host_get_disks(const_sg_host_t host, unsigned int* disk_count, sg_disk_t** disks);
 
 // =========== user-level functions ===============
index 05b5fdd..fc8d09a 100644 (file)
@@ -19,7 +19,7 @@ namespace resource {
 
 /** @ingroup SURF_interface
  * @brief SURF resource interface class
- * @details This is the ancestor class of every resources in SimGrid, such as links, CPU or storage
+ * @details This is the ancestor class of every resources in SimGrid, such as links, CPU or disk
  */
 class XBT_PUBLIC Resource {
   std::string name_;
index e90f849..5880037 100644 (file)
@@ -67,12 +67,6 @@ typedef sg_file_t msg_file_t; // MSG backwards compatibility
 #define MSG_file_rmove(file, host, fullpath) sg_file_rmove((file), (host), (fullpath))
 
 #define MSG_storage_file_system_init() sg_storage_file_system_init()
-#define MSG_storage_get_free_size(st) sg_storage_get_size_free(st)
-#define MSG_storage_get_used_size(st) sg_storage_get_size_used(st)
-#define MSG_storage_get_size(st) sg_storage_get_size(st)
-#define MSG_storage_get_content(st) sg_storage_get_content(st)
-
-#define MSG_host_get_storage_content(st) sg_host_get_storage_content(st)
 
 #endif // SIMGRID_HAVE_MSG
 
@@ -94,9 +88,7 @@ namespace s4u {
  *
  * Used to simulate the time it takes to access to a file, but does not really store any information.
  *
- * They are located on @ref simgrid::s4u::Storage that are accessed from a given @ref simgrid::s4u::Host through
- * mountpoints.
- * For now, you cannot change the mountpoints programmatically, and must declare them from your platform file.
+ * They are located on @ref simgrid::s4u::Disk that are accessed from a given @ref simgrid::s4u::Host
  */
 class XBT_PUBLIC File : public xbt::Extendable<File> {
   sg_size_t size_ = 0;
index 541e93e..cfabc7b 100644 (file)
@@ -187,7 +187,6 @@ XBT_PUBLIC xbt_dynar_t SD_PTG_dotload(const char* filename);
 #define SD_route_get_latency sg_host_get_route_latency
 #define SD_route_get_bandwidth sg_host_get_route_bandwidth
 
-#define SD_workstation_get_mounted_storage_list sg_host_get_mounted_storage_list // XBT_ATTRIB_DEPRECATED_v330
 // Lost functions
 //SD_workstation_get_access_mode
 //SD_workstation_set_access_mode
index af3dcd8..7868a31 100644 (file)
@@ -15,7 +15,7 @@ namespace xbt {
 
 /** @brief a PropertyHolder can be given a set of textual properties
  *
- * Common PropertyHolders are elements of the platform file, such as Host, Link or Storage.
+ * Common PropertyHolders are elements of the platform file, such as Host, Link, or Disk.
  */
 class PropertyHolder { // DO NOT DERIVE THIS CLASS, or the diamond inheritance mayhem will get you
   std::unique_ptr<std::unordered_map<std::string, std::string>> properties_ = nullptr;
index f857937..75df2d4 100644 (file)
@@ -54,9 +54,6 @@ JNIEXPORT jdouble JNICALL Java_org_simgrid_msg_Host_getSpeed (JNIEnv *env, jobje
 JNIEXPORT jdouble JNICALL Java_org_simgrid_msg_Host_getCoreNumber (JNIEnv *env, jobject jhost);
 JNIEXPORT jobject JNICALL Java_org_simgrid_msg_Host_getProperty(JNIEnv *env, jobject jhost, jobject jname);
 JNIEXPORT void JNICALL Java_org_simgrid_msg_Host_setProperty(JNIEnv *env, jobject jhost, jobject jname, jobject jvalue);
-JNIEXPORT jobjectArray JNICALL Java_org_simgrid_msg_Host_getMountedStorage(JNIEnv * env, jobject jhost);
-JNIEXPORT jobjectArray JNICALL Java_org_simgrid_msg_Host_getAttachedStorage(JNIEnv * env, jobject jhost);
-JNIEXPORT jobjectArray JNICALL Java_org_simgrid_msg_Host_getStorageContent(JNIEnv * env, jobject jhost);
 JNIEXPORT jobjectArray JNICALL Java_org_simgrid_msg_Host_all(JNIEnv *env, jclass cls);
 JNIEXPORT void JNICALL Java_org_simgrid_msg_Host_setAsyncMailbox(JNIEnv * env, jclass cls_arg, jobject jname);
 
index 44ae1b9..146fb31 100644 (file)
@@ -74,7 +74,7 @@ public:
   lmm::Constraint* get_read_constraint() const { return constraint_read_; }
   lmm::Constraint* get_write_constraint() const { return constraint_write_; }
 
-  /** @brief Check if the Storage is used (if an action currently uses its resources) */
+  /** @brief Check if the Disk is used (if an action currently uses its resources) */
   bool is_used() const override;
   void apply_event(profile::Event* event, double value) override;
   void turn_on() override;
@@ -100,11 +100,11 @@ public:
   /**
    * @brief diskAction constructor
    *
-   * @param model The StorageModel associated to this DiskAction
+   * @param model The DiskModel associated to this DiskAction
    * @param cost The cost of this DiskAction in bytes
    * @param failed [description]
    * @param var The lmm variable associated to this DiskAction if it is part of a LMM component
-   * @param storage The Storage associated to this DiskAction
+   * @param disk The Disk associated to this DiskAction
    * @param type [description]
    */
   DiskAction(kernel::resource::Model* model, double cost, bool failed, kernel::lmm::Variable* var, DiskImpl* disk,
index 933e6fc..bdb796f 100644 (file)
@@ -39,7 +39,7 @@ public:
  ************/
 /** @ingroup SURF_host_interface
  * @brief SURF Host interface class
- * @details A host represents a machine with an aggregation of a Cpu, a RoutingEdge and a Storage
+ * @details A host represents a machine with an aggregation of a Cpu, a RoutingEdge and Disk(s)
  */
 class XBT_PRIVATE HostImpl : public xbt::PropertyHolder {
   std::vector<kernel::actor::ProcessArg*> actors_at_boot_;
index 8dae75c..1c3168b 100644 (file)
@@ -121,27 +121,6 @@ public:
   double lat;
 };
 
-class StorageCreationArgs {
-public:
-  std::string filename;
-  int lineno;
-  std::string id;
-  std::string type_id;
-  std::string content;
-  std::unordered_map<std::string, std::string>* properties;
-  std::string attach;
-};
-
-class StorageTypeCreationArgs {
-public:
-  std::string id;
-  std::string model;
-  std::string content;
-  std::unordered_map<std::string, std::string>* properties;
-  std::unordered_map<std::string, std::string>* model_properties;
-  sg_size_t size;
-};
-
 class DiskCreationArgs {
 public:
   std::string id;
@@ -150,12 +129,6 @@ public:
   double write_bw;
 };
 
-class MountCreationArgs {
-public:
-  std::string storageId;
-  std::string name;
-};
-
 class ProfileCreationArgs {
 public:
   std::string id;