Skip to contentSkip to conversation

BUG: pty.c in linux 2.6.31-rc5+

DISCUSSION MRU.Ilya 3 participants 3 messages Aug 4, 2011

lore
Share & tools
lore

MRU.Ilya

author
#0 lore raw

Synchronic's line disciplines(ppp_synctty, hdlc) can't work with pseudo terminals.

If frames send very fast from one side(master/slave) to other one,
frames merge in one (merge point "flip buffer") and the other side can read only one big frame. But sync line discipline need one frame in one read.

For push one frame to ldisc I propose small patch, but this regression for commit e043e42bdb66885b3ac10d27a01ccb9972e2b0a3.

I don't find any mutexes and other things forbidden in not user context, in function flush_to_ldisc() , and I think this patch
is true and this function can called from any context. But I not sure.

Relative commit 86d23a057e718f73adc15e463c643d6e014a19cd.

#1 lore raw
Quoted 8 lines · MRU.Ilya

Synchronic's line disciplines(ppp_synctty, hdlc) can't work with pseudo terminals.

If frames send very fast from one side(master/slave) to other one, frames merge in one (merge point "flip buffer") and the other side can read only one big frame.
But sync line discipline need one frame in one read.

Quoted 8 lines

For push one frame to ldisc I propose small patch, but this regression for commit e043e42bdb66885b3ac10d27a01ccb9972e2b0a3.

I don't find any mutexes and other things forbidden in not user context, in function flush_to_ldisc() , and I think this patch
is true and this function can called from any context. But I not sure.

Relative commit 86d23a057e718f73adc15e463c643d6e014a19cd.

I don't see a patch here.

<formletter>

This is not the correct way to submit patches for inclusion in the stable kernel tree. Please read Documentation/stable_kernel_rules.txt for how to do this properly.

</formletter>

#2 lore raw
In reply to MRU.Ilya
Quoted 2 lines · On Thu, 04 Aug 2011 10:43:19 +0400 "MRU.Ilya" <[email protected]> wrote:

Synchronic's line disciplines(ppp_synctty, hdlc) can't work with pseudo terminals.

Correct - tty devices do not support framing. This is intentional.