A few comments:<div><br></div><div>1.) In base_task.c and base_mt_task.c, you replaced tabs with spaces while defining PERIOD and EXEC_COST</div><div><br></div><div>2.) Why are you requiring that deadline not be more than period?  This constraint is only necessary for HRT systems.  For SRT, you just ensure a tardiness bound of at least $C_i - D_i$.  There are a couple of reasons you could want to do this:</div>
<div>a.) A system designer may have a deadline that is larger than the average-case execution time, but smaller than the WCET, if doing something stochastic.</div><div>b.) If the deadlines aren't really deadlines but are just priority points.  For example, without your restriction, G-FL can be implemented by requesting $D_i - (m-1)/m C_i$ as the "deadline", but with your restriction, a task can get rejected unnecessarily.  Technically this case could actually apply to HRT systems as well.</div>
<div><br></div><div>Otherwise, looks good.</div><div><br></div><div>-Jeremy Erickson<br><br><div class="gmail_quote">On Tue, Jul 17, 2012 at 5:37 PM, Glenn Elliott <span dir="ltr"><<a href="mailto:gelliott@cs.unc.edu" target="_blank">gelliott@cs.unc.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks for the feedback, everyone.<br>
<br>
Please see the revised patches attached.<br>
<span class="HOEnZb"><font color="#888888"><br>
-Glenn<br>
</font></span><br>_______________________________________________<br>
litmus-dev mailing list<br>
<a href="mailto:litmus-dev@lists.litmus-rt.org">litmus-dev@lists.litmus-rt.org</a><br>
<a href="https://lists.litmus-rt.org/listinfo/litmus-dev" target="_blank">https://lists.litmus-rt.org/listinfo/litmus-dev</a><br>
<br></blockquote></div><br></div>