diff options
| author | Kaike Wan <[email protected]> | 2019-07-15 12:45:40 -0400 |
|---|---|---|
| committer | Jason Gunthorpe <[email protected]> | 2019-07-22 14:57:55 -0300 |
| commit | f4d46119f214f9a7620b0d18b153d7e0e8c90b4f (patch) | |
| tree | ebfbba88f950a93cfde560a4aefa14d07687f193 /net/switchdev/switchdev.c | |
| parent | dc25b239ebeaa3c58e5ceaa732140427d386aa16 (diff) | |
IB/hfi1: Drop all TID RDMA READ RESP packets after r_next_psn
When a TID sequence error occurs while receiving TID RDMA READ RESP
packets, all packets after flow->flow_state.r_next_psn should be dropped,
including those response packets for subsequent segments.
The current implementation will drop the subsequent response packets for
the segment to complete next, but may accept packets for subsequent
segments and therefore mistakenly advance the r_next_psn fields for the
corresponding software flows. This may result in failures to complete
subsequent segments after the current segment is completed.
The fix is to only use the flow pointed by req->clear_tail for checking
KDETH PSN instead of finding a flow from the request's flow array.
Fixes: b885d5be9ca1 ("IB/hfi1: Unify the software PSN check for TID RDMA READ/WRITE")
Cc: <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Reviewed-by: Mike Marciniszyn <[email protected]>
Signed-off-by: Kaike Wan <[email protected]>
Signed-off-by: Mike Marciniszyn <[email protected]>
Signed-off-by: Jason Gunthorpe <[email protected]>
Diffstat (limited to 'net/switchdev/switchdev.c')
0 files changed, 0 insertions, 0 deletions