[Mesa-dev] [PATCH 1/4] mapi_abi: Allow more dynamic entries

[Tapani Pälli]

On 02/22/2016 09:54 PM, Ian Romanick wrote:
> NAK.
>
> The purpose of the dynamic entries is to allow a newer version of the GL
> driver to work with a slightly older libGL.  There is no reason to
> support more than 256 new functions... update your libGL.  If there are
> functions that libGL doesn't support, just add them.

How about the proposal where we could calculate the amount based on the 
Khronos XML?

https://lists.freedesktop.org/archives/mesa-dev/2016-February/107922.html


> On 02/11/2016 04:03 AM, Tapani Pälli wrote:
>> From: Bernard Kilarski <bernard.r.kilarski at intel.com>
>>
>> Signed-off-by: Bernard Kilarski <bernard.r.kilarski at intel.com>
>> Signed-off-by: Matt Roper <matthew.d.roper at intel.com>
>> Cc: "11.0 11.1" <mesa-stable at lists.freedesktop.org
>> ---
>>   src/mapi/mapi_abi.py | 2 +-
>>   src/mapi/u_execmem.c | 2 +-
>>   2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/src/mapi/mapi_abi.py b/src/mapi/mapi_abi.py
>> index 012a5c3..9eafb94 100644
>> --- a/src/mapi/mapi_abi.py
>> +++ b/src/mapi/mapi_abi.py
>> @@ -38,7 +38,7 @@ import glX_XML
>>
>>
>>   # number of dynamic entries
>> -ABI_NUM_DYNAMIC_ENTRIES = 256
>> +ABI_NUM_DYNAMIC_ENTRIES = 512
>>
>>   class ABIEntry(object):
>>       """Represent an ABI entry."""
>> diff --git a/src/mapi/u_execmem.c b/src/mapi/u_execmem.c
>> index 89d5c1d..f8334f9 100644
>> --- a/src/mapi/u_execmem.c
>> +++ b/src/mapi/u_execmem.c
>> @@ -37,7 +37,7 @@
>>   #include "u_execmem.h"
>>
>>
>> -#define EXEC_MAP_SIZE (4*1024)
>> +#define EXEC_MAP_SIZE (8*1024)
>>
>>   static mtx_t exec_mutex = _MTX_INITIALIZER_NP;
>>
>>
>