* bug#24124: emacs terminal escape command error @ 2016-08-01 14:34 modem 2016-08-01 15:18 ` Eli Zaretskii 2016-08-03 13:21 ` modem 0 siblings, 2 replies; 7+ messages in thread From: modem @ 2016-08-01 14:34 UTC (permalink / raw) To: 24124 Hi, I am facing a problem with emacs inside tmux. The problem can be reproduced easily. Emacs seems to have an terminal issue when being back from background mode on shell. Let me explain: When I put emacs in background (^z) and then in foreground again ("fg", bash) sometime emacs will not reset properly the terminal and I won't be able to scroll up (using PageUp/PageDown). After debugging the issue with a tmux developer, we tracked it down to emacs and believe this is an emacs issue with terminal handling when going back from background mode. I am attaching a snippet of the technical detail about this issue : > When you ^Z, tmux gets 112 bytes in all cases: > > input_parse: %0 ground, 40 bytes: \n\n\033[K\033[?1l\033>\033[34h\033[?25h\033[?1049l\033[39;49m\r > input_parse: %0 ground, 72 bytes: \r\n[1]+ Stopped emacs large_text_file.txt\r\nroot@milu:~# > > \033[?1049l is rmcup which exits cursor positioning (alternate screen) mode. > > Then when you type fg, it gets: > > input_parse: %0 ground, 694 bytes: \033[?1049h\033[34l\033[?1h\033=\033[24;1H\033[K\033[H\033[?25l\033[3mFile Edit Options Buffers ... > > \033[?1049h is smcup which turns it on again. > > But sometimes, it gets: > > input_parse: %0 ground, 674 bytes: \033[24;1H\033[K\033[H\033[?25l\033[3mFile Edit Options Buffers ... > > Where emacs is not sending \033[1049h, or in fact any of the smcup, > cvvis, smkx sequence it normally sends. This can be reproduced by hitting ^z and then "fg" in a loop for a couples of minutes. If you run tmux with "tmux -vvvv" you will be able to obtain the logs mentionned above. To observe the problem you can just press PageUp each time you are back in emacs, when the bug will reproduce you won't pageUp inside emacs but enter the tmux's copy-mode. My ~/.tmux.conf only has 1 line: bind-key -T root PPage if-shell -F "#{alternate_on}" "send-keys PPage" "copy-mode -e -u" If you need me to apply some patch and/or reproduce just let me know I will happily helps. Thanks a lot, -- modem ^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#24124: emacs terminal escape command error 2016-08-01 14:34 bug#24124: emacs terminal escape command error modem @ 2016-08-01 15:18 ` Eli Zaretskii 2016-08-03 13:21 ` modem 1 sibling, 0 replies; 7+ messages in thread From: Eli Zaretskii @ 2016-08-01 15:18 UTC (permalink / raw) To: modem; +Cc: 24124 > From: modem <modem@member.fsf.org> > Date: Mon, 1 Aug 2016 16:34:43 +0200 > > When I put emacs in background (^z) and then in foreground again ("fg", > bash) sometime emacs will not reset properly the terminal and I won't be > able to scroll up (using PageUp/PageDown). > > After debugging the issue with a tmux developer, we tracked it down to > emacs and believe this is an emacs issue with terminal handling when > going back from background mode. > > I am attaching a snippet of the technical detail about this issue : > > > When you ^Z, tmux gets 112 bytes in all cases: > > > > input_parse: %0 ground, 40 bytes: > \n\n\033[K\033[?1l\033>\033[34h\033[?25h\033[?1049l\033[39;49m\r > > input_parse: %0 ground, 72 bytes: \r\n[1]+ Stopped > emacs large_text_file.txt\r\nroot@milu:~# > > > > \033[?1049l is rmcup which exits cursor positioning (alternate > screen) mode. > > > > Then when you type fg, it gets: > > > > input_parse: %0 ground, 694 bytes: > \033[?1049h\033[34l\033[?1h\033=\033[24;1H\033[K\033[H\033[?25l\033[3mFile > Edit Options Buffers ... > > > > \033[?1049h is smcup which turns it on again. > > > > But sometimes, it gets: > > > > input_parse: %0 ground, 674 bytes: > \033[24;1H\033[K\033[H\033[?25l\033[3mFile Edit Options Buffers ... > > > > Where emacs is not sending \033[1049h, or in fact any of the smcup, > > cvvis, smkx sequence it normally sends. Probably because Emacs tries to optimize the cursor motion commands for minimal cost. See the code of the cmgoto function (in cm.c in Emacs sources). But you didn't tell what terminfo entry is used by Emacs on your system, and didn't tell what the above terminal commands do on your terminal, so it's hard to proceed in investigating this issue. Thanks. ^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#24124: emacs terminal escape command error 2016-08-01 14:34 bug#24124: emacs terminal escape command error modem 2016-08-01 15:18 ` Eli Zaretskii @ 2016-08-03 13:21 ` modem 2016-08-04 17:07 ` Eli Zaretskii 1 sibling, 1 reply; 7+ messages in thread From: modem @ 2016-08-03 13:21 UTC (permalink / raw) To: 24124 [-- Attachment #1: Type: text/plain, Size: 1978 bytes --] > [Please keep the bug address on the CC list.] > >> Thanks you for your advises. >> >> My terminfo: "echo $TERM" return "screen". Is that it ? > > Probably. > >> I recompiled emacs after commenting a few optimization in src/cm.c (cmgoto). >> Unfortunately I can still reproduce the bug with this binary. >> >> I am attaching you the patch I did. > > I think you disabled the wrong code. What you need is force Emacs to > always do this: > > if (cost <= relcost)œ > { /* really is cheaper */ > emacs_tputs (tty, p, 1, cmputc); > curY (tty) = row, curX (tty) = col; > return; > } Hi Eli, Thanks a lot. So I did another patch and another try. I can still reproduce the bug. Any more advices on stuff I should try are welcomed ;-) Here is the patch: --- emacs24-24.4+1.orig/src/cm.c 2014-10-21 23:45:42.000000000 +0200 +++ /dev/shm/cm.c 2016-08-03 15:10:55.803193217 +0200 @@ -327,10 +327,12 @@ char *p; const char *dcm; +#if 0 /* First the degenerate case */ if (row == curY (tty) && col == curX (tty)) /* already there */ return; - +#endif // XXX MODEM + if (curY (tty) >= 0 && curX (tty) >= 0) { /* We may have quick ways to go to the upper-left, bottom-left, @@ -375,7 +377,7 @@ * In the following comparison, the = in <= is because when the costs * are the same, it looks nicer (I think) to move directly there. */ - if (directcost <= relcost) + if (directcost <= relcost || 1) // XXX MODEM { /* compute REAL direct cost */ cost = 0; @@ -383,14 +385,14 @@ ? tgoto (dcm, row, col) : tgoto (dcm, col, row)); emacs_tputs (tty, p, 1, evalcost); - if (cost <= relcost) + if (cost <= relcost || 1) // XXX MODEM { /* really is cheaper */ emacs_tputs (tty, p, 1, cmputc); curY (tty) = row, curX (tty) = col; return; } } - + switch (use) { case USEHOME: [-- Attachment #2: second_try.patch --] [-- Type: text/x-patch, Size: 1158 bytes --] --- emacs24-24.4+1.orig/src/cm.c 2014-10-21 23:45:42.000000000 +0200 +++ /dev/shm/cm.c 2016-08-03 15:10:55.803193217 +0200 @@ -327,10 +327,12 @@ char *p; const char *dcm; +#if 0 /* First the degenerate case */ if (row == curY (tty) && col == curX (tty)) /* already there */ return; - +#endif // XXX MODEM + if (curY (tty) >= 0 && curX (tty) >= 0) { /* We may have quick ways to go to the upper-left, bottom-left, @@ -375,7 +377,7 @@ * In the following comparison, the = in <= is because when the costs * are the same, it looks nicer (I think) to move directly there. */ - if (directcost <= relcost) + if (directcost <= relcost || 1) // XXX MODEM { /* compute REAL direct cost */ cost = 0; @@ -383,14 +385,14 @@ ? tgoto (dcm, row, col) : tgoto (dcm, col, row)); emacs_tputs (tty, p, 1, evalcost); - if (cost <= relcost) + if (cost <= relcost || 1) // XXX MODEM { /* really is cheaper */ emacs_tputs (tty, p, 1, cmputc); curY (tty) = row, curX (tty) = col; return; } } - + switch (use) { case USEHOME: ^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#24124: emacs terminal escape command error 2016-08-03 13:21 ` modem @ 2016-08-04 17:07 ` Eli Zaretskii 0 siblings, 0 replies; 7+ messages in thread From: Eli Zaretskii @ 2016-08-04 17:07 UTC (permalink / raw) To: modem; +Cc: 24124 > From: modem <modem@member.fsf.org> > Date: Wed, 3 Aug 2016 15:21:24 +0200 > > > I think you disabled the wrong code. What you need is force Emacs to > > always do this: > > > > if (cost <= relcost)œ > > { /* really is cheaper */ > > emacs_tputs (tty, p, 1, cmputc); > > curY (tty) = row, curX (tty) = col; > > return; > > } > > Hi Eli, > > Thanks a lot. So I did another patch and another try. I can still > reproduce the bug. > > Any more advices on stuff I should try are welcomed ;-) I'm stumped. It could be some problem with some other piece of software, not with Emacs. Is there someone you could ask to produce a human-readable description of the command Emacs sent to the terminal driver, both in the "good" and in the "bad" cases? I mean this citation which you brought originally, but without any details: > I am attaching a snippet of the technical detail about this issue : > > > When you ^Z, tmux gets 112 bytes in all cases: > > > > input_parse: %0 ground, 40 bytes: > \n\n\033[K\033[?1l\033>\033[34h\033[?25h\033[?1049l\033[39;49m\r > > input_parse: %0 ground, 72 bytes: \r\n[1]+ Stopped > emacs large_text_file.txt\r\nroot@milu:~# > > > > \033[?1049l is rmcup which exits cursor positioning (alternate > screen) mode. > > > > Then when you type fg, it gets: > > > > input_parse: %0 ground, 694 bytes: > \033[?1049h\033[34l\033[?1h\033=\033[24;1H\033[K\033[H\033[?25l\033[3mFile > Edit Options Buffers ... > > > > \033[?1049h is smcup which turns it on again. > > > > But sometimes, it gets: > > > > input_parse: %0 ground, 674 bytes: > \033[24;1H\033[K\033[H\033[?25l\033[3mFile Edit Options Buffers ... > > > > Where emacs is not sending \033[1049h, or in fact any of the smcup, > > cvvis, smkx sequence it normally sends. It would be nice to be able to understand what was meant by whoever wrote this, as smcup, cvvis, etc. don't tell me much (I'm not a terminfo expert). Thanks. ^ permalink raw reply [flat|nested] 7+ messages in thread
[parent not found: <57A06C4B.70603@member.fsf.org>]
* bug#24124: emacs terminal escape command error [not found] <57A06C4B.70603@member.fsf.org> @ 2016-08-02 15:39 ` Eli Zaretskii 2019-11-08 3:54 ` Stefan Kangas 0 siblings, 1 reply; 7+ messages in thread From: Eli Zaretskii @ 2016-08-02 15:39 UTC (permalink / raw) To: modem; +Cc: 24124 [Please keep the bug address on the CC list.] > From: modem <modem@member.fsf.org> > Date: Tue, 2 Aug 2016 11:47:55 +0200 > > Thanks you for your advises. > > My terminfo: "echo $TERM" return "screen". Is that it ? Probably. > I recompiled emacs after commenting a few optimization in src/cm.c (cmgoto). > Unfortunately I can still reproduce the bug with this binary. > > I am attaching you the patch I did. I think you disabled the wrong code. What you need is force Emacs to always do this: if (cost <= relcost) { /* really is cheaper */ emacs_tputs (tty, p, 1, cmputc); curY (tty) = row, curX (tty) = col; return; } ^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#24124: emacs terminal escape command error 2016-08-02 15:39 ` Eli Zaretskii @ 2019-11-08 3:54 ` Stefan Kangas 2020-01-18 10:51 ` Stefan Kangas 0 siblings, 1 reply; 7+ messages in thread From: Stefan Kangas @ 2019-11-08 3:54 UTC (permalink / raw) To: Eli Zaretskii; +Cc: 24124, modem Eli Zaretskii <eliz@gnu.org> writes: > [Please keep the bug address on the CC list.] > >> From: modem <modem@member.fsf.org> >> Date: Tue, 2 Aug 2016 11:47:55 +0200 >> >> Thanks you for your advises. >> >> My terminfo: "echo $TERM" return "screen". Is that it ? > > Probably. > >> I recompiled emacs after commenting a few optimization in src/cm.c (cmgoto). >> Unfortunately I can still reproduce the bug with this binary. >> >> I am attaching you the patch I did. > > I think you disabled the wrong code. What you need is force Emacs to > always do this: > > if (cost <= relcost) > { /* really is cheaper */ > emacs_tputs (tty, p, 1, cmputc); > curY (tty) = row, curX (tty) = col; > return; > } That was 3 years ago. Did you ever get a chance to try the suggested fix by Eli above? Best regards, Stefan Kangas ^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#24124: emacs terminal escape command error 2019-11-08 3:54 ` Stefan Kangas @ 2020-01-18 10:51 ` Stefan Kangas 0 siblings, 0 replies; 7+ messages in thread From: Stefan Kangas @ 2020-01-18 10:51 UTC (permalink / raw) To: Eli Zaretskii; +Cc: 24124-done, modem Stefan Kangas <stefan@marxist.se> writes: > That was 3 years ago. Did you ever get a chance to try the suggested > fix by Eli above? More information was requested, but none was given within 10 weeks, so I'm closing this bug. If this is still an issue, please reply to this email (use "Reply to all" in your email client) and we can reopen the bug report. Best regards, Stefan Kangas ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2020-01-18 10:51 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-08-01 14:34 bug#24124: emacs terminal escape command error modem 2016-08-01 15:18 ` Eli Zaretskii 2016-08-03 13:21 ` modem 2016-08-04 17:07 ` Eli Zaretskii [not found] <57A06C4B.70603@member.fsf.org> 2016-08-02 15:39 ` Eli Zaretskii 2019-11-08 3:54 ` Stefan Kangas 2020-01-18 10:51 ` Stefan Kangas
Code repositories for project(s) associated with this external index https://git.savannah.gnu.org/cgit/emacs.git https://git.savannah.gnu.org/cgit/emacs/org-mode.git This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.