[PATCH] os: Don't write the reply that overflows output buffer
Keith Packard
keithp at keithp.com
Mon Aug 2 11:36:27 PDT 2010
On Mon, 2 Aug 2010 11:29:55 -0400, Kristian Høgsberg <krh at bitplanet.net> wrote:
>
> Here's the other approach for fixing damage vs flush client. In writing the
> commit message, it certainly does feel like damage should just be fixed to
> not reply on that behaviour. And writing the patch, I realize that the
> optimization is not just about saving a syscall, but also about avoiding
> having to realloc the output buffer to hold a request that's bigger than
> BUFSIZE (hello, XkbSendMap()). The current code only does that when it
> fails to write the output buffer + extra request and the remaining bytes
> don't fit in a standard sized buffer.
This doesn't really fix the issue as we're potentially sending a stream
of events from the damage extension. We'd have to buffer all of them
until FlushAllOutput, I think.
It is a nice cleanup though; let's look at this post-1.9 separately.
--
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg/attachments/20100802/9c0cbfea/attachment.pgp>
More information about the xorg
mailing list