All of lore.kernel.org
 help / color / mirror / Atom feed
From: James Smart <James.Smart@Emulex.Com>
To: Mike Christie <michaelc@cs.wisc.edu>
Cc: linux-scsi@vger.kernel.org
Subject: Re: [PATCH] [REPOST] SCSI and FC Transport: add netlink support for posting of transport events
Date: Sat, 19 Aug 2006 07:56:07 -0400	[thread overview]
Message-ID: <44E6FC57.5050502@emulex.com> (raw)
In-Reply-To: <44E68F65.2000701@cs.wisc.edu>

How big of a number do you need ?  48 bits ?
We can up to 64bits, but I'd reserve 8bits for a "type" field.
(ugh, sounds like I'm redefining naming authorities...)

On a side thought - is the mac address really the right thing to
use for a vendor id. Wouldn't you be extracting the vendor id from
the mac address ?

-- james

Mike Christie wrote:
> James Smart wrote:
>> +/*
>> + * Vendor ID:
>> + *   If transports post vendor-unique events, they must pass a well-known
>> + *   32-bit vendor identifier. This identifier consists of 8 bits indicating
>> + *   the "type" of identifier contained, and 24 bits of id data.
>> + *
>> + *   Identifiers for each type:
>> + *    PCI :  ID data is the 16 bit PCI Registered Vendor ID
>> + */
>> +#define SCSI_NL_VID_ID_MASK			0x00FFFFFF
>> +#define SCSI_NL_VID_TYPE_MASK			0xFF000000
>> +#define SCSI_NL_VID_TYPE_PCI			0x01000000
>> +
> 
> Would it be possible to make this a little larger? For iscsi we wanted
> to use the MAC address. It can wait. I do not care if it goes in now or
> later. I just don't want to be the first one to break apps by changing
> the number and seeing how many checked version numbers :)
> 

  reply	other threads:[~2006-08-19 12:11 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-08-18 21:30 [PATCH] [REPOST] SCSI and FC Transport: add netlink support for posting of transport events James Smart
2006-08-19  4:11 ` Mike Christie
2006-08-19 11:56   ` James Smart [this message]
2006-08-20  2:26     ` Mike Christie
2006-08-20 13:15       ` James Smart
2006-08-20 13:30         ` James Smart
2006-08-20 22:35         ` Mike Christie
2006-08-20 22:46           ` Mike Christie
2006-08-20 23:01           ` Mike Christie
2006-08-21  0:03         ` Mike Christie
2006-08-30 17:40 ` James Bottomley
2006-08-30 17:51   ` James Smart

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=44E6FC57.5050502@emulex.com \
    --to=james.smart@emulex.com \
    --cc=linux-scsi@vger.kernel.org \
    --cc=michaelc@cs.wisc.edu \
    /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.