librbd: journal::Replay no longer holds lock while completing callback

Fixes: http://tracker.ceph.com/issues/16433
Signed-off-by: Jason Dillaman <dillaman@redhat.com>

Merge pull request #9451 from s09816/master

librbd: enable/disable of features is not allowed when already enabled/disabled

Reviewed-by: Jason Dillaman <dillaman@redhat.com>

Merge pull request #9715 from trociny/wip-16321

librbd: re-register watch on old format image rename

Reviewed-by: Jason Dillaman <dillaman@redhat.com>

Merge pull request #9874 from athanatos/wip-16185

OSDMonitor::prepare_pgtemp: only update up_thru if newer

Reviewed-by: Sage Weil <sage@redhat.com>

Merge pull request #9875 from athanatos/wip-16127

OSDMonitor: drop pg temp from sources other than the current primary

Reviewed-by: Sage Weil <sage@redhat.com>

Merge pull request #9850 from dillaman/wip-16404

librbd: flag image as updated after proxying maintenance op

Reviewed-by: Mykola Golub <mgolub@mirantis.com>

Merge pull request #8370 from jack-changtao/pg_del

osd/PG: cleanup: delete find_best_info again

Reviewed-by: Samuel Just <sjust@redhat.com>

Merge pull request #9802 from yuyuyu101/wip-16378

msg/async: fix test stuck into STATE_WAIT

Reviewed-by: Samuel Just <sjust@redhat.com>

Merge pull request #8195 from tchaikov/wip-cmake-visibility

cmake changes

Reviewed-by: Ali Maredia <amaredia@redhat.com>

cmake: use ${CEPH_SHARED} when adding libradosstriper

Signed-off-by: Kefu Chai <kchai@redhat.com>

Merge pull request #9827 from liewegas/wip-bluestore-invalidate

os/bluestore: discard unreferenced buffers as they are overwritten

Reviewed-by: Igor Fedotov <ifedotov@mirantis.com>

Merge pull request #9868 from liewegas/wip-bluestore-lba

include/small_encoding: fix lba encoding with many low zeros

Reviewed-by: Igor Fedotov <ifedotov@mirantis.com>

Merge pull request #9863 from trociny/fixup-comp-warn

test: fix compilation warnings

Reviewed-by: Jason Dillaman <dillaman@redhat.com>

Merge pull request #9821 from dx9/wip-ac-alpine

configure.ac: add --with-reentrant-strsignal and --with-thread-safe-r…

Reviewed-by: Kefu Chai <kchai@redhat.com>

os/bluestore: discard unreferenced buffers as they are overwritten

Signed-off-by: Sage Weil <sage@redhat.com>

include/small_encoding: fix lba encoding with many low zeros

Signed-off-by: Sage Weil <sage@redhat.com>

msg/async: recover lock to continue later

Signed-off-by: Haomai Wang <haomai@xsky.com>

Merge pull request #9865 from liewegas/wip-bluestore-encoding

os/bluestore: fix blob_t encoding test cases, and clean up initialization

Reviewed-by: Igor Fedotov <ifedotov@mirantis.com>

configure.ac: add --with-reentrant-strsignal and --with-thread-safe-res-query options for Alpine support

Signed-off-by: John Coyle <dx9err@gmail.com>

test: fix compilation warnings

Signed-off-by: Mykola Golub <mgolub@mirantis.com>

os/bluestore/bluestore_types: fix blob initializers, test instances

- initialize members in c++11 style
- drop pextent ctor
- fix up test instances
- make csum_order default to 0, so it matches the
  CSUM_NONE decoding logic.

Signed-off-by: Sage Weil <sage@redhat.com>

os/bluestore/bluestore_types: drop unused var

Signed-off-by: Sage Weil <sage@redhat.com>

Merge pull request #9813 from jcsp/wip-kernel-doc

doc/cephfs: add advice about kernel versions

Merge pull request #9823 from dillaman/wip-16362

rbd-mirror: keep events from different epochs independent

Reviewed-by: Mykola Golub <mgolub@mirantis.com>

cmake: include Group.cc in rbd cli

to match with automake, and the "group" command is missing in the cmake
build of "rbd" cli tool.

Signed-off-by: Kefu Chai <kchai@redhat.com>

cmake: install dso into ${CMAKE_INSTALL_LIBDIR} instead of "lib"

Signed-off-by: Kefu Chai <kchai@redhat.com>

cmake: add visibility flags for exported dso

