osdc/Objecter: wire up native Dispatcher fast dispatch

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

osdc/Objecter: make Objecter a Dispatcher

Note that it's not actually doing it yet, though!

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

mds: add finisher

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

objecter: take write lock in ms_handle_reset()

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

rwlock: assert on unlock if not locked

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

objecter: fix locking

ms_handle_reset() takes the session lock, needed to be able to call
_kick_requetss() with it held.

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

objecter: split objecter initialization

Separate objecter initialization to non cluster related work (e.g.,
internal data structures, other registrations), and to operations that
can initiate cluster interaction. This is so that we don't hit a rare
race where we can get called indirectly from one of the dispatcher callbacks
e.g., into handle_osd_map() when not yet being initialized.
This requires that objecter->init() should be called before
messenger->add_dispatcher_head(), and objecter->start() after it.

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

objecter: Objecter::Op is refcounted

Cancellation of lingerin ops collided with ops that we sent osdmap on,
needed to add a refcount for correct object delete.

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

objecter: don't serialize responses if there's no object name

This implicitly fixes an issue with the list_objects() being reentrant,
and triggers a lock dependency issue. The better solution would be to
have the callback context specify whether it's reentrant or not, but
will require a much bigger change.

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

objecter: shard completion_lock

Object ops responses are sharded, lock hashed by object name. This
guarantees ordering on the same object. Cross object order is not
guaranteed anymore.

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

objecter: a major refactoring

Fixes: #7619
Removed the client_lock (that used to pass in as a param) and replaced
it with a read-write lock (completely controlled by the objecter). Also
added a per-session read-write lock. Adapt code to use the new locking
scheme, removed locking where not needed. Replaced various counters to
atomics instead of grabbing the lock for updates. Moved ops to live
under the session.

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
Signed-off-by: Sage Weil <sage@inktank.com>

timer: fix RWTimer shutdown

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

librados: cancel operation

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

test_mon_workloadgen: fix for new objecter interface

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

rgw: abort early from logging function

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

osd: adapt to new Objecter interface

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

librados: remove warning

Seems that the current locking scheme is ok. Objecter handles its
locking internally, librados needs to hold its lock.

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

librados: push snap related functions down to objecter

Need to take appropriate lock.

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

librados: call objecter->handle_osd_map() under lock

The lock is not shared with the objecter anymore so we need to call
it under this lock, otherwise we're going to race with
wait_for_osdmap().

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

librados: adapt to new objecter interface

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

librados: don't hold client lock in certain operations

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

librados: don't hold lock in dispatch()

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

mds: adapt to new objecter interface

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

client: adapt to new objecter interface

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

time: create RWTimer

a timer implementation that uses RWLock

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

common/RWLock: add lock context

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

common/RWLock: add explicit unlock() to *Lockers

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

common/RWLock: add get()

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

common/RWLock: check return values

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

common/RWLock: track read/write locks via atomics for assertion checks

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

Merge pull request #2138 from xiaoxichen/fix_crush_ruleset

mon/OSDMonitor : Use user provided ruleset for replicated pool

Reviewed-by: Loic Dachary <loic-201408@dachary.org>

Merge branch 'wip-weak-refs'

vstart.sh: debug rgw = 20 on -d

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

Merge pull request #2292 from lpabon/rgw-doc

vstart.sh to enable support for simple Rados

Tested-by: GWabhishek.lekshmanan@gmail.com

common/shared_cache: dump weak refs on shutdown

Before asserting, dump what the refs are along with the counts.

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

Merge pull request #2304 from majianpeng/fix1

small bug fix for journal

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

common/shared_cache: take a cct

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

Merge pull request #2291 from ceph/wip-fingerprint

mon: add a unique clsuter fingerprint

Reviewed-by: Joao Eduardo Luis <joao.luis@inktank.com>

Merge pull request #2307 from dmsimard/crushdoc

Doc: Add config option to turn off crush update

Reviewed-by: Loic Dachary <loic-201408@dachary.org>

Merge pull request #2288 from xiaoxichen/fix_add_simple_ruleset

CrushWrapper: pick a ruleset same as rule_id

Reviewed-by: Loic Dachary <loic-201408@dachary.org>

Doc: Add config option to turn off crush update

Signed-off-by: David Moreau Simard <dmsimard@iweb.com>

CrushWrapper: pick a ruleset same as rule_id

Originally in the add_simple_ruleset funtion, the ruleset_id
is not reused but rule_id is reused. So after some add/remove
against rules, the newly created rule likely to have
ruleset!=rule_id.

We dont want this happen because we are trying to hold the constraint
that ruleset == rule_id.

Signed-off-by: Xiaoxi Chen <xiaoxi.chen@intel.com>

Merge pull request #2273 from dachary/wip-mailmap

mailmap updates

Merge remote-tracking branch 'gh/wip-filejournal'

