#2885 closed defect (fixed)
Fix rtems_rate_monotonic_postponed_job_count() prototype
Reported by: | Sebastian Huber | Owned by: | Sebastian Huber |
---|---|---|---|
Priority: | normal | Milestone: | 5.1 |
Component: | score | Version: | 5 |
Severity: | normal | Keywords: | |
Cc: | Blocked By: | ||
Blocking: |
Description
rtems_rate_monotonic_postponed_job_count() should return an RTEMS status code. It should be renamed to rtems_rate_monotonic_get_postponed_job_count() or rtems_rate_monotonic_get_postponed_jobs() (similar to rtems_rate_monotonic_get_statistics()).
Change History (9)
comment:1 Changed on 01/24/17 at 14:13:27 by Sebastian Huber
comment:2 Changed on 01/24/17 at 15:44:34 by Kuan-Hsun Chen
I think we should rename it as rtems_rate_monotonic_get_postponed_job_count(uint32_t *count), though the name is really long.
The RTEMS status codes could be:
RTEMS_SUCCESSFUL - count returned successfully
RTEMS_INVALID_ID - invalid rate monotonic period id
RTEMS_INVALID_ADDRESS - invalid address of count
comment:3 Changed on 01/25/17 at 06:28:28 by Sebastian Huber
My preferred option: just add this to rtems_rate_monotonic_period_status and return it via rtems_rate_monotonic_get_status().
comment:4 Changed on 01/25/17 at 08:28:16 by Kuan-Hsun Chen
I can do this.
I will add the count into the structure of rtems_rate_monotonic_period_status, remove rtems_rate_monotonic_get_postponed_job_count(), and extend rtems_rate_monotonic_get_status().
comment:5 Changed on 01/25/17 at 13:19:19 by Kuan-Hsun Chen
Sp69 is also planned to be updated, which is the test for rtems_rate_monotonic_get_status()
comment:6 Changed on 01/30/17 at 06:53:30 by Kuan-Hsun Chen <c0066c@…>
comment:7 Changed on 01/30/17 at 07:06:48 by Sebastian Huber
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:8 Changed on 05/11/17 at 07:31:02 by Sebastian Huber
Milestone: | 4.12 → 4.12.0 |
---|
comment:9 Changed on 11/09/17 at 06:27:14 by Sebastian Huber
Milestone: | 4.12.0 → 5.1 |
---|
Milestone renamed
There is no test case for this function.