* no regression found, and there's a substantial reduction in the size of
  generated librados library (both stripped and not).
* also remove the duplicated set_target_properties(librbd...) command

Signed-off-by: Kefu Chai <kchai@redhat.com>

cmake: move snap_set_diff.cc out of librados

snap_set_diff.cc is used internally and is not exposed via librados. we
cannot use the function implemented in it by linking against librados. so
compile it as an OBJECT target and link against it individualy if
necessary.

Signed-off-by: Kefu Chai <kchai@redhat.com>

Merge branch 'wip-dmick-install-deps-master'

install-deps.sh: use mk-build-deps instead of processing control

Reviewed-by: Erwan Velu <erwan@redhat.com>
Reviewed-by: Josh Durgin <jdurgin@redhat.com>

install-deps.sh: use mk-build-deps instead of processing control

mk-build-deps is designed to handle installing build dependencies;
use that instead, so '|' indications are handled properly.

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #9791 from ceph/wip-cmake-compile-flags

cmake: Wip cmake compile flags

Reviewed-by: Casey Bodley <cbodley@redhat.com>

Merge pull request #9481 from yehudasa/wip-rgw-hold-lease-1

rgw: stop bucket lease only after draining object sync operations

Reviewed-by: Casey Bodley <cbodley@redhat.com>

cmake: CMAKE_BUILD_TYPE defaulted to RelWithDebInfo

Default build type is RelWithDebInfo unless
CMAKE_BUILD_TYPE is specified when running the
`cmake` command.

Signed-off-by: Ali Maredia <amaredia@redhat.com>

Merge pull request #9066 from yangdongsheng/rbd_bench

rbd: introduce a simple bench for read

Reviewed-by: Jason Dillaman <dillaman@redhat.com>
Reviewed-by: Mykola Golub <mgolub@mirantis.com>

librbd: flag image as updated after proxying maintenance op

Fixes: http://tracker.ceph.com/issues/16404
Signed-off-by: Jason Dillaman <dillaman@redhat.com>

cmake: eliminate FORTIFY_SOURCE warnings

"-werr" was added to CHECK_C_COMPILER_FLAG to
make sure FORTIFY_SOURCE is only turned on if
the flag generates no warnings.

Added "-U_FORTIFY_SOURCE" to CMAKE_C_FLAGS to
mimic PR #9761.

Signed-off-by: Ali Maredia <amaredia@redhat.com>

Merge pull request #9728 from liewegas/wip-bluestore-encoding

os/bluestore: space efficient int encoding

Reviewed-by: Igor Fedotov <ifedotov@mirantis.com>

rbd-mirror: keep events from different epochs independent

Fixes: http://tracker.ceph.com/issues/16362
Signed-off-by: Jason Dillaman <dillaman@redhat.com>

os/bluestore/bluestore_types: delta encode for ref_map offsets

Signed-off-by: Sage Weil <sage@redhat.com>

os/bluestore: make blob csum info encoding conditional

Signed-off-by: Sage Weil <sage@redhat.com>

os/bluestore: use FLAG_CSUM to indicate if checksums are present

Signed-off-by: Sage Weil <sage@redhat.com>

os/bluestore: fsck: print intervals in hex

Signed-off-by: Sage Weil <sage@redhat.com>

os/bluestore: use varint for pextent count

Signed-off-by: Sage Weil <sage@redhat.com>

os/bluestore: varint or lba encode pextent, lextent, ref_map, onode

Certainly not done here, but this is a start.

Signed-off-by: Sage Weil <sage@redhat.com>

include/small_encoding: small encode/decode helpers

Signed-off-by: Sage Weil <sage@redhat.com>

Merge pull request #9819 from dillaman/wip-16363

librbd: fix lockdep issue when duplicate event detected

Reviewed-by: Mykola Golub <mgolub@mirantis.com>

Merge pull request #9816 from dillaman/wip-16198

librbd: potential race when replaying journal ops

Reviewed-by: Mykola Golub <mgolub@mirantis.com>

Merge pull request #9788 from dillaman/wip-16230

rbd-mirror: potential race condition accessing local image journal

Reviewed-by: Mykola Golub <mgolub@mirantis.com>

Merge pull request #9787 from batrick/mds-rename-enotempty

mds: improve comments for ENOTEMPTY checks

Reviewed-by: John Spray <john.spray@redhat.com>

Merge pull request #9840 from xiexingguo/xxg-wip-fix-bmap-fm

