diff options
author | Wouter Verhelst <w@uter.be> | 2024-08-12 15:20:42 +0200 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2024-09-06 08:31:40 -0600 |
commit | 296dbc72d29085d5fc34430d0760423071e9e81d (patch) | |
tree | 16ad23f771b23beda1874c6527b06fda165a80ec /drivers/block | |
parent | 41372f5c9a866365e212809b3543ae8cb5b2542b (diff) | |
download | lwn-296dbc72d29085d5fc34430d0760423071e9e81d.tar.gz lwn-296dbc72d29085d5fc34430d0760423071e9e81d.zip |
nbd: correct the maximum value for discard sectors
The version of the NBD protocol implemented by the kernel driver
currently has a 32 bit field for length values. As the NBD protocol uses
bytes as a unit of length, length values larger than 2^32 bytes cannot
be expressed.
Update the max_hw_discard_sectors field to match that.
Signed-off-by: Wouter Verhelst <w@uter.be>
Fixes: 268283244c0f ("nbd: use the atomic queue limits API in nbd_set_size")
Reviewed-by: Damien Le Moal <dlemoal@kernel.org>
Cc: Eric Blake <eblake@redhat.Com>
Link: https://lore.kernel.org/r/20240812133032.115134-8-w@uter.be
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'drivers/block')
-rw-r--r-- | drivers/block/nbd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c index 18b94d7c795a..b852050d8a96 100644 --- a/drivers/block/nbd.c +++ b/drivers/block/nbd.c @@ -350,7 +350,7 @@ static int __nbd_set_size(struct nbd_device *nbd, loff_t bytesize, lim = queue_limits_start_update(nbd->disk->queue); if (nbd->config->flags & NBD_FLAG_SEND_TRIM) - lim.max_hw_discard_sectors = UINT_MAX; + lim.max_hw_discard_sectors = UINT_MAX >> SECTOR_SHIFT; else lim.max_hw_discard_sectors = 0; if (!(nbd->config->flags & NBD_FLAG_SEND_FLUSH)) { |