[Bug 1573] Per CPU Patch -- SMP Step #1

bugzilla-daemon at rtems.org bugzilla-daemon at rtems.org
Wed Jun 23 17:08:23 UTC 2010


https://www.rtems.org/bugzilla/show_bug.cgi?id=1573

--- Comment #7 from Joel Sherrill <joel.sherrill at oarcorp.com> 2010-06-23 12:08:22 CDT ---
Thanks for the reviews.  I have tried to address various suggestions.

(1) Name change
  rtems_per_cpu_t -> Per_CPU_Control
  rtems_per_cpu_info -> _Per_cpu_Information
  In other places, "Information" denotes the collection of instances
  and when SMP is in place, this will be an array.

(2) typedef struct Thread_Control_struct Thread_Control moved
  here because the structure needs pointers to Thread_Control for
  executing, heir, and idle.  thread.h needs a LOT to include it.
  This reduced .h file dependency.

  Similarly, percpu.h is included from <rtems/system.h> and needed
  to be very simple.

(3) SYM is only used by asm files so it is the responsibility of
  all the .S file to include it.  Just like rtems/system.h or <config.h>
  in other places.

(4) Context_switch_necessary needs to be renamed to dispatch needed.
  We are doing more than a simple context switch.  Gedare's work is
  breaking out a Scheduler Core object and I want to draw a strong
  distinction between scheduling, dispatching, and a simple context switch.

  Gedare.. can this name change be done as part of your Scheduler work?

-- 
Configure bugmail: https://www.rtems.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.



More information about the bugs mailing list