os/bluestore: minor improvements for bitmap-freelist-manager

Reviewed-by: Sage Weil <sage@redhat.com>

msg/async/AsyncConnection: shorter handshake step deps

Originally connect side will wait for accept side sending banner, actually
we don't need this logic to ensure anything.

So connect side will issue banner after tcp handshake straightway. It will
help for reducing session handshake period, then it avoid peer side drop this
connection in low level but connect side don't know right now.

Signed-off-by: Haomai Wang <haomai@xsky.com>

Merge pull request #9815 from xiexingguo/xxg-wip-fix-blue-cache

os/bluestore: cache relevant cleanups

Reviewed-by: Igor Fedotov <ifedotov@mirantis.com>
Reviewed-by: Sage Weil <sage@redhat.com>

os/bluestore: make bytes_per_key a global member

So we don't have to figure it out everywhere.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>

os/bluestore: fix a typo(nonexistant->nonexistent)

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>

os/bluestore: simplify BufferSpace::read() a little

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>

os/bluestore: simplify BufferSpace::read() a little

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>

os/bluestore: change type of length of region_t from uint32_t to uint64_t

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>

os/bluestore: add sanity check when try to rm_buffer

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>

Merge pull request #9828 from liewegas/wip-bluestore-read-before-write

os/bluestore: limit read-amp on overwrite

Reviewed-by: Varada Kari <varada.kari@sandisk.com>

Merge pull request #9826 from liewegas/wip-bluefs

os/bluestore/BlueFS: instrument which file types we write to

Reviewed-by: Varada Kari <varada.kari@sandisk.com>

Merge pull request #9834 from tchaikov/wip-silence-warnings

test, bluestore: silence warnings

Reviewed-by: Ramesh Chander <Ramesh.Chander@sandisk.com>

Merge pull request #9635 from liewegas/wip-make-dist

make-dist: include src/.git_version and src/ceph_ver.h

Reviewed-by: Kefu Chai <kchai@redhat.com>

Merge pull request #9836 from dx9/wip-hardcoded-tcmalloc

cmake: fix hard coded tcmalloc lib

Reviewed-by: Kefu Chai <kchai@redhat.com>

cmake: fix hard coded tcmalloc lib

Signed-off-by: John Coyle <dx9err@gmail.com>

Merge pull request #9670 from ceph/wip-debian-clean

debian: silence couple warnings

Reviewed-by: Dan Mick <dmick@redhat.com>

bluestore/BitAllocator: silence warnings of method hidden

is_allocated(int64_t *blocks, int64_t num_blocks, int blk_off) only gets
called in BitMapAreaIN and its child classes. so it's safe to move this
method down to BitMapAreaIN.

Signed-off-by: Kefu Chai <kchai@redhat.com>

ceph_test_objectstore: remove unused function

its caller was removed in 8a42e61

Signed-off-by: Kefu Chai <kchai@redhat.com>

Merge pull request #9832 from ceph/wip-cmake-librbd-unittest-fix

cmake: unittest_librbd missing a source file

cmake: unittest_librbd missing a source file

src/test/librbd/test_ConsistencyGroups.cc missing
from ${unittest_librbd_srcs}.

Signed-off-by: Ali Maredia <amaredia@redhat.com>

Merge pull request #9531 from yehudasa/wip-16175

rgw: data sync debug logging

Reviewed-by: Casey Bodley <cbodley@redhat.com>

Merge pull request #9676 from efirs/fix_bitmap_freelist

os/bluestore: Fix enumeration when only single key in BitmapFreelistManager

os/bluestore/BlueFS: instrument which file types we write to

Signed-off-by: Sage Weil <sage@redhat.com>

os/bluestore: limit read-amp on overwrite

If we are doing a small overwrite over a blob with a large chunk size
(say, due to a large csum order), we are better off writing into a new
allocation than doing a large read/modify/write.  If the read amp will be
more than min_alloc_size, skip the read entirely and write into a new
blob.

Signed-off-by: Sage Weil <sage@redhat.com>

test: remove remaining references to compression_dir

Removed in 38c7f4a06673e4806d327931744338a7393b45ef

Signed-off-by: Sage Weil <sage@redhat.com>

Merge pull request #9829 from ceph/wip-cmake-readme

cmake: updated CMake README instructions

Merge pull request #9814 from songbaisen/a11

mon: remove the repeat quorum size jugement

Reviewed-by: Sage Weil <sage@redhat.com>

Merge pull request #9818 from ifed01/wip-zero-length-writes

