> On Feb 18, 2020, at 3:58 PM, Jason Gunthorpe <jgg@xxxxxxxx> wrote: > > On Tue, Feb 18, 2020 at 03:53:45PM -0500, Andrew Boyer wrote: >> There is an ib_ud_header_unpack() in core/ud_header.c, but it has no consumers. >> >> Would I be allowed to add a roce version alongside it? > > Why? Personally I loath these accessors > > I have been thinking of dropping all of them in favour of the stuff in > include/rdma/iba.h, which has really been a good improvement to the cm > >> May I do that now or must it wait until a consumer is ready to be checked in? > > New stuff always needs in-tree users. > > You can send a patch to delete ib_ud_header_unpack() though > > Jason OK. It was being used for query_ah and query_qp, but I can design that out, no problem. Are we still permitted to use ib_ud_header_pack() or should we avoid that too? -Andrew
