public inbox for linux-rdma@vger.kernel.org
 help / color / mirror / Atom feed
From: Or Gerlitz <ogerlitz-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
To: Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
Cc: linux-rdma <linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Michael Wang <yun.wang-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
Subject: Re: [PATCH v7 00/23] IB/Verbs: IB Management Helpers
Date: Thu, 30 Apr 2015 08:40:20 +0300	[thread overview]
Message-ID: <5541C044.7090800@mellanox.com> (raw)
In-Reply-To: <1430324895.102408.23.camel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>

On 4/29/2015 7:28 PM, Doug Ledford wrote:
> On Tue, 2015-04-28 at 17:10 +0200, Michael Wang wrote:
>> >Since v6:
>> >   * Thanks to Ira, Devesh for the review and testing:-)
>> >   * Thanks for the comments from Sean, Tom, Jason, Doug, Devesh, Ira,
>> >     Liran:-)  Please remind me if anything missed:-P
>> >   * Use query_protocol() and enum protocol type in 1#
>> >   * Use rdma_protocol_XX() in 2#
>> >   * Drop cma_set_legacy_transport()
>> >   * Reserve rdma_ib_or_iboe() and rdma_node_get_transport()
>> >   * Updated github repository to v7
> I've taken your patchset and threw it into a for-4.2 branch in my repo.
> This will get it 0day testing.  I've also pulled it into my test cluster
> and done minimal testing (bootup, finds devices, gets IP address via dhcp).

Doug,

As I wrote to Michael, this series carries way (way) over Cc: pointers 
in each it's commits. We have never had something like this since this 
stack was formed 10y ago and accepted into 2.6.12 --- and I didn't hear 
any good reason for why doing so (see the thread  for "[PATCH v7 04/23] 
IB/Verbs: Reform IB-core cm") -- can you please provide your opinion on 
the matter?

Or.
--
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:[~2015-04-30  5:40 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-28 15:10 [PATCH v7 00/23] IB/Verbs: IB Management Helpers Michael Wang
2015-04-28 15:10 ` [PATCH v7 01/23] IB/Verbs: Implement new callback query_protocol() Michael Wang
     [not found] ` <1430233823-7075-1-git-send-email-yun.wang-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
2015-04-28 15:10   ` [PATCH v7 02/23] IB/Verbs: Implement raw management helpers Michael Wang
2015-04-28 15:10   ` [PATCH v7 03/23] IB/Verbs: Reform IB-core mad/agent/user_mad Michael Wang
2015-04-28 15:10   ` [PATCH v7 10/23] IB/Verbs: Reform cm related part in IB-core cma/ucm Michael Wang
2015-04-28 15:10   ` [PATCH v7 11/23] IB/Verbs: Reform route related part in IB-core cma Michael Wang
2015-04-28 15:10   ` [PATCH v7 14/23] IB/Verbs: Reform rest " Michael Wang
2015-04-28 15:10   ` [PATCH v7 15/23] IB/Verbs: Use management helper cap_ib_mad() Michael Wang
2015-04-28 15:10   ` [PATCH v7 21/23] IB/Verbs: Use management helper cap_read_multi_sge() Michael Wang
2015-04-28 15:10   ` [PATCH v7 22/23] IB/Verbs: Use management helper cap_af_ib() Michael Wang
2015-04-28 15:10   ` [PATCH v7 23/23] IB/Verbs: Use management helper cap_eth_ah() Michael Wang
2015-05-01  6:34   ` [PATCH v7 00/23] IB/Verbs: IB Management Helpers ira.weiny
     [not found]     ` <20150501063428.GA30098-W4f6Xiosr+yv7QzWx2u06xL4W9x8LtSr@public.gmane.org>
2015-05-01 16:24       ` Hefty, Sean
2015-05-01 16:27         ` Weiny, Ira
2015-05-04  7:36       ` Michael Wang
2015-04-28 15:10 ` [PATCH v7 04/23] IB/Verbs: Reform IB-core cm Michael Wang
2015-04-28 19:02   ` Or Gerlitz
     [not found]     ` <CAJ3xEMhOg1xh5dXburjavqcy3crwWs6OoF2qEojiDxDhT-MCDw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-04-29  7:40       ` Michael Wang
2015-04-29  8:17         ` Or Gerlitz
2015-04-29 15:48         ` Or Gerlitz
2015-04-30  7:28           ` Michael Wang
2015-04-28 15:10 ` [PATCH v7 05/23] IB/Verbs: Reform IB-core sa_query Michael Wang
2015-04-28 15:10 ` [PATCH v7 06/23] IB/Verbs: Reform IB-core multicast Michael Wang
2015-04-28 15:10 ` [PATCH v7 07/23] IB/Verbs: Reform IB-ulp ipoib Michael Wang
2015-04-28 15:10 ` [PATCH v7 08/23] IB/Verbs: Reform IB-ulp xprtrdma Michael Wang
2015-04-28 15:10 ` [PATCH v7 09/23] IB/Verbs: Reform IB-core verbs Michael Wang
2015-04-28 15:10 ` [PATCH v7 12/23] IB/Verbs: Reform mcast related part in IB-core cma Michael Wang
2015-04-28 15:10 ` [PATCH v7 13/23] IB/Verbs: Reform cma_acquire_dev() Michael Wang
2015-04-28 15:10 ` [PATCH v7 16/23] IB/Verbs: Use management helper cap_ib_smi() Michael Wang
2015-04-28 15:10 ` [PATCH v7 17/23] IB/Verbs: Use management helper cap_ib_cm() Michael Wang
2015-04-28 15:10 ` [PATCH v7 18/23] IB/Verbs: Use management helper cap_iw_cm() Michael Wang
2015-04-28 15:10 ` [PATCH v7 19/23] IB/Verbs: Use management helper cap_ib_sa() Michael Wang
2015-04-28 15:10 ` [PATCH v7 20/23] IB/Verbs: Use management helper cap_ib_mcast() Michael Wang
2015-04-29 16:28 ` [PATCH v7 00/23] IB/Verbs: IB Management Helpers Doug Ledford
     [not found]   ` <1430324895.102408.23.camel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-04-30  5:40     ` Or Gerlitz [this message]
     [not found]       ` <5541C044.7090800-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-04-30  7:00         ` Doug Ledford
     [not found]           ` <1430377216.102408.47.camel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2015-04-30  7:24             ` Or Gerlitz
     [not found]               ` <5541D8CB.8050305-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-04-30  8:11                 ` Michael Wang
     [not found]                   ` <5541E3BE.2080708-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>
2015-04-30  8:23                     ` Or Gerlitz
     [not found]                       ` <5541E67E.4030906-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2015-04-30  8:28                         ` Michael Wang
2015-04-30  7:48     ` Michael Wang

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=5541C044.7090800@mellanox.com \
    --to=ogerlitz-vpraknaxozvwk0htik3j/w@public.gmane.org \
    --cc=dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=yun.wang-EIkl63zCoXaH+58JC4qpiA@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