From ec84342fd8a2667646f1fe689a23af70fc7ed32a Mon Sep 17 00:00:00 2001 From: Ulrich Weigand Date: Mon, 2 Sep 2019 21:26:16 +0200 Subject: [PATCH] bluestore: Fix incorrect use of __le16/32/64 Use ceph_le16/32/64 instead of __le16/32/64 (which are no-op outside of kernel code). Fixes (partially): https://tracker.ceph.com/issues/41605 Signed-off-by: Ulrich Weigand --- src/os/bluestore/BlueStore.cc | 6 +++--- src/os/bluestore/bluestore_types.h | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc index 9a613e57d91..7936db74375 100644 --- a/src/os/bluestore/BlueStore.cc +++ b/src/os/bluestore/BlueStore.cc @@ -610,9 +610,9 @@ struct Int64ArrayMergeOperator : public KeyValueDB::MergeOperator { ceph_assert(llen == rlen); ceph_assert((rlen % 8) == 0); new_value->resize(rlen); - const __le64* lv = (const __le64*)ldata; - const __le64* rv = (const __le64*)rdata; - __le64* nv = &(__le64&)new_value->at(0); + const ceph_le64* lv = (const ceph_le64*)ldata; + const ceph_le64* rv = (const ceph_le64*)rdata; + ceph_le64* nv = &(ceph_le64&)new_value->at(0); for (size_t i = 0; i < rlen >> 3; ++i) { nv[i] = lv[i] + rv[i]; } diff --git a/src/os/bluestore/bluestore_types.h b/src/os/bluestore/bluestore_types.h index 10c62c00ae3..63f7c0fd8ab 100644 --- a/src/os/bluestore/bluestore_types.h +++ b/src/os/bluestore/bluestore_types.h @@ -758,11 +758,11 @@ public: case 1: return reinterpret_cast(p)[i]; case 2: - return reinterpret_cast(p)[i]; + return reinterpret_cast(p)[i]; case 4: - return reinterpret_cast(p)[i]; + return reinterpret_cast(p)[i]; case 8: - return reinterpret_cast(p)[i]; + return reinterpret_cast(p)[i]; default: ceph_abort_msg("unrecognized csum word size"); } -- 2.39.5