public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
From: Patrick Mansfield <patmans@us.ibm.com>
To: Martin Peschke3 <MPESCHKE@de.ibm.com>
Cc: Luben Tuikov <luben@splentec.com>,
	Christoph Hellwig <hch@infradead.org>,
	linux-scsi@vger.kernel.org
Subject: Re: [PATCH] SCSI Core patches
Date: Tue, 14 Jan 2003 10:56:17 -0800	[thread overview]
Message-ID: <20030114105617.B24400@beaverton.ibm.com> (raw)
In-Reply-To: <OF17AF196C.AE6043CD-ONC1256CAE.00549A53@de.ibm.com>; from MPESCHKE@de.ibm.com on Tue, Jan 14, 2003 at 05:19:30PM +0100

On Tue, Jan 14, 2003 at 05:19:30PM +0100, Martin Peschke3 wrote:
> 
> I've been pleading for 64 bit values, too.
> The FCP driver for zSeries I am maintaining for 2.4 uses a mapping
> between 64 bit and 32 bit values. This is ugly and I'd like to get
> rid of it in its 2.5 port if possible.
> 
> Such a change would raise an issue related to device discovery.
> A sparsely populated name space for SCSI ID and LUN makes SCSI
> device detection harder. For LUN, this should be fixed by the usage
> of REPORT_LUNS. Additionally, lldd's could specify the size or
> range of SCSI IDs and LUNs which apply to them in order to reduce
> discovery overhead.

Martin -

The current 2.5 scsi code uses REPORT LUNS, and has a 64 bit to 32 bit
converter (scsi 64 bit lun to linux sdev->lun, the sdev->lun is host byte
ordered), look at scsi_scan.c::scsilun_to_int. scsilun_to_int (it doesn't
handle the address method in the first two bits of the LUN, probably a bug
for some targets).

You can set the shost->max_id and shost->max_lun, and as an ugly hack set
them before the scsi scan occurrs (after scsi_register, before returning
from the detect or before calling the new for 2.5 scsi_add_host).

> I think, the SCSI ID for FCP should also be 64 bit in size and
> ought to be comprised of the World-Wide Port Name (WWPN) rather
> than the more volatile 24 bit Destination ID (D_ID).

I would rather the SCSI target ID be like an address rather than an
identifier (such as might be in a scsi_nexus), right now we have a
sdev->name that currently holds a device id.

-- Patrick Mansfield

  parent reply	other threads:[~2003-01-14 18:56 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-01-14 16:19 [PATCH] SCSI Core patches Martin Peschke3
2003-01-14 16:51 ` Tony Battersby
2003-01-14 18:56 ` Patrick Mansfield [this message]
  -- strict thread matches above, loose matches on Subject: below --
2003-01-15 15:35 Martin Peschke3
2003-01-15 15:52 ` James Bottomley
2003-01-15 17:12   ` Mike Anderson
2003-01-15 17:40     ` Luben Tuikov
2003-01-14 21:29 Martin Peschke3
2003-01-14 22:16 ` Patrick Mansfield
2003-01-14 20:37 Martin Peschke3
2003-01-14 21:27 ` Patrick Mansfield
2003-01-14 20:01 Martin Peschke3
2003-01-14 20:17 ` Patrick Mansfield
2003-01-07 13:56 Luben Tuikov
2003-01-07 18:21 ` Patrick Mansfield
2003-01-07 19:23   ` Luben Tuikov
2003-01-07 20:33     ` Patrick Mansfield
2003-01-07 22:14       ` Luben Tuikov
2003-01-08  1:36         ` Patrick Mansfield
2003-01-08  5:13           ` Luben Tuikov
2003-01-11 18:12           ` Christoph Hellwig
2003-01-13 20:33             ` Patrick Mansfield
2003-01-13 21:30               ` Luben Tuikov
2003-01-14 18:49                 ` Patrick Mansfield
2003-01-14 19:52                   ` Luben Tuikov
2003-01-07 19:44   ` Doug Ledford
2003-01-07 22:53     ` Patrick Mansfield
2003-01-08 17:33       ` Luben Tuikov
2003-01-08 21:13         ` Mike Anderson
2003-01-10 12:35           ` Andre Hedrick
2003-01-10 17:06           ` James Bottomley
2003-01-10 19:23             ` Luben Tuikov
2003-01-10 20:05               ` James Bottomley

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20030114105617.B24400@beaverton.ibm.com \
    --to=patmans@us.ibm.com \
    --cc=MPESCHKE@de.ibm.com \
    --cc=hch@infradead.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=luben@splentec.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox