On Thu, Dec 19, 2019 at 10:36:45AM +0530, Selvin Xavier wrote: > > Instead I guess a new symbol as rdma_get_gid_attr_context() can be added > > too. > I am okay with both adding context to gid_attr struct or adding a symbol. > Let me know your preference. > Or shall i handle this inside bnxt_re itself. Not sure whether any > other drivers intend to use this. Having a function to return the same void * that is passed to the driver ops functions seems reasonable and small to me. But you could also spruce this up a bit and have it work more like a true 'priv' and get rid of that void *.. The container_of thing is really odd Jason
