New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

os/bluestore: batch throttle #15284

Merged
merged 3 commits into from May 28, 2017

Conversation

Projects
None yet
4 participants
@majianpeng
Member

majianpeng commented May 25, 2017

1:make throttle.put batch handle
2: avoid call the same osd qcond.notify_all many times.

@ifed01

ifed01 approved these changes May 25, 2017

LGTM except that attomics use seems excessive

@liewegas liewegas changed the title from Bluestore batch throotle to os/bluestore: batch throotle May 25, 2017

@liewegas liewegas changed the title from os/bluestore: batch throotle to os/bluestore: batch throttle May 25, 2017

@liewegas

This comment has been minimized.

Show comment
Hide comment
@liewegas

liewegas May 25, 2017

Member

the rest looks good!

Member

liewegas commented May 25, 2017

the rest looks good!

@majianpeng

This comment has been minimized.

Show comment
Hide comment
@majianpeng

majianpeng May 26, 2017

Member

update.

Member

majianpeng commented May 26, 2017

update.

majianpeng added some commits May 25, 2017

os/bluestore/BlueStore: batch handle Throttle.put.
Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>
os/bluestore/BlueStore: Only call once notify_all.
All txcs of deferred_running have the same osr, so only call once
notify_call.

Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>
os/bluestore/BlueStore: avoid iterating to get ios/cost in _kv_sync_t…
…hread.

Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>

@liewegas liewegas merged commit 0b0c7e5 into ceph:master May 28, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@majianpeng majianpeng deleted the majianpeng:bluestore-batch-throotle branch Jun 1, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment