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 <bart.vanass...@sandisk.com>
> Cc: Christoph Hellwig <h...@lst.de>
> Cc: Hannes Reinecke <h...@suse.com>
> Cc: Johannes Thumshirn <jthumsh...@suse.de>
> ---
>  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 <h...@suse.com>

Cheers,

Hannes

Reply via email to