Bug 25270 - The server needs 32 bit regions
Summary: The server needs 32 bit regions
Status: RESOLVED MOVED
Alias: None
Product: xorg
Classification: Unclassified
Component: Server/General (show other bugs)
Version: git
Hardware: x86 (IA32) Linux (All)
: medium major
Assignee: Adam Jackson
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-11-24 19:51 UTC by David Ronis
Modified: 2018-12-13 22:21 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
gdb output captured with logging (28.65 KB, application/octet-stream)
2009-11-29 11:53 UTC, David Ronis
no flags Details
ddd/gdb log file (21.62 KB, application/octet-stream)
2009-12-06 11:21 UTC, David Ronis
no flags Details

Description David Ronis 2009-11-24 19:51:34 UTC
I've been following the git master of Xorg.   I tried viewing a magicpoint presentation today and X aborted.   The first slide displays properly (it's basically text).  The second is also simple text, displays as it should, but then X crashes.  I captured the message from startx shown in the summary on a remote screen.   I very vaguely recall having a similar problem with pixman several months ago.  

This happens only on one of the machines I tried it on.   This uses the radeon ati driver.
Comment 1 David Ronis 2009-11-25 10:52:37 UTC
I rebuilt pixman and xserver with debugging symbols (-O0 -g), attached gdb to X and triggered the bug.   r looks reasonable, except that 

r->x1 = 1224 (r->y1=0), r->x2=1220 (r->y2=768)

Comment 2 Søren Sandmann Pedersen 2009-11-27 10:42:22 UTC
Thanks for the bug report.

Can you post a backtrace, and also the complete contents of the 'r' region (ie., "p *r" in gdb).

Generally, when assertions are triggered in pixman it has been because the X server is doing something wrong, so it would be useful to see where this is coming from.


Comment 3 David Ronis 2009-11-29 11:53:24 UTC
Created attachment 31554 [details]
gdb output captured with logging

I logged the gdb session.   The information you want is probably at the end.  

Note:  p *r gave the same information I'd manually entered in the initial report.
Comment 4 Søren Sandmann Pedersen 2009-12-03 03:56:14 UTC
Could you try compiling pixman again with "-g -O0", but this time also adding

    #define PIXMAN_REGION_DEBUG

at the top of pixman-region.c?
Comment 5 Søren Sandmann Pedersen 2009-12-03 03:56:54 UTC
Hopefully that will produce some useful debug spew in the Xorg log file.
Comment 6 David Ronis 2009-12-06 11:18:58 UTC
I added the define as per your request and reran the program.   I still get an abort at the same point in magicpoint, but now at a different point in  pixman.  I'll attach the log file generated from within ddd/gdb.  There was no extra output in Xorg's log file.

Note that there is some print out towards then end.  This is the output of print *reg[12].
Comment 7 David Ronis 2009-12-06 11:21:14 UTC
Created attachment 31784 [details]
ddd/gdb log file
Comment 8 Søren Sandmann Pedersen 2009-12-16 13:36:54 UTC
What's going on here is that the 65532 in this:

#7  0x08121e03 in damageCopyArea (pSrc=0x86a4378, pDst=0x86a4378, pGC=0x87562c0, srcx=1224, srcy=0, width=65532, height=768, d
stx=1232, dsty=0) at damage.c:949

ends up generating a region containing a box with negative width. That is what causes the assert in pixman to fail.

Overflows of signed types are undefined in C, but assuming the obvious behavior, this actually does work, so in the recently-released pixman 0.16.4, the assert is turned off.

I don't think the server should be relying on this kind of thing, but it is difficult to see any way to fix this apart from moving to 32 bit regions, so I am moving and retitling the bug to reflect that.
Comment 9 David Ronis 2009-12-16 13:43:39 UTC
Thanks

Is there a simple workaround?   This is my main presentation software.

Comment 10 Søren Sandmann Pedersen 2009-12-17 13:41:32 UTC
I forgot to mention that newly-released pixman 0.16.4 disables this assert, so installing that should make it work (if it doesn't, please let me know).
Comment 11 David Ronis 2009-12-19 21:26:25 UTC
I just upgraded to the git/master of pixman (as well as xserver and several other modules).  The problem remains.
Comment 12 David Ronis 2009-12-29 14:03:22 UTC
Forget comment #11.  Turns out it was finding another (older) copy of pixman.  When I upgraded it the problem goes away.

Comment 13 GitLab Migration User 2018-12-13 22:21:56 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/xorg/xserver/issues/386.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.