[PATCH] RDMA/nes: Remove unused field sent_ts

2015-02-03 Thread Tina Ruchandani
This patch removes the unused field sent_ts from struct nes_cm_tcp_context.
This patch is part of an effort to clean up instances of timekeeping data-types
like 'struct timeval' which will overflow on 32-bit systems in year 2038 and
beyond.

Signed-off-by: Tina Ruchandani ruchandani.t...@gmail.com
---
 drivers/infiniband/hw/nes/nes_cm.h | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/infiniband/hw/nes/nes_cm.h 
b/drivers/infiniband/hw/nes/nes_cm.h
index f522cf6..98948f1 100644
--- a/drivers/infiniband/hw/nes/nes_cm.h
+++ b/drivers/infiniband/hw/nes/nes_cm.h
@@ -279,7 +279,6 @@ struct nes_cm_tcp_context {
u8  rcv_wscale;
 
struct nes_cm_tsa_context tsa_cntxt;
-   struct timevalsent_ts;
 };
 
 
-- 
2.2.0.rc0.207.ga3a616c

--
To unsubscribe from this list: send the line unsubscribe linux-rdma in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] RDMA/nes: Remove unused field sent_ts

2015-02-03 Thread Arnd Bergmann
On Tuesday 03 February 2015 17:17:47 Tina Ruchandani wrote:
 This patch removes the unused field sent_ts from struct nes_cm_tcp_context.
 This patch is part of an effort to clean up instances of timekeeping 
 data-types
 like 'struct timeval' which will overflow on 32-bit systems in year 2038 and
 beyond.
 
 Signed-off-by: Tina Ruchandani ruchandani.t...@gmail.com
 

Acked-by: Arnd Bergmann a...@arndb.de
--
To unsubscribe from this list: send the line unsubscribe linux-rdma in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [PATCH] RDMA/nes: Remove unused field sent_ts

2015-02-03 Thread Latif, Faisal

 From: Tina Ruchandani [mailto:ruchandani.t...@gmail.com]
 Sent: Tuesday, February 03, 2015 5:48 AM
 
 This patch removes the unused field sent_ts from struct nes_cm_tcp_context.
 This patch is part of an effort to clean up instances of timekeeping 
 data-types
 like 'struct timeval' which will overflow on 32-bit systems in year 2038 and
 beyond.
 
 Signed-off-by: Tina Ruchandani ruchandani.t...@gmail.com
 ---
 --
 2.2.0.rc0.207.ga3a616c

Acked-by: Faisal Latif faisal.la...@intel.com

--
To unsubscribe from this list: send the line unsubscribe linux-rdma in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [PATCH] IB/mad: fix ifnullfree.cocci warnings

2015-02-03 Thread Weiny, Ira


 -Original Message-
 From: Wu, Fengguang
 Sent: Tuesday, January 27, 2015 1:36 PM
 To: Weiny, Ira
 Cc: kbuild-...@01.org; Roland Dreier; Hefty, Sean; Hal Rosenstock; Or Gerlitz;
 Yan Burman; linux-rdma@vger.kernel.org; linux-ker...@vger.kernel.org
 Subject: [PATCH] IB/mad: fix ifnullfree.cocci warnings
 
 drivers/infiniband/core/mad.c:2088:3-8: WARNING: NULL check before freeing
 functions like kfree, debugfs_remove, debugfs_remove_recursive or
 usb_free_urb is not needed. Maybe consider reorganizing relevant code to
 avoid passing NULL values.
 
  NULL check before some freeing functions is not needed.
 
  Based on checkpatch warning
  kfree(NULL) is safe this check is probably not required
  and kfreeaddr.cocci by Julia Lawall.
 
 Generated by: scripts/coccinelle/free/ifnullfree.cocci
 
 Signed-off-by: Fengguang Wu fengguang...@intel.com


Signed-off-by: Ira Weiny ira.we...@intel.com


 ---
 
  mad.c |3 +--
  1 file changed, 1 insertion(+), 2 deletions(-)
 
 --- a/drivers/infiniband/core/mad.c
 +++ b/drivers/infiniband/core/mad.c
 @@ -2084,8 +2084,7 @@ out:
   /* Post another receive request for this QP */
   if (response) {
   ib_mad_post_receive_mads(qp_info, response);
 - if (recv)
 - kfree(recv);
 + kfree(recv);
   } else
   ib_mad_post_receive_mads(qp_info, recv);  }
