Bug 542 - RFE: Investigate GLX support in Xprt
Summary: RFE: Investigate GLX support in Xprt
Status: RESOLVED WONTFIX
Alias: None
Product: xprint
Classification: Unclassified
Component: Server: Other (show other bugs)
Version: unspecified
Hardware: All All
: lowest enhancement
Assignee: Roland Mainz
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 558 559
Blocks:
  Show dependency treegraph
 
Reported: 2004-04-23 18:16 UTC by Roland Mainz
Modified: 2011-09-13 12:57 UTC (History)
5 users (show)

See Also:
i915 platform:
i915 features:


Attachments
Patch to modify "glxgears" for print output (9.42 KB, patch)
2004-04-23 18:18 UTC, Roland Mainz
no flags Details | Splinter Review
Patch for xc/programs/Xserver/ sources to enable GLX for Xprint (2.17 KB, patch)
2004-04-23 18:22 UTC, Roland Mainz
no flags Details | Splinter Review
PDF output of modified glxgears, 22 pages. DIN-A5, 300DPI (888.94 KB, application/pdf)
2004-04-23 18:34 UTC, Roland Mainz
no flags Details
Fix bugs in X visual setup, and enable GLX extension (5.19 KB, patch)
2004-04-26 07:44 UTC, Alan Hourihane
no flags Details | Splinter Review
reduce warnings during compilation patch (15.03 KB, patch)
2004-04-26 07:45 UTC, Alan Hourihane
no flags Details | Splinter Review
Fix bugs in X visual setup, and enable GLX extension [checked-in] (6.37 KB, patch)
2004-04-26 21:03 UTC, Roland Mainz
no flags Details | Splinter Review

Description Roland Mainz 2004-04-23 18:16:59 UTC
RFE: Investigate whether it is possible/usefull to support GLX in Xprt.

I've played a little bit around in the last hour and it seems to work... but I
am still not sure if all the stuff supported by GLX/OpenGL will work well with
"paginated devices".
Comment 1 Roland Mainz 2004-04-23 18:18:53 UTC
Created attachment 229 [details] [review]
Patch to modify "glxgears" for print output
Comment 2 Roland Mainz 2004-04-23 18:22:29 UTC
Created attachment 230 [details] [review]
Patch for xc/programs/Xserver/ sources to enable GLX for Xprint
Comment 3 Roland Mainz 2004-04-23 18:34:59 UTC
Created attachment 231 [details]
PDF output of modified glxgears, 22 pages. DIN-A5, 300DPI
Comment 4 Roland Mainz 2004-04-23 21:16:11 UTC
Comment on attachment 231 [details]
PDF output of modified glxgears, 22 pages. DIN-A5, 300DPI

OK, it seems that this sample PDF job is for the trashcan.

SuSE 8.2's /usr/lib/libGL.so links to /usr/lib/GL/libGL.so.1.4.mesasoft by
default which bypasses the GLX protocol completely and just renders via
XPutImage().
Adjusting the link to /usr/lib/GL/libGL.so.1.2.xf86_glx results in a nice
"Error: couldn't get an RGB, Double-buffered visual" from "glxgears".

