Project

General

Profile

Bug #1564

Inline entry methods don't respect group dependence from CkEntryOptions

Added by Phil Miller about 2 years ago. Updated over 1 year ago.

Status:
Merged
Priority:
Normal
Assignee:
Category:
-
Target version:
Start date:
05/17/2017
Due date:
% Done:

0%


Description

When the target object exists locally and delivery happens inline, the CkEntryOptions are ignored, even if it indicates an unsatisfied group dependence.


Related issues

Related to Charm++ - Bug #1563: Chare Array construction doesn't respect setGroupDepID in CkEntryOptions (or any other entry method options) Merged 05/17/2017

History

#1 Updated by Phil Miller about 2 years ago

  • Status changed from New to Implemented

#2 Updated by Phil Miller about 2 years ago

  • Target version changed from 6.8.0 to 6.9.0

#3 Updated by Phil Miller over 1 year ago

  • Status changed from Implemented to New

This says 'Implemented', but I can't find code where I've actually done it. Shouldn't be too hard, but it's not something already done.

Note when doing this that the generated method definitions for chares, arrays, and groups each come form different places in charmxi, so implementation will have to touch each of them (or extract their commonality)

#4 Updated by Phil Miller over 1 year ago

  • Assignee changed from Phil Miller to Nitin Bhat

Per discussion with Eric B, assigning to Nitin, based on recent work around CkEntryOptions handling

#5 Updated by Nitin Bhat over 1 year ago

  • Status changed from New to In Progress

Is there a test/example that can reproduce this bug? (just to be sure that the fix works correctly).
I am not able to reproduce this issue with a test that calls an inline entry method with a group dependence. (probably because in all my test cases, the system favors the default case of getting the group creation message before the entry method message).

#6 Updated by Nitin Bhat over 1 year ago

  • Related to Bug #1563: Chare Array construction doesn't respect setGroupDepID in CkEntryOptions (or any other entry method options) added

#7 Updated by Nitin Bhat over 1 year ago

  • Status changed from In Progress to Implemented

#8 Updated by Sam White over 1 year ago

  • Status changed from Implemented to Merged

Also available in: Atom PDF