Re: [PATCH] Make sure ipc functions return CS_ERR_TRY_AGAIN and not CS_ERR_TIMEOUT

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



ACK, tested and all, it appears to fix also
https://bugzilla.redhat.com/show_bug.cgi?id=787725


On 2/7/2012 4:27 AM, Angus Salkeld wrote:
> This is because most applications that use corosync do not test
> for TIMEOUT but only for TRY_AGAIN.
> 
> Signed-off-by: Angus Salkeld <asalkeld@xxxxxxxxxx>
> ---
>  include/corosync/corotypes.h |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/include/corosync/corotypes.h b/include/corosync/corotypes.h
> index 74183d8..c67bf29 100644
> --- a/include/corosync/corotypes.h
> +++ b/include/corosync/corotypes.h
> @@ -151,6 +151,7 @@ static inline cs_error_t qb_to_cs_error (int result)
>  	case ENOMEM:
>  		err = CS_ERR_NO_MEMORY;
>  		break;
> +	case ETIMEDOUT:
>  	case EAGAIN:
>  		err = CS_ERR_TRY_AGAIN;
>  		break;
> @@ -158,7 +159,6 @@ static inline cs_error_t qb_to_cs_error (int result)
>  		err = CS_ERR_FAILED_OPERATION;
>  		break;
>  	case ETIME:
> -	case ETIMEDOUT:
>  		err = CS_ERR_TIMEOUT;
>  		break;
>  	case EINVAL:

_______________________________________________
discuss mailing list
discuss@xxxxxxxxxxxx
http://lists.corosync.org/mailman/listinfo/discuss


[Index of Archives]     [Linux Clusters]     [Corosync Project]     [Linux USB Devel]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Linux Kernel]     [Linux SCSI]     [X.Org]

  Powered by Linux