git: hammer2 - Revamp flush and xopq mechanism, stabilization
Matthew Dillon
dillon at crater.dragonflybsd.org
Mon Nov 28 12:29:37 PST 2016
commit 2085215738c03d949e60de63843cb91e84836eb9
Author: Matthew Dillon <dillon at apollo.backplane.com>
Date: Mon Nov 28 12:18:54 2016 -0800
hammer2 - Revamp flush and xopq mechanism, stabilization
* Originally the xopq mechanism was meant to allow multiple worker
threads to share the same queue in a NxM configuration, but for
all intents and purposes we changed to a one-queue-per-thread
mechanic. Formalize this by associating the queue directly with
the thread structure.
* Separate out strategy-related XOPs from vnop-related XOPs, using half
the worker threads for strategy XOPs and the other half for vnop XOPs.
This fixes at least one deadlock which could occur if a strategy XOP
was queued after a vnop on the same worker thread. Since the
buffers related to the strategy XOP were locked, it could prevent
the vnop ahead of it from being able to execute.
* Cleanup the xopq sleep/wakeup mechanism and also fix a 30-60 second
stall when a xop-xop dependency is present on the same queue. The
clearing of the older xop was not waking up the worker thread.
* Revamp the transaction code, putpages via UIO_NOCOPY vop_write's,
and vop_strategy, to allow all buffer-cache-related I/Os to execute
concurrent with a flush. This fixes several deadlocks and long stalls
that could occur during flushes. Nominal modifying vnops will still
interlock with flushes, for now.
* Remove the PREFLUSH flag, it is no longer relevant.
* NOTE - Flush code probably doesn't completely handle modifying races
yet and needs more work. However, such races should be restricted to
buffer-cache-related I/Os only and thus it should be possible to
interlock them on an inode-by-inode basis.
Summary of changes:
sys/vfs/hammer2/hammer2.h | 14 +++----
sys/vfs/hammer2/hammer2_admin.c | 85 +++++++++++++++++++++-----------------
sys/vfs/hammer2/hammer2_flush.c | 43 ++++++++++---------
sys/vfs/hammer2/hammer2_io.c | 4 ++
sys/vfs/hammer2/hammer2_strategy.c | 14 ++-----
sys/vfs/hammer2/hammer2_vfsops.c | 30 ++++----------
sys/vfs/hammer2/hammer2_vnops.c | 11 ++++-
7 files changed, 102 insertions(+), 99 deletions(-)
http://gitweb.dragonflybsd.org/dragonfly.git/commitdiff/2085215738c03d949e60de63843cb91e84836eb9
--
DragonFly BSD source repository
More information about the Commits
mailing list