X-Git-Url: http://bilbo.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/a882344bc908a258a991701e226556ec23fd7cd6..9db79d70d969bdb58f26c689ad4f965b3d9e80cf:/src/kernel/resource/DiskImpl.hpp diff --git a/src/kernel/resource/DiskImpl.hpp b/src/kernel/resource/DiskImpl.hpp index af027a7503..a1bdb8ae50 100644 --- a/src/kernel/resource/DiskImpl.hpp +++ b/src/kernel/resource/DiskImpl.hpp @@ -1,28 +1,21 @@ -/* Copyright (c) 2019-2022. 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/s4u/Disk.hpp" #include "simgrid/s4u/Io.hpp" #include "src/kernel/resource/Resource.hpp" -#include "src/surf/surf_interface.hpp" #include "xbt/PropertyHolder.hpp" #include -#ifndef DISK_IMPL_HPP_ -#define DISK_IMPL_HPP_ - -/********* - * Model * - *********/ - -namespace simgrid { -namespace kernel { -namespace resource { +namespace simgrid::kernel::resource { /*********** * Classes * ***********/ @@ -34,13 +27,9 @@ 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; - - virtual DiskAction* io_start(const DiskImpl* disk, sg_size_t size, s4u::Io::OpType type) = 0; }; /************ @@ -61,6 +50,7 @@ class DiskImpl : public Resource_T, public xbt::PropertyHolder { 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(); @@ -71,6 +61,17 @@ public: 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_; } @@ -78,13 +79,13 @@ public: DiskImpl* set_host(s4u::Host* host); s4u::Host* get_host() const { return host_; } - virtual void set_read_bandwidth(double value) { read_bw_.peak = value; } + void set_read_bandwidth(double value); double get_read_bandwidth() const { return read_bw_.peak * read_bw_.scale; } - virtual void set_write_bandwidth(double value) { write_bw_.peak = value; } + void set_write_bandwidth(double value); double get_write_bandwidth() const { return write_bw_.peak * write_bw_.scale; } - virtual void set_readwrite_bandwidth(double value) { readwrite_bw_ = value; } + void set_readwrite_bandwidth(double value); double get_readwrite_bandwidth() const { return readwrite_bw_; } DiskImpl* set_read_constraint(lmm::Constraint* constraint_read); @@ -113,6 +114,8 @@ public: 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; }; /********** @@ -125,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 +} // namespace simgrid::kernel::resource #endif /* DISK_IMPL_HPP_ */