[PATCH] os/bluestore/BlueFS: use uint64_t for `len`
authorKefu Chai <kchai@redhat.com>
Fri, 28 Jun 2019 03:35:54 +0000 (11:35 +0800)
committerBernd Zeimetz <bzed@debian.org>
Fri, 31 Jan 2020 23:47:27 +0000 (23:47 +0000)
commit9edcb60a19025c1f230fb1482b9bcc4d8ef265a1
tree7e1c2e74c1e50da2ee92f2b6df49938ee8209dee
parent368996c0161df55c1f6d372fdb1711c993c0b3a6
[PATCH] os/bluestore/BlueFS: use uint64_t for `len`

change the type of parameter `len` of `BlueFS::_read_random()` from
`size_t` to `uint64_t`.

i think the type of `size_t` comes from
`rocksdb::RandomAccessFile::Read(uint64_t offset, size_t n,
rocksdb::Slice* result, char* scratch)`. and when we implement this
method, we continued using `n`'s type. but, we are using it with
`std::min()`, for instance, where the template parameter type deduction
fails if the lhs and rhs parameters' types are different. so probaly the
better solution is to use `uint64_t` directly to avoid the the cast and
specializing the template.

Signed-off-by: Kefu Chai <kchai@redhat.com>
Gbp-Pq: Name bluefs-use-uint64_t-for-len.patch
src/os/bluestore/BlueFS.cc
src/os/bluestore/BlueFS.h