Looks good to me. I would add some checks regarding the number and size of the
segments and the allocated space (MCA_BTL_SEG_MAX_SIZE) to make sure we never
hit the corner case where there are too many segments compared with the
available space. And add a huge comment in the btl.h about the fact
Hi Josh
I had a chance to review your code this morning, and generally find it is okay
with me. I see a couple of things that appear to limit it, though they may be
intentional:
1. the call to pineapple_init really needs a third flag to define the process
type. Locking the underlying orte_init
Over the next month, there will be significant changes to ORTE both in terms of
framework APIs and internal behavior. This work will focus on a few areas:
1. launch scalability and timing. I try to review our status on this whenever
we prepare for the start of a new release series, and as usual