Skip to content
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

luminous: os/bluestore: >2GB bluefs writes #28965

Merged
merged 3 commits into from Jul 11, 2019

Conversation

liewegas
Copy link
Member

@liewegas liewegas commented Jul 10, 2019

liewegas and others added 3 commits July 10, 2019 10:21
KernelDevice::aio_{submit,write,read}() are critical paths. calling
cct->_conf->subsys.should_gather() multi-times is not optimal. the
downside of this issue is that if the aio is printed, the size of
buffer in PrebufferedStreambuf could be large if the number of iov is
large, that could be a heavy load to the memory subsystem.

Signed-off-by: Kefu Chai <kchai@redhat.com>
(cherry picked from commit 67c6f76)

# Conflicts:
#	src/os/bluestore/aio.cc
 - adjacent to aio_queue_t::submit()
On Linux, write() (and similar system calls) will transfer at most
0x7ffff000 (2,147,479,552) bytes, it will cap data if aio pwritev
more than 0x7ffff000, so we have the split the data to more aio submit.

Signed-off-by: kungf <yang.wang@easystack.cn>
(cherry picked from commit 4d33114)

- conflict due to fd_direct instead of choose_fd()
Write 2GB to verify the blockdevice aio splitting.

Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit 1974f01)
- fixed get_val/set_val calls
- add_block_device doesn't take third arg in luminous
@liewegas liewegas changed the title luminous: qa/valgrind: backport suppressions luminous: os/bluestore: >2GB bluefs writes Jul 10, 2019
@liewegas liewegas added this to the luminous milestone Jul 10, 2019
@neha-ojha
Copy link
Member

@yuriw
Copy link
Contributor

yuriw commented Jul 10, 2019

@neha-ojha neha-ojha merged commit 998d4fc into ceph:luminous Jul 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants