OSDN Git Service

IB/cm: Remove unused and erroneous msg sequence encoding
authorHåkon Bugge <haakon.bugge@oracle.com>
Wed, 4 Jul 2018 10:48:01 +0000 (12:48 +0200)
committerJason Gunthorpe <jgg@mellanox.com>
Mon, 9 Jul 2018 17:39:28 +0000 (11:39 -0600)
commit87a37ce9e400e40daee537ff95343e3c94743c6d
tree8da1daac23eb0c86a5d84ff6bbd76c025c5e4131
parent76bc79ccceea09a19f681dff7fd633fb8fb95c79
IB/cm: Remove unused and erroneous msg sequence encoding

In cm_form_tid(), a two bit message sequence number is OR'ed into bit
31-30 of the lower TID value.

After commit f06d26537559 ("IB/cm: Randomize starting comm ID"), the
local_id is XOR'ed with a 32-bit random value. Hence, bit 31-30 in the
lower TID now has an arbitrarily value and it makes no sense to OR in
the message sequence number.

Adding to that, the evolution in use of IDR routines in cm_alloc_id()
has always had the possibility of returning a value with bit 30 set.

In addition, said bits are never checked.

Hence, remove the encoding and the corresponding enum.

Signed-off-by: Håkon Bugge <haakon.bugge@oracle.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
drivers/infiniband/core/cm.c
drivers/infiniband/core/cm_msgs.h