--
To unsubscribe from this list: send the line unsubscribe linux-rdma in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [PATCH] IB/mad: fix ifnullfree.cocci warnings

2015-02-03 Thread Weiny, Ira
 
  drivers/infiniband/core/mad.c:2088:3-8: WARNING: NULL check before
  freeing functions like kfree, debugfs_remove, debugfs_remove_recursive
  or usb_free_urb is not needed. Maybe consider reorganizing relevant
  code to avoid passing NULL values.
 
   NULL check before some freeing functions is not needed.
 
   Based on checkpatch warning
   kfree(NULL) is safe this check is probably not required
   and kfreeaddr.cocci by Julia Lawall.
 
  Generated by: scripts/coccinelle/free/ifnullfree.cocci
 
  Signed-off-by: Fengguang Wu fengguang...@intel.com
 
 
 Signed-off-by: Ira Weiny ira.we...@intel.com

Sorry this should have been:

Reviewed-by: Ira Weiny ira.we...@intel.com
Tested-by: Ira Weiny ira.we...@intel.com


 
 
  ---
 
   mad.c |3 +--
   1 file changed, 1 insertion(+), 2 deletions(-)
 
  --- a/drivers/infiniband/core/mad.c
  +++ b/drivers/infiniband/core/mad.c
  @@ -2084,8 +2084,7 @@ out:
  /* Post another receive request for this QP */
  if (response) {
  ib_mad_post_receive_mads(qp_info, response);
  -   if (recv)
  -   kfree(recv);
  +   kfree(recv);
  } else
  ib_mad_post_receive_mads(qp_info, recv);  }
 --
 To unsubscribe from this list: send the line unsubscribe linux-rdma in the 
 body
 of a message to majord...@vger.kernel.org More majordomo info at
 http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line unsubscribe linux-rdma in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[GIT PULL] please pull infiniband.git

2015-02-03 Thread Roland Dreier
Hi Linus,

Please pull from

git://git.kernel.org/pub/scm/linux/kernel/git/roland/infiniband.git 
tags/rdma-for-linus


Last minute InfiniBand/RDMA changes for 3.19:
 - Revert IPoIB driver back to 3.18 state.  We had a number of fixes go
   into 3.19, but they introduced regressions.  We tried to get everything
   fixed up but ran out of time, so we'll try again for 3.20.
 - Similarly, turn off the new extended query port verb.  Late in the
   cycle we realized the ABI is not quite right, and rather than freeze
   something in a rush and make a mistake, we'll take a bit more time
   and get it right in 3.20.


Haggai Eran (1):
  IB/core: Temporarily disable ex_query_device uverb

Roland Dreier (9):
  Revert IPoIB: No longer use flush as a parameter
  Revert IPoIB: Make ipoib_mcast_stop_thread flush the workqueue
  Revert IPoIB: Use dedicated workqueues per interface
  Revert IPoIB: change init sequence ordering
  Revert IPoIB: fix mcast_dev_flush/mcast_restart_task race
  Revert IPoIB: fix MCAST_FLAG_BUSY usage
  Revert IPoIB: Make the carrier_on_task race aware
  Revert IPoIB: Consolidate rtnl_lock tasks in workqueue
  Merge branches 'ipoib' and 'odp' into for-next

 drivers/infiniband/core/uverbs_main.c  |   1 -
 drivers/infiniband/ulp/ipoib/ipoib.h   |  19 +-
 drivers/infiniband/ulp/ipoib/ipoib_cm.c|  18 +-
 drivers/infiniband/ulp/ipoib/ipoib_ib.c|  27 +--
 drivers/infiniband/ulp/ipoib/ipoib_main.c  |  49 ++---
 drivers/infiniband/ulp/ipoib/ipoib_multicast.c | 239 +
 drivers/infiniband/ulp/ipoib/ipoib_verbs.c |  22 +--
 7 files changed, 134 insertions(+), 241 deletions(-)
--
To unsubscribe from this list: send the line unsubscribe linux-rdma in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html