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

[Sam Lang]

On Feb 26, 2008, at 12:17 PM, Pete Wyckoff wrote:

> 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

Agreed.  I've merged them.  Thanks for getting my back!
-sam

> .
>
> 		-- Pete
>