Reviewed-by: Sage Weil <sage@redhat.com>
Tested-by: Mark Kirkwood <mark.kirkwood@catalyst.net.nz>

Merge remote-tracking branch 'gh/wip-lttng'

Revert "Merge pull request #2253 from adamcrume/wip-lttng"

This reverts commit 5b9ade5058f72116eb3dc920f442361c2a33b46b, reversing
changes made to b0aa846b3f81225a779de00100e15334fb8156b3.

I merged the wrong version of wip-lttng.

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

Merge pull request #2274 from ceph/wip-9144

filestore: fix force_sync race with mount + umount + mount

Reviewed-by: David Zafman <dzafman@redhat.com>

Merge pull request #2253 from adamcrume/wip-lttng

RBD-replay

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

mon: generate cluster_fingerprint if null

This triggers after an upgrade of a legacy cluster that has no fingerprint.

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

mon: add a cluster fingerprint

Generate it on cluster creations with the initial monmap.  Include it in
the report.  Provide no way for this uuid to be fed in to the cluster
(intentionally or not) so that it can be assumed to be a truly unique
identifier for the cluster.

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

rbd_replay: Add unit test for batch_unreachable_from

This requires a fair amount of code reorganization, since the types in
the function signature were previously not in a header file.

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Add unit test for Ser

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Split --with-lttng and --with-babeltrace options

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Remove tracing-specfic local variables when lttng disabled

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Add informational messages to configure.ac

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Only install rbd-replay-prep when enabled

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Check for babeltrace when autoconfiguring lttng

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Check that the main babeltrace header is present

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Test for sane lttng-gen-tp

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Don't build trace libraries when lttng is disabled

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Fix build error on 32-bit systems

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Remove -lboost_regex from unit test

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Don't build trace headers when lttng is disabled

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Update packaging

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Only require lttng for Fedora >= 20 and RHEL == 6 for RPM distros

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Add distro packaging

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Fix "make distcheck"

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Support --with-lttng=check

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Remove tracing from libcommon

This is a short-term fix for issues caused by tracepoints in libcommon.
Code crashes at runtime if the same tracepoints are linked into the
program multiple times.  This happens with libcommon because it is
statically linked into dynamic libraries such as librados, then
statically linked into executables because symbols from libcommon are
not exposed in librados.  Therefore, any programs that use librados and
libcommon would crash because of duplicate tracepoints.

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Fix bug in rbd-replay-prep, Thread::issued_io with wrong IO

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Remove extent tracepoints and inline extents

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Move tracepoints from internal.cc to librbd.cc

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Add man page and package binaries for rbd-replay-prep

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Convert prep-for-replay.py to rbd-replay-prep.cc

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Add man page, and add binaries to Debian packaging

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Remove outdated message that says regexes are used

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Replace Boost dependencies with custom string code

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Remove boost_regex from LIBADD and LDADD

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Fix compiler warning in unit tests

Was getting:
test/test_rbd_replay.cc:44:3: warning: converting ‘false’ to pointer type for argument 1 of ‘char testing::internal::IsNullLiteralHelper(testing::internal::Secret*)’ [-Wconversion-null]

Fixed by changing EXPECT_EQ(false, xxx) to EXPECT_FALSE(xxx).
For completeness, also changed EXPECT_EQ(true, xxx) to EXPECT_TRUE(xxx).

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Switch ImageNameMap from regex to plain string matching

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Switch ImageNameMap to using rbd_loc

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Add rbd_replay::rbd_loc

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Use standard image@snap instead of image/snap

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Fix ceph_ctf_stringp

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Support replaying partial traces

Tracing may start after the application is started, and image open calls
may missed. To support replaying these traces, additional information is
traced, allowing missing open calls to be generated.

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Use non-zero fake data

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Add unit tests

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Preload liblttng-ust-fork.so in TESTS_ENVIRONMENT

This adds LD_PRELOAD=liblttng-ust-fork.so to TESTS_ENVIRONMENT.
This prevents lttng from complaining when processes are forked.
The complaints otherwise taint the output and cause tests to fail.

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Disable LTTng by default, add --with-lttng configure option

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Fix 'make check' by setting up fake HOME

If the HOME variable is not set, LTTng complains, and the extra output
fails to match what the tests expect, so they fail.

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Add --map-image option for image name mapping

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Add --pool option

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Add assertions

Signed-off-by: Adam Crume <adamcrume@gmail.com>

lttng: Default to not storing buffer content

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Add command-line arguments to prep-for-replay.py

Includes:
--print-on-read
--print-on-write
--window

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Fix async writes in prep-for-replay.py

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Add --read-only flag

Removes ignoreWrites flag in prep-for-replay.py

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Fix examples/rbd-replay/replay

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Clean up prep-for-replay.py

Signed-off-by: Adam Crume <adamcrume@gmail.com>

rbd-replay: Hash completions across multiple mutexes to reduce contention

Signed-off-by: Adam Crume <adamcrume@gmail.com>