[PATCH] IB/mlx5: fix null dereference

2015-11-22 Thread Saurabh Sengar
removing null dereference Signed-off-by: Saurabh Sengar --- is there any better way to fix it ? as if there is no 'x' there is no way we can access 'x->lock' drivers/infiniband/hw/mlx5/qp.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/infiniband

Re: [PATCH] IB/sa: replace GFP_KERNEL with GFP_ATOMIC

2015-11-13 Thread Saurabh Sengar
On Fri, 13 Nov 2015 at 10:47:52 +, Wan, Kaike wrote: > I don't think so. > The following patch has rendered this patch unnecessary: > https://patchwork.kernel.org/patch/7526811/ > Kaike OK, could you please use "Reported-by" tag of my name in your patch Regards, Saurabh -- To unsubscribe fro

Re: [PATCH] IB/sa: replace GFP_KERNEL with GFP_ATOMIC

2015-11-12 Thread Saurabh Sengar
On Wed, 28 Oct 2015 at 04:30:10 +, Weiny, Ira wrote: > Until we can remove the spinlock the current proposed patch should be applied > in the interim. Sorry for the noise before. > Reviewed-By: Ira Weiny Hi, is this patch expected to be applied ? -- To unsubscribe from this list: send th

[PATCH] IB/ipoib: optimized the function ipoib_mcast_alloc

2015-10-31 Thread Saurabh Sengar
ipoib_mcast_alloc is called only in atomic context, hence removing the extra check. Signed-off-by: Saurabh Sengar --- Hi, Even if in future, if there are some functions expected to call it in normal context(not atomic), we can pass the GFP_KERNEL or GFP_ATOMIC directly to function call instead

[PATCH] IB/sa: replace GFP_KERNEL with GFP_ATOMIC

2015-10-27 Thread Saurabh Sengar
replace GFP_KERNEL with GFP_ATOMIC, as code while holding a spinlock should be atomic GFP_KERNEL may sleep and can cause deadlock, where as GFP_ATOMIC may fail but certainly avoids deadlock Signed-off-by: Saurabh Sengar --- drivers/infiniband/core/sa_query.c | 4 ++-- 1 file changed, 2