On Mon, Dec 18, 2017 at 11:25:11AM -0700, Jason Gunthorpe wrote: > On Wed, Dec 13, 2017 at 04:17:59PM -0800, Bryan Tan wrote: > > > > Do some cleanup and fixes related to the comments received during SRQ > > support upstreaming that applied to existing code. Also add in two > > missed macros used by the userlevel library for SRQs, a missed > > ib_umem_release, an incorrect usage of the new refcount_t type, > > and switched from use of wait queues to completions. > > > > v0 -> v1 changelog: > > - Removed use of BIT() in UAPI header > > - Make setting/usage of is_kernel consistent between QP/CQ/SRQ > > - Use completions instead of wait queues for resource destroy > > - Cleaned up commit messages > > This series would be fine for -next, the patches look OK to me now. > > But there are too many non-rc things to be for-rc. > > If you want this in -rc for some reason then you need to send a series > with just those commits, otherwise it will go to -next. I will send out a series with the following commits: > RDMA/vmw_pvrdma: Call ib_umem_release on destroy QP path > RDMA/vmw_pvrdma: Use refcount_dec_and_test to avoid warning > RDMA/vmw_pvrdma: Use completion instead of wait queue > RDMA/vmw_pvrdma: Add UAR SRQ macros in ABI header file And I will clarify the commit message in the last commit there. Thanks! Bryan -- 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