diff options
| author | NeilBrown <neilb@suse.de> | 2012-07-31 09:08:14 +0200 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2012-07-31 09:08:14 +0200 |
| commit | 0021b7bc045e4b0b85d8c53614342aaf84ca96a5 (patch) | |
| tree | 1432761eec4c49bbacea55df083e73599e18ea1c /drivers/md/raid1.c | |
| parent | 01ff5dbc0925d11c8ad76eed3bdd02d0c7e1e0f5 (diff) | |
| download | op-kernel-dev-0021b7bc045e4b0b85d8c53614342aaf84ca96a5.zip op-kernel-dev-0021b7bc045e4b0b85d8c53614342aaf84ca96a5.tar.gz | |
md: remove plug_cnt feature of plugging.
This seemed like a good idea at the time, but after further thought I
cannot see it making a difference other than very occasionally and
testing to try to exercise the case it is most likely to help did not
show any performance difference by removing it.
So remove the counting of active plugs and allow 'pending writes' to
be activated at any time, not just when no plugs are active.
This is only relevant when there is a write-intent bitmap, and the
updating of the bitmap will likely introduce enough delay that
the single-threading of bitmap updates will be enough to collect large
numbers of updates together.
Removing this will make it easier to centralise the unplug code, and
will clear the other for other unplug enhancements which have a
measurable effect.
Signed-off-by: NeilBrown <neilb@suse.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'drivers/md/raid1.c')
| -rw-r--r-- | drivers/md/raid1.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/md/raid1.c b/drivers/md/raid1.c index cacd008..36a8fc0 100644 --- a/drivers/md/raid1.c +++ b/drivers/md/raid1.c @@ -2173,8 +2173,7 @@ static void raid1d(struct mddev *mddev) blk_start_plug(&plug); for (;;) { - if (atomic_read(&mddev->plug_cnt) == 0) - flush_pending_writes(conf); + flush_pending_writes(conf); spin_lock_irqsave(&conf->device_lock, flags); if (list_empty(head)) { |
