OK. Since for now I don't need it I'll remove this patch but it might be worth considering adding them as a standalone patch.
> -----Original Message----- > From: [email protected] [mailto:[email protected]] On > Behalf Of Bart Van Assche > Sent: Monday, August 15, 2011 4:13 AM > To: [email protected] > Cc: [email protected] > Subject: Re: [patch v2 02/37] add opcodes to ib_pack.h > > On Sun, Jul 24, 2011 at 9:43 PM, <[email protected]> wrote: > > > > Bring up to date with the current version of the IBTA spec. > > - add new opcodes for RC and RD > > - add new groups of opcodes for CN and XRC > > > > Signed-off-by: Bob Pearson <[email protected]> > > > > --- > > include/rdma/ib_pack.h | 39 > ++++++++++++++++++++++++++++++++++++++- > > 1 file changed, 38 insertions(+), 1 deletion(-) > > > > Index: infiniband/include/rdma/ib_pack.h > > > ========================================================== > ========= > > --- infiniband.orig/include/rdma/ib_pack.h > > +++ infiniband/include/rdma/ib_pack.h > > @@ -75,6 +75,8 @@ enum { > > IB_OPCODE_UC = 0x20, > > IB_OPCODE_RD = 0x40, > > IB_OPCODE_UD = 0x60, > > + IB_OPCODE_CN = 0x80, > > + IB_OPCODE_XRC = 0xA0, > > A convention when submitting Linux kernel patches is to add only those > constants that are actually used in the code submitted in the same > patch series. So I'm not sure it's fine to add the XRC constants in > ib_pack.h through the rxe patch series. > > Bart. -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
