change log for rtems (2010-06-22)

Chris Johns chrisj at rtems.org
Wed Jun 23 01:49:45 UTC 2010


On 23/06/10 6:12 AM, rtems-vc at rtems.org wrote:
> diff -u rtems/cpukit/libcsupport/include/rtems/libio_.h:1.37 rtems/cpukit/libcsupport/include/rtems/libio_.h:1.38
> --- rtems/cpukit/libcsupport/include/rtems/libio_.h:1.37	Mon Jun 14 08:35:45 2010
> +++ rtems/cpukit/libcsupport/include/rtems/libio_.h	Tue Jun 22 15:03:41 2010
> @@ -148,13 +148,7 @@
>    *  Macro to free a node.
>    */
>
> -#define rtems_filesystem_freenode( _node ) \
> -  do { \
> -    if ( (_node)->ops )\
> -      if ( (_node)->ops->freenod_h ) \
> -        (*(_node)->ops->freenod_h)( (_node) ); \
> -  } while (0)
> -
> +void rtems_filesystem_freenode( rtems_filesystem_location_info_t*  node );
>

Why has this been done ?

This code is a hot spot in the file system. Not all file systems need to 
free a node but they now suffer an extra call. Maybe the overhead is not 
too much but it does add up.

Chris



More information about the vc mailing list