* 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
[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-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
* 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.