From: James Smart <James.Smart@Emulex.Com>
To: Eddie Williams <Eddie.Williams@steeleye.com>
Cc: Matthew Wilcox <matthew@wil.cx>, linux-scsi@vger.kernel.org
Subject: Re: [PATCH] Print SCSI Inquiry data more compactly
Date: Wed, 21 Jun 2006 13:18:43 -0400 [thread overview]
Message-ID: <44997F73.7040903@emulex.com> (raw)
In-Reply-To: <1150909340.20655.37.camel@gator.sc.steeleye.com>
I agree with Eddie's points, with another small recommendation:
scsi <2:0:1:0> x00:HP 18.2G:ATLAS10K3_18_SCA:HP05:02:
where x00 is byte 1 of the inquiry data - device type and qualifier.
Clarifies the H:C:T:L as well.
-- james
Eddie Williams wrote:
> Do you need the labels? How about
> scsi 2:0:1:0:DASD:HP 18.2G:ATLAS10K3_18_SCA:HP05:02:
>
> This keeps it short. For those that are probably looking at this data
> the labels are not needed anyway. It is easy to parse if someone wants.
> I used "DASD" just because that is what was referenced before. I think
> I would prefer the number or "direct access" but either way I would not
> lose any sleep.
>
> Eddie
> On Wed, 2006-06-21 at 10:50 -0600, Matthew Wilcox wrote:
>> On Wed, Jun 21, 2006 at 12:41:50PM -0400, Philip R. Auld wrote:
>>>> scsi 2:0:1:0: Device: DASD HP 18.2G ATLAS10K3_18_SCA HP05 ANSI ver: 02
>>> Parsing this, (and the Matt's earlier version), there are 5 tokens
>>> there in the middle for 4 fields. How does one know what the vendor
>>> string is for example?
>> 1) This is not for parsing, it's informational for users. Do tools
>> really still parse through /var/log/dmesg?
>>
>> 2) This information is available other ways, such as through sysfs and
>> procfs.
>>
>> 3) I know it's not obvious from the above example (damn HP firmware),
>> but these are actually fixed-size fields. The first is 8 bytes, "HP 18.2G"
>> the second 16 bytes are "ATLAS10K3_18_SCA", the third is "HP05" and then
>> the ANSI ver is at the end.
>> -
>> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
next prev parent reply other threads:[~2006-06-21 17:18 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-06-21 15:42 [PATCH] Print SCSI Inquiry data more compactly Salyzyn, Mark
2006-06-21 16:41 ` Philip R. Auld
2006-06-21 16:50 ` Matthew Wilcox
2006-06-21 17:02 ` Eddie Williams
2006-06-21 17:18 ` James Smart [this message]
2006-06-21 17:33 ` Matthew Wilcox
2006-06-21 18:04 ` Stefan Richter
2006-06-21 19:03 ` Matthew Wilcox
2006-06-21 22:36 ` Stefan Richter
2006-06-22 2:55 ` Douglas Gilbert
2006-06-21 17:47 ` Philip R. Auld
2006-06-21 22:14 ` Stefan Richter
2006-06-21 22:23 ` Matthew Wilcox
-- strict thread matches above, loose matches on Subject: below --
2006-06-20 22:27 Matthew Wilcox
2006-06-21 14:50 ` Douglas Gilbert
2006-06-21 15:47 ` Matthew Wilcox
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=44997F73.7040903@emulex.com \
--to=james.smart@emulex.com \
--cc=Eddie.Williams@steeleye.com \
--cc=linux-scsi@vger.kernel.org \
--cc=matthew@wil.cx \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.