mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Alan Cox <alan@lxorguk.ukuu.org.uk>
To: Thomas Pfaff <tpfaff@pcs.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: Question about TTY_DO_WRITE_WAKEUP
Date: Thu, 20 Nov 2008 14:31:47 +0000	[thread overview]
Message-ID: <20081120143147.5e8d6c20@lxorguk.ukuu.org.uk> (raw)
In-Reply-To: <Pine.LNX.4.64.0811201446380.21561@lxtpfaff.pcs.ditec.de>

> Is the TTY_DO_WRITE_WAKEUP bit some kind of cruft that can be removed entirely 
> or should i generate a patch for serial devices only ?

It shouldn't happen at the serial level. The line discipline sets the
flag as and when it has data for the process above.

The expected flow in normal use is

	app writes to tty
		data goes to n_tty ldisc
			goes to tty
		n_tty ldisc seems tty blocked
		n_tty ldisc sets DO_WRITE_WAKEUP

	tty gets ucongested
		calls n_tty write_wakeup
			SIGIO sent
			Flag set again

The important part of the logic is in n_tty.c and tty_io.c

Alan

  reply	other threads:[~2008-11-20 14:32 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-20 10:06 Thomas Pfaff
2008-11-20 13:17 ` Alan Cox
2008-11-20 14:09   ` Thomas Pfaff
2008-11-20 14:31     ` Alan Cox [this message]
2008-11-20 15:12       ` Thomas Pfaff
2008-11-20 15:55         ` Alan Cox
2008-11-20 16:39           ` Thomas Pfaff
2008-11-20 16:47             ` Alan Cox
2008-11-20 16:58             ` Alan Cox

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20081120143147.5e8d6c20@lxorguk.ukuu.org.uk \
    --to=alan@lxorguk.ukuu.org.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tpfaff@pcs.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®