[ofw] [PATCH] use macro for tmp file path

Sasha Khapyorsky sashak at voltaire.com
Tue May 25 04:02:06 PDT 2010


On 09:51 Fri 21 May     , Smith, Stan wrote:
> 
> Use defined macro for tmp file path
> 
> signed-off-by: stan smith <stan.smith at intel.com>
> 

The patch is whitespece-mangled.

> diff --git a/opensm/opensm/st.c b/opensm/opensm/st.c
> index ea76038..2d39117 100644
> --- a/opensm/opensm/st.c
> +++ b/opensm/opensm/st.c
> @@ -174,7 +174,7 @@ static int init_st = 0;
> 
>  static void stat_col()
>  {
> -       FILE *f = fopen("/var/log/osm_st_col", "w");
> +       FILE *f = fopen( OSM_DEFAULT_TMP_DIR "osm_st_col", "w");

I think it was discussed somehow in the past.

Shouldn't this be:

	OSM_DEFAULT_TMP_DIR "/osm_st_col"

, so that trailing '/' will not be mandatory in directory define?

Sasha

>         fprintf(f, "collision: %d\n", collision);
>         fclose(f);
>  }
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 



More information about the ofw mailing list