[PATCH 26] SMP: remove unused code in simple SMP scheduler

Gedare Bloom gedare at rtems.org
Thu Mar 29 19:48:42 UTC 2012


Agreed this looks like it was a stub that was filled in and then left
in place. So it is probably dead code and we should prune it.

Looks good.
-Gedare

On Thu, Mar 29, 2012 at 7:52 AM, Daniel Hellstrom <daniel at gaisler.com> wrote:
> A similar if-statement is just above, I can see no reason why
> to have this double code.
>
> Signed-off-by: Daniel Hellstrom <daniel at gaisler.com>
> ---
>  cpukit/score/src/schedulersimplesmpschedule.c |    3 ---
>  1 files changed, 0 insertions(+), 3 deletions(-)
>
> diff --git a/cpukit/score/src/schedulersimplesmpschedule.c b/cpukit/score/src/schedulersimplesmpschedule.c
> index d8fd06c..95be98a 100644
> --- a/cpukit/score/src/schedulersimplesmpschedule.c
> +++ b/cpukit/score/src/schedulersimplesmpschedule.c
> @@ -155,9 +155,6 @@ bool _Scheduler_simple_smp_Assign(
>       continue;
>     }
>
> -    if ( h->current_priority > pheir->current_priority )
> -      continue;
> -
>     /*
>      *  If heir of potential CPU and of the current CPU are of the SAME
>      *  priority, then which has been running longer?
> --
> 1.7.0.4
>
> _______________________________________________
> rtems-devel mailing list
> rtems-devel at rtems.org
> http://www.rtems.org/mailman/listinfo/rtems-devel




More information about the devel mailing list