X-Git-Url: http://bilbo.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/340910247cb7a827db19aaf0e9f6c0714d606eac..5843ccab4e336d47ca34f54e68760ac78d242f36:/src/kernel/resource/DiskImpl.hpp diff --git a/src/kernel/resource/DiskImpl.hpp b/src/kernel/resource/DiskImpl.hpp index 9898cb747f..a1bdb8ae50 100644 --- a/src/kernel/resource/DiskImpl.hpp +++ b/src/kernel/resource/DiskImpl.hpp @@ -1,28 +1,21 @@ -/* Copyright (c) 2019-2021. The SimGrid Team. All rights reserved. */ +/* Copyright (c) 2019-2023. 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. */ +#ifndef DISK_IMPL_HPP_ +#define DISK_IMPL_HPP_ + #include "simgrid/kernel/resource/Action.hpp" #include "simgrid/kernel/resource/Model.hpp" -#include "simgrid/kernel/resource/Resource.hpp" #include "simgrid/s4u/Disk.hpp" #include "simgrid/s4u/Io.hpp" -#include "src/surf/surf_interface.hpp" -#include +#include "src/kernel/resource/Resource.hpp" +#include "xbt/PropertyHolder.hpp" #include -#ifndef DISK_INTERFACE_HPP_ -#define DISK_INTERFACE_HPP_ - -/********* - * Model * - *********/ - -namespace simgrid { -namespace kernel { -namespace resource { +namespace simgrid::kernel::resource { /*********** * Classes * ***********/ @@ -34,9 +27,7 @@ class DiskAction; *********/ class DiskModel : public Model { public: - explicit DiskModel(const std::string& name); - DiskModel(const DiskModel&) = delete; - DiskModel& operator=(const DiskModel&) = delete; + using Model::Model; virtual DiskImpl* create_disk(const std::string& name, double read_bandwidth, double write_bandwidth) = 0; }; @@ -45,23 +36,42 @@ public: * Resource * ************/ class DiskImpl : public Resource_T, public xbt::PropertyHolder { - s4u::Host* host_ = nullptr; s4u::Disk piface_; - double read_bw_ = -1.0; - double write_bw_ = 1.0; + s4u::Host* host_ = nullptr; lmm::Constraint* constraint_write_ = nullptr; /* Constraint for maximum write bandwidth*/ - lmm::Constraint* constraint_read_ = nullptr; /* Constraint for maximum read bandwidth*/ + lmm::Constraint* constraint_read_ = nullptr; /* Constraint for maximum read bandwidth*/ + std::unordered_map sharing_policy_ = { + {s4u::Disk::Operation::READ, s4u::Disk::SharingPolicy::LINEAR}, + {s4u::Disk::Operation::WRITE, s4u::Disk::SharingPolicy::LINEAR}, + {s4u::Disk::Operation::READWRITE, s4u::Disk::SharingPolicy::LINEAR}}; + std::unordered_map sharing_policy_cb_ = {}; + std::function factor_cb_ = {}; + + Metric read_bw_ = {0.0, 0, nullptr}; + Metric write_bw_ = {0.0, 0, nullptr}; + double readwrite_bw_ = -1; /* readwrite constraint bound, usually max(read, write) */ + std::atomic_int_fast32_t refcount_{0}; + + void apply_sharing_policy_cfg(); protected: ~DiskImpl() override = default; // Disallow direct deletion. Call destroy() instead. public: - DiskImpl(const std::string& name, double read_bandwidth, double write_bandwidth) - : Resource_T(name), piface_(name, this), read_bw_(read_bandwidth), write_bw_(write_bandwidth) - { - } + explicit DiskImpl(const std::string& name, double read_bandwidth, double write_bandwidth); DiskImpl(const DiskImpl&) = delete; DiskImpl& operator=(const DiskImpl&) = delete; + friend void intrusive_ptr_add_ref(DiskImpl* disk) + { + disk->refcount_.fetch_add(1, std::memory_order_acq_rel); + } + friend void intrusive_ptr_release(DiskImpl* disk) + { + if (disk->refcount_.fetch_sub(1, std::memory_order_release) == 1) { + std::atomic_thread_fence(std::memory_order_acquire); + delete disk; + } + } /** @brief Public interface */ const s4u::Disk* get_iface() const { return &piface_; } @@ -69,11 +79,14 @@ public: DiskImpl* set_host(s4u::Host* host); s4u::Host* get_host() const { return host_; } - DiskImpl* set_read_bandwidth(double read_bw); - double get_read_bandwidth() const { return read_bw_; } + void set_read_bandwidth(double value); + double get_read_bandwidth() const { return read_bw_.peak * read_bw_.scale; } - DiskImpl* set_write_bandwidth(double write_bw); - double get_write_bandwidth() const { return write_bw_; } + void set_write_bandwidth(double value); + double get_write_bandwidth() const { return write_bw_.peak * write_bw_.scale; } + + void set_readwrite_bandwidth(double value); + double get_readwrite_bandwidth() const { return readwrite_bw_; } DiskImpl* set_read_constraint(lmm::Constraint* constraint_read); lmm::Constraint* get_read_constraint() const { return constraint_read_; } @@ -81,17 +94,28 @@ public: DiskImpl* set_write_constraint(lmm::Constraint* constraint_write); lmm::Constraint* get_write_constraint() const { return constraint_write_; } - /** @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; + profile::Event* get_read_event() const { return read_bw_.event; } + void unref_read_event() { tmgr_trace_event_unref(&read_bw_.event); } + + profile::Event* get_write_event() const { return write_bw_.event; } + void unref_write_event() { tmgr_trace_event_unref(&write_bw_.event); } + + DiskImpl* set_read_bandwidth_profile(profile::Profile* profile); + DiskImpl* set_write_bandwidth_profile(profile::Profile* profile); + + void set_sharing_policy(s4u::Disk::Operation op, s4u::Disk::SharingPolicy policy, const s4u::NonLinearResourceCb& cb); + s4u::Disk::SharingPolicy get_sharing_policy(s4u::Disk::Operation op) const; + + void set_factor_cb(const std::function& cb); + const std::function& get_factor_cb() const { return factor_cb_; } + void turn_on() override; void turn_off() override; void seal() override; void destroy(); // Must be called instead of the destructor + virtual DiskAction* io_start(sg_size_t size, s4u::Io::OpType type) = 0; - virtual DiskAction* read(sg_size_t size) = 0; - virtual DiskAction* write(sg_size_t size) = 0; }; /********** @@ -104,9 +128,8 @@ public: using Action::Action; void set_state(simgrid::kernel::resource::Action::State state) override; + void update_remains_lazy(double now) override; }; -} // namespace resource -} // namespace kernel -} // namespace simgrid -#endif /* DISK_INTERFACE_HPP_ */ +} // namespace simgrid::kernel::resource +#endif /* DISK_IMPL_HPP_ */