<p dir="ltr"><br>
On Dec 21, 2015 1:07 PM, "Sebastian Huber" <<a href="mailto:sebastian.huber@embedded-brains.de">sebastian.huber@embedded-brains.de</a>> wrote:<br>
><br>
> Do we want to remove the task note functions so that an application gets compiler and linker errors or warnings? Or do we make the functions empty and return an error status code?</p>
<p dir="ltr">These were really bad APIs long before SMP was a consideration. I would be happy to see them completely gone.</p>
<p dir="ltr">We need to capture their removal in a 4.12 release notes with transition info. We weren't good at this for 4.11.  Perhaps the manual can replace the text section on them with a note they are gone as of 4.12 and you should use TLS or keys.</p>
<p dir="ltr">><br>
> ----- Am 21. Dez 2015 um 19:32 schrieb Joel Sherrill <a href="mailto:joel@rtems.org">joel@rtems.org</a>:<br>
><br>
> > To remove notepads fully, the documentation and testsuites<br>
> > should also be touched. There should be more in the patch.<br>
> ><br>
> > --joel<br>
> ><br>
> > On Mon, Dec 21, 2015 at 1:15 AM, Sebastian Huber <<br>
> > <a href="mailto:sebastian.huber@embedded-brains.de">sebastian.huber@embedded-brains.de</a>> wrote:<br>
> ><br>
> >> On 21/12/15 02:13, Aun-Ali Zaidi wrote:<br>
> >><br>
> >>> updates #2493.<br>
> >>> ---<br>
> >>>   cpukit/rtems/include/rtems/rtems/tasks.h | 2 +-<br>
> >>>   cpukit/rtems/src/taskgetnote.c           | 2 --<br>
> >>>   cpukit/rtems/src/tasksetnote.c           | 2 --<br>
> >>>   3 files changed, 1 insertion(+), 5 deletions(-)<br>
> >>><br>
> >>> diff --git a/cpukit/rtems/include/rtems/rtems/tasks.h<br>
> >>> b/cpukit/rtems/include/rtems/rtems/tasks.h<br>
> >>> index 1ad537d..114624d 100644<br>
> >>> --- a/cpukit/rtems/include/rtems/rtems/tasks.h<br>
> >>> +++ b/cpukit/rtems/include/rtems/rtems/tasks.h<br>
> >>> @@ -678,7 +678,7 @@ typedef struct {<br>
> >>>      *<br>
> >>>      *  @note MUST BE LAST ENTRY.<br>
> >>>      */<br>
> >>> -  uint32_t Notepads[ RTEMS_ZERO_LENGTH_ARRAY ] RTEMS_DEPRECATED;<br>
> >>> +  //uint32_t Notepads[ RTEMS_ZERO_LENGTH_ARRAY ] RTEMS_DEPRECATED;<br>
> >>><br>
> >><br>
> >> Please don't use comments or #if 0 to remove things.<br>
> >><br>
> >>   }  RTEMS_API_Control;<br>
> >>>     /**<br>
> >>> diff --git a/cpukit/rtems/src/taskgetnote.c<br>
> >>> b/cpukit/rtems/src/taskgetnote.c<br>
> >>> index c097d9c..ba40d2e 100644<br>
> >>> --- a/cpukit/rtems/src/taskgetnote.c<br>
> >>> +++ b/cpukit/rtems/src/taskgetnote.c<br>
> >>> @@ -60,7 +60,6 @@ rtems_status_code rtems_task_get_note(<br>
> >>>     if ( _Objects_Are_ids_equal( id, OBJECTS_ID_OF_SELF ) ||<br>
> >>>          _Objects_Are_ids_equal( id, executing->Object.id ) ) {<br>
> >>>         api = executing->API_Extensions[ THREAD_API_RTEMS ];<br>
> >>> -      *note = api->Notepads[ notepad ];<br>
> >>>         return RTEMS_SUCCESSFUL;<br>
> >>>     }<br>
> >>>   @@ -69,7 +68,6 @@ rtems_status_code rtems_task_get_note(<br>
> >>>         case OBJECTS_LOCAL:<br>
> >>>         api = the_thread->API_Extensions[ THREAD_API_RTEMS ];<br>
> >>> -      *note = api->Notepads[ notepad ];<br>
> >>>         _Objects_Put( &the_thread->Object );<br>
> >>>         return RTEMS_SUCCESSFUL;<br>
> >>><br>
> >>><br>
> >><br>
> >> What is the purpose of this function after your change?<br>
> >><br>
> >> --<br>
> >> Sebastian Huber, embedded brains GmbH<br>
> >><br>
> >> Address : Dornierstr. 4, D-82178 Puchheim, Germany<br>
> >> Phone   : +49 89 189 47 41-16<br>
> >> Fax     : +49 89 189 47 41-09<br>
> >> E-Mail  : <a href="mailto:sebastian.huber@embedded-brains.de">sebastian.huber@embedded-brains.de</a><br>
> >> PGP     : Public key available on request.<br>
> >><br>
> >> Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.<br>
> >><br>
> >><br>
> >> _______________________________________________<br>
> >> devel mailing list<br>
> >> <a href="mailto:devel@rtems.org">devel@rtems.org</a><br>
> >> <a href="http://lists.rtems.org/mailman/listinfo/devel">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</p>