On 09/04/2015 12:27 PM, Jeff Squyres (jsquyres) wrote:
On Sep 4, 2015, at 12:27 PM, Rob Latham <[email protected]> wrote:
We have a formal workflow for frequent contributors that involves sign-offs and such. For occasional contributions, sending a patch to devel@mpich (as you did) works great.
Ken's going to review this.
Thanks!
This patch is important to us because Cisco's usNIC libfabric support is actual UDP under the covers, hence, if you want ordering (and you really do when you use the tag-matching interface; it doesn't make sense, otherwise), then you need to ask for it.
Other libfabric interfaces that provide the tag-matching support (e.g., PSM) are inherently ordered already, so even if you didn't ask for it, you got it, and it all worked out ok.
Jeff, This patch looks correct. I'm just running it through our Jenkins to sanity check it. A few minor issues. 1. I'd like to omit the whitespace modifications so it's clear that it's just the 2 lines that were added for this change. 2. Our project uses "Signed-off-by" as an indication of code review. While we are aware this is contrary to how most git projects work, this is the current situation. For consistency, I would remove your signoff and add my own to indicate I reviewed the patch before pushing to master. Does that sound okay? Thanks, Ken