Re: [PATCH for-next V5 00/12] Move RoCE GID management to IB/Core

2015-06-11 Thread Matan Barak
On Thu, Jun 11, 2015 at 1:01 AM, Jason Gunthorpe jguntho...@obsidianresearch.com wrote: On Wed, Jun 10, 2015 at 11:19:03PM +0300, Matan Barak wrote: Sure gid_type is gone, but I didn't say roceve2 specific, I said latent elements. ie I'm assuming reasons for the scary locking are because

Re: [PATCH for-next V5 00/12] Move RoCE GID management to IB/Core

2015-06-11 Thread Matan Barak
On Thu, Jun 11, 2015 at 6:57 AM, Jason Gunthorpe jguntho...@obsidianresearch.com wrote: On Wed, Jun 10, 2015 at 09:06:28PM -0400, Doug Ledford wrote: People tend to push the patches should be small, self contained, incremental ideal. In some cases, that gets carried to an extreme. In this

Re: [PATCH for-next V5 00/12] Move RoCE GID management to IB/Core

2015-06-11 Thread Matan Barak
On Thu, Jun 11, 2015 at 3:15 AM, Doug Ledford dledf...@redhat.com wrote: On Wed, 2015-06-10 at 09:00 -0600, Jason Gunthorpe wrote: On Wed, Jun 10, 2015 at 11:53:15AM +0300, Or Gerlitz wrote: Jason, can you ack that this post addressed your comments? Well, I asked for a cleanup series,

Re: [PATCH for-next V5 00/12] Move RoCE GID management to IB/Core

2015-06-11 Thread Matan Barak
On Thu, Jun 11, 2015 at 4:06 AM, Doug Ledford dledf...@redhat.com wrote: On Wed, 2015-06-10 at 12:49 -0600, Jason Gunthorpe wrote: On Wed, Jun 10, 2015 at 06:08:30PM +0300, Matan Barak wrote: It isn't really a cleanup because the whole gid table is new code and has latent elements for rocev2

Re: [patch -next] net/mlx5_core: fix an error code

2015-06-11 Thread Or Gerlitz
On Thu, Jun 11, 2015 at 11:50 AM, Dan Carpenter dan.carpen...@oracle.com wrote: We return success if mlx5e_alloc_sq_db() fails but we should return an error code. Fixes: f62b8bb8f2d3 ('net/mlx5: Extend mlx5_core to support ConnectX-4 Ethernet functionality') Signed-off-by: Dan Carpenter

[patch -next] net/mlx5_core: fix an error code

2015-06-11 Thread Dan Carpenter
We return success if mlx5e_alloc_sq_db() fails but we should return an error code. Fixes: f62b8bb8f2d3 ('net/mlx5: Extend mlx5_core to support ConnectX-4 Ethernet functionality') Signed-off-by: Dan Carpenter dan.carpen...@oracle.com diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_main.c

RE: [PATCH v4 1/1] ibacm: Add support for pathrecord query through netlink

2015-06-11 Thread Wan, Kaike
From: linux-rdma-ow...@vger.kernel.org [mailto:linux-rdma- ow...@vger.kernel.org] On Behalf Of Hal Rosenstock Sent: Thursday, June 11, 2015 10:15 AM On 6/9/2015 10:57 AM, kaike@intel.com wrote: From: Kaike Wan kaike@intel.com This patch enables ibacm to process pathrecord

Re: [PATCH v4 1/1] ibacm: Add support for pathrecord query through netlink

2015-06-11 Thread Hal Rosenstock
On 6/9/2015 10:57 AM, kaike@intel.com wrote: From: Kaike Wan kaike@intel.com This patch enables ibacm to process pathrecord queries through netlink. Since ibacm can cache pathrecords, this implementation provides an easy pathrecord cache for kernel components and therefore offers

Re: [PATCH for-4.1 0/2] Adds support for user mode bar2 mapping and bar2 qid density

2015-06-11 Thread Doug Ledford
On Thu, 2015-06-11 at 09:08 -0500, Steve Wise wrote: -Original Message- From: linux-rdma-ow...@vger.kernel.org [mailto:linux-rdma-ow...@vger.kernel.org] On Behalf Of Doug Ledford Sent: Thursday, June 11, 2015 12:01 AM To: Hariprasad Shenai Cc: linux-rdma@vger.kernel.org;

RE: [PATCH for-4.1 0/2] Adds support for user mode bar2 mapping and bar2 qid density

2015-06-11 Thread Steve Wise
-Original Message- From: linux-rdma-ow...@vger.kernel.org [mailto:linux-rdma-ow...@vger.kernel.org] On Behalf Of Doug Ledford Sent: Thursday, June 11, 2015 12:01 AM To: Hariprasad Shenai Cc: linux-rdma@vger.kernel.org; sw...@opengridcomputing.com; lee...@chelsio.com;

