[Mesa-dev] [PATCH 12/15] intel: Downsample on DRI2 flush

Chad Versace chad.versace at linux.intel.com
Mon Jul 23 08:00:46 PDT 2012


On 07/23/2012 06:30 AM, Brian Paul wrote:
> On Sat, Jul 21, 2012 at 6:36 PM, Chad Versace
> <chad.versace at linux.intel.com> wrote:
>> CC: Eric Anholt <eric at anholt.net>
>> CC: Paul Berry <stereotype441 at gmail.com>
>> Signed-off-by: Chad Versace <chad.versace at linux.intel.com>
>> ---
>>  src/mesa/drivers/dri/intel/intel_screen.c | 31 +++++++++++++++++++++++++++++++
>>  1 file changed, 31 insertions(+)
>>
>> diff --git a/src/mesa/drivers/dri/intel/intel_screen.c b/src/mesa/drivers/dri/intel/intel_screen.c
>> index 4a12b67..958ff9f 100644
>> --- a/src/mesa/drivers/dri/intel/intel_screen.c
>> +++ b/src/mesa/drivers/dri/intel/intel_screen.c
>> @@ -177,6 +177,36 @@ static const __DRItexBufferExtension intelTexBufferExtension = {
>>  };
>>
>>  static void
>> +intel_downsample_for_dri2_flush(struct intel_context *intel,
>> +                                __DRIdrawable *drawable)
>> +{
>> +   if (intel->gen < 6) {
>> +      /* MSAA is not supported, so don't waste time checking for
>> +       * a multisample buffer.
>> +       */
>> +      return;
>> +   }
>> +
>> +   struct gl_framebuffer *fb = drawable->driverPrivate;
>> +   struct intel_renderbuffer *rb;
> 
> Declarations following code will cause a compiler error (depending on
> the compiler and flags).  You might want to fix that, just in case.

It's ok here. The Intel driver is compiled with -std=c99 and uses
code-after-declarations in other places too.


More information about the mesa-dev mailing list