emo/thin-provisioning-tools
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[space-maps/recursive] lift a lock, and add a lock

Browse Source
This commit is contained in:
Joe Thornber 2020-05-26 08:45:35 +01:00
parent 0e54de95e1
commit 12b6157c5b
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
persistent-data/space-maps/recursive.cc
View File

@ -88,6 +88,7 @@ namespace {
} }
virtual ref_t get_count(block_address b) const { virtual ref_t get_count(block_address b) const {
recursing_const_lock lock(*this);
return modify_count(b, sm_->get_count(b)); return modify_count(b, sm_->get_count(b));
} }
@ -123,7 +124,6 @@ namespace {
} }
} }
// FIXME: double check this
virtual maybe_block virtual maybe_block
find_free(span_iterator &it) { find_free(span_iterator &it) {
recursing_lock lock(*this); recursing_lock lock(*this);
@ -208,11 +208,9 @@ namespace {
} }
void flush_ops_() { void flush_ops_() {
// FIXME: we need to remove entries as we act upon them recursing_lock lock(*this);
for (auto const &p : ops_) { for (auto const &p : ops_) {
// FIXME: lift outside the loop?
recursing_lock lock(*this);
block_address b = p.first; block_address b = p.first;
auto const &op = p.second; auto const &op = p.second;