Re: [PATCH v4 11/17] nvme-fabrics: Simplify ib_post_(send|recv|srq_recv)() calls

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux