Re: [PATCH v2] RDMA/rxe: Set wqe->status correctly if an unexpected response is received

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

 



On 06/28/18 01:58, Moni Shoua wrote:
wqe->status is already set before the code that you mentioned has been
reached.
>
can you be more specific?
for instance I didn't find similar behavior for the example below
(also in check_ack())

         case IB_OPCODE_RC_ATOMIC_ACKNOWLEDGE:
                 syn = aeth_syn(pkt);

                 if ((syn & AETH_TYPE_MASK) != AETH_ACK)
                         return COMPST_ERROR;

                 if (wqe->wr.opcode != IB_WR_ATOMIC_CMP_AND_SWP &&
                     wqe->wr.opcode != IB_WR_ATOMIC_FETCH_AND_ADD)
                         return COMPST_ERROR;   <<<<<<<<<<<<<<<<<<<<<<<<<<
                 reset_retry_counters(qp);
                 return COMPST_ATOMIC;

Hello Moni,

I may have been wrong when I claimed that all other paths that return COMPST_ERROR already set wqe->status to an error value. However, I only have a test case for the path I modified. Hence my patch.

Bart.



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux