On Tue 15-04-25 16:55:06, Thomas Weißschuh wrote:
> Remove the suppression of the uevents before scanning for partitions.
> The partitions inherit their suppression settings from their parent device,
> which lead to the uevents being dropped.
>
> This is similar to the same changes for LOOP_CONFIGURE done in
> commit bb430b694226 ("loop: LOOP_CONFIGURE: send uevents for partitions").
>
> Fixes: 498ef5c777d9 ("loop: suppress uevents while reconfiguring the device")
> Cc: stable@xxxxxxxxxxxxxxx
> Signed-off-by: Thomas Weißschuh <thomas.weissschuh@xxxxxxxxxxxxx>
Looks good. Feel free to add:
Reviewed-by: Jan Kara <jack@xxxxxxx>
Honza
> ---
> Changes in v3:
> - Rebase onto block/block-6.15
> - Drop already applied patch "loop: properly send KOBJ_CHANGED uevent for disk device"
> - Add patch to fix partition uevents for LOOP_SET_FD
> - Link to v2: https://lore.kernel.org/r/20250415-loop-uevent-changed-v2-1-0c4e6a923b2a@xxxxxxxxxxxxx
>
> Changes in v2:
> - Use correct Fixes tag
> - Rework commit message slightly
> - Rebase onto v6.15-rc1
> - Link to v1: https://lore.kernel.org/r/20250317-loop-uevent-changed-v1-1-cb29cb91b62d@xxxxxxxxxxxxx
> ---
> drivers/block/loop.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/block/loop.c b/drivers/block/loop.c
> index 3be7f00e7fc740da2745ffbccfcebe53eef2ddaa..e9ec7a45f3f2d1dd2a82b3506f3740089a20ae05 100644
> --- a/drivers/block/loop.c
> +++ b/drivers/block/loop.c
> @@ -662,12 +662,12 @@ static int loop_change_fd(struct loop_device *lo, struct block_device *bdev,
> * dependency.
> */
> fput(old_file);
> + dev_set_uevent_suppress(disk_to_dev(lo->lo_disk), 0);
> if (partscan)
> loop_reread_partitions(lo);
>
> error = 0;
> done:
> - dev_set_uevent_suppress(disk_to_dev(lo->lo_disk), 0);
> kobject_uevent(&disk_to_dev(lo->lo_disk)->kobj, KOBJ_CHANGE);
> return error;
>
> @@ -675,6 +675,7 @@ static int loop_change_fd(struct loop_device *lo, struct block_device *bdev,
> loop_global_unlock(lo, is_loop);
> out_putf:
> fput(file);
> + dev_set_uevent_suppress(disk_to_dev(lo->lo_disk), 0);
> goto done;
> }
>
>
> ---
> base-commit: 7ed2a771b5fb3edee9c4608181235c30b40bb042
> change-id: 20250307-loop-uevent-changed-aa3690f43e03
>
> Best regards,
> --
> Thomas Weißschuh <thomas.weissschuh@xxxxxxxxxxxxx>
>
--
Jan Kara <jack@xxxxxxxx>
SUSE Labs, CR
