X-Git-Url: http://bilbo.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/6c1fdfa949304917158331410c2e91b0ca1b2e90..9db79d70d969bdb58f26c689ad4f965b3d9e80cf:/src/kernel/resource/DiskImpl.hpp diff --git a/src/kernel/resource/DiskImpl.hpp b/src/kernel/resource/DiskImpl.hpp index 4fb041d07a..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(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; } + + void set_write_bandwidth(double value); + double get_write_bandwidth() const { return write_bw_.peak * write_bw_.scale; } - DiskImpl* set_write_bandwidth(double write_bw); - double get_write_bandwidth() const { return write_bw_; } + 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; }; /********** @@ -102,30 +126,10 @@ class DiskAction : public Action { public: static xbt::signal on_state_change; - DiskAction(Model* model, double cost, bool failed, DiskImpl* disk, s4u::Io::OpType type) - : Action(model, cost, failed), type_(type), disk_(disk){}; - - /** - * @brief diskAction constructor - * - * @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 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, - s4u::Io::OpType type) - : Action(model, cost, failed, var), type_(type), disk_(disk){}; - + using Action::Action; void set_state(simgrid::kernel::resource::Action::State state) override; - - s4u::Io::OpType type_; - DiskImpl* disk_; + 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_ */