All of lore.kernel.org
 help / color / mirror / Atom feed
From: Or Gerlitz <ogerlitz-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
To: Roland Dreier <roland-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Cc: Jack Morgenstein
	<jackm-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>,
	linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	liranl-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org,
	yevgenyp-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org,
	dotanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org,
	tziporet-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org
Subject: Re: [PATCH for-next 00/29] Add SRIOV support for IB interfaces
Date: Tue, 12 Jun 2012 10:22:46 +0300	[thread overview]
Message-ID: <4FD6EE46.20900@mellanox.com> (raw)
In-Reply-To: <CAL1RGDVsGUBHUBOaajnzO4NcFjA1xBZczLA=vxV3=oRKme+LrQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>

On 6/12/2012 10:00 AM, Roland Dreier wrote:
> Jack Morgenstein<jackm-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>  wrote:
>> several of the patches have notations indicating things that will be fixed in V1
>
> Not sure what you want me to do with this -- it seems you yourself are saying this series is not ready to merge yet?

Roland,

We observed few last minute issues and preferred not to further defer 
submission,
with the reasons being:

1. be sure that the submitted code passed through regression, not even a 
bit change since

2. the issues we spotted are all fairly simple to fix and V1 will be out 
quickly

3. the design --> resulted code is a bit complex, wanted it to see the 
day light such that review can start (thank you for the quick feedback 
on patch 03)

All in all, V1 is coming soon, and if you prefer starting from there, 
let it be, but
still, will love to get any comments / questions / corrections.

Or.

The known issues to be fixed in V1, with NNN being the patch number:

002 illegal use here of the MSB of the port_num field in the ib core ah 
structure

013 add patch which will indicate some of the most-significant bits in
     ib_qp_create_flags as reserved for low-level driver use.

015 modify the slave_num macro defined here to be usable in mlx4_core as 
well

022 require some slight cleanup

024 require some slight cleanup

029 change the patch to reflect that RoCE is still not supported on slaves

also librdmacm will currently not support multiple VF/PF on the same host,
we will change the patch set to solve that, no change in the library