os/store_test: uniform zero length writes handling

Reviewed-by: Sage Weil <sage@redhat.com>

cmake: updated CMake README instructions

Updated testing and cluster start up instructions
for CMake in README.md

Signed-off-by: Ali Maredia <amaredia@redhat.com>

Merge pull request #9605 from liewegas/wip-statfs

os: simplify store_statfs_t

Reviewed-by: Igor Fedotov <ifedotov@mirantis.com>

Merge pull request #9261 from prallabh/user_quota

rgw: Let the default quota settings take effect during user creation

Merge pull request #9817 from theanalyst/wip/boost-optional

rgw_swift: newer versions of boost/utility no longer include in_place
Reviewed-by: Orit Wasserman <owasserm@redhat.com>

librbd: fix lockdep issue when duplicate event detected

Fixes: http://tracker.ceph.com/issues/16363
Signed-off-by: Jason Dillaman <dillaman@redhat.com>

os/store_test: fix synthetic test cases to handle zero-writes properly, i.e. do not increase object size.

Signed-off-by: Igor Fedotov <ifedotov@mirantis.com>

os/memstore: zero-length write should not effect object size

Signed-off-by: Sage Weil <sage@redhat.com>

ceph_test_objectstore: zero-length write should not affect object size

Signed-off-by: Sage Weil <sage@redhat.com>

rgw_swift: newer versions of boost/utility no longer include in_place

boost > 1.58 no longer includes in_place in boost/utility, we need to
include in_place_factory explicitly. This causes build failures in
distros that ship with a higher version of boost. Since the only call is for
swift_ver_location, another possibility is to use emplace()
instead (though this requires boost ~ 1.56)

Fixes: http://tracker.ceph.com/issues/16391
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>

Merge pull request #9786 from pritha-srivastava/wip-multisite-error-msgs

rgw: modifying multi-site log messages.

Reviewed-by: Casey Bodley <cbodley@redhat.com>

librbd: potential race when replaying journal ops

Fixes: http://tracker.ceph.com/issues/16198
Signed-off-by: Jason Dillaman <dillaman@redhat.com>

Merge pull request #9812 from zhjwpku/cephx_protocol

doc: cephx_protocol typos

doc/cephfs: add advice about kernel versions

Signed-off-by: John Spray <john.spray@redhat.com>

  mon: remove the repeat quorum size jugement

  Signed-off-by:song baisen <song.baisen@zte.com.cn>

rgw: Add documentation for RGW default quota

Signed-off-by: Pavan Rallabhandi <PRallabhandi@walmartlabs.com>

Merge pull request #9188 from rzarzynski/wip-rgw-swift-versioning-new-2

rgw: improve support for Swift's object versioning.
Reviewed-by: Orit Wasserman <owasserm@redhat.com>

tools/rbd: introduce a simple bench for read

Currently, we have a bench-write for in rbd tool, but we don't have
a simple bench for read.

This patch add a new subcommand of bench, with an option of --io-type.
Then we can do a simple bench for write or read.

Signed-off-by: Dongsheng Yang <dongsheng.yang@easystack.cn>

tools/rbd: make bench-write invisible to user

rbd bench-write is going to be deprecated. This patch
makes it invisible in help message and manpage. But it
is still working.

Signed-off-by: Dongsheng Yang <dongsheng.yang@easystack.cn>

tools/rbd: introduce a new field of visible in Action Class

The new field means whether we want to show this Action in
command help. Sometimes, we want to make one action to be
deprecated and invisible to user, but we have to keep it
working for compatibility.

Then we need to support this action but make it invisible.

Signed-off-by: Dongsheng Yang <dongsheng.yang@easystack.cn>

rgw: modifying multi-site log messages.

Increasing the log level and removing 'ERROR' from messages that appear during normal multi-site operation.
Fixes: http://tracker.ceph.com/issues/16121
Signed-off-by: Pritha Srivastava <prsrivas@redhat.com>

Merge pull request #9804 from ceph/revert-9801-msg_msg_types_entity_name_t_makeprivate

Revert "msg/msg_types.h: make members private in ..."

Reviewed-by: Kefu Chai <kchai@redhat.com>

Revert "msg/msg_types.h: make members private in ..."

Merge pull request #9801 from stiopaa1/msg_msg_types_entity_name_t_makeprivate

msg/msg_types.h: make members private in `entity_name_t`

Reviewed-by: Kefu Chai <kchai@redhat.com>