[ofiwg] [ofiwg/libfabric] Ofi bgq provider (#2500)
Jeff Squyres (jsquyres)
jsquyres at cisco.com
Wed Nov 2 15:21:24 PDT 2016
On Nov 2, 2016, at 6:01 PM, Coffman, Paul K. <pcoffman at anl.gov> wrote:
>
> Jeff –
>
> Do all of my new files need to have this dual BSD and GPLv2 license copyright in the header?
Yes; it's OpenFabrics rules.
> And is it ok for me to signoff on my own commits?
Yep!
I'm just about to turn in a review for your code.
In general, I think it looks awesome -- thanks for submitting! I obviously can't comment on 90% of the code because I don't have a BGQ to run with, and I have no way of testing / etc. I am providing some stylistic norms to conform to, and some comments on the code that I *can* read/grok (e.g., the configury, etc.). Most of my comments are pretty minor and easily addressed. Please take them all as constructive criticism!
--
Jeff Squyres
jsquyres at cisco.com
For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/
More information about the ofiwg
mailing list