On 7/17/2018 2:58 AM, Bart Van Assche wrote:
Instead of declaring and passing a dummy 'bad_wr' pointer, pass NULL as third argument to ib_post_(send|recv|srq_recv)(). Signed-off-by: Bart Van Assche <bart.vanassche@xxxxxxx> Cc: Christoph Hellwig <hch@xxxxxx> Cc: Sagi Grimberg <sagi@xxxxxxxxxxx> --- drivers/nvme/host/rdma.c | 11 +++++------ drivers/nvme/target/rdma.c | 10 ++++------ 2 files changed, 9 insertions(+), 12 deletions(-)
I would prefer dividing it to 2 different commits like you did in iser/isert & srp/srpt (and update the subject as well).
the code looks fine. -- 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
