diff options
| author | Mikulas Patocka <mpatocka@redhat.com> | 2025-11-17 21:42:02 +0100 |
|---|---|---|
| committer | Mikulas Patocka <mpatocka@redhat.com> | 2025-11-21 12:51:41 +0100 |
| commit | fe680d8c747f4e676ac835c8c7fb0f287cd98758 (patch) | |
| tree | 5f1dd6654819d7c5ec1e21e1fb397fd550ddb6b5 | |
| parent | a612d24e850b28c16384cf071636034935447e43 (diff) | |
dm-verity: fix unreliable memory allocation
GFP_NOWAIT allocation may fail anytime. It needs to be changed to
GFP_NOIO. There's no need to handle an error because mempool_alloc with
GFP_NOIO can't fail.
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Cc: stable@vger.kernel.org
Reviewed-by: Eric Biggers <ebiggers@kernel.org>
| -rw-r--r-- | drivers/md/dm-verity-fec.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/md/dm-verity-fec.c b/drivers/md/dm-verity-fec.c index d382a390d39a..72047b47a7a0 100644 --- a/drivers/md/dm-verity-fec.c +++ b/drivers/md/dm-verity-fec.c @@ -320,11 +320,7 @@ static int fec_alloc_bufs(struct dm_verity *v, struct dm_verity_fec_io *fio) if (fio->bufs[n]) continue; - fio->bufs[n] = mempool_alloc(&v->fec->prealloc_pool, GFP_NOWAIT); - if (unlikely(!fio->bufs[n])) { - DMERR("failed to allocate FEC buffer"); - return -ENOMEM; - } + fio->bufs[n] = mempool_alloc(&v->fec->prealloc_pool, GFP_NOIO); } /* try to allocate the maximum number of buffers */ |