[Mesa-dev] [PATCH 6/8] i965: Convert brw state dirty bits to 64-bits

Kenneth Graunke kenneth at whitecape.org
Wed Aug 27 17:41:37 PDT 2014


On Wednesday, August 27, 2014 05:10:36 PM Jordan Justen wrote:
> r-b with these changes?
> 
> By the way, chrisf pointed out on irc that we should extend
> ctx->NewDriverState to 64-bit as well. I'll look into that, but I
> don't think it needs to block this change. Although, I'd be willing to
> pull that in here as well if anyone prefers it.
> 
> -Jordan

If you're going to add BRW_NEW_* bits beyond (1 << 31), then you need to expand ctx->NewDriverState and all of the values in struct gl_driver_flags to be GLbitfield64/uint64_t as well.  We may want to widen the bitfields in the Gallium state tracker to match.

Failing to do so creates an awkward situation where the DriverFlags mechanism can only use half of the available dirty bits, so we'd have to carefully order bits to avoid truncation problems.  I don't like that at all.

Moving to a 64-bit value makes sense.  But, we need to do both at the same time.  Ideally, I would prefer to see that done first, outside of the rest of your series.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20140827/fb3970af/attachment.sig>


More information about the mesa-dev mailing list