Re: [PATCH v3 7/7] ALSA: seq: Don't lose partial read failure

[Cezary Rojewski <cezary.rojewski@intel.com>]

On 10/7/2026 7:25 PM, Takashi Iwai wrote:

> --- a/sound/core/seq/seq_clientmgr.c
> +++ b/sound/core/seq/seq_clientmgr.c
> @@ -480,11 +480,11 @@ static ssize_t snd_seq_read(struct file *file, char __user *buf, size_t count,
>  	if (err < 0) {
>  		if (cell)
>  			snd_seq_fifo_cell_putback(fifo, cell);
> -		if (err == -EAGAIN && result > 0)
> -			err = 0;
>  	}
>  
> -	return (err < 0) ? err : result;
> +	if (result > 0)
> +		return result;
> +	return err < 0 ? err : 0;

Can 'err' even be positive? Looks to me as if 'result' holds the bytes
and flat 'return err' suffices.

>  }
>  
>