It seems that the Xserver-side patch still needs work... ;-(
Comment 5 Roland Mainz 2004-04-23 22:05:51 UTC
It seems there is something special required to mark a visual as
"GLX-enabled"... does anyone have a clue what needs to be done for that ?
Comment 6 Roland Mainz 2004-04-24 06:32:40 UTC
Some additiona info:
glxinfo output is interesting:
-- snip --
% glxinfo -display :50 -t
name of display: :50.0
Error: couldn't find RGB GLX visual

Vis  Vis   Visual Trans  buff lev render DB ste  r   g   b   a  aux dep ste 
accum buffers  MS   MS
 ID Depth   Type  parent size el   type     reo sz  sz  sz  sz  buf th  ncl  r  
g   b   a  num bufs
----------------------------------------------------------------------------------------------------
0x21 24 TrueColor    1      0  0  ci     0   0   0   0   0   0   0    0  0   0  
0   0   0   0   0
0x22  8 PseudoColor  1      0  0  ci     0   0   0   0   0   0   0    0  0   0  
0   0   0   0   0
-- snip --

Note the "ci" (color-indexed visual) entry for the "TrueColor" line - that
cannot be right. More digging in the glxinfo sources shows that all the data are
simply "0", which means: Not initalised. Something in the Xserver-side GLX
extension code is going wrong...

... then I added  some debug code to xc/programs/Xserver/GL/glx/glxscreens.c -
it seems that in |void __glXScreenInit(GLint numscreens)|
|__glXActiveScreens[i].numUsableVisuals| is set to |0|, e.g. no "useable GL
visuals". Fun... ;-(
Maybe |__MESA_initVisuals| is broken somehow...
Comment 7 Alan Hourihane 2004-04-24 10:29:49 UTC
I've fixed this and will commit the patch once Egbert has finished with the trunk.
Comment 8 Roland Mainz 2004-04-24 10:42:01 UTC
Alan Hourihane wrote:
> I've fixed this and will commit the patch once Egbert has finished with the >
> trunk.

Two minor things:
1. Can you attach the patch here that I can take a look at it first, please ?
2. Please don't commit the glxgears.c patch... I have to clean it up and adjust
it that it allows both "display" and "print" mode... the current patch
(attachment 229 [details] [review]) only supports printer output...
Comment 9 Alan Hourihane 2004-04-24 16:35:04 UTC
Actually, all the problems are with Xprint itself and nothing to do with Mesa or
GLX.
Comment 10 Alan Hourihane 2004-04-26 07:44:17 UTC
Created attachment 237 [details] [review]
Fix bugs in X visual setup, and enable GLX extension

This patch fixes bugs in Xprint's visuals setup code and also enables the GLX
extension
Comment 11 Alan Hourihane 2004-04-26 07:45:39 UTC
Created attachment 238 [details] [review]
reduce warnings during compilation patch

When compiling Xprint, it outputs lots and lots of warnings. This patch clears
up some of the more painful ones.
Comment 12 Roland Mainz 2004-04-26 20:31:45 UTC
Alan Hourihane wrote:
> Created an attachment (id=237)
> Fix bugs in X visual setup, and enable GLX extension
> This patch fixes bugs in Xprint's visuals setup code and also enables the GLX
> extension

The patch gets "glxinfo" working... but unfortunately the Xprint-enabled
"glxgears" version (from attachment 229 [details] [review]) crashes Xprt.

On the client ("glxgears") side the stack trace look like this (I added a
XSyncronize() to ensure the requests are in sync):
-- snip --
0x402d5342 in select () from /lib/libc.so.6
(gdb) where
#0  0x402d5342 in select () from /lib/libc.so.6
#1  0x40186fe8 in __JCR_LIST__ () from /usr/X11R6/lib/libX11.so.6
#2  0x400f1298 in _XRead () from /usr/X11R6/lib/libX11.so.6
#3  0x400f1dbf in _XReply () from /usr/X11R6/lib/libX11.so.6
#4  0x400ecce5 in XSync () from /usr/X11R6/lib/libX11.so.6
#5  0x400ecda5 in _XSyncFunction () from /usr/X11R6/lib/libX11.so.6
#6  0x40086f26 in __glXFlushRenderBuffer () from /usr/lib/libGL.so.1
#7  0x40059b98 in __indirect_glEndList () from /usr/lib/libGL.so.1
#8  0x0804b3ef in init () at glxgears.c:304
#9  0x0804bef2 in main (argc=5, argv=0xbffff684) at glxgears.c:626
#10 0x4020f8ae in __libc_start_main () from /lib/libc.so.6
-- snip --
... e.g. it crashes in the |glEndList()| call in line 304:
-- snip --
299   /* make the gears */
300   gear1 = glGenLists(1);
301   glNewList(gear1, GL_COMPILE);
302   glMaterialfv(GL_FRONT, GL_AMBIENT_AND_DIFFUSE, red);
303   gear(1.0, 4.0, 1.0, 20, 0.7);
304   glEndList();
305
306   gear2 = glGenLists(1);
-- snip --

On the Xserver side the stack trace look like this:
-- snip --
(gdb) where
#0  0x0839371e in NORMALF (x=0, y=0, z=1) at t_imm_api.c:603
#1  0x083937b5 in _tnl_Normal3fv (v=0x8ce0cd0) at t_imm_api.c:624
#2  0x0822e85b in glNormal3fv (v=0x8ce0cd0) at
../../../../../extras/Mesa/src/glapitemp.h:348
#3  0x081f83b5 in __glXDisp_Normal3fv (pc=0x8ce0cd0 "") at g_render.c:410
#4  0x0821a689 in __glXRender (cl=0x8a1a910, pc=0x8ce0ccc "\020") at
glxcmds.c:1320
#5  0x081f6bcb in __glXDispatch (client=0x8b2f348) at glxext.c:434
#6  0x08074179 in Dispatch () at dispatch.c:454
#7  0x0805b1e9 in main (argc=6, argv=0xbffff6b4, envp=0xbffff6d0) at main.c:442
#8  0x400c48ae in __libc_start_main () from /lib/libc.so.6
-- snip --
I have turned NORMALF from a C macro into a C function - after that gdb gave me
the hint that |IM| is |NULL|:
-- snip --
Program received signal SIGSEGV, Segmentation fault.
0x0839371e in NORMALF (x=0, y=0, z=1) at t_imm_api.c:603
603        count = IM->Count;
(gdb) print IM
$1 = (struct immediate *) 0x0
-- snip --
Comment 13 Roland Mainz 2004-04-26 21:03:31 UTC
Created attachment 243 [details] [review]
Fix bugs in X visual setup, and enable GLX extension [checked-in]

I'll check this patch in for now since it seems to be a good idea to get the
visual setup fixed anyway.
Comment 14 Roland Mainz 2004-04-26 21:08:50 UTC
Comment on attachment 243 [details] [review]
Fix bugs in X visual setup, and enable GLX extension [checked-in]

Attachment #243 [details] checked-in

Checking in ChangeLog;
/cvs/xorg/xc/ChangeLog,v  <--  ChangeLog
new revision: 1.9; previous revision: 1.8
done
Checking in programs/Xserver/Xprint/Imakefile;
/cvs/xorg/xc/programs/Xserver/Xprint/Imakefile,v  <--  Imakefile
new revision: 1.3; previous revision: 1.2
done
Checking in programs/Xserver/Xprint/ps/Imakefile;
/cvs/xorg/xc/programs/Xserver/Xprint/ps/Imakefile,v  <--  Imakefile
new revision: 1.3; previous revision: 1.2
done
Checking in programs/Xserver/Xprint/ps/PsInit.c;
/cvs/xorg/xc/programs/Xserver/Xprint/ps/PsInit.c,v  <--  PsInit.c
new revision: 1.3; previous revision: 1.2
done
Checking in programs/Xserver/mi/miinitext.c;
/cvs/xorg/xc/programs/Xserver/mi/miinitext.c,v	<--  miinitext.c
new revision: 1.3; previous revision: 1.2
done
Mailing the commit message to xorg-commit@pdx.freedesktop.org...
Comment 15 Roland Mainz 2004-04-26 21:11:18 UTC
Alan:
Any idea what is going wrong in comment #12 ?
Comment 16 Alan Hourihane 2004-04-26 22:47:16 UTC
Mesa in the X.Org tree is still based on 5.0.2, and there was a fix in Mesa 
6.0 which creates the buffers properly.

I've copied that fix to the Mesa 5.0.2 file and glxgears (from attachment 229 [details] [review]) 
works here now.

So I've committed it.
Comment 17 Alan Hourihane 2004-04-26 22:52:02 UTC
One thing to note in this Roland, is that Mesa has hardcoded maximum 
dimensions for width and height. In Mesa 5.0.2 that's only 2048x2048, and in 
Mesa 6.0 it's 4096x4096.

This is a far cry from printer resolutions. 
Comment 18 Roland Mainz 2004-04-26 22:55:07 UTC
Alan Hourihane wrote:
> Mesa in the X.Org tree is still based on 5.0.2, and there was a fix in Mesa 
> 6.0 which creates the buffers properly.
> I've copied that fix to the Mesa 5.0.2 file and glxgears (from attachment 229 [details] [review]) 
> works here now.

Nice! Thanks!

> So I've committed it.

Erm... there is a problem with the commit:
-- snip --
Modified files:
xc/extras/Mesa/src/X/:    Tag: XORG-CURRENT
xm_dd.c
-- snip --
If I understand this right XORG-CURRENT content was moved to "trunk" and
XORG-CURRENT is now frozen, obsolete and checkins forbidden... ;-(
Comment 19 Roland Mainz 2004-04-26 22:56:34 UTC
Alan Hourihane wrote:
> One thing to note in this Roland, is that Mesa has hardcoded maximum 
> dimensions for width and height. In Mesa 5.0.2 that's only 2048x2048, and in 
> Mesa 6.0 it's 4096x4096.
> This is a far cry from printer resolutions.

Can that limit be bumped to a higher value somehow ?
Comment 20 Roland Mainz 2004-04-26 23:28:09 UTC
Alan:
BTW: For the PostScript DDX it may be worth to check whether
http://www.geuz.org/gl2ps/ can be integrated. The tricky part may be that this
would mean that different screens use different OpenGL engines... and I am not
sure whether this is possible...
Comment 21 Alan Hourihane 2004-04-26 23:30:05 UTC
In reply to comment #19

The values can be bumped, but Mesa creates buffers. For example if you bumped 
it to 8192x8192 you'll end up creating a couple of 256MB buffers at a depth 24 
resolution.

Also, at some point you'll run into rendering errors because of precision 
problems by increasing it too far. That's why we only bumped it to 4Kx4K in 
Mesa 6.0 as that's reasonable - any furthur and it could get sticky.
Comment 22 Alan Hourihane 2004-04-26 23:31:22 UTC
In reply to comment #18

I've committed the fix to the trunk now
Comment 23 Alan Hourihane 2004-04-26 23:33:06 UTC
In reply to comment #20

That program seems to have some severe limitations.
Comment 24 Roland Mainz 2004-04-27 02:09:14 UTC
Alan Hourihane wrote:
> In reply to comment #18
>
> I've committed the fix to the trunk now

Thank you very much! :)
Comment 25 Roland Mainz 2004-04-27 02:19:52 UTC
Alan Hourihane wrote:
> In reply to comment #19
>
> The values can be bumped, but Mesa creates buffers. For example if you bumped 
> it to 8192x8192 you'll end up creating a couple of 256MB buffers at a depth 24 
> resolution.

Well, mmap() of temporary files could be used for those buffers... =:-)
OKOK, _bad_ idea since the PS output will still be gigantic... I still hope
there is a way to hook-up a OpenGL--->PS engine which translates the OpenGL
rendering calls into PostScript code... the Mesa code would then only be used
for the other DDX (PCL, Raster, etc.).

