On Wed, Oct 19, 2016 at 04:08:12PM +0000, Amrani, Ram wrote: > Hi Jason, > I've created a user account on github, forked rdma_plumbing and added libqedr in multiple patches, > after I've verified that it builds warning free and in-line with previous cleanups. > I've sent a pull request to linux-rdma rather than rdma-plumbing. Is that OK? So far so good. As Leon said, post your patches to the mailing list. You also need to make sure it builds, Travis says the 32 bit builds are no good (look at the pull request and click on the red X) The github process from here is to make changes and then update your branch on your github, that will reflect in the pull request. Eg you can immediately fix the 32 bit issues and see that travis goes green. I left some minor notes for you on github, the build system stuff looks fine to me, and I didn't notice anything too unusual in a casual browse. Didn't check if the code was any good.. Jason -- 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
