speakup crashes with kernel 5.4.69

John Covici covici at ccs.covici.com
Tue Nov 3 14:21:10 EST 2020


I am sorry to  report that your patch does not work, probably because
it references drivers/accessibility rather than staging.  If that is
the case, I could change it and try again, or let me know.

Thanks.

On Mon, 02 Nov 2020 04:10:16 -0500,
Samuel Thibault wrote:
> 
> [1  <text/plain; us-ascii (7bit)>]
> John Covici, le lun. 02 nov. 2020 03:47:47 -0500, a ecrit:
> > Looks like the patch is working!
> 
> Ok, good. Could you try the (much simpler) attached patch instead?
> 
> Samuel
> [2 patch <text/plain; us-ascii (7bit)>]
> --- a/drivers/accessibility/speakup/spk_ttyio.c
> +++ b/drivers/accessibility/speakup/spk_ttyio.c
> @@ -298,11 +298,13 @@ static unsigned char ttyio_in(int timeou
>  	struct spk_ldisc_data *ldisc_data = speakup_tty->disc_data;
>  	char rv;
>  
> -	if (wait_for_completion_timeout(&ldisc_data->completion,
> +	if (!timeout) {
> +		if (!try_wait_for_completion(&ldisc_data->completion))
> +			return 0xff;
> +	} else if (wait_for_completion_timeout(&ldisc_data->completion,
>  					usecs_to_jiffies(timeout)) == 0) {
> -		if (timeout)
> -			pr_warn("spk_ttyio: timeout (%d)  while waiting for input\n",
> -				timeout);
> +		pr_warn("spk_ttyio: timeout (%d)  while waiting for input\n",
> +			timeout);
>  		return 0xff;
>  	}
>  

-- 
Your life is like a penny.  You're going to lose it.  The question is:
How do
you spend it?

         John Covici wb2una
         covici at ccs.covici.com


More information about the Speakup mailing list