Re: [PATCH] Move the eh_deadline module parameter definition to the top of hosts.c

[Martin K. Petersen]

Bart,

> The eh_deadline definition occurs in the middle of the code for
> releasing a host. Avoid splitting the host release code by moving
> the definition of the eh_deadline parameter to the top of the
> hosts.c source file.

Applied to 4.17/scsi-queue. Thank you!

-- 
Martin K. Petersen  Oracle Linux Engineering

Re: [PATCH] Move the eh_deadline module parameter definition to the top of hosts.c

[Johannes Thumshirn]

Thanks Bart,
Reviewed-by: Johannes Thumshirn 

-- 
Johannes Thumshirn                                          Storage
jthu
msh...@suse.de                                +49 911 74053 689
SUSE
LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Felix Imendörffer, Jane
Smithard, Graham Norton
HRB 21284 (AG Nürnberg)
Key fingerprint = EC38
9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850

Re: [PATCH] Move the eh_deadline module parameter definition to the top of hosts.c

[Hannes Reinecke]

On 02/12/2018 07:41 PM, Bart Van Assche wrote:
> The eh_deadline definition occurs in the middle of the code for
> releasing a host. Avoid splitting the host release code by moving
> the definition of the eh_deadline parameter to the top of the
> hosts.c source file.
> 
> Signed-off-by: Bart Van Assche 
> Cc: Christoph Hellwig 
> Cc: Hannes Reinecke 
> Cc: Johannes Thumshirn 
> ---
>  drivers/scsi/hosts.c | 12 ++--
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/scsi/hosts.c b/drivers/scsi/hosts.c
> index 57bf43e34863..a0a7e4ff255c 100644
> --- a/drivers/scsi/hosts.c
> +++ b/drivers/scsi/hosts.c
> @@ -42,6 +42,12 @@
>  #include "scsi_logging.h"
>  
>  
> +static int shost_eh_deadline = -1;
> +
> +module_param_named(eh_deadline, shost_eh_deadline, int, S_IRUGO|S_IWUSR);
> +MODULE_PARM_DESC(eh_deadline,
> +  "SCSI EH timeout in seconds (should be between 0 and 2^31-1)");
> +
>  static DEFINE_IDA(host_index_ida);
>  
>  
> @@ -358,12 +364,6 @@ static void scsi_host_dev_release(struct device *dev)
>   kfree(shost);
>  }
>  
> -static int shost_eh_deadline = -1;
> -
> -module_param_named(eh_deadline, shost_eh_deadline, int, S_IRUGO|S_IWUSR);
> -MODULE_PARM_DESC(eh_deadline,
> -  "SCSI EH timeout in seconds (should be between 0 and 2^31-1)");
> -
>  static struct device_type scsi_host_type = {
>   .name = "scsi_host",
>   .release =  scsi_host_dev_release,
> 
Reviewed-by: Hannes Reinecke 

Cheers,

Hannes