Skip to content

Commit 844e80b

Browse files
authored
Merge pull request ceph#62185 from ronen-fr/wip-rf-more8-keys
kernel/KernelDevice: replace obsolete get_tracked_conf_keys() Reviewed-by: Matt Vandermeulen <[email protected]> Reviewed-by: Radoslaw Zarzynski <[email protected]>
2 parents 7a036c6 + c2b2d33 commit 844e80b

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

src/blk/kernel/KernelDevice.cc

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ using std::list;
5252
using std::map;
5353
using std::string;
5454
using std::vector;
55+
using namespace std::literals;
5556

5657
using ceph::bufferlist;
5758
using ceph::bufferptr;
@@ -1562,14 +1563,13 @@ int KernelDevice::invalidate_cache(uint64_t off, uint64_t len)
15621563
return r;
15631564
}
15641565

1565-
const char** KernelDevice::get_tracked_conf_keys() const
1566+
std::vector<std::string> KernelDevice::get_tracked_keys()
1567+
const noexcept
15661568
{
1567-
static const char* KEYS[] = {
1568-
"bdev_async_discard_threads",
1569-
"bdev_enable_discard",
1570-
NULL
1569+
return {
1570+
"bdev_async_discard_threads"s,
1571+
"bdev_enable_discard"s
15711572
};
1572-
return KEYS;
15731573
}
15741574

15751575
void KernelDevice::handle_conf_change(const ConfigProxy& conf,

src/blk/kernel/KernelDevice.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -166,7 +166,7 @@ class KernelDevice : public BlockDevice,
166166
void close() override;
167167

168168
// config observer bits
169-
const char** get_tracked_conf_keys() const override;
169+
std::vector<std::string> get_tracked_keys() const noexcept override;
170170
void handle_conf_change(const ConfigProxy& conf,
171171
const std::set <std::string> &changed) override;
172172
};

0 commit comments

Comments
 (0)