On Wed, 2018-07-18 at 17:56 +-0300, Max Gurtovoy wrote: +AD4- +AD4- On 7/17/2018 2:58 AM, Bart Van Assche wrote: +AD4- +AD4- Instead of declaring and passing a dummy 'bad+AF8-wr' pointer, pass NULL +AD4- +AD4- as third argument to ib+AF8-post+AF8-(send+AHw-recv+AHw-srq+AF8-recv)(). +AD4- +AD4- +AD4- +AD4- Signed-off-by: Bart Van Assche +ADw-bart.vanassche+AEA-wdc.com+AD4- +AD4- +AD4- Cc: Christoph Hellwig +ADw-hch+AEA-lst.de+AD4- +AD4- +AD4- Cc: Sagi Grimberg +ADw-sagi+AEA-grimberg.me+AD4- +AD4- +AD4- --- +AD4- +AD4- drivers/nvme/host/rdma.c +AHw- 11 +-+-+-+-+------- +AD4- +AD4- drivers/nvme/target/rdma.c +AHw- 10 +-+-+-+------- +AD4- +AD4- 2 files changed, 9 insertions(+-), 12 deletions(-) +AD4- +AD4- +AD4- +AD4- I would prefer dividing it to 2 different commits like you did in +AD4- iser/isert +ACY- srp/srpt (and update the subject as well). Hello Max, That sounds like a good idea to me. I will split this patch. Bart.-- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html