[PATCH 13/15] loop: cleanup loop_config_discard

2024-02-12 Thread Christoph Hellwig
Initialize the local variables for the discard max sectors and granularity to zero as a sensible default, and then merge the calls assigning them to the queue limits. Signed-off-by: Christoph Hellwig Reviewed-by: Keith Busch Reviewed-by: Chaitanya Kulkarni Reviewed-by: Ming Lei Reviewed-by: Da

[PATCH 13/15] loop: cleanup loop_config_discard

2024-02-11 Thread Christoph Hellwig
Initialize the local variables for the discard max sectors and granularity to zero as a sensible default, and then merge the calls assigning them to the queue limits. Signed-off-by: Christoph Hellwig Reviewed-by: Keith Busch Reviewed-by: Chaitanya Kulkarni Reviewed-by: Ming Lei Reviewed-by: Da

Re: [PATCH 13/15] loop: cleanup loop_config_discard

2024-01-23 Thread Hannes Reinecke
On 1/22/24 18:36, Christoph Hellwig wrote: Initialize the local variables for the discard max sectors and granularity to zero as a sensible default, and then merge the calls assigning them to the queue limits. Signed-off-by: Christoph Hellwig --- drivers/block/loop.c | 27

Re: [PATCH 13/15] loop: cleanup loop_config_discard

2024-01-23 Thread Chaitanya Kulkarni
On 1/22/24 09:36, Christoph Hellwig wrote: > Initialize the local variables for the discard max sectors and > granularity to zero as a sensible default, and then merge the > calls assigning them to the queue limits. > > Signed-off-by: Christoph Hellwig > Looks good. Reviewed-by: Chaitanya Kulkar

Re: [PATCH 13/15] loop: cleanup loop_config_discard

2024-01-22 Thread Damien Le Moal
On 1/23/24 02:36, Christoph Hellwig wrote: > Initialize the local variables for the discard max sectors and > granularity to zero as a sensible default, and then merge the > calls assigning them to the queue limits. > > Signed-off-by: Christoph Hellwig Looks OK to me. Reviewed-by: Damien Le Moa

[PATCH 13/15] loop: cleanup loop_config_discard

2024-01-22 Thread Christoph Hellwig
Initialize the local variables for the discard max sectors and granularity to zero as a sensible default, and then merge the calls assigning them to the queue limits. Signed-off-by: Christoph Hellwig --- drivers/block/loop.c | 27 --- 1 file changed, 8 insertions(+), 19 d