Re: [PATCH 05/36] HMM: introduce heterogeneous memory management v3.

2015-06-11 Thread Jerome Glisse
On Wed, Jun 10, 2015 at 06:15:08PM -0700, Mark Hairgrove wrote: [...] There is no race here, the mirror struct will only be freed once as again the list is a synchronization point. Whoever remove the mirror from the list is responsible to drop the list reference. In the fixed code the

Re: [PATCH v4 1/1] ibacm: Add support for pathrecord query through netlink

2015-06-11 Thread Hal Rosenstock
On 6/11/2015 10:25 AM, Wan, Kaike wrote: +static void acm_nl_process_resolve(struct acmc_client *client, + struct acm_nl_msg *acmnlmsg) +{ + struct acm_msg msg; + struct nlattr *attr; + int payload_len; + int rem; + int total_attr_len; + +

Re: [PATCH for-next V5 07/12] IB/core: Add RoCE table bonding support

2015-06-11 Thread Matan Barak
On Thu, Jun 11, 2015 at 9:18 AM, Jason Gunthorpe jguntho...@obsidianresearch.com wrote: On Mon, Jun 08, 2015 at 05:12:10PM +0300, Matan Barak wrote: +static enum bonding_slave_state is_eth_active_slave_of_bonding(struct net_device *idev, +

RE: [PATCH v4 1/1] ibacm: Add support for pathrecord query through netlink

2015-06-11 Thread Wan, Kaike
From: Hal Rosenstock [mailto:h...@dev.mellanox.co.il] Sent: Thursday, June 11, 2015 11:53 AM To: Wan, Kaike Cc: linux-rdma@vger.kernel.org; Fleck, John; Weiny, Ira Subject: Re: [PATCH v4 1/1] ibacm: Add support for pathrecord query through netlink On 6/11/2015 10:25 AM, Wan, Kaike wrote:

RE: [PATCH v4 4/4] IB/sa: Route SA pathrecord query through netlink

2015-06-11 Thread Weiny, Ira
If user space PR capabilities (unicast, multicast, both) is supported, it affects the API. How? If we can query SA for multicast PRs without joining the multicast groups, what additional changes in netlink API do we need to support both? Nothing to support both but if we wanted to

[PATCH] IB/ucm: ib_ucm_release_dev clears wrong bit if devnum is greater than IB_UCM_MAX_DEVICES

2015-06-11 Thread clsoto
From: Carol L Soto cls...@linux.vnet.ibm.com ib_ucm_release_dev clears wrong bit if devnum is greater than IB_UCM_MAX_DEVICES. Signed-off-by: Carol L Soto cls...@linux.vnet.ibm.com --- drivers/infiniband/core/ucm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [PATCH for-next V5 00/12] Move RoCE GID management to IB/Core

2015-06-11 Thread Jason Gunthorpe
On Thu, Jun 11, 2015 at 12:49:14PM +0300, Matan Barak wrote: No, that's not true. For example cm_init_av_by_path calls ib_find_cached_gid *in a spinlock* which in turns go to the roce_gid_table_find_gid. So it's obvious you can't use a semaphore/mutex. Of course, spin lock is the appropriate

[PATCH v5 1/4] IB/netlink: Add defines for local service requests through netlink

2015-06-11 Thread kaike . wan
From: Kaike Wan kaike@intel.com This patch adds netlink defines for SA client, local service group, local service operations, and related attributes. Signed-off-by: Kaike Wan kaike@intel.com Signed-off-by: John Fleck john.fl...@intel.com Signed-off-by: Ira Weiny ira.we...@intel.com

[PATCH v5 2/4] IB/core: Check the presence of netlink multicast group listeners

2015-06-11 Thread kaike . wan
From: Kaike Wan kaike@intel.com This patch adds a function to check if listeners for a netlink multicast group are present. Signed-off-by: Kaike Wan kaike@intel.com Signed-off-by: John Fleck john.fl...@intel.com Signed-off-by: Ira Weiny ira.we...@intel.com Reviewed-by: Sean Hefty

[PATCH v5 3/4] IB/sa: Allocate SA query with kzalloc

2015-06-11 Thread kaike . wan
From: Kaike Wan kaike@intel.com Replace kmalloc with kzalloc so that all uninitialized fields in SA query will be zero-ed out to avoid unintentional consequence. This prepares the SA query structure to accept new fields in the future. Signed-off-by: Kaike Wan kaike@intel.com

[PATCH v5 0/4] Sending kernel pathrecord query to user cache server

2015-06-11 Thread kaike . wan
From: Kaike Wan kaike@intel.com A SA cache is undeniably critical for fabric scalability and performance. In user space, the ibacm application provides a good example of pathrecord cache for address and route resolution. With the recent implementation of the provider architecture, ibacm

[PATCH v5 4/4] IB/sa: Route SA pathrecord query through netlink

2015-06-11 Thread kaike . wan
From: Kaike Wan kaike@intel.com This patch routes a SA pathrecord query to netlink first and processes the response appropriately. If a failure is returned, the request will be sent through IB. The decision whether to route the request to netlink first is determined by the presence of a

[PATCH v5 1/1] ibacm: Add support for pathrecord query through netlink

2015-06-11 Thread kaike . wan
From: Kaike Wan kaike@intel.com This patch enables ibacm to process pathrecord queries through netlink. Since ibacm can cache pathrecords, this implementation provides an easy pathrecord cache for kernel components and therefore offers great performance advantage on large fabric systems.

Re: [PATCH for-next V4 1/5] IB/uverbs: Fix reference counting usage of event files

2015-06-11 Thread Jason Gunthorpe
On Thu, Jun 11, 2015 at 03:56:50PM +0300, Yishai Hadas wrote: - file-async_file = filp-private_data; - - INIT_IB_EVENT_HANDLER(file-event_handler, file-device-ib_dev, - ib_uverbs_event_handler); - ret = ib_register_event_handler(file-event_handler); -

Re: [PATCH v6 0/3] linux: address broken PAT drivers

2015-06-11 Thread Luis R. Rodriguez
On Mon, Jun 08, 2015 at 09:57:12PM -0300, Mauro Carvalho Chehab wrote: Em Mon, 08 Jun 2015 17:20:19 -0700 Luis R. Rodriguez mcg...@do-not-panic.com escreveu: From: Luis R. Rodriguez mcg...@suse.com Mauro, since the ivtv patch is already acked by the driver maintainer and depends

Re: [PATCH for-next V4 3/5] IB/uverbs: Enable device removal when there are active user space applications

2015-06-11 Thread Jason Gunthorpe
On Thu, Jun 11, 2015 at 03:56:52PM +0300, Yishai Hadas wrote: Enables the uverbs_remove_one to succeed despite the fact that there are running IB applications working with the given ib device. This functionality enables a HW device to be unbind/reset despite the fact that there are running

Re: [PATCH for-next V3 0/8] Add completion timestamping support

2015-06-11 Thread Jason Gunthorpe
On Thu, Jun 11, 2015 at 04:35:19PM +0300, Or Gerlitz wrote: Jason had another comment calling for re-thinking / questioning the need for a dedicated uverbs_ex_cmd_mask in the IB device. This goes beyond the scope of this specific series. How is properly setting up the new verbs you added

Re: [PATCH for-next V3 1/8] IB/core: Change provider's API of create_cq to be extendible

2015-06-11 Thread Jason Gunthorpe
On Thu, Jun 11, 2015 at 04:35:20PM +0300, Or Gerlitz wrote: From: Matan Barak mat...@mellanox.com Add a new ib_cq_init_attr structure which contains the previous cqe (minimum number of CQ entries) and comp_vector (completion vector) in addition to a new flags field. All vendors' create_cq

[PATCH v6 1/3] ivtv: use arch_phys_wc_add() and require PAT disabled

2015-06-11 Thread Luis R. Rodriguez
From: Luis R. Rodriguez mcg...@suse.com We are burrying direct access to MTRR code support on x86 in order to take advantage of PAT. In the future we also want to make the default behaviour of ioremap_nocache() to use strong UC, use of mtrr_add() on those systems would make write-combining void.

[PATCH v6 0/3] linux: address broken PAT drivers

2015-06-11 Thread Luis R. Rodriguez
From: Luis R. Rodriguez mcg...@suse.com Mauro, since the ivtv patch is already acked by the driver maintainer and depends on an x86 symbol that went through Boris' tree are you OK in it going through Boris' tree? Boris, provided the outcome of the above maintainer's preference for you to merge

[PATCH v6 3/3] IB/ipath: use arch_phys_wc_add() and require PAT disabled

2015-06-11 Thread Luis R. Rodriguez
From: Luis R. Rodriguez mcg...@suse.com We are burrying direct access to MTRR code support on x86 in order to take advantage of PAT. In the future we also want to make the default behaviour of ioremap_nocache() to use strong UC, use of mtrr_add() on those systems would make write-combining void.

[PATCH v6 2/3] IB/ipath: add counting for MTRR

2015-06-11 Thread Luis R. Rodriguez
From: Luis R. Rodriguez mcg...@suse.com There is no good reason not to, we eventually delete it as well. Cc: Toshi Kani toshi.k...@hp.com Cc: Roland Dreier rol...@kernel.org Cc: Sean Hefty sean.he...@intel.com Cc: Hal Rosenstock hal.rosenst...@gmail.com Cc: Suresh Siddha sbsid...@gmail.com Cc:

RE: [PATCH 14/14] IB/mad: Add final OPA MAD processing

2015-06-11 Thread Liran Liss
From: Doug Ledford [mailto:dledf...@redhat.com] OPA cannot impersonate IB; OPA node and link types have to be designated as such. In terms of MAD processing flows, both explicit (as in the handle_opa_smi() call below) and implicit code paths (which share IB flows - there are

RE: [PATCH v5 1/1] ibacm: Add support for pathrecord query through netlink

2015-06-11 Thread Wan, Kaike
From: Jason Gunthorpe [mailto:jguntho...@obsidianresearch.com] Sent: Thursday, June 11, 2015 2:00 PM To: Wan, Kaike Cc: linux-rdma@vger.kernel.org; Fleck, John; Weiny, Ira Subject: Re: [PATCH v5 1/1] ibacm: Add support for pathrecord query through netlink On Thu, Jun 11, 2015 at

RE: [PATCH v5 1/4] IB/netlink: Add defines for local service requests through netlink

2015-06-11 Thread Wan, Kaike
From: Jason Gunthorpe [mailto:jguntho...@obsidianresearch.com] Sent: Thursday, June 11, 2015 1:58 PM To: Wan, Kaike Cc: linux-rdma@vger.kernel.org; Fleck, John; Weiny, Ira Subject: Re: [PATCH v5 1/4] IB/netlink: Add defines for local service requests through netlink On Thu, Jun 11, 2015

Re: [PATCH v5 1/4] IB/netlink: Add defines for local service requests through netlink

2015-06-11 Thread Jason Gunthorpe
On Thu, Jun 11, 2015 at 06:32:44PM +, Wan, Kaike wrote: Why are you sending this again without responding the comments I provided on this patch? Are you referring to the mandatory/option attributes comments? Hum weird, I typed in a message yesterday but it seems to have evaporated. I

Re: [PATCH v5 1/1] ibacm: Add support for pathrecord query through netlink

2015-06-11 Thread Jason Gunthorpe
On Thu, Jun 11, 2015 at 06:31:53PM +, Wan, Kaike wrote: 1. all mandatory attributes must be present in the message. Otherwise, the request is invalid. All mandatory attributes that are present must be supported or the user space side cannot answer the request. 2. Optional attributes are

Re: [PATCH v5 1/4] IB/netlink: Add defines for local service requests through netlink

2015-06-11 Thread Jason Gunthorpe
On Thu, Jun 11, 2015 at 06:32:44PM +, Wan, Kaike wrote: From: Jason Gunthorpe [mailto:jguntho...@obsidianresearch.com] Sent: Thursday, June 11, 2015 1:58 PM To: Wan, Kaike Cc: linux-rdma@vger.kernel.org; Fleck, John; Weiny, Ira Subject: Re: [PATCH v5 1/4] IB/netlink: Add defines for

RE: [PATCH v5 1/1] ibacm: Add support for pathrecord query through netlink

2015-06-11 Thread Wan, Kaike
From: Jason Gunthorpe [mailto:jguntho...@obsidianresearch.com] Sent: Thursday, June 11, 2015 2:47 PM To: Wan, Kaike Cc: linux-rdma@vger.kernel.org; Fleck, John; Weiny, Ira Subject: Re: [PATCH v5 1/1] ibacm: Add support for pathrecord query through netlink On Thu, Jun 11, 2015 at

RE: [PATCH v5 1/4] IB/netlink: Add defines for local service requests through netlink

2015-06-11 Thread Wan, Kaike
From: Jason Gunthorpe [mailto:jguntho...@obsidianresearch.com] Sent: Thursday, June 11, 2015 2:52 PM To: Wan, Kaike Cc: linux-rdma@vger.kernel.org; Fleck, John; Weiny, Ira Subject: Re: [PATCH v5 1/4] IB/netlink: Add defines for local service requests through netlink On Thu, Jun 11, 2015

Re: [PATCH v5 1/1] ibacm: Add support for pathrecord query through netlink

2015-06-11 Thread Jason Gunthorpe
On Thu, Jun 11, 2015 at 06:52:19PM +, Wan, Kaike wrote: From: Jason Gunthorpe [mailto:jguntho...@obsidianresearch.com] Sent: Thursday, June 11, 2015 2:47 PM To: Wan, Kaike Cc: linux-rdma@vger.kernel.org; Fleck, John; Weiny, Ira Subject: Re: [PATCH v5 1/1] ibacm: Add support for

RE: [PATCH v5 1/1] ibacm: Add support for pathrecord query through netlink

2015-06-11 Thread Wan, Kaike
From: Jason Gunthorpe [mailto:jguntho...@obsidianresearch.com] Sent: Thursday, June 11, 2015 3:27 PM To: Wan, Kaike Cc: linux-rdma@vger.kernel.org; Fleck, John; Weiny, Ira Subject: Re: [PATCH v5 1/1] ibacm: Add support for pathrecord query through netlink On Thu, Jun 11, 2015 at

RE: [PATCH for-next V2 0/9] Add completion timestamping support

2015-06-11 Thread Hefty, Sean
Intel is supporting multicast in hardware. Its just a bad implementation (broadcast and filtering MC groups in the HCA or what was that?) and there is no plan to fix the issues despite the problem being known for quite some time. Also does this mean that libfabric only to supports the

Re: [PATCH v5 1/4] IB/netlink: Add defines for local service requests through netlink

2015-06-11 Thread Jason Gunthorpe
On Thu, Jun 11, 2015 at 07:21:11PM +, Wan, Kaike wrote: Recommend a 'QP Type' with options of: - RC QP: Return up to the full 6 path tuple with full APM data - UD QP: Return a single non-reversible path - GMP QP: Return up to two reversible GMP paths. If it is hard to get

Re: [PATCH v5 1/1] ibacm: Add support for pathrecord query through netlink

2015-06-11 Thread Jason Gunthorpe
On Thu, Jun 11, 2015 at 07:35:37PM +, Wan, Kaike wrote: #define RDMA_NLA_F_MANDATORY (1 13) #define RDMA_NLA_TYPE_MASK ~(NLA_F_NESTED | NLA_F_NET_BYTEORDER | RDMA_NLA_F_MANDATORY) In this case, kernel will use (RDMA_NLA_F_MANDATORY | type) to inform user space that this attribute

Re: [PATCH v6 2/3] IB/ipath: add counting for MTRR

2015-06-11 Thread Borislav Petkov
On Thu, Jun 11, 2015 at 10:50:01AM -0700, Luis R. Rodriguez wrote: From: Luis R. Rodriguez mcg...@suse.com There is no good reason not to, we eventually delete it as well. Cc: Toshi Kani toshi.k...@hp.com Cc: Roland Dreier rol...@kernel.org Cc: Sean Hefty sean.he...@intel.com Cc: Hal

Re: [PATCH v6 0/3] linux: address broken PAT drivers

2015-06-11 Thread Luis R. Rodriguez
On Thu, Jun 11, 2015 at 10:49:59AM -0700, Luis R. Rodriguez wrote: From: Luis R. Rodriguez mcg...@suse.com Mauro, since the ivtv patch is already acked by the driver maintainer and depends on an x86 symbol that went through Boris' tree are you OK in it going through Boris' tree? Sorry I

RE: [PATCH v4 0/4] Sending kernel pathrecord query to user cache server

2015-06-11 Thread Wan, Kaike
From: Jason Gunthorpe [mailto:jguntho...@obsidianresearch.com] Sent: Wednesday, June 10, 2015 3:15 PM On Wed, Jun 10, 2015 at 05:04:55PM +, Hefty, Sean wrote: Not in the patches themselves but in the general issue when a PR changes. Do you think this needs addressing or are

RE: [PATCH v4 4/4] IB/sa: Route SA pathrecord query through netlink

2015-06-11 Thread Wan, Kaike
From: Hal Rosenstock [mailto:h...@dev.mellanox.co.il] Sent: Thursday, June 11, 2015 8:24 AM To: Weiny, Ira Cc: Jason Gunthorpe; Wan, Kaike; linux-rdma@vger.kernel.org; Fleck, John Subject: Re: [PATCH v4 4/4] IB/sa: Route SA pathrecord query through netlink On 6/10/2015 5:09 PM, Weiny, Ira

[PATCH for-next V3 2/8] IB/core: Change ib_create_cq to use struct ib_cq_init_attr

2015-06-11 Thread Or Gerlitz
From: Matan Barak mat...@mellanox.com Currently, ib_create_cq uses cqe and comp_vecotr instead of the extendible ib_cq_init_attr struct. Earlier patches already changed the vendors to work with ib_cq_init_attr. This patch changes the consumers too. Signed-off-by: Matan Barak mat...@mellanox.com

[PATCH for-next V3 0/8] Add completion timestamping support

2015-06-11 Thread Or Gerlitz
Hi Doug, This patchset adds the kernel control path for completion timestamping support by user-space verbs consumers. Timestamping is used by applications in order to know when a WQE was received/transmitted by the HW. The value is given is HCA hardware cycles, but could be easily converted

[PATCH for-next V3 8/8] IB/mlx4: Add support for CQ time-stamping

2015-06-11 Thread Or Gerlitz
From: Matan Barak mat...@mellanox.com This includes: * support allocation of CQ with the TIMESTAMP_COMPLETION creation flag. * add timestamp_mask and hca_core_clock to query_device, reporting the number of supported timestamp bits (mask) and the hca_core_clock frequency. * return hca core

[PATCH for-next V3 5/8] IB/core: Add timestamp_mask and hca_core_clock to query_device

2015-06-11 Thread Or Gerlitz
From: Matan Barak mat...@mellanox.com In order to expose timestamp we need to expose two new attributes in query_device to be used for CQ completion time-stamping: timestamp_mask - how many bits are valid in the timestamp, where timestamp values could be 64bits the most. hca_core_clock -

[PATCH for-next V3 7/8] IB/mlx4: Add mmap call to map the hardware clock

2015-06-11 Thread Or Gerlitz
From: Matan Barak mat...@mellanox.com In order to read the HCA's cycle counter efficiently in user space, we need to map the HCA's register. This is done through mmap call. Signed-off-by: Matan Barak mat...@mellanox.com Signed-off-by: Or Gerlitz ogerl...@mellanox.com ---

[PATCH for-next V4 2/5] IB/uverbs: Explicitly pass ib_dev to uverbs commands

2015-06-11 Thread Yishai Hadas
From: Shachar Raindel rain...@mellanox.com Done in preparation for deploying RCU for the device removal flow. Allows isolating the RCU handling to the uverb_main layer and keeping the uverbs_cmd code as is. Signed-off-by: Shachar Raindel rain...@mellanox.com Signed-off-by: Yishai Hadas

[PATCH for-next V4 3/5] IB/uverbs: Enable device removal when there are active user space applications

2015-06-11 Thread Yishai Hadas
Enables the uverbs_remove_one to succeed despite the fact that there are running IB applications working with the given ib device. This functionality enables a HW device to be unbind/reset despite the fact that there are running user space applications using it. It exposes a new IB kernel API

[PATCH for-next V4 4/5] IB/mlx4_ib: Disassociate support

2015-06-11 Thread Yishai Hadas
Implements the IB core disassociate_ucontext API. The driver detaches the HW resources for a given user context to prevent a dependency between application termination and device disconnecting. This is done by managing the VMAs that were mapped to the HW bars such as door bell and blueflame. When

[PATCH for-next V4 1/5] IB/uverbs: Fix reference counting usage of event files

2015-06-11 Thread Yishai Hadas
Fix the reference counting usage to be handled in the event file creation/destruction function, instead of being done by the caller. This is done for both async/non-async event files. Based on Jason Gunthorpe report at https://www.mail-archive.com/ linux-rdma@vger.kernel.org/msg24680.html: The

[PATCH for-next V4 5/5] IB/ucma: HW Device hot-removal support

2015-06-11 Thread Yishai Hadas
Currently, IB/cma remove_one flow blocks until all user descriptor managed by IB/ucma are released. This prevents hot-removal of IB devices. This patch allows IB/cma to remove devices regardless of user space activity. Upon getting the RDMA_CM_EVENT_DEVICE_REMOVAL event we close all the underlying

[PATCH for-next V4 0/5] HW Device hot-removal support

2015-06-11 Thread Yishai Hadas
Currently, if there is any user space application using an IB device, it is impossible to unload the HW device driver for this device. Similarly, if the device is hot-unplugged or reset, the device driver hardware removal flow blocks until all user contexts are destroyed. This patchset removes

Re: [PATCH v4 4/4] IB/sa: Route SA pathrecord query through netlink

2015-06-11 Thread Hal Rosenstock
On 6/11/2015 8:54 AM, Wan, Kaike wrote: From: Hal Rosenstock [mailto:h...@dev.mellanox.co.il] Sent: Thursday, June 11, 2015 8:24 AM To: Weiny, Ira Cc: Jason Gunthorpe; Wan, Kaike; linux-rdma@vger.kernel.org; Fleck, John Subject: Re: [PATCH v4 4/4] IB/sa: Route SA pathrecord query through

[PATCH for-next V3 1/8] IB/core: Change provider's API of create_cq to be extendible

2015-06-11 Thread Or Gerlitz
From: Matan Barak mat...@mellanox.com Add a new ib_cq_init_attr structure which contains the previous cqe (minimum number of CQ entries) and comp_vector (completion vector) in addition to a new flags field. All vendors' create_cq callbacks are changed in order to work with the new API. This

Re: [PATCH v4 4/4] IB/sa: Route SA pathrecord query through netlink

2015-06-11 Thread Hal Rosenstock
On 6/10/2015 5:09 PM, Weiny, Ira wrote: On 6/10/2015 3:10 PM, Jason Gunthorpe wrote: On Wed, Jun 10, 2015 at 01:47:36PM -0400, Hal Rosenstock wrote: On 6/9/2015 10:57 AM, kaike@intel.com wrote: From: Kaike Wan kaike@intel.com This patch routes a SA pathrecord query to netlink first

RE: [PATCH v4 1/4] IB/netlink: Add defines for local service requests through netlink

2015-06-11 Thread Wan, Kaike
From: Hefty, Sean Sent: Wednesday, June 10, 2015 4:32 PM To: Hal Rosenstock; Wan, Kaike Cc: linux-rdma@vger.kernel.org; Fleck, John; Weiny, Ira Subject: RE: [PATCH v4 1/4] IB/netlink: Add defines for local service requests through netlink +/* Local Service Reversible attribute */ struct

[PATCH for-next V3 6/8] IB/core: Pass hardware specific data in query_device

2015-06-11 Thread Or Gerlitz
From: Matan Barak mat...@mellanox.com Vendors should be able to pass vendor specific data to/from user-space via query_device uverb. In order to do this, we need to pass the vendors' specific udata. Signed-off-by: Matan Barak mat...@mellanox.com Signed-off-by: Or Gerlitz ogerl...@mellanox.com

[PATCH for-next V3 4/8] IB/core: Extend ib_uverbs_create_cq

2015-06-11 Thread Or Gerlitz
From: Matan Barak mat...@mellanox.com ib_uverbs_ex_create_cq follows the extension verbs mechanism. New features (for example, CQ creation flags field which is added in a downstream patch) could used via user-space libraries without breaking the ABI. Signed-off-by: Matan Barak

[PATCH for-next V3 3/8] IB/core: Add CQ creation time-stamping flag

2015-06-11 Thread Or Gerlitz
From: Matan Barak mat...@mellanox.com Add CQ creation flag which dictates that the created CQ will report completion time-stamp value in the WC. Signed-off-by: Matan Barak mat...@mellanox.com Signed-off-by: Or Gerlitz ogerl...@mellanox.com --- include/rdma/ib_verbs.h |4 1 files

RE: [PATCH v4 0/4] Sending kernel pathrecord query to user cache server

2015-06-11 Thread Wan, Kaike
-Original Message- From: Hal Rosenstock [mailto:h...@dev.mellanox.co.il] Sent: Wednesday, June 10, 2015 3:52 PM On 6/10/2015 1:04 PM, Hefty, Sean wrote: Not in the patches themselves but in the general issue when a PR changes. Do you think this needs addressing or are things

RE: [PATCH v5 1/1] ibacm: Add support for pathrecord query through netlink

2015-06-11 Thread Weiny, Ira
On Thu, Jun 11, 2015 at 01:04:25PM -0400, kaike@intel.com wrote: +static int acm_nl_parse_path_attr(struct nlattr *attr, + struct acm_ep_addr_data *data) + switch (attr-nla_type NLA_TYPE_MASK) { + default: + acm_log(1, WARN: unknown attr

[PATCH v7 0/3] linux: address broken PAT drivers

2015-06-11 Thread Luis R. Rodriguez
From: Luis R. Rodriguez mcg...@suse.com Boris, the following patches make use of the newly exported pat_enabled() which went in through your tree. All driver and respective subsystem maintainers have Acked these patches and are OK for them to go in through your tree. Please let me know if there

[PATCH v7 1/3] ivtv: use arch_phys_wc_add() and require PAT disabled

2015-06-11 Thread Luis R. Rodriguez
From: Luis R. Rodriguez mcg...@suse.com We are burrying direct access to MTRR code support on x86 in order to take advantage of PAT. In the future we also want to make the default behaviour of ioremap_nocache() to use strong UC, use of mtrr_add() on those systems would make write-combining void.

[PATCH v7 2/3] IB/ipath: add counting for MTRR

2015-06-11 Thread Luis R. Rodriguez
From: Luis R. Rodriguez mcg...@suse.com There is no good reason not to, we eventually delete it as well. Cc: Toshi Kani toshi.k...@hp.com Cc: Roland Dreier rol...@kernel.org Cc: Sean Hefty sean.he...@intel.com Cc: Hal Rosenstock hal.rosenst...@gmail.com Cc: Suresh Siddha sbsid...@gmail.com Cc:

[PATCH v7 3/3] IB/ipath: use arch_phys_wc_add() and require PAT disabled

2015-06-11 Thread Luis R. Rodriguez
From: Luis R. Rodriguez mcg...@suse.com We are burrying direct access to MTRR code support on x86 in order to take advantage of PAT. In the future we also want to make the default behaviour of ioremap_nocache() to use strong UC, use of mtrr_add() on those systems would make write-combining void.

RE: [PATCH v5 1/1] ibacm: Add support for pathrecord query through netlink

2015-06-11 Thread Weiny, Ira
On Thu, Jun 11, 2015 at 08:16:41PM +, Weiny, Ira wrote: For instance if we add SL it would be mandatory, but policy information like requesting net_device would be optional. Why would SL be mandatory? If the kernel asks for a specific SL then user space must respect that and

RE: [PATCH 14/14] IB/mad: Add final OPA MAD processing

2015-06-11 Thread Hefty, Sean
cap_is_switch_smi would be a nice refinement to let us drop nodetype. Exactly, we need a bit added to the immutable data bits, and a new cap_ helper, and then nodetype is ready to be retired. Add a bit, drop a u8 ;-) I agree that the node type enum isn't particularly useful and should be

[PATCH 09/41] IB/hfi1: add low lower receive functions

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

[PATCH 12/41] IB/hfi1: add firmware hooks

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

[PATCH 00/41] Add OPA gen1 driver

2015-06-11 Thread Mike Marciniszyn
This patch series adds the OPA gen1 driver. The patch depends on the this MAD patch series: https://patchwork.kernel.org/patch/6584551/ The patches add the files and headers and the last patch add the build hooks. --- Jubin John (1): IB/core: Add opa driver to kbuild Mike Marciniszyn

[PATCH 01/41] IB/core: Add OPA Port header definitions

2015-06-11 Thread Mike Marciniszyn
Add common OPA header definitions for driver build: - opa_port_info.h - opa_smi.h - hfi1_user.sh Signed-off-by: Ira Weiny ira.we...@intel.com Signed-off-by: Jubin John jubin.j...@intel.com Signed-off-by: Mike Marciniszyn mike.marcinis...@intel.com --- include/rdma/opa_port_info.h | 452

[PATCH 03/41] IB/hfi1: add common header file definitions

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

[PATCH 04/41] IB/hfi1: add completion queue processing

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

[PATCH 05/41] IB/hfi1: add debugfs handling

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

[PATCH 10/41] IB/hfi1: add eeprom hooks

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

RE: [PATCH v5 1/4] IB/netlink: Add defines for local service requests through netlink

2015-06-11 Thread Wan, Kaike
From: Jason Gunthorpe [mailto:jguntho...@obsidianresearch.com] Sent: Thursday, June 11, 2015 3:52 PM To: Wan, Kaike Cc: linux-rdma@vger.kernel.org; Fleck, John; Weiny, Ira Subject: Re: [PATCH v5 1/4] IB/netlink: Add defines for local service requests through netlink On Thu, Jun 11, 2015

Re: [PATCH 05/36] HMM: introduce heterogeneous memory management v3.

2015-06-11 Thread Mark Hairgrove
On Thu, 11 Jun 2015, Jerome Glisse wrote: On Wed, Jun 10, 2015 at 06:15:08PM -0700, Mark Hairgrove wrote: [...] There is no race here, the mirror struct will only be freed once as again the list is a synchronization point. Whoever remove the mirror from the list is responsible to

Re: [patch -next] net/mlx5_core: fix an error code

2015-06-11 Thread David Miller
From: Dan Carpenter dan.carpen...@oracle.com Date: Thu, 11 Jun 2015 11:50:01 +0300 We return success if mlx5e_alloc_sq_db() fails but we should return an error code. Fixes: f62b8bb8f2d3 ('net/mlx5: Extend mlx5_core to support ConnectX-4 Ethernet functionality') Signed-off-by: Dan

[PATCH 26/41] IB/hfi1: add qsfp handling

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

[PATCH 37/41] IB/hfi1: add PSM sdma hooks

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

[PATCH 38/41] IB/hfi1: add general verbs handling

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

[PATCH 31/41] IB/hfi1: add sysfs routines

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

[PATCH 36/41] IB/hfi1: add low level page locking

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

[PATCH 30/41] IB/hfi1: add SRQ handling

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

[PATCH 27/41] IB/hfi1: add RC QP handling

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

[PATCH 34/41] IB/hfi1: add UC QP handling

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

[PATCH 35/41] IB/hfi1: add UD QP handling

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

[PATCH 28/41] IB/hfi1: add routines for RC/UC

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

[PATCH 33/41] IB/hfi1: add QSFP twsi routines

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

[PATCH 39/41] IB/hfi1: add multicast routines

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

[PATCH 41/41] IB/core: Add opa driver to kbuild

2015-06-11 Thread Mike Marciniszyn
From: Jubin John jubin.j...@intel.com Signed-off-by: Mike Marciniszyn mike.marcinis...@intel.com --- drivers/infiniband/Kconfig |1 + drivers/infiniband/hw/Makefile |1 + 2 files changed, 2 insertions(+) diff --git a/drivers/infiniband/Kconfig b/drivers/infiniband/Kconfig index

[PATCH 40/41] IB/hfi1: add driver make/config files

2015-06-11 Thread Mike Marciniszyn
Signed-off-by: Andrew Friedley andrew.fried...@intel.com Signed-off-by: Arthur Kepner arthur.kep...@intel.com Signed-off-by: Brendan Cunningham brendan.cunning...@intel.com Signed-off-by: Brian Welty brian.we...@intel.com Signed-off-by: Caz Yokoyama caz.yokoy...@intel.com Signed-off-by: Dean Luick

  1   2   >