Re: [PATCHv4 1/1] SCSI: hosts: update to use ida_simple for host_no management

From	Ewan Milne <emilne@redhat.com>
Newsgroups	linux.kernel
Subject	Re: [PATCHv4 1/1] SCSI: hosts: update to use ida_simple for host_no management
Date	2015-12-11 16:40 +0100
Message-ID	<qExXA-6lB-25@gated-at.bofh.it> (permalink)
References	(6 earlier) <quuy0-3qR-59@gated-at.bofh.it> <qvqVk-7cN-29@gated-at.bofh.it> <qvzOV-4k5-5@gated-at.bofh.it> <qvXRg-3dL-13@gated-at.bofh.it> <qEhg6-3j4-19@gated-at.bofh.it>
Organization	Red Hat

Show all headers | View raw

On Thu, 2015-12-10 at 13:48 -0800, Lee Duncan wrote:
> On 11/17/2015 03:20 PM, Martin K. Petersen wrote:
> >>>>>> "Lee" == Lee Duncan <lduncan@suse.com> writes:
> > 
> > Lee> Martin: I will be glad to update the patch, creating a modprobe
> > Lee> parameter as suggested, if you find this acceptable.
> > 
> > For development use a module parameter would be fine. But I am concerned
> > about our support folks that rely on the incrementing host number when
> > analyzing customer log files.
> > 
> > Ewan: How do you folks feel about this change?
> > 
> 
> Ewan?

Personally, I think having host numbers that increase essentially
without limit (I think I've seen this with iSCSI sessions) are a
problem, the numbers start to lose meaning for people when they
are not easily recognizable.  Yes, it can help when you're analyzing
a log file, but it seems to me that you would want to track the
host state throughout anyway, so you could just follow the number
as it changes.

If we change the behavior, we have to change documentation, and
our support people will get calls.  But that's not a reason not
to do it.

-Ewan

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

Re: [PATCHv4 1/1] SCSI: hosts: update to use ida_simple for host_no  management Lee Duncan <lduncan@suse.com> - 2015-12-10 22:50 +0100
  Re: [PATCHv4 1/1] SCSI: hosts: update to use ida_simple for  host_no management Ewan Milne <emilne@redhat.com> - 2015-12-11 16:40 +0100
    Re: [PATCHv4 1/1] SCSI: hosts: update to use ida_simple for host_no  management Lee Duncan <lduncan@suse.com> - 2015-12-13 20:20 +0100
      Re: [PATCHv4 1/1] SCSI: hosts: update to use ida_simple for  host_no management Ewan Milne <emilne@redhat.com> - 2015-12-14 16:10 +0100
        Re: [PATCHv4 1/1] SCSI: hosts: update to use ida_simple for host_no  management Hannes Reinecke <hare@suse.de> - 2015-12-14 16:30 +0100
          Re: [PATCHv4 1/1] SCSI: hosts: update to use ida_simple for host_no management "Martin K. Petersen" <martin.petersen@oracle.com> - 2015-12-15 03:00 +0100
            Re: [PATCHv4 1/1] SCSI: hosts: update to use ida_simple for host_no  management Lee Duncan <lduncan@suse.com> - 2015-12-17 20:30 +0100

csiph-web