From: ira.weiny-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org
To: dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org
Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
dgoodell-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org,
devesh.sharma-1wcpHE2jlwO1Z/+hSey0Gg@public.gmane.org,
faisal.latif-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org,
matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org,
leonro-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org,
swise-ut6Up61K2wZBDgjK7y7TUQ@public.gmane.org,
Ira Weiny <ira.weiny-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Subject: [PATCH 00/13] Create device FW version call back
Date: Thu, 9 Jun 2016 17:53:55 -0400 [thread overview]
Message-ID: <1465509248-11324-1-git-send-email-ira.weiny@intel.com> (raw)
From: Ira Weiny <ira.weiny-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Currently there are 3 ways to get the firmware version for an
RDMA device.
1) verbs query device
2) ethtool
3) sysfs entry
Both 1 and 2 are expecting a 64bit field which is vendor specific
and encodes a 3 number version. Some hardware, like OPA, only
has 2 numbers in it's version, while other hardware like cxgb4
has 4.
Support for the sysfs entry was left to the devices but generally
done in a "common" way.
This series moves to unify firmware version reporting by creating
the ib_get_device_fw_str method within the ib_device. With this
we can then do 3 things.
1) Support device dependent formats of version strings with
ethtool (IPoIB)
2) Support a common sysfs entry in the core for backwards
compatibility which can either be changed to a common
FW version string with the new verbs interface or simply
removed in favor of using ethtool.
3) Eventually drop the sysfs entry once we can get user space
tools off of it. If we decide not to have an interface
through verbs (require the use of ethtool via IPoIB) iWarp
and RoCE devices don't even have to have this support at
all and the code can be eliminated for those devices.
1 and 2 are done with this patch series. 3 needs to be decided
by the community if, and when, sysfs support can be dropped.
NOTE: I have tested these patches with mlx4, qib, and hfi1.
Other drivers were compile tested only.
Ira Weiny (13):
IB/core: Add get FW version string to the core
IB/cxgb3: Support device FW version string
IB/cxgb4: Support device FW version string
IB/i40iw: Support device FW version string
IB/mlx4: Support device FW version string
IB/mlx5: Support device FW version string
IB/mthca: Supprot device FW version string
IB/nes: Support device FW version string
IB/ocrdma: Support device FW version string
IB/usnic: Support device FW version string
IB/ipoib: Use new device FW version string
IB/core: Export a common fw_ver sysfs entry
IB/hfi1: Add device FW version string
drivers/infiniband/core/device.c | 9 ++++++++
drivers/infiniband/core/sysfs.c | 16 +++++++++++++-
drivers/infiniband/hw/cxgb3/iwch_provider.c | 27 +++++++++++------------
drivers/infiniband/hw/cxgb4/provider.c | 31 +++++++++++++-------------
drivers/infiniband/hw/hfi1/hfi.h | 2 ++
drivers/infiniband/hw/hfi1/verbs.c | 15 +++++++++++++
drivers/infiniband/hw/i40iw/i40iw_verbs.c | 24 +++++++++-----------
drivers/infiniband/hw/mlx4/main.c | 24 ++++++++++----------
drivers/infiniband/hw/mlx5/main.c | 21 +++++++++---------
drivers/infiniband/hw/mthca/mthca_provider.c | 24 ++++++++++----------
drivers/infiniband/hw/nes/nes_verbs.c | 33 ++++++++++++----------------
drivers/infiniband/hw/ocrdma/ocrdma_main.c | 19 ++++++++--------
drivers/infiniband/hw/usnic/usnic_ib_main.c | 16 ++++++++++++++
drivers/infiniband/hw/usnic/usnic_ib_sysfs.c | 17 --------------
drivers/infiniband/ulp/ipoib/ipoib_ethtool.c | 6 ++---
include/rdma/ib_verbs.h | 3 +++
16 files changed, 157 insertions(+), 130 deletions(-)
--
1.8.2
--
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
next reply other threads:[~2016-06-09 21:53 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-09 21:53 ira.weiny-ral2JQCrhuEAvxtiuMwx3w [this message]
[not found] ` <1465509248-11324-1-git-send-email-ira.weiny-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2016-06-09 21:53 ` [PATCH 01/13] IB/core: Add get FW version string to the core ira.weiny-ral2JQCrhuEAvxtiuMwx3w
[not found] ` <1465509248-11324-2-git-send-email-ira.weiny-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2016-06-09 22:29 ` Jason Gunthorpe
[not found] ` <20160609222919.GA13366-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-06-14 2:45 ` ira.weiny
2016-06-09 21:53 ` [PATCH 02/13] IB/cxgb3: Support device FW version string ira.weiny-ral2JQCrhuEAvxtiuMwx3w
[not found] ` <1465509248-11324-3-git-send-email-ira.weiny-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2016-06-10 13:49 ` Steve Wise
2016-06-09 21:53 ` [PATCH 03/13] IB/cxgb4: " ira.weiny-ral2JQCrhuEAvxtiuMwx3w
[not found] ` <1465509248-11324-4-git-send-email-ira.weiny-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2016-06-10 13:50 ` Steve Wise
2016-06-09 21:53 ` [PATCH 04/13] IB/i40iw: " ira.weiny-ral2JQCrhuEAvxtiuMwx3w
2016-06-09 21:54 ` [PATCH 05/13] IB/mlx4: " ira.weiny-ral2JQCrhuEAvxtiuMwx3w
2016-06-09 21:54 ` [PATCH 06/13] IB/mlx5: " ira.weiny-ral2JQCrhuEAvxtiuMwx3w
2016-06-09 21:54 ` [PATCH 07/13] IB/mthca: Supprot " ira.weiny-ral2JQCrhuEAvxtiuMwx3w
2016-06-09 21:54 ` [PATCH 08/13] IB/nes: Support " ira.weiny-ral2JQCrhuEAvxtiuMwx3w
2016-06-09 21:54 ` [PATCH 09/13] IB/ocrdma: " ira.weiny-ral2JQCrhuEAvxtiuMwx3w
2016-06-09 21:54 ` [PATCH 10/13] IB/usnic: " ira.weiny-ral2JQCrhuEAvxtiuMwx3w
2016-06-09 21:54 ` [PATCH 11/13] IB/ipoib: Use new " ira.weiny-ral2JQCrhuEAvxtiuMwx3w
2016-06-09 21:54 ` [PATCH 12/13] IB/core: Export a common fw_ver sysfs entry ira.weiny-ral2JQCrhuEAvxtiuMwx3w
[not found] ` <1465509248-11324-13-git-send-email-ira.weiny-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2016-06-09 22:40 ` Jason Gunthorpe
[not found] ` <20160609224038.GB13366-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2016-06-10 14:00 ` Leon Romanovsky
[not found] ` <20160610140059.GB5408-2ukJVAZIZ/Y@public.gmane.org>
2016-06-14 2:56 ` ira.weiny
2016-06-14 2:54 ` ira.weiny
2016-06-09 21:54 ` [PATCH 13/13] IB/hfi1: Add device FW version string ira.weiny-ral2JQCrhuEAvxtiuMwx3w
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=1465509248-11324-1-git-send-email-ira.weiny@intel.com \
--to=ira.weiny-ral2jqcrhueavxtiumwx3w@public.gmane.org \
--cc=devesh.sharma-1wcpHE2jlwO1Z/+hSey0Gg@public.gmane.org \
--cc=dgoodell-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org \
--cc=dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
--cc=faisal.latif-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
--cc=leonro-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
--cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
--cc=swise-ut6Up61K2wZBDgjK7y7TUQ@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