#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

There is no test case for this function.

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().

Last edited on 01/25/17 at 06:29:06 by Sebastian Huber (previous) (diff)

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

In 0794197f729f5ee9551dc7b2e14867f394cbc327/rtems:

rtems: Fix _Rate_monotonic_Renew_deadline()

Prepare a precondition to prevent the potential integer overflow.

Remove one redundant parameter in _Rate_monotonic_Renew_deadline().

sptests/sprmsched02: Create
A test case for checking the overflow condition of postponed_jobs
in rtems_rate_monotonic_period_status.

Update #2885.

comment:7 Changed on 01/30/17 at 07:06:48 by Sebastian Huber

Resolution: fixed
Status: newclosed

comment:8 Changed on 05/11/17 at 07:31:02 by Sebastian Huber

Milestone: 4.124.12.0

comment:9 Changed on 11/09/17 at 06:27:14 by Sebastian Huber

Milestone: 4.12.05.1

Milestone renamed

Note: See TracTickets for help on using tickets.