[rtems commit] tmtests/tm26: Avoid NULL pointer access
Sebastian Huber
sebastian.huber at embedded-brains.de
Fri May 29 15:45:27 UTC 2015
----- Joel Sherrill <joel.sherrill at oarcorp.com> schrieb:
>
>
> On 5/29/2015 8:37 AM, Sebastian Huber wrote:
> > Module: rtems
> > Branch: master
> > Commit: 5611839a7e2e371dd1f327c336c785095f634e55
> > Changeset: http://git.rtems.org/rtems/commit/?id=5611839a7e2e371dd1f327c336c785095f634e55
> >
> > Author: Sebastian Huber <sebastian.huber at embedded-brains.de>
> > Date: Fri May 29 14:18:32 2015 +0200
> >
> > tmtests/tm26: Avoid NULL pointer access
> >
> > ---
> >
> > testsuites/tmtests/tm26/task1.c | 3 ---
> > 1 file changed, 3 deletions(-)
> >
> > diff --git a/testsuites/tmtests/tm26/task1.c b/testsuites/tmtests/tm26/task1.c
> > index 6cec7da..ee66218 100644
> > --- a/testsuites/tmtests/tm26/task1.c
> > +++ b/testsuites/tmtests/tm26/task1.c
> > @@ -439,9 +439,6 @@ rtems_task Low_task(
> > thread_disable_dispatch();
> >
> > benchmark_timer_initialize();
> > -#if (CPU_HARDWARE_FP == 1) || (CPU_SOFTWARE_FP == 1)
> > - _Context_Restore_fp( &_Thread_Get_executing()->fp_context );
> > -#endif
>
> What impact does this have on the case being timed?
>
> How did this fault? What configuration?
Sorry, I didn't fault. I was stupid enough to look at "executing->fp_context" in the GDB. I will revert this patch.
The main problem is that this test restores an FP context without the FPU enabled. I have to figure out how we can avoid this.
>
> > _Context_Switch(
> > &executing->Registers,
> > &_Thread_Get_executing()->Registers
> >
> > _______________________________________________
> > vc mailing list
> > vc at rtems.org
> > http://lists.rtems.org/mailman/listinfo/vc
> >
>
> --
> Joel Sherrill, Ph.D. Director of Research & Development
> joel.sherrill at OARcorp.com On-Line Applications Research
> Ask me about RTEMS: a free RTOS Huntsville AL 35805
> Support Available (256) 722-9985
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
--
Sebastian Huber, embedded brains GmbH
Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone : +49 89 189 47 41-16
Fax : +49 89 189 47 41-09
E-Mail : sebastian.huber at embedded-brains.de
PGP : Public key available on request.
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.
More information about the devel
mailing list