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
FileJournal: New journal padding mode #6856
Conversation
Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com>
407c9cb
to
8c70cd4
Compare
Benchmark result: Intel(R) Xeon(R) CPU E5-2620 v2 @ 2.10GHz |
@liewegas , Please review it. Thanks. |
8c70cd4
to
86d3882
Compare
Current, because the requirement of DirectIo, the padding mode is adding padding data for every entry. In fact, we fetch a couple of entries as a system write as possible as can. So we only make sure system-write content met the requirement of DirectIO. We adding a padding entry at the end of system-write content(for most case, this include a couple of entries). User could set the osd_journal_padding_ahead to decide which mode. osd_journal_padding_ahead = true is good choice for pcie-ssd. Signed-off-by: Jianpeng Ma <jianpeng.ma@intel.com> Signed-off-by: Xinze Chi <xinze@xsky.com>
86d3882
to
33ab6a3
Compare
@@ -643,7 +644,11 @@ int FileJournal::_fdump(Formatter &f, bool simple) | |||
&bl, | |||
&seq, | |||
&ss); | |||
if (result != SUCCESS) { | |||
if (result == SKIP) { | |||
seq = pre_seq; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would it be simpler to make do_read_entry not clobber &seq when returning SKIP?
is this still current ? If yes, it needs a rebase ;-) |
We want to focus efforts on optimizations with BlueStore. Close it. |
Current, because the requirement of DirectIo, the padding mode
is adding padding data for every entry.
In fact, we fetch a couple of entries as a system write as possible
as can. So we only make sure system-write content met the requirement
of DirectIO. We adding a padding entry at the end of system-write
content(for most case, this include a couple of entries).
User could set the osd_journal_padding_ahead to decide which mode.
osd_journal_padding_ahead = true is good choice for pcie-ssd.
This work is base on #5877.
Signed-off-by: Jianpeng Ma jianpeng.ma@intel.com
Signed-off-by: Xinze Chi xinze@xsky.com