RE: non-const pointer void * addr in rdma_reg_* and rdma_post_[send|write]

2013-11-27 Thread Hefty, Sean
> I just started working with librdmacm and I was wondering if there is a > specific reason why rdma_reg_* functions and rdma_post_send/write functions > take the local memory address as non-const pointer "void * addr". These > functions shouldn't and don't change the memory pointed to by addr. I t

non-const pointer void * addr in rdma_reg_* and rdma_post_[send|write]

2013-11-27 Thread Hannes Weisbach
Hi, I just started working with librdmacm and I was wondering if there is a specific reason why rdma_reg_* functions and rdma_post_send/write functions take the local memory address as non-const pointer "void * addr". These functions shouldn't and don't change the memory pointed to by addr. I t

Re: [PATCH] osm_sm_state_mgr.c Fix handling of polling retry number

2013-11-27 Thread Line Holen
On 11/27/13 13:16, Hal Rosenstock wrote: On 11/15/2013 7:15 AM, Line Holen wrote: The retry counter is now only updated if a packet is actually sent. (But as before the initial request is also counted.) Prior to this change the actual maximum number of packets sent were polling retry number min

Re: [PATCH for v3.13 2/7] IB/uverbs: remove implicit cast in INIT_UDATA()

2013-11-27 Thread Yann Droneaud
Hi Matan, Le mercredi 27 novembre 2013 à 10:21 +0200, Matan Barak a écrit : > On 27/11/2013 12:02 AM, Yann Droneaud wrote: ... > > INIT_UDATA(&udata, buf + sizeof cmd, > > - (unsigned long) cmd.response + sizeof resp, > > + (void __user *)(unsigned long)cmd.response +

Re: [PATCH] osm_sm_state_mgr.c Fix handling of polling retry number

2013-11-27 Thread Hal Rosenstock
On 11/15/2013 7:15 AM, Line Holen wrote: > The retry counter is now only updated if a packet is actually sent. > (But as before the initial request is also counted.) > > Prior to this change the actual maximum number of packets sent were > polling retry number minus one. > > Signed-off-by: Line H

Re: [PATCH for v3.13 0/7] create_flow/destroy_flow fixes for v3.13

2013-11-27 Thread Matan Barak
On 27/11/2013 12:02 AM, Yann Droneaud wrote: Hi, Please find a patchset against create_flow/destroy_flow and associated extended command scheme. These are fixes that must be applied before making the new uverbs widely available. This patchset gather some patches already sent independently: - T

Re: [PATCH for v3.13 2/7] IB/uverbs: remove implicit cast in INIT_UDATA()

2013-11-27 Thread Matan Barak
On 27/11/2013 12:02 AM, Yann Droneaud wrote: Currently, INIT_UDATA() does an implicit cast to a pointer, so that 'response' address, eg. output buffer, can be used as is to initialize a struct ib_udata: do {\ (udata)->