diff options
| author | Damien Le Moal <[email protected]> | 2024-09-17 13:32:31 +0000 |
|---|---|---|
| committer | Jens Axboe <[email protected]> | 2024-09-17 14:34:00 +0000 |
| commit | e3accac1a976e65491a9b9fba82ce8ddbd3d2389 (patch) | |
| tree | c71c9eef7ee3b820bbf6950620e0a9c32a596455 /drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | |
| parent | Merge tag 'v6.11' into for-6.12/block (diff) | |
| download | kernel-e3accac1a976e65491a9b9fba82ce8ddbd3d2389.tar.gz kernel-e3accac1a976e65491a9b9fba82ce8ddbd3d2389.zip | |
block: Fix elv_iosched_local_module handling of "none" scheduler
Commit 734e1a860312 ("block: Prevent deadlocks when switching
elevators") introduced the function elv_iosched_load_module() to allow
loading an elevator module outside of elv_iosched_store() with the
target device queue not frozen, to avoid deadlocks. However, the "none"
scheduler does not have a module and as a result,
elv_iosched_load_module() always returns an error when trying to switch
to this valid scheduler.
Fix this by ignoring the return value of the request_module() call
done by elv_iosched_load_module(). This restores the behavior before
commit 734e1a860312, which was to ignore the request_module() result and
instead rely on elevator_change() to handle the "none" scheduler case.
Reported-by: Shin'ichiro Kawasaki <[email protected]>
Fixes: 734e1a860312 ("block: Prevent deadlocks when switching elevators")
Cc: [email protected]
Signed-off-by: Damien Le Moal <[email protected]>
Reviewed-by: Christoph Hellwig <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Jens Axboe <[email protected]>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_object.c')
0 files changed, 0 insertions, 0 deletions
