[Mesa-dev] [PATCH 5/7] i965: use _mesa_geometry_width/height/layers/samples for programming geometry of framebuffer to GEN
Rogovin, Kevin
kevin.rogovin at intel.com
Fri Apr 24 09:42:45 PDT 2015
> My point specifically was that you are also updating atoms that _are not_ re-used.
> And as those changes are not really needed, I wouldn't take the risk of changing
> something in vain. I would introduce them only when you have patches to really enable older generations.
My take is the following:
1. Tracking (and guaranteeing) that those function left unchanged as is are
exactly just those for before Gen7 is a pain. Much easier, and more reliable
to hit them all instead. A significant number of functions in i965 are not emit
functions of any atom but emit functions of atoms map to them. Again, more
reliable and -safer- to change them all, then just the bare minimum.
2. The change is benign. If _HasAttachments is true, then the function substitution
gives the same value. For Gens not supporting the extension there is no effect.
3. Lastly, as stated: for later it leaves the option to enable it for Gen6 and below,
it is just trivial change, but it needs testing on hardware.
When I writing this work, I originally had it for all Gens, but changed to support
only Gen7and higher because that is all on which I can test it.
-Kevin
More information about the mesa-dev
mailing list