> Also, at some point you'll run into rendering errors because of precision 
> problems by increasing it too far. That's why we only bumped it to 4Kx4K in 
> Mesa 6.0 as that's reasonable - any furthur and it could get sticky.

I have tried 8192x8192 pixels with the current Mesa version in "trunk" and that
seems to work perfectly... and is more than enougth for DIN-A4 at 600DPI.

There is still the issue what will happen if the size is "too large" - will an
error be generated and forwarded to the client side (e.g. something like
"RequestTooLarge"), will the GLX drawable size silenty be truncated or what will
happen ?

BTW: Is there a way to peek the maximum dimensions supported by the rasterizer
at the client side ?
Comment 26 Roland Mainz 2004-04-27 02:23:37 UTC
Alan Hourihane wrote:
> In reply to comment #20
> 
> That program seems to have some severe limitations.

1. Can you make a small summary what exactly is "missing" in "gl2ps", please ?
2. Do you know any "better" solution than using "gl2ps" ?
Comment 27 Alan Hourihane 2004-04-27 02:29:38 UTC
In reply to comment #25

Running 8192x8192 may not show anomalies with all applications, it really 
depends on what they're doing.

As for interrogating the GL's maximum viewport do :-

   GLint max[2];
   glGetIntegerv(GL_MAX_VIEWPORT_DIMS, max);
