[Bug 1789] Add Simple SMP Scheduler

bugzilla-daemon at rtems.org bugzilla-daemon at rtems.org
Wed May 11 19:26:40 UTC 2011


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

--- Comment #1 from Gedare <giddyup44 at yahoo.com> 2011-05-11 14:26:40 CDT ---
Some of these changes have been committed independently to separate the smp
files. The 'simple smp' scheduler should be documented a little bit, with an
explanation at least of the algorithmic overheads and general approach taken in
scheduling. Other detailed comments follow.

schedulersimplesmpschedule.c:
+/*
+ *  The following is very useful on the scheduler simulator when debugging
+ *  this algorithm.  You are not likely to be able to print like this when
+ *  running on a real system.
+ */
+#if 0
+#define D(format,...) printf( format, __VA_ARGS__)
+#else
+#define D(format,...) 
+#endif
Cut/replace these (#ifdef RTEMS_DEBUG ..printf/printk)

+bool _Scheduler_simple_smp_Assign(
Need some comments on what this function is doing, what the return value means.

+  for (n = _Chain_First( chain ); !_Chain_Is_tail(chain, n); n = n->next) {
Should use _Chain_Next(n)

_Scheduler_simple_Ready_queue_Enqueue()
This one slipped by me before. It should not have a capital E in enqueue. The
same goes for the other functions named similarly (grep simple_Ready) that are
already in the CVS.

-- 
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