On Tue, Jul 25, 2017 at 4:39 PM, Andrew Boyer <andrew.boyer@xxxxxxxx> wrote: > To successfully match an IPv6 path, the path cookie must match. Store it > in the QP so that the IPv6 path can be reused. > > Replace open-coded version of dst_check() with the actual call, fixing the > logic. The open-coded version skips the check call if dst->obsolete is 0 > (DST_OBSOLETE_NONE), proceeding to replace the route. DST_OBSOLETE_NONE > means that the route may continue to be used, though. > > Add dst_clone() in prepare_ipv6_hdr(). Otherwise the reference count goes > negative as packets complete. > > Replace sk_dst_get()/dst_release() in rxe_qp_cleanup() with sk_dst_reset(). > sk_dst_get() takes a new reference on dst, so the dst_release() doesn't > actually release the original reference, which was the design intent. > > Remove unneeded initialization in prepare6(). > > Fixes: 4ed6ad1eb30e ("IB/rxe: Cache dst in QP instead of getting it...") > Signed-off-by: Andrew Boyer <andrew.boyer@xxxxxxxx> > --- > drivers/infiniband/sw/rxe/rxe_net.c | 9 ++++++--- > drivers/infiniband/sw/rxe/rxe_qp.c | 9 ++------- > drivers/infiniband/sw/rxe/rxe_verbs.h | 1 + > 3 files changed, 9 insertions(+), 10 deletions(-) > Fixes seem to be correct but I identify here 3 unrelated patches. Please split -- 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