bug#18912: 24.4; mode-line corruption on graphical frames in dual-headed display

["Bruno Félix Rezende Ribeiro" <oitofelix@gnu.org>]

[-- Attachment #1.1: Type: text/plain, Size: 4639 bytes --]

Eli Zaretskii wrote:
> I'd like to solve this problem, not paper over it.  I hope you agree
> that solving it is better than working around it.

Sure, given that it's practical to solve.

> I find it hard to believe that we won't find the root cause, so I
> think we don't need to consider that possibility.

I'm glad to hear it!  I'll do whatever I can to help you figure this
out. :-)

> Here are the first instructions to get information that will help us
> start digging into this.  (These are in addition to what Martin
> already requested.)

I've replied to Martin's request already.

> First, please build Emacs without optimizations and with extra
> checking code enabled:
> 
>   CFLAGS='-O0 -g3' ./configure --prefix=/home/felix/opt/emacs-24.4 --with-x-toolkit=athena --enable-checking='yes,glyphs'
> 
> (I only care about CFLAGS and the --enable-checking= option; the rest
> I just copied from your original report, but feel free to change them
> if you want.  The --enable-checking= option is required to compile the
> dump-glyph-matrix command I ask you to use below.)

Here is what Emacs says about the configuration step:

  Configured using:
   `configure --prefix=/home/felix/opt/emacs-24.4-debug
  --with-x-toolkit=athena --enable-checking=yes,glyphs 'CFLAGS=-O0 -g3''

I did a VPATH build, though.

> After 'configure' finishes, invoke "make".  You don't have to invoke
> "make install", as Emacs can be run from the src/ directory where it
> was built.

I did that: ran it right away from the build directory.

> Now start Emacs you've just build with "emacs -Q", type the "C-x d"
> command that you say is sufficient to reproduce the problem, and then
> type this:
> 
>   M-x dump-glyph-matrix RET
> 
> This outputs to the standard error stream the description of the
> internal data structure, called "glyph matrix", which Emacs maintains
> for each window on a GUI frame.  Please post that output (you may wish
> to redirect stderr to a file when you invoke Emacs, to make that
> easier).
> 
> Please do this once for the "good" configuration, and another time for
> the "bad" configuration, but please make sure you invoke Emacs the
> same in both cases and type the same "C-x d" command in each case.

The outputs of the 'dump-glyph-matrix' command for both the good and bad
configurations are in the attached files 'dump-glyph-matrix-good.txt'
and 'dump-glyph-matrix-bad.txt', respectively; they are identical, though.

> Next, in the "bad" configuration and with the listing of /dev and
> corrupted mode line displayed, type this:
> 
>   M-: (setq frame-resize-pixelwise t) RET
> 
> Now carefully decrease the frame's height by dragging its upper or
> lower edge with the mouse.  Please drag it slowly and carefully, so
> that the frame is resized one pixel at a time.  After each resize,
> please see if the mode line is no longer corrupted.  Perhaps kill the
> buffer and type "C-x d" again to make sure the corruption disappeared
> for good.  The first time the mode line stops being corrupted, type
> 
>   M-x dump-glyph-matrix RET
> 
> again, and include this (3rd) output in your message.  Also, please
> tell how many pixels did you subtract from the original frame height
> before the corruption disappeared (assuming it ever does).

For the sake of precision I used the method suggested by Martin:
evaluating the expression

(while (y-or-n-p "decrease?")
  (set-frame-height nil (1- (frame-text-height)) nil t))

in the requested condition.  In order to not be fooled by a spurious
mode-line redrawing, I killed the buffer and invoked 'C-x d /dev RET'
again and the glitch was gone, after one iteration (minus 1 pixel).  The
output of the command 'dump-glyph-matrix' is in the attached file named
'dump-glyph-matrix-bad-resized.txt'.  This one is different from the
other two.

Then, to be sure I executed immediately the inverse procedure by
evaluating the expression

(while (y-or-n-p "increase?")
  (set-frame-height nil (1+ (frame-text-height)) nil t))

and therefore incrementing the frame's height by 1 pixel.  After killing
and getting again to the Dired buffer, the glitch was back.

> Where are the instructions to use the debugger, you ask?  Well, not
> yet, but maybe after we see what the above tells us.

I'm waiting for further instructions. :-P

-- 
 ,= ,-_-. =.  Bruno Félix Rezende Ribeiro (oitofelix) [0x28D618AF]
((_/)o o(\_)) There is no system but GNU;
 `-'(. .)`-'  GNU Linux-Libre is one of its official kernels;
     \_/      All software must be free as in freedom;

