Opened 9 years ago

Last modified 21 months ago

#123 new bug

MPIDU_Yield and MPID_Thread_yield

Reported by: Pavan Balaji <balaji@…> Owned by:
Priority: long-term Milestone: future
Component: mpich Keywords:
Cc:

Description (last modified by balaji)

MPIDU_Yield has been implemented as a mpid/common/locks utility function
for a number of platforms. MPID_Thread_yield is implemented at the MPI
top-level, but only for a subset of cases (e.g., sched_yield and yield;
no windows version or select version is present).

It's probably a better idea to move the MPIDU_Yield function as a
top-level utility as MPID_Yield, and allow MPID_Thread_yield to use
MPID_Yield internally. This means that the internal usage of the
MPIDU_Yield function has to change as well, and probably the header file
dependencies too.

Sending this email as a place holder for this fix.

-- Pavan

--
Pavan Balaji
http://www.mcs.anl.gov/~balaji

Change History (13)

comment:1 Changed 9 years ago by Pavan Balaji

  • id set to 123

This message has 0 attachment(s)

comment:2 Changed 9 years ago by balaji

  • Owner set to balaji

comment:3 Changed 9 years ago by balaji

  • Milestone set to mpich2-1.1b1

comment:4 Changed 9 years ago by balaji

  • Milestone mpich2-1.1b1 deleted
  • Priority changed from major to long-term

This is not a critical problem. Changing this to a long-term feature.

comment:5 Changed 9 years ago by balaji

  • Milestone set to mpich2-1.1.1

comment:6 Changed 8 years ago by balaji

  • Milestone changed from mpich2-1.1.1 to mpich2-1.1.2

comment:7 Changed 8 years ago by balaji

  • Description modified (diff)
  • Resolution set to fixed
  • Status changed from new to closed

comment:8 Changed 8 years ago by balaji

  • Description modified (diff)
  • Resolution fixed deleted
  • Status changed from closed to reopened

Reverted [3c8f0731870ca76a48aeb4d8928f7049d37b2f3a] in [57e5f21d36d06abe9b12e0fb60064582a31cd8f4]. This was causing device-specific definitions of thread yield to break. We'll need to modify this more carefully, probably by removing device-specific definitions totally, and making thread yield completely a MPIU_ level function.

Reopening this ticket.

comment:9 Changed 8 years ago by balaji

  • Milestone changed from mpich2-1.1.2 to mpich2-1.2

Milestone mpich2-1.1.2 deleted

comment:10 Changed 8 years ago by balaji

  • Description modified (diff)
  • Milestone changed from mpich2-1.2 to mpich2-1.2.1

comment:11 Changed 8 years ago by balaji

  • Milestone changed from mpich2-1.2.1 to future

Since this is not critical, I'm moving it to "future".

comment:12 Changed 4 years ago by balaji

  • Description modified (diff)
  • Owner balaji deleted
  • Status changed from reopened to new

comment:13 Changed 21 months ago by jhammond

I wonder if this can be closed:

Jeffs-MacBook-Air:git jhammond$ git grep MPID_Thread_yield
Jeffs-MacBook-Air:git jhammond$ git grep MPIDU_Yield
maint/extractstates.in:                    'MPID_STATE_MPIDU_YIELD'       => 'MPIDU_Yield',

All references to yield are now in the MPIU namespace...

Note: See TracTickets for help on using tickets.