Revert "dm cache: do not wake_worker() in free_migration()"
authorMike Snitzer <snitzer@redhat.com>
Wed, 29 Jul 2015 18:11:28 +0000 (14:11 -0400)
committerMike Snitzer <snitzer@redhat.com>
Wed, 29 Jul 2015 18:32:08 +0000 (14:32 -0400)
commit3508e6590d4729ac07f01f7ae2256c2f9dc738b8
tree46cbef379c3f3b9a37cc59ff2c9fe5fb6c7d6694
parent6ed443c07f1e7f819983422b16598facb152250d
Revert "dm cache: do not wake_worker() in free_migration()"

This reverts commit 386cb7cdeeef97e0bf082a8d6bbfc07a2ccce07b.

Taking the wake_worker() out of free_migration() will slow writeback
dramatically, and hence adaptability.

Say we have 10k blocks that need writing back, but are only able to
issue 5 concurrently due to the migration bandwidth: it's imperative
that we wake_worker() immediately after migration completion; waiting
for the next 1 second wake up (via do_waker) means it'll take a long
time to write that all back.

Reported-by: Joe Thornber <ejt@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
drivers/md/dm-cache-target.c