From: Howard Chu <hyc@symas.com>
To: Andi Kleen <andi@firstfloor.org>
Cc: linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: EXTPROC, telnetd LINEMODE, revisited
Date: Fri, 11 Jun 2010 04:17:32 -0700 [thread overview]
Message-ID: <4C121B4C.2000702@symas.com> (raw)
In-Reply-To: <87631pq2aj.fsf@basil.nowhere.org>
Andi Kleen wrote:
> Howard Chu<hyc@symas.com> writes:
>
>> It's been over 10 years since I looked at this last
>>
>> http://lkml.indiana.edu/hypermail/linux/kernel/9911.3/0650.html
>
> I would suggest you repost the patch.
Looks like my email got posted twice already (oops). The updated patch was
attached each time, you didn't get it?
>> From a quick look it looks straight forward enough.
The patch I posted still isn't quite right; it lets all of the input fall thru
the regular tty input code. If ICANON is set then the tty driver will parse
and act on any control characters in the input, but since the input was
already fully processed on the client, any control characters remaining in the
input should just be passed through literally. That should be an easy thing to
fix though.
>> Despite the fact that most people today have ready access to high
>> speed broadband networking today, I think the motivation for local
>> character processing is as great now as it was 10 years ago. I
>> regularly use an ssh client on an Android phone to keep tabs on my
>> servers when I'm away from my home base, and sometimes cellphone
>> connectivity can be extremely lossy, networks can be heavily
>> congested, etc... Waiting for character-at-a-time packet turnarounds
>> in these conditions can be pretty aggravating. Also, if you're
>
> I agree that it would be sometimes useful, I also had these
> problems.
>
>> unfortunate enough to need to get access to a machine while you're
>> roaming away from your home network, the per-byte roaming fees can be
>> murder. Both of these pain points can be minimized by using local
>> character processing and only sending complete lines to the remote
>> server.
>
> I have some doubts this really needs to be implemented in the kernel.
> Back in the old days it was important to save round trips
> to user space because CPUs were so slow, but these days I don't think
> that's an issue anymore for mere typing.
>
> Couldn't you implement it in screen or a similar pty based tool?
From how I see it at the moment, everything still depends on the tty driver.
Even if you devise some other mechanism, you still have to be able to
intercept any ioctl's issued on the pty slave and Do The Right Thing with them
in the daemon on the pty master. I guess, alternatively, you could set an
environment variable in the child process that inherits the pty slave, that
tells applications how to send commands out of band to the master, but that
will require a lot more app-level coordination.
>> Another feature with readline is a command history buffer that can be
>> reviewed using Cursor Up/Down. Can/should we define this in the tty
>> driver too? Or perhaps rely on the client to implement its own command
>> buffer, and never mention this aspect on the wire protocol. Again,
>> given the purpose, it makes most sense to me to keep this feature on
>> the client side. But some coordination with the server would still be
>> useful. E.g., different programs can maintain their own persistent
>> command history files. It might be nice to have a way for the app to
>> signal to the client which command context to use for the current
>> history buffer, and keep them all separate. (Or not, I can see other
>> times where you'd just rather have it all as one stack.)
>
> e.g. history management is definitely something that should not
> be done in the kernel.
I definitely wasn't trying to suggest that the management occur in the kernel.
Only that some mechanism for telling the client to toggle the feature on or
off would be useful.
>> PS: if anyone knows where to send the patches for telnetd, please
>> email me. Looks like the upstream source hasn't been touched since
>> 2000.
>
> I think they're defacto maintained by the distributions.
>
> I would submit them to one of the big distributions and let
> that maintainer figure it out.
OK. Since it looks like the source code I'm working with came from Debian I'll
start there, thanks.
> -Andi
--
-- Howard Chu
CTO, Symas Corp. http://www.symas.com
Director, Highland Sun http://highlandsun.com/hyc/
Chief Architect, OpenLDAP http://www.openldap.org/project/
next prev parent reply other threads:[~2010-06-11 11:17 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-11 10:18 EXTPROC, telnetd LINEMODE, revisited Howard Chu
2010-06-11 10:41 ` Andi Kleen
2010-06-11 11:17 ` Howard Chu [this message]
2010-06-11 20:22 ` Howard Chu
2010-06-12 1:36 ` [PATCH] " Howard Chu
2010-06-15 18:08 ` [PATCH 0/2] tty: add EXTPROC support for LINEMODE Howard Chu
2010-06-15 18:10 ` [PATCH 1/2] " Howard Chu
2010-06-15 18:13 ` [PATCH 2/2] tty: Add arch-specific EXTPROC definitions Howard Chu
2010-06-15 18:35 ` [PATCH 0/2] tty: add EXTPROC support for LINEMODE Greg KH
2010-06-15 18:46 ` Howard Chu
-- strict thread matches above, loose matches on Subject: below --
2010-06-11 9:54 EXTPROC, telnetd LINEMODE, revisited Howard Chu
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=4C121B4C.2000702@symas.com \
--to=hyc@symas.com \
--cc=andi@firstfloor.org \
--cc=linux-kernel@vger.kernel.org \
/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