Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1288953 > unrolled thread

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

Started byLee Duncan <lduncan@suse.com>
First post2015-12-10 22:50 +0100
Last post2015-12-11 16:40 +0100
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  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

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

FromLee Duncan <lduncan@suse.com>
Date2015-12-10 22:50 +0100
SubjectRe: [PATCHv4 1/1] SCSI: hosts: update to use ida_simple for host_no management
Message-ID<qEhg6-3j4-19@gated-at.bofh.it>
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?

-- 
Lee Duncan

--
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/

[toc] | [next] | [standalone]


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

FromEwan Milne <emilne@redhat.com>
Date2015-12-11 16:40 +0100
SubjectRe: [PATCHv4 1/1] SCSI: hosts: update to use ida_simple for host_no management
Message-ID<qExXA-6lB-25@gated-at.bofh.it>
In reply to#1288953
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/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web