If we throttle at num_threads, then nothing gets into the workqueue until
a worker thread is idle, which means you pay the latency of setting it up
and queueing it. This way we keep some requests ready to go.
Signed-off-by: Sage Weil <sage@newdream.net>
Document it as with the tcp stuff, remove an if(0)'d debugging block,
and remove the useless "sd" parameter since it's always the same as
the Pipe's sd member.
Signed-off-by: Greg Farnum <gregory.farnum@dreamhost.com>
Why were all these data members public? They're accessed by Pipes
and the Accepter and stuff, so maybe that's why...but that's all
internal interface stuff.
Signed-off-by: Greg Farnum <gregory.farnum@dreamhost.com>
Convert ms_addr and _my_name to be references to their fields in
the entity_inst_t my_inst.
This way we can use const references for accessing all of them,
instead of the bizarre distinction we had before for get_myinst().
Signed-off-by: Greg Farnum <gregory.farnum@dreamhost.com>
Return a const reference to the actual address, instead of copying it.
All current users are happy with this, and I can't see a good reason
to copy it instead.
Signed-off-by: Greg Farnum <gregory.farnum@dreamhost.com>
Making an allowance for lossy server connections is silly. Just don't
ask for the Connection in that case. (There aren't any users who
rely on the previous behavior.)
Document that requirement in Messenger.h!
Signed-off-by: Greg Farnum <gregory.farnum@dreamhost.com>
Beforehand the return code was ignored, so fixed the cases
where we erroneously return error instead of success.
Signed-off-by: Yehuda Sadeh <yehuda.sadeh@dreamhost.com>
This fixes#1856: looking up subuser that doesn't exist returns
user as long as subuser prefix defined existing user.
Signed-off-by: Yehuda Sadeh <yehuda@hq.newdream.net>
fsid is a field in the CephContext _conf structure and is parsed by
the standard options parsing library before it gets to the ceph_mon
custom parsing.
Instead do the standard parsing, and check that member directly
to decide if we want to (over)write the monmap's fsid.
Fixes one part of #2221.
Signed-off-by: Greg Farnum <gregory.farnum@dreamhost.com>
Update pg stats on any op completion (read or write), not just writes. Do
the calls with log_op_stats() for consistency's sake. Skip if the request
was an error.
Fixes: #2209
Signed-off-by: Sage Weil <sage.weil@dreamhost.com>
Reviewed-by: Samuel Just <samuel.just@dreamhost.com>
Assign a (unique) name to each Sequencer. This will aid in debugging, and
can be useful when dumping traces of FileStore workloads.
Signed-off-by: Sage Weil <sage.weil@dreamhost.com>
If toctree is inside a section, the subtree is inside the section too.
We don't want all of dev/* to be under "Mailing list".
I have not found a decent workaround for this. The toplevel toctree
avoids this purely by the fact that it is the topmost toctree. Right
now that means you should 1) avoid having more than a few paragraphs of
text before the toctree for that subtree (put most of the content after
the toctree; clumsy if the toctree is long), or 2) put the toptree
immediately after the document title, make it :hidden:, and let the
reader use links in the text or the ToC in the sidebar to navigate.
See start/index for an example of this.
Signed-off-by: Tommi Virtanen <tommi.virtanen@dreamhost.com>