[-- Attachment #1.2: dump-glyph-matrix-bad.txt --]
[-- Type: text/plain, Size: 3639 bytes --]

PT = 214, BEGV = 1. ZV = 10370
Cursor x = 468, y = 72, hpos = 52, vpos = 4
=============================================
  0: (1) ''
  0: (1) '  /dev:[\n]'
  0: (1) ''
  1: (1) ''
  1: (1) '  total used in directory 4 available 10240[\n]'
  1: (1) ''
  2: (1) ''
  2: (1) '  drwxr-xr-x  19 root root        3500 Nov  1 21:13 .[\n]'
  2: (1) ''
  3: (1) ''
  3: (1) '  drwxr-xr-x  22 root root        4096 Out 27 17:58 ..[\n]'
  3: (1) ''
  4: (1) ''
  4: (1) '  crw-rw----   1 root video    10, 175 Out 30 22:18 agpgart[\n]'
  4: (1) ''
  5: (1) ''
  5: (1) '  crw-rw----+  1 root audio    14,   4 Out 30 22:18 audio[\n]'
  5: (1) ''
  6: (1) ''
  6: (1) '  crw-------   1 root root     10, 235 Out 30 22:18 autofs[\n]'
  6: (1) ''
  7: (1) ''
  7: (1) '  drwxr-xr-x   2 root root         400 Out 30 22:18 block[\n]'
  7: (1) ''
  8: (1) ''
  8: (1) '  drwxr-xr-x   2 root root         100 Out 30 22:16 bsg[\n]'
  8: (1) ''
  9: (1) ''
  9: (1) '  crw-------   1 root root     10, 234 Out 30 22:17 btrfs-control[\n]'
  9: (1) ''
 10: (1) ''
 10: (1) '  drwxr-xr-x   3 root root          60 Out 30 22:16 bus[\n]'
 10: (1) ''
 11: (1) ''
 11: (1) '  drwxr-xr-x   2 root root        3340 Nov  3 15:28 char[\n]'
 11: (1) ''
 12: (1) ''
 12: (1) '  crw-------   1 root root      5,   1 Out 30 22:18 console[\n]'
 12: (1) ''
 13: (1) ''
 13: (1) '  lrwxrwxrwx   1 root root          11 Out 30 22:16 core -> /proc/kcore[\n]'
 13: (1) ''
 14: (1) ''
 14: (1) '  drwxr-xr-x   2 root root          60 Out 30 22:16 cpu[\n]'
 14: (1) ''
 15: (1) ''
 15: (1) '  crw-------   1 root root     10,  62 Out 30 22:18 cpu_dma_latency[\n]'
 15: (1) ''
 16: (1) ''
 16: (1) '  crw-rw-rw-   1 root root     10, 203 Out 30 22:18 cuse[\n]'
 16: (1) ''
 17: (1) ''
 17: (1) '  drwxr-xr-x   5 root root         100 Out 30 22:17 disk[\n]'
 17: (1) ''
 18: (1) ''
 18: (1) '  brw-rw----   1 root disk    254,   0 Out 30 22:18 dm-0[\n]'
 18: (1) ''
 19: (1) ''
 19: (1) '  brw-rw----   1 root disk    254,   1 Out 30 22:18 dm-1[\n]'
 19: (1) ''
 20: (1) ''
 20: (1) '  brw-rw----   1 root disk    254,   2 Out 30 22:18 dm-2[\n]'
 20: (1) ''
 21: (1) ''
 21: (1) '  brw-rw----   1 root disk    254,   3 Out 30 22:18 dm-3[\n]'
 21: (1) ''
 22: (1) ''
 22: (1) '  drwxr-xr-x   2 root root          80 Out 30 22:18 dri[\n]'
 22: (1) ''
 23: (1) ''
 23: (1) '  crw-rw----+  1 root audio    14,   3 Out 30 22:18 dsp[\n]'
 23: (1) ''
 24: (1) ''
 24: (1) '  crw-rw----   1 root video    29,   0 Out 30 22:18 fb0[\n]'
 24: (1) ''
 25: (1) ''
 25: (1) '  lrwxrwxrwx   1 root root          13 Out 30 22:16 fd -> /proc/self/fd[\n]'
 25: (1) ''
 26: (1) ''
 26: (1) '  drwxr-xr-x   2 root root         100 Out 30 22:17 felix-laptop[\n]'
 26: (1) ''
 27: (1) ''
 27: (1) '  crw-rw-rw-   1 root root      1,   7 Out 30 22:18 full[\n]'
 27: (1) ''
 28: (1) ''
 28: (1) '  crw-rw-rw-   1 root root     10, 229 Out 30 22:18 fuse[\n]'
 28: (1) ''
 29: (1) ''
 29: (1) '  crw-------   1 root root    251,   0 Nov  1 21:13 hidraw0[\n]'
 29: (1) ''
 30: (1) ''
 30: (1) '  crw-------   1 root root    251,   1 Nov  1 00:43 hidraw1[\n]'
 30: (1) ''
 31: (1) ''
 31: (1) '  crw-------   1 root root     10, 228 Out 30 22:18 hpet[\n]'
 31: (1) ''
 32: (1) ''
 32: (1) '  drwxr-xr-x   2 root root           0 Out 30 22:17 hugepages[\n]'
 32: (1) ''
 33: (1) ''
 33: (1) '  lrwxrwxrwx   1 root root          25 Out 30 22:17 initctl -> /run/systemd/init'
 33: (1) ''
 34: (0) ''
 34: (0) ''
 34: (0) ''
 35: (0) ''
 35: (0) ''
 35: (0) ''
 36: (0) ''
 36: (0) ''
 36: (0) ''
 37: (1) ''
 37: (1) ' U:%%-  dev                 Top L5     (Dired by name)                              '
 37: (1) ''

[-- Attachment #1.3: dump-glyph-matrix-bad-resized.txt --]
[-- Type: text/plain, Size: 3687 bytes --]

PT = 214, BEGV = 1. ZV = 10370
Cursor x = 468, y = 72, hpos = 52, vpos = 4
=============================================
  0: (1) ''
  0: (1) '  /dev:[\n]'
  0: (1) ''
  1: (1) ''
  1: (1) '  total used in directory 4 available 10240[\n]'
  1: (1) ''
  2: (1) ''
  2: (1) '  drwxr-xr-x  19 root root        3500 Nov  1 21:13 .[\n]'
  2: (1) ''
  3: (1) ''
  3: (1) '  drwxr-xr-x  22 root root        4096 Out 27 17:58 ..[\n]'
  3: (1) ''
  4: (1) ''
  4: (1) '  crw-rw----   1 root video    10, 175 Out 30 22:18 agpgart[\n]'
  4: (1) ''
  5: (1) ''
  5: (1) '  crw-rw----+  1 root audio    14,   4 Out 30 22:18 audio[\n]'
  5: (1) ''
  6: (1) ''
  6: (1) '  crw-------   1 root root     10, 235 Out 30 22:18 autofs[\n]'
  6: (1) ''
  7: (1) ''
  7: (1) '  drwxr-xr-x   2 root root         400 Out 30 22:18 block[\n]'
  7: (1) ''
  8: (1) ''
  8: (1) '  drwxr-xr-x   2 root root         100 Out 30 22:16 bsg[\n]'
  8: (1) ''
  9: (1) ''
  9: (1) '  crw-------   1 root root     10, 234 Out 30 22:17 btrfs-control[\n]'
  9: (1) ''
 10: (1) ''
 10: (1) '  drwxr-xr-x   3 root root          60 Out 30 22:16 bus[\n]'
 10: (1) ''
 11: (1) ''
 11: (1) '  drwxr-xr-x   2 root root        3340 Nov  3 15:28 char[\n]'
 11: (1) ''
 12: (1) ''
 12: (1) '  crw-------   1 root root      5,   1 Out 30 22:18 console[\n]'
 12: (1) ''
 13: (1) ''
 13: (1) '  lrwxrwxrwx   1 root root          11 Out 30 22:16 core -> /proc/kcore[\n]'
 13: (1) ''
 14: (1) ''
 14: (1) '  drwxr-xr-x   2 root root          60 Out 30 22:16 cpu[\n]'
 14: (1) ''
 15: (1) ''
 15: (1) '  crw-------   1 root root     10,  62 Out 30 22:18 cpu_dma_latency[\n]'
 15: (1) ''
 16: (1) ''
 16: (1) '  crw-rw-rw-   1 root root     10, 203 Out 30 22:18 cuse[\n]'
 16: (1) ''
 17: (1) ''
 17: (1) '  drwxr-xr-x   5 root root         100 Out 30 22:17 disk[\n]'
 17: (1) ''
 18: (1) ''
 18: (1) '  brw-rw----   1 root disk    254,   0 Out 30 22:18 dm-0[\n]'
 18: (1) ''
 19: (1) ''
 19: (1) '  brw-rw----   1 root disk    254,   1 Out 30 22:18 dm-1[\n]'
 19: (1) ''
 20: (1) ''
 20: (1) '  brw-rw----   1 root disk    254,   2 Out 30 22:18 dm-2[\n]'
 20: (1) ''
 21: (1) ''
 21: (1) '  brw-rw----   1 root disk    254,   3 Out 30 22:18 dm-3[\n]'
 21: (1) ''
 22: (1) ''
 22: (1) '  drwxr-xr-x   2 root root          80 Out 30 22:18 dri[\n]'
 22: (1) ''
 23: (1) ''
 23: (1) '  crw-rw----+  1 root audio    14,   3 Out 30 22:18 dsp[\n]'
 23: (1) ''
 24: (1) ''
 24: (1) '  crw-rw----   1 root video    29,   0 Out 30 22:18 fb0[\n]'
 24: (1) ''
 25: (1) ''
 25: (1) '  lrwxrwxrwx   1 root root          13 Out 30 22:16 fd -> /proc/self/fd[\n]'
 25: (1) ''
 26: (1) ''
 26: (1) '  drwxr-xr-x   2 root root         100 Out 30 22:17 felix-laptop[\n]'
 26: (1) ''
 27: (1) ''
 27: (1) '  crw-rw-rw-   1 root root      1,   7 Out 30 22:18 full[\n]'
 27: (1) ''
 28: (1) ''
 28: (1) '  crw-rw-rw-   1 root root     10, 229 Out 30 22:18 fuse[\n]'
 28: (1) ''
 29: (1) ''
 29: (1) '  crw-------   1 root root    251,   0 Nov  1 21:13 hidraw0[\n]'
 29: (1) ''
 30: (1) ''
 30: (1) '  crw-------   1 root root    251,   1 Nov  1 00:43 hidraw1[\n]'
 30: (1) ''
 31: (1) ''
 31: (1) '  crw-------   1 root root     10, 228 Out 30 22:18 hpet[\n]'
 31: (1) ''
 32: (1) ''
 32: (1) '  drwxr-xr-x   2 root root           0 Out 30 22:17 hugepages[\n]'
 32: (1) ''
 33: (0) ''
 33: (0) '  lrwxrwxrwx   1 root root          25 Out 30 22:17 initctl -> /run/systemd/init'
 33: (0) ''
 34: (0) ''
 34: (0) ''
 34: (0) ''
 35: (0) ''
 35: (0) ' U:%%-  dev                 Top L5     (Dired by name)                              '
 35: (0) ''
 36: (1) ''
 36: (1) ' U:%%-  dev                 Top L5     (Dired by name)                              '
 36: (1) ''

[-- Attachment #1.4: dump-glyph-matrix-good.txt --]
[-- Type: text/plain, Size: 3639 bytes --]

PT = 214, BEGV = 1. ZV = 10370
Cursor x = 468, y = 72, hpos = 52, vpos = 4
=============================================
  0: (1) ''
  0: (1) '  /dev:[\n]'
  0: (1) ''
  1: (1) ''
  1: (1) '  total used in directory 4 available 10240[\n]'
  1: (1) ''
  2: (1) ''
  2: (1) '  drwxr-xr-x  19 root root        3500 Nov  1 21:13 .[\n]'
  2: (1) ''
  3: (1) ''
  3: (1) '  drwxr-xr-x  22 root root        4096 Out 27 17:58 ..[\n]'
  3: (1) ''
  4: (1) ''
  4: (1) '  crw-rw----   1 root video    10, 175 Out 30 22:18 agpgart[\n]'
  4: (1) ''
  5: (1) ''
  5: (1) '  crw-rw----+  1 root audio    14,   4 Out 30 22:18 audio[\n]'
  5: (1) ''
  6: (1) ''
  6: (1) '  crw-------   1 root root     10, 235 Out 30 22:18 autofs[\n]'
  6: (1) ''
  7: (1) ''
  7: (1) '  drwxr-xr-x   2 root root         400 Out 30 22:18 block[\n]'
  7: (1) ''
  8: (1) ''
  8: (1) '  drwxr-xr-x   2 root root         100 Out 30 22:16 bsg[\n]'
  8: (1) ''
  9: (1) ''
  9: (1) '  crw-------   1 root root     10, 234 Out 30 22:17 btrfs-control[\n]'
  9: (1) ''
 10: (1) ''
 10: (1) '  drwxr-xr-x   3 root root          60 Out 30 22:16 bus[\n]'
 10: (1) ''
 11: (1) ''
 11: (1) '  drwxr-xr-x   2 root root        3340 Nov  3 15:28 char[\n]'
 11: (1) ''
 12: (1) ''
 12: (1) '  crw-------   1 root root      5,   1 Out 30 22:18 console[\n]'
 12: (1) ''
 13: (1) ''
 13: (1) '  lrwxrwxrwx   1 root root          11 Out 30 22:16 core -> /proc/kcore[\n]'
 13: (1) ''
 14: (1) ''
 14: (1) '  drwxr-xr-x   2 root root          60 Out 30 22:16 cpu[\n]'
 14: (1) ''
 15: (1) ''
 15: (1) '  crw-------   1 root root     10,  62 Out 30 22:18 cpu_dma_latency[\n]'
 15: (1) ''
 16: (1) ''
 16: (1) '  crw-rw-rw-   1 root root     10, 203 Out 30 22:18 cuse[\n]'
 16: (1) ''
 17: (1) ''
 17: (1) '  drwxr-xr-x   5 root root         100 Out 30 22:17 disk[\n]'
 17: (1) ''
 18: (1) ''
 18: (1) '  brw-rw----   1 root disk    254,   0 Out 30 22:18 dm-0[\n]'
 18: (1) ''
 19: (1) ''
 19: (1) '  brw-rw----   1 root disk    254,   1 Out 30 22:18 dm-1[\n]'
 19: (1) ''
 20: (1) ''
 20: (1) '  brw-rw----   1 root disk    254,   2 Out 30 22:18 dm-2[\n]'
 20: (1) ''
 21: (1) ''
 21: (1) '  brw-rw----   1 root disk    254,   3 Out 30 22:18 dm-3[\n]'
 21: (1) ''
 22: (1) ''
 22: (1) '  drwxr-xr-x   2 root root          80 Out 30 22:18 dri[\n]'
 22: (1) ''
 23: (1) ''
 23: (1) '  crw-rw----+  1 root audio    14,   3 Out 30 22:18 dsp[\n]'
 23: (1) ''
 24: (1) ''
 24: (1) '  crw-rw----   1 root video    29,   0 Out 30 22:18 fb0[\n]'
 24: (1) ''
 25: (1) ''
 25: (1) '  lrwxrwxrwx   1 root root          13 Out 30 22:16 fd -> /proc/self/fd[\n]'
 25: (1) ''
 26: (1) ''
 26: (1) '  drwxr-xr-x   2 root root         100 Out 30 22:17 felix-laptop[\n]'
 26: (1) ''
 27: (1) ''
 27: (1) '  crw-rw-rw-   1 root root      1,   7 Out 30 22:18 full[\n]'
 27: (1) ''
 28: (1) ''
 28: (1) '  crw-rw-rw-   1 root root     10, 229 Out 30 22:18 fuse[\n]'
 28: (1) ''
 29: (1) ''
 29: (1) '  crw-------   1 root root    251,   0 Nov  1 21:13 hidraw0[\n]'
 29: (1) ''
 30: (1) ''
 30: (1) '  crw-------   1 root root    251,   1 Nov  1 00:43 hidraw1[\n]'
 30: (1) ''
 31: (1) ''
 31: (1) '  crw-------   1 root root     10, 228 Out 30 22:18 hpet[\n]'
 31: (1) ''
 32: (1) ''
 32: (1) '  drwxr-xr-x   2 root root           0 Out 30 22:17 hugepages[\n]'
 32: (1) ''
 33: (1) ''
 33: (1) '  lrwxrwxrwx   1 root root          25 Out 30 22:17 initctl -> /run/systemd/init'
 33: (1) ''
 34: (0) ''
 34: (0) ''
 34: (0) ''
 35: (0) ''
 35: (0) ''
 35: (0) ''
 36: (0) ''
 36: (0) ''
 36: (0) ''
 37: (1) ''
 37: (1) ' U:%%-  dev                 Top L5     (Dired by name)                              '
 37: (1) ''

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]