[LITMUS^RT] proposing liblitmus API additions and changes
Glenn Elliott
gelliott at cs.unc.edu
Wed Mar 6 00:50:53 CET 2013
On Mar 1, 2013, at 5:35 PM, Björn Brandenburg <bbb at mpi-sws.org> wrote:
> Hi Glenn,
>
> thanks a lot for your patches! I'm very glad to see more "infrastructure improvements" patches that make the system easier to use or more robust. Comments inline.
>
> On Mar 1, 2013, at 5:35 AM, Glenn Elliott <gelliott at cs.unc.edu> wrote:
>
>> #2: prop/pthreadification
>> This branch introduces a new pthread-styled API to setting up real-time task parameters. The sporadic_* functions and macros are taking ever more parameters. Worse, these parameters do not always apply to the active scheduler (ex. fixed priority). This API should hopefully simply things with a familiar interface and make code more expressive.
<snip>
> Personally, I don't like the pthreads API, so this isn't really a huge improvement to me. I would prefer not having to link against pthreads for "regular" tasks. Error reporting is important.
>
> I would suggest to merge a simplified and rebased version of the migration API changes, and to wait for consensus regarding the attr patch set.
>
> Thanks,
> Björn
I've taken a step back from the aggressive pthread-ification of the Litmus API. I propose a compromise. It is more robust than simply calling memset() to initialize rt_task, but not nearly verbose as my pthread-idifed Litmus API. Instead of either of these options, a user calls init_rt_task_param(struct rt_task*) to set up default real-time behaviors.
Please see prop/rt_task-init on github for the proposed changes.
This branch includes several patches that fall into one of three categories:
1) Updates to add/use init_rt_task_param().
2) Updates to make all code clearly express time in nanoseconds (or use conversion macros).
3) Updates to support clustered scheduling in litmus /bin apps.
(Sorry for putting non-rt_task-init related patches on this branch. It was just easier this way to maintain dependencies.)
Thanks,
Glenn
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.litmus-rt.org/pipermail/litmus-dev/attachments/20130305/056fedb0/attachment.html>
More information about the litmus-dev
mailing list