From: Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
To: Matan Barak <matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Cc: linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Or Gerlitz <ogerlitz-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
Christoph Lameter <cl-vYTEC60ixJUAvxtiuMwx3w@public.gmane.org>,
Eran Ben Elisha <eranbe-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Subject: Re: [PATCH 0/3] IB/mlx4: Fix extended counters support
Date: Thu, 11 Feb 2016 11:08:53 -0500 [thread overview]
Message-ID: <56BCB215.4050802@redhat.com> (raw)
In-Reply-To: <1455179084-4778-1-git-send-email-matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
[-- Attachment #1: Type: text/plain, Size: 1788 bytes --]
On 02/11/2016 03:24 AM, Matan Barak wrote:
> From: Eran Ben Elisha <eranbe-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
>
> Hi Doug,
>
> This series fixes an issue introduced in 4.5 after applying Christoph Lameter's
> "IB 64 bit counter support" series. Christoph used a mad in order to query the
> vendor driver for extended counters support, however this mad wasn't handled in
> mlx4 when link layer is Ethernet. Thus, extended counters support was broken.
> In addition, this series fixes a bug in IB/core sysfs which was also introduced
> in "IB 64 bit counter support" series.
>
> Patch 0001 fixes wrong handling of port info capability mask, which caused
> setting of extended counters sysfs even if underlying device doesn't support
> it.
> Patch 0002-0003 align mlx4 counters sysfs for extended counters scheme in case
> link layer is Ethernet.
>
> Thanks,
> Eran and Matan.
>
> Eran Ben Elisha (3):
> IB/core: Fix reading capability mask of the port info class
> IB/mlx4: Add support for extended counters over RoCE ports
> IB/mlx4: Add support for the port info class for RoCE ports
>
> drivers/infiniband/core/sysfs.c | 5 ++--
> drivers/infiniband/hw/mlx4/mad.c | 63 +++++++++++++++++++++++++++++++---------
> 2 files changed, 52 insertions(+), 16 deletions(-)
>
Hi Matan,
The first patch is a true bug fix while the next two add the feature to
RoCE mode. Based upon the code patches, the RoCE code should work fine
for 4.5 without the new support, there isn't a bug there (they simply
won't respond to the ext counters port request). So I took the first
patch for-rc and the next two I'll put in for-next.
--
Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
GPG KeyID: 0E572FDD
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 884 bytes --]
next prev parent reply other threads:[~2016-02-11 16:08 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-11 8:24 [PATCH 0/3] IB/mlx4: Fix extended counters support Matan Barak
[not found] ` <1455179084-4778-1-git-send-email-matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2016-02-11 8:24 ` [PATCH 1/3] IB/core: Fix reading capability mask of the port info class Matan Barak
[not found] ` <1455179084-4778-2-git-send-email-matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2016-02-11 13:53 ` Christoph Lameter
2016-02-11 8:24 ` [PATCH 2/3] IB/mlx4: Add support for extended counters over RoCE ports Matan Barak
2016-02-11 8:24 ` [PATCH 3/3] IB/mlx4: Add support for the port info class for " Matan Barak
2016-02-11 16:08 ` Doug Ledford [this message]
[not found] ` <56BCB215.4050802-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2016-02-11 16:49 ` [PATCH 0/3] IB/mlx4: Fix extended counters support Matan Barak
[not found] ` <CAAKD3BDQ7x2JzeWrPz+bzG_otRzWmhEmh8cMJ_DXkYGH+SqAzw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-02-11 17:00 ` Christoph Lameter
2016-02-11 17:46 ` Doug Ledford
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=56BCB215.4050802@redhat.com \
--to=dledford-h+wxahxf7alqt0dzr+alfa@public.gmane.org \
--cc=cl-vYTEC60ixJUAvxtiuMwx3w@public.gmane.org \
--cc=eranbe-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
--cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=matanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
--cc=ogerlitz-VPRAkNaXOzVWk0Htik3J/w@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;
as well as URLs for NNTP newsgroup(s).