[Pvfs2-developers] Re: [Pvfs2-cvs] commit by slang in pvfs2/src/io/job: job.c

[Pete Wyckoff]

cvs at parl.clemson.edu wrote on Tue, 26 Feb 2008 10:54 -0500:
> Update of /projects/cvsroot/pvfs2/src/io/job
> In directory parlweb1:/tmp/cvs-serv3926/src/io/job
> 
> Modified Files:
>       Tag: small-file-branch
> 	job.c 
> Log Message:
> fix return codes for job layer to prevent returning negative values from state actions.
> 
> 
> Index: job.c
> ===================================================================
> RCS file: /projects/cvsroot/pvfs2/src/io/job/job.c,v
> diff -p -u -r1.177.2.23 -r1.177.2.24
> --- job.c	26 Feb 2008 14:31:53 -0000	1.177.2.23
> +++ job.c	26 Feb 2008 15:54:57 -0000	1.177.2.24
> @@ -383,7 +383,8 @@ int job_bmi_send(PVFS_BMI_addr_t addr,
>      jd = alloc_job_desc(JOB_BMI);
>      if (!jd)
>      {
> -        return (-errno);
> +        out_status_p->error_code = -PVFS_ERROR_CODE(errno);
> +        return 1;

Would some of this go well in HEAD now too?  Maybe avoid a big
scattered merge all over the tree if you sneak in nice little
fixes like this immediatly.

		-- Pete