Comment 28 Alan Hourihane 2004-04-27 02:30:35 UTC
In reply to comment #26

I don't really have the time to go through that application,  but the lack of 
texture support is major.
Comment 29 Roland Mainz 2004-04-27 02:34:15 UTC
For the log:
I filed two spin-off bugs to move the work on glxgears and the build warnings
out of this bug:
1. Bug 558 - 'RFE: Add Xprint support to "glxgears"'
2. Bug 559 - 'RFE: Fix some build warnings in Xprint code'
Comment 30 Roland Mainz 2004-04-27 03:06:57 UTC
Alan Hourihane wrote:
> In reply to comment #26
> I don't really have the time to go through that application,  but the lack of 
> texture support is major.

OK... would it still be possible to add the gs2ps engine to Xprt ?
If "yes"... is it possible to have two (or more) different OpenGL engines (like
"Mesa" and "gl2ps") working on different screens at the same time in the same
Xserver instance ?
Comment 31 Alan Hourihane 2004-04-27 06:48:56 UTC
You can't have two OpenGL engines operating at the same time - no.
Comment 32 Roland Mainz 2004-04-27 08:39:47 UTC
I am not sure whether this is related:
Tody I noticd a weired behaviour of Xprt:
The Xprt process starts to consume nearly 100% CPU time doing nothing.
Trying to get stack trace samples I am getting this:
-- snip --
0x40163041 in gettimeofday () from /lib/libc.so.6
(gdb) where
#0  0x40163041 in gettimeofday () from /lib/libc.so.6
#1  0x0804c1c2 in GetTimeInMillis ()
#2  0x08079a6d in WaitForSomething ()
#3  0x0806cb49 in Dispatch ()
#4  0x0805a63b in main ()
#5  0x400de8ae in __libc_start_main () from /lib/libc.so.6
-- snip ..

