All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mike Christie <michaelc@cs.wisc.edu>
To: James.Smart@Emulex.Com
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 22:26:54 -0400	[thread overview]
Message-ID: <44E7C86E.1030302@cs.wisc.edu> (raw)
In-Reply-To: <44E6FC57.5050502@emulex.com>

James Smart wrote:
> 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 ?
> 

I was looking for a persistent per hba value for some other long reason
that we can work around in userspace so I take back that comment.

Are you only sending the vendor id to match some vendor specific
userspace code with the event? Currently, for iscsi we only send the
host no then match the driver, host and some driver or vendor specific
code by using the host no and proc_name attr. This is because for
software iscsi and iser we do not know the pci device that will be used.
For software iscsi it could be a bonded device or multiple devices
depending on tables getting updated. What would we use for the vendor id
in this case? Is there a null value or some special unused one we can
abuse or why do you need the vendor id value if you can look that up in
sysfs by just following the host no to the scsi host dir then going to
the pci device?

  reply	other threads:[~2006-08-20  3:27 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
2006-08-20  2:26     ` Mike Christie [this message]
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=44E7C86E.1030302@cs.wisc.edu \
    --to=michaelc@cs.wisc.edu \
    --cc=James.Smart@Emulex.Com \
    --cc=linux-scsi@vger.kernel.org \
    /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.