--
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:[~2012-06-12  7:22 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-11 10:45 [PATCH for-next 00/29] Add SRIOV support for IB interfaces Jack Morgenstein
     [not found] ` <1339411570-4689-1-git-send-email-jackm-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>
2012-06-11 10:45   ` [PATCH for-next 01/29] net/mlx4_core: Pass an invalid PCI id number to VFs Jack Morgenstein
2012-06-11 10:45   ` [PATCH for-next 02/29] IB/mlx4: Mask out high order bit of port_num in mlx4_ib_create_ah Jack Morgenstein
2012-06-11 10:45   ` [PATCH for-next 03/29] IB/mlx4: Add run-time switchable error path debug output capability Jack Morgenstein
     [not found]     ` <1339411570-4689-4-git-send-email-jackm-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org>
2012-06-12  7:00       ` Roland Dreier
2012-06-11 10:45   ` [PATCH for-next 04/29] IB/core: change pkey table lookups to support full and partial membership for the same pkey Jack Morgenstein
2012-06-11 10:45   ` [PATCH for-next 05/29] IB/core: Add ib_find_exact_cached_pkey() to search for 16-bit pkey match Jack Morgenstein
2012-06-11 10:45   ` [PATCH for-next 06/29] IB/sa: Add GuidInfoRecord query support Jack Morgenstein
2012-06-11 10:45   ` [PATCH for-next 07/29] IB/core: move macros from cm_msgs.h to ib_cm.h Jack Morgenstein
2012-06-11 10:45   ` [PATCH for-next 08/29] {NET,IB}/mlx4: Use port management change event instead of smp_snoop Jack Morgenstein
2012-06-11 10:45   ` [PATCH for-next 09/29] net/mlx4_core: For SRIOV, initialize ib port-capabilities for all slaves Jack Morgenstein
2012-06-11 10:45   ` [PATCH for-next 10/29] net/mlx4_core: Implement mechanism for reserved qkeys Jack Morgenstein
2012-06-11 10:45   ` [PATCH for-next 11/29] net/mlx4_core: Allow guests to support IB ports Jack Morgenstein
2012-06-11 10:45   ` [PATCH for-next 12/29] net/mlx4_core: place phys gid and pkey tbl sizes in mlx4_phys_caps struct and paravirtualize them Jack Morgenstein
2012-06-11 10:45   ` [PATCH for-next 13/29] IB/mlx4: SRIOV IB context objects and proxy/tunnel sqp support Jack Morgenstein
2012-06-11 10:45   ` [PATCH for-next 14/29] net/mlx4_core: Add proxy and tunnel QPs to the reserved QP area Jack Morgenstein
2012-06-11 10:45   ` [PATCH for-next 15/29] IB/mlx4: Initialize SRIOV IB support for slaves in master context Jack Morgenstein
2012-06-11 10:45   ` [PATCH for-next 16/29] {NET,IB}/mlx4: Implement QP paravirtualization Jack Morgenstein
2012-06-11 10:45   ` [PATCH for-next 17/29] IB/mlx4: SRIOV multiplex and demultiplex MADs Jack Morgenstein
2012-06-11 10:45   ` [PATCH for-next 18/29] {NET,IB}/mlx4: MAD_IFC paravirtualization Jack Morgenstein
2012-06-11 10:46   ` [PATCH for-next 19/29] IB/mlx4: Added Multicast Groups (MCG) para-virtualization for SRIOV Jack Morgenstein
2012-06-11 10:46   ` [PATCH for-next 20/29] IB/mlx4: Add CM paravirtualization Jack Morgenstein
2012-06-11 10:46   ` [PATCH for-next 21/29] net/mlx4_core: Add IB port-state machine, and port mgmt event propagation infrastructure Jack Morgenstein
2012-06-11 10:46   ` [PATCH for-next 22/29] {NET,IB}/mlx4: Add alias_guid mechanism Jack Morgenstein
2012-06-11 10:46   ` [PATCH for-next 23/29] IB/mlx4: Propagate pkey and guid change port management events to slaves Jack Morgenstein
2012-06-11 10:46   ` [PATCH for-next 24/29] IB/mlx4: Add iov directory in sysfs under the ib device Jack Morgenstein
2012-06-11 10:46   ` [PATCH for-next 25/29] net/mlx4_core: Adjustments to SET_PORT for SRIOV-IB Jack Morgenstein
2012-06-11 10:46   ` [PATCH for-next 26/29] IB/mlx4: Initialize guid-cache index 0 (default guid) Jack Morgenstein
2012-06-11 10:46   ` [PATCH for-next 27/29] net/mlx4_core: INIT/CLOSE port logic for IB ports in SRIOV mode Jack Morgenstein
2012-06-11 10:46   ` [PATCH for-next 28/29] IB/mlx4: Miscellaneous adjustments to SRIOV IB support Jack Morgenstein
2012-06-11 10:46   ` [PATCH for-next 29/29] {NET,IB}/mlx4: Activate SRIOV mode for IB Jack Morgenstein
2012-06-12  7:00   ` [PATCH for-next 00/29] Add SRIOV support for IB interfaces Roland Dreier
     [not found]     ` <CAL1RGDVsGUBHUBOaajnzO4NcFjA1xBZczLA=vxV3=oRKme+LrQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-06-12  7:22       ` Or Gerlitz [this message]
  -- strict thread matches above, loose matches on Subject: below --
2012-06-14 13:21 Yann Droneaud
     [not found] ` <9e8965c740ff0e889a097a722b9fff90.squirrel-2RFepEojUI2lDZmfZ6uX/xeHL2rgt/dS@public.gmane.org>
2012-06-17 14:39   ` Or Gerlitz
     [not found]     ` <4FDDEC16.9060702-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2012-06-18  9:43       ` Yann Droneaud

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=4FD6EE46.20900@mellanox.com \
    --to=ogerlitz-vpraknaxozvwk0htik3j/w@public.gmane.org \
    --cc=dotanb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=jackm-LDSdmyG8hGV8YrgS2mwiifqBs+8SCbDb@public.gmane.org \
    --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=liranl-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=roland-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
    --cc=tziporet-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=yevgenyp-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 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.