Alanh:
Is it possible that this issue is related to te GLX changes or is this a
different problem ?
Comment 33 Roland Mainz 2004-04-27 08:43:07 UTC
Roland Mainz wrote:
> The Xprt process starts to consume nearly 100% CPU time doing nothing.

... and running "xplsprinters" on that server gets rid of the problem - the
server stops hogging the CPU. Weired.
Comment 34 Roland Mainz 2004-04-28 06:56:25 UTC
I filed bug 567 ("Xorg Xprt starts to consume 100% CPU when being idle for some
time") for the problem found in comment #32.
Comment 35 Roland Mainz 2004-04-30 07:22:28 UTC
Alan Hourihane wrote:
> You can't have two OpenGL engines operating at the same time - no.

Is it possible that Mesa can handle different (Mesa) drivers on different
screens ?
Comment 36 Alan Hourihane 2004-04-30 08:43:13 UTC
Yes, that's possible
Comment 37 Roland Mainz 2004-04-30 08:50:34 UTC
Alan Hourihane wrote:
> Yes, that's possible

Is it possible to modify gl2ps to work as Mesa driver ?
Comment 38 Alan Hourihane 2004-04-30 18:14:02 UTC
Anything's possible
Comment 39 Corbin Simpson 2011-09-13 12:57:40 UTC
Closing WONTFIX because nobody cares about Xprint. Reopen if you plan to address this bug.


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.