Re: serial port regression caused by "Char: tty_ioctl, use wait_event_interruptible_timeout" patch

[Paul Fulghum <paulkf@microgate.com>]

Paul Fulghum wrote:
> Instead of reverting the patch can you try modifying
> this part of the patch:
> 
> +	if (wait_event_interruptible_timeout(tty->write_wait,
> +			!tty->driver->chars_in_buffer(tty), timeout))
> +		return;
> 
> by changing it to:
> 
> +	if (wait_event_interruptible_timeout(tty->write_wait,
> +			!tty->driver->chars_in_buffer(tty), timeout) < 0)
> +		return;
> 
> It looks like the patch changed the behavior of
> tty_wait_until_sent by not calling the driver
> specific wait_until_sent if a timeout occurs.

I mispoke, the patch changed the behavior by not
calling the driver specific wait_until_sent if
the condition is true.

Original behavior:
call driver->wait_until_sent() on
timeout or true condition
(skip for signal)

Patch behavior:
call driver->wait_until_sent() only
on timeout (rc == 0)
(skip for signal or true)

By modifying the patch as described above,
the original behavior is restored.

-- 
Paul Fulghum
Microgate Systems, Ltd.