Closed jdinan closed 3 years ago
Vote | Count |
---|---|
Fix Now (either option:1(a), option:1(b), option:2, option:5) | 0 |
Fix Later (either option:3, option:4, option:6) | 0 |
Fix Now
:Vote | Count |
---|---|
Option:1 (a) | 0 |
Option:1 (b) | 0 |
Option:2 | 0 |
Option:5 | 0 |
Fix Later
:No need to select the option now, we can do that later.
@naveen-rn If I understood correctly, the preference from last week was to implement Option 2 for OpenSHMEM 1.5:
int shmem_ctx_create(long options, shmem_ctx_t *ctx);
int shmem_team_create_ctx(shmem_team_t team, long options, shmem_ctx_t *ctx, long mask, const shmem_ctx_opt_t *opt);
@naveen-rn What's the status of this issue? Are we adding the above for OpenSHMEM 1.5?
@manjugv Is still having mixed thoughts on this approach.
closing this issue, as it is handled as part of https://github.com/openshmem-org/specification/issues/455.
This ISSUE describes the various options that can be used to link a context to a particular team and space. Now, the default
shmem_ctx_create
creates a context with a set ofoptions
, andshmem_team_create_ctx
creates a context with a set ofoptions
on a particularteam
.How do we handle when we introduce space in OpenSHMEM-1.6? Do we introduce a new API, break compatibility and fix the existing APIs? Or do we need to make the current APIs extendable? Or do we fix the current APIs?
Different API options available for us are listed below from the Feb/6/2020 F2F discussion: