diff options
author | Mikulas Patocka <mpatocka@redhat.com> | 2021-02-10 15:26:23 -0500 |
---|---|---|
committer | Mike Snitzer <snitzer@redhat.com> | 2021-02-11 09:45:28 -0500 |
commit | a666e5c05e7c4aaabb2c5d58117b0946803d03d2 (patch) | |
tree | 1a9dc24b70359cd4f268966d99763d79bea28926 /drivers/md/dm-crypt.c | |
parent | e3290b9491ff5b7ee40f9e0a4c06821988a2a2bf (diff) | |
download | lwn-a666e5c05e7c4aaabb2c5d58117b0946803d03d2.tar.gz lwn-a666e5c05e7c4aaabb2c5d58117b0946803d03d2.zip |
dm: fix deadlock when swapping to encrypted device
The system would deadlock when swapping to a dm-crypt device. The reason
is that for each incoming write bio, dm-crypt allocates memory that holds
encrypted data. These excessive allocations exhaust all the memory and the
result is either deadlock or OOM trigger.
This patch limits the number of in-flight swap bios, so that the memory
consumed by dm-crypt is limited. The limit is enforced if the target set
the "limit_swap_bios" variable and if the bio has REQ_SWAP set.
Non-swap bios are not affected becuase taking the semaphore would cause
performance degradation.
This is similar to request-based drivers - they will also block when the
number of requests is over the limit.
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Cc: stable@vger.kernel.org
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
Diffstat (limited to 'drivers/md/dm-crypt.c')
-rw-r--r-- | drivers/md/dm-crypt.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/md/dm-crypt.c b/drivers/md/dm-crypt.c index ae0f0a4e3689..11c105ecd165 100644 --- a/drivers/md/dm-crypt.c +++ b/drivers/md/dm-crypt.c @@ -3342,6 +3342,7 @@ static int crypt_ctr(struct dm_target *ti, unsigned int argc, char **argv) wake_up_process(cc->write_thread); ti->num_flush_bios = 1; + ti->limit_swap_bios = true; return 0; |