mirror of
https://github.com/matrix-construct/construct
synced 2025-02-18 09:40:12 +01:00
ircd::db: Improve opts related for random_access_file; minor cleanup.
This commit is contained in:
parent
7698759d04
commit
5239e49f4f
2 changed files with 19 additions and 11 deletions
|
@ -21,7 +21,10 @@ struct ircd::db::database::env::random_access_file final
|
|||
using Status = rocksdb::Status;
|
||||
using Slice = rocksdb::Slice;
|
||||
|
||||
static const fs::fd::opts default_opts;
|
||||
|
||||
database &d;
|
||||
fs::fd::opts opts;
|
||||
fs::fd fd;
|
||||
|
||||
bool use_direct_io() const noexcept override;
|
||||
|
|
27
ircd/db.cc
27
ircd/db.cc
|
@ -1890,8 +1890,8 @@ noexcept
|
|||
|
||||
rocksdb::Status
|
||||
ircd::db::database::env::NewRandomAccessFile(const std::string& name,
|
||||
std::unique_ptr<RandomAccessFile>* r,
|
||||
const EnvOptions& options)
|
||||
std::unique_ptr<RandomAccessFile> *const r,
|
||||
const EnvOptions &options)
|
||||
noexcept
|
||||
{
|
||||
ctx::uninterruptible::nothrow ui;
|
||||
|
@ -3056,8 +3056,8 @@ const noexcept
|
|||
// random_access_file
|
||||
//
|
||||
|
||||
const ircd::fs::fd::opts
|
||||
_random_access_file_opts{[]
|
||||
decltype(ircd::db::database::env::random_access_file::default_opts)
|
||||
ircd::db::database::env::random_access_file::default_opts{[]
|
||||
{
|
||||
ircd::fs::fd::opts ret{std::ios_base::in};
|
||||
ret.direct = true;
|
||||
|
@ -3066,25 +3066,30 @@ _random_access_file_opts{[]
|
|||
|
||||
ircd::db::database::env::random_access_file::random_access_file(database *const &d,
|
||||
const std::string &name,
|
||||
const EnvOptions &opts)
|
||||
const EnvOptions &env_opts)
|
||||
:d
|
||||
{
|
||||
*d
|
||||
}
|
||||
,opts{[&env_opts]
|
||||
{
|
||||
fs::fd::opts ret{default_opts};
|
||||
ret.direct = env_opts.use_direct_reads;
|
||||
return ret;
|
||||
}()}
|
||||
,fd
|
||||
{
|
||||
name, _random_access_file_opts
|
||||
name, this->opts
|
||||
}
|
||||
{
|
||||
#ifdef RB_DEBUG_DB_ENV
|
||||
log::debug
|
||||
{
|
||||
log, "'%s': opened rfile:%p fd:%d '%s' opts:%p",
|
||||
log, "'%s': opened rfile:%p fd:%d '%s'",
|
||||
d->name,
|
||||
this,
|
||||
int(fd),
|
||||
name,
|
||||
&opts
|
||||
name
|
||||
};
|
||||
#endif
|
||||
}
|
||||
|
@ -3261,14 +3266,14 @@ bool
|
|||
ircd::db::database::env::random_access_file::use_direct_io()
|
||||
const noexcept
|
||||
{
|
||||
return _random_access_file_opts.direct;
|
||||
return opts.direct;
|
||||
}
|
||||
|
||||
size_t
|
||||
ircd::db::database::env::random_access_file::GetRequiredBufferAlignment()
|
||||
const noexcept
|
||||
{
|
||||
return 4096;
|
||||
return ircd::info::page_size;
|
||||
}
|
||||
|
||||
//
|
||||
|
|
Loading…
Add table
Reference in a new issue