diff options
author | 2007-07-12 17:28:25 +0100 | |
---|---|---|
committer | 2007-07-12 15:01:08 -0700 | |
commit | 943317efdbc295e8a28df3f5cbd549d066ee8b4a (patch) | |
tree | 63eea4f14c4b3913b6755d802afb7631007a9e0e /crypto/algapi.c | |
parent | dm snapshot: permit invalid activation (diff) | |
download | linux-rng-943317efdbc295e8a28df3f5cbd549d066ee8b4a.tar.xz linux-rng-943317efdbc295e8a28df3f5cbd549d066ee8b4a.zip |
dm raid1: clear region outside spinlock
A clear_region function is permitted to block (in practice, rare) but gets
called in rh_update_states() with a spinlock held.
The bits being marked and cleared by the above functions are used
to update the on-disk log, but are never read directly. We can
perform these operations outside the spinlock since the
bits are only changed within one thread viz.
- mark_region in rh_inc()
- clear_region in rh_update_states().
So, we grab the clean_regions list items via list_splice() within the
spinlock and defer clear_region() until we iterate over the list for
deletion - similar to how the recovered_regions list is already handled.
We then move the flush() call down to ensure it encapsulates the changes
which are done by the later calls to clear_region().
Signed-off-by: Jonathan Brassow <jbrassow@redhat.com>
Signed-off-by: Alasdair G Kergon <agk@redhat.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'crypto/algapi.c')
0 files changed, 0 insertions, 0 deletions