[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