[RFC] swap event handling fixes

Jesse Barnes jbarnes at virtuousgeek.org
Fri Apr 29 14:37:14 PDT 2011


On Thu, 28 Apr 2011 13:27:18 -0700
Jesse Barnes <jbarnes at virtuousgeek.org> wrote:

> I obviously failed to count the swap event structure size after adding
> and removing fields a few times, and didn't even account for padding.  The
> end result is that clients today won't receive the sbc_lo field at all,
> and so will likely stuff junk into that field on the client side (or
> zero at best).
> 
> This patchset fixes up the structure definitions, bumps the proto
> levels, and adds server and client handling code for it all.  It should
> be forward and backward compatible, but as always review and testing
> appreciated.
> 
> I think the event_type checking on the client side still needs work;
> the field is split now so I need to check the right one on old servers.
> I'll also add swap support for the new requests in case people ever want
> to run this stuff between big and little endian machines.

Btw, I've been trying to test these fixes but I can't make the existing
code break; I'm getting a nicely incrementing sbc count using the
piglit event test (after adding some code to dump the swap event
fields), so somehow the bits are getting through.  I just don't know
how yet...

-- 
Jesse Barnes, Intel Open Source Technology Center


More information about the dri-devel mailing list