[PATCH linux dev-4.10] drivers (fsi): occ: Change errno for releasing client during transfer

Christopher Bostic cbostic at linux.vnet.ibm.com
Wed Nov 1 08:03:35 AEDT 2017


Reviewed-by: Christopher Bostic <cbostic at linux.vnet.ibm.com>


On 10/31/17 12:34 PM, Eddie James wrote:
> From: "Edward A. James" <eajames at us.ibm.com>
>
> Clients were receiving the ECANCELED errno when the client is removed
> during a transfer. Use ENODEV instead, as ECANCELED is for AIO ops.
>
> Signed-off-by: Edward A. James <eajames at us.ibm.com>
> ---
>   drivers/fsi/occ.c | 4 +---
>   1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/fsi/occ.c b/drivers/fsi/occ.c
> index 1fe1a54..adc64f3 100644
> --- a/drivers/fsi/occ.c
> +++ b/drivers/fsi/occ.c
> @@ -240,9 +240,7 @@ static ssize_t occ_read_common(struct occ_client *client, char __user *ubuf,
>   		spin_lock_irq(&client->lock);
>
>   		if (!test_bit(XFR_COMPLETE, &xfr->flags)) {
> -			if (test_bit(XFR_CANCELED, &xfr->flags))
> -				rc = -ECANCELED;
> -			else if (occ->cancel)
> +			if (occ->cancel || test_bit(XFR_CANCELED, &xfr->flags))
>   				rc = -ENODEV;
>   			else
>   				rc = -EINTR;



More information about the openbmc mailing list