[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