Roland Mainz wrote:
[snip]
> > > The PsPutImage() function is an internal server function in the Xprint
> > > module.  I have no idea why that would be getting called.
> >
> > I've just had a look at the code and did some modifications which
> > improve the performance a lot - its now almost twice as fast.
> 
> I am looking into the patch right now... the performance increment isn't
> that surprising as you have elimited some
> |sprintf()|/|strlen()|/|strcat()| or replaced it with less heavy code.
> And you removed the |ferror()| call which likely saves a couple of
> syscalls, too.
> 
> > Can
> > anyone pls look at the patch, make sure its correct and then apply it?
> 
> Yes, I'll do that. I filed
> https://bugs.freedesktop.org/show_bug.cgi?id=3001
> ("|glXSwapBuffers()|/|XPutImage()| are slow") to track the further
> progress...
> ... thank you very much for the patch! :)

Just for the log: I've checked the patch in a few secs ago.

----

Bye,
Roland

-- 
  __ .  . __
 (o.\ \/ /.o) [EMAIL PROTECTED]
  \__\/\/__/  MPEG specialist, C&&JAVA&&Sun&&Unix programmer
  /O /==\ O\  TEL +49 641 7950090
 (;O/ \/ \O;)
--- Begin Message ---
CVSROOT:        /cvs/xorg
Module name:    xc
Changes by:     [EMAIL PROTECTED]       05/04/12 17:05:37

Log message:
  2005-04-12 Roland Mainz <[EMAIL PROTECTED]>
    * xc/programs/Xserver/Xprint/ps/psout.c
    bugzilla #3001 (https://bugs.freedesktop.org/show_bug.cgi?id=3001)
    attachment #2404 (https://bugs.freedesktop.org/attachment.cgi?id=2404)
    Improve rendering performance when glXSwapBuffers()| or |XPutImage()|
    are called for the PostScript DDX via optimizing the codepath around
    |PsOut_OutImageBytes()|.
    Patch by Simon Toedt <[EMAIL PROTECTED]> and Roland Mainz
    <[EMAIL PROTECTED]>.

Modified files:
      ./:
        ChangeLog 
      xc/programs/Xserver/Xprint/ps/:
        psout.c 
  
  Revision      Changes    Path
  1.871         +11 -1     xc/ChangeLog
  1.6           +55 -18    xc/programs/Xserver/Xprint/ps/psout.c

_______________________________________________
xorg-commit mailing list
[EMAIL PROTECTED]
http://lists.freedesktop.org/mailman/listinfo/xorg-commit

--- End Message ---

Reply via email to