[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