Skip to content
Snippets Groups Projects
Commit 5ac60242 authored by Uday Shankar's avatar Uday Shankar Committed by Jens Axboe
Browse files

ublk: set_params: properly check if parameters can be applied


The parameters set by the set_params call are only applied to the block
device in the start_dev call. So if a device has already been started, a
subsequently issued set_params on that device will not have the desired
effect, and should return an error. There is an existing check for this
- set_params fails on devices in the LIVE state. But this check is not
sufficient to cover the recovery case. In this case, the device will be
in the QUIESCED or FAIL_IO states, so set_params will succeed. But this
success is misleading, because the parameters will not be applied, since
the device has already been started (by a previous ublk server). The bit
UB_STATE_USED is set on completion of the start_dev; use it to detect
and fail set_params commands which arrive too late to be applied (after
start_dev).

Signed-off-by: default avatarUday Shankar <ushankar@purestorage.com>
Fixes: 0aa73170 ("ublk_drv: add SET_PARAMS/GET_PARAMS control command")
Reviewed-by: default avatarMing Lei <ming.lei@redhat.com>
Link: https://lore.kernel.org/r/20250304-set_params-v1-1-17b5e0887606@purestorage.com


Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent b654f7a5
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment