[PATCH] staging/hfi1: replace indent spaces with tabs

2015-08-21 Thread Jeff Becker
Running checkpatch.pl on mad.c produces several ERROR: code indent should use tabs where possible messages. This patch fixes these. Signed-off-by: Jeff Becker jeffrey.c.bec...@nasa.gov --- drivers/staging/hfi1/mad.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git

Re: [PATCH WIP 28/43] IB/core: Introduce new fast registration API

2015-08-21 Thread Jason Gunthorpe
On Thu, Aug 20, 2015 at 11:34:58PM -0700, Christoph Hellwig wrote: How is this going to work for drivers that might consumer multiple MRs per request like SRP or similar upcoming block drivers? Unless you want to allocate a potentially large number of MRs for each request that scheme doesn't

[PATCH 1/2] IB/ipoib: Clean up send-only multicast joins

2015-08-21 Thread Jason Gunthorpe
Even though we don't expect the group to be created by the SM we sill need to provide all the parameters to force the SM to validate they are correct. Signed-off-by: Jason Gunthorpe jguntho...@obsidianresearch.com --- drivers/infiniband/ulp/ipoib/ipoib_multicast.c | 47 +-

[PATCH 2/2] IB/ipoib: Suppress warning for send only join failures

2015-08-21 Thread Jason Gunthorpe
We expect send only joins to fail, it just means there are no listeners for the group. The correct thing to do is silently drop the packet at source. Eg avahi will full join 224.0.0.251 which causes a send only IGMP packet to 224.0.0.22, and then a warning level kmessage like this: ib0:

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

2015-08-21 Thread Jason Gunthorpe
On Fri, Aug 14, 2015 at 08:52:09AM -0400, kaike@intel.com wrote: +static LIST_HEAD(ib_nl_request_list); +static DEFINE_SPINLOCK(ib_nl_request_lock); +static atomic_t ib_nl_sa_request_seq; +static struct workqueue_struct *ib_nl_wq; +static struct delayed_work ib_nl_timed_work; +static

Re: [PATCH v9 0/4] Sending kernel pathrecord query to user cache server

2015-08-21 Thread Jason Gunthorpe
On Fri, Aug 14, 2015 at 08:52:05AM -0400, kaike@intel.com wrote: Some tests with namespace have been performed: 1. An unprivileged user cannot bind to the RDMA_NL_GROUP_LS multicast group; 2. An unprivileged user cannot create a new network namespace. However, it can create a new

Re: [PATCH WIP 28/43] IB/core: Introduce new fast registration API

2015-08-21 Thread Christoph Hellwig
On Thu, Aug 20, 2015 at 01:04:13PM -0600, Jason Gunthorpe wrote: Trying to decouple the sub resources, ie by separately pooling the MR/SQE/etc, is just unnecessary complexity, IMHO.. NFS client already had serioues bugs in this area. So, I turn to the idea that every ULP should work as the