public inbox for linux-rdma@vger.kernel.org
 help / color / mirror / Atom feed
From: Roland Dreier <rdreier-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org>
To: Eli Cohen <eli-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>
Cc: vlad-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org,
	linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH] Add new device IDs for ConnectX VPI HCAs
Date: Thu, 25 Feb 2010 15:28:16 -0800	[thread overview]
Message-ID: <adapr3sx5db.fsf@roland-alpha.cisco.com> (raw)
In-Reply-To: <20100225210850.GA26139-8YAHvHwT2UEvbXDkjdHOrw/a8Rv0c6iv@public.gmane.org> (Eli Cohen's message of "Thu, 25 Feb 2010 23:08:50 +0200")

 > > The kernel calls this device ID "MT26438 ConnectX EN 40GigE PCIe gen2
 > > 5GT/s".  Maybe we should just delete these comments, since we can't seem
 > > to get them right?
 > 
 > I guess it would be best if we take the description from
 > http://pciids.sourceforge.net/v2.2/pci.ids

I wonder whether it's really worth having the text at all.  Maybe it's a
bit useful, I'm not sure.  But the pciids text could probably use some
cleaning up, eg:

	634a  MT25418 [ConnectX VPI PCIe 2.0 2.5GT/s - IB DDR / 10GigE]
	6368  MT25448 [ConnectX EN 10GigE, PCIe 2.0 2.5GT/s]

it would be nice to decide on a standard order of whether the PCIe or
the network info comes first.

And then to make things even better we have:

		103c 3313  HP NC542m Dual Port Flex-10 10GbE BLc Adapter

so we can't even decide on 10GigE vs. 10GbE.

But yeah, patches to get the ID list into shape with at least the right
list of device IDs for kernel and userspace would be welcome.
-- 
Roland Dreier  <rolandd-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org>
For corporate legal information go to:
http://www.cisco.com/web/about/doing_business/legal/cri/index.html
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2010-02-25 23:28 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-08 14:04 [PATCH] Add new device IDs for ConnectX VPI HCAs Vladimir Sokolovsky
2010-02-24 22:07 ` Roland Dreier
     [not found]   ` <adazl2yxp7r.fsf-BjVyx320WGW9gfZ95n9DRSW4+XlvGpQz@public.gmane.org>
2010-02-25 21:08     ` Eli Cohen
     [not found]       ` <20100225210850.GA26139-8YAHvHwT2UEvbXDkjdHOrw/a8Rv0c6iv@public.gmane.org>
2010-02-25 23:28         ` Roland Dreier [this message]
     [not found]           ` <adapr3sx5db.fsf-BjVyx320WGW9gfZ95n9DRSW4+XlvGpQz@public.gmane.org>
2010-02-26 13:55             ` Eli Cohen

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=adapr3sx5db.fsf@roland-alpha.cisco.com \
    --to=rdreier-fyb4gu1cfyuavxtiumwx3w@public.gmane.org \
    --cc=eli-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org \
    --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=vlad-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox