[Mesa-dev] [PATCH 00/13] ARB_debug_output

2012-02-15 Thread Marek Olšák
Hi everyone,

this series adds the ARB_debug_output extension. It implements the minimum 
feature set required by the spec, which is GL API error logging.

I've added a new piglit test for this: arb_debug_output-api_error. I'd like to 
push this series in a week if there are no concerns.

The whole series is also available here:
  git://people.freedesktop.org/~mareko/mesa arb-debug-output

Marek Olšák (3):
  mesa: print GL errors via debug_output
  mesa: display list dispatch for ARB_debug_output
  mesa: expose ARB_debug_output

nobled (10):
  mesa: split error handling into its own file
  glapi: add ARB_debug_output.xml
  mesa: add infrastructure for GL_ARB_debug_output
  mesa: add some GL_ARB_debug_output functions
  mesa: add message-toggle booleans for GL_ARB_debug_output
  mesa: add glDebugMessageControlARB
  mesa: add yet more context fields for GL_ARB_debug_output
  mesa: add control for categories of application-provided messages
  mesa: add struct for managing client debug namespaces
  mesa: implement the last of GL_ARB_debug_output

Marek
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [PATCH 00/13] ARB_debug_output

2012-02-15 Thread Matt Turner
On Wed, Feb 15, 2012 at 8:28 AM, Marek Olšák  wrote:
> nobled (10):

Linus doesn't even accept patches from people who don't set their name
in ~/.gitconfig. Can you please set your name?
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [PATCH 00/13] ARB_debug_output

2012-03-07 Thread Brian Paul
On Wed, Feb 15, 2012 at 6:28 AM, Marek Olšák  wrote:
> Hi everyone,
>
> this series adds the ARB_debug_output extension. It implements the minimum 
> feature set required by the spec, which is GL API error logging.
>
> I've added a new piglit test for this: arb_debug_output-api_error. I'd like 
> to push this series in a week if there are no concerns.
>
> The whole series is also available here:
>  git://people.freedesktop.org/~mareko/mesa arb-debug-output
>
> Marek Olšák (3):
>  mesa: print GL errors via debug_output
>  mesa: display list dispatch for ARB_debug_output
>  mesa: expose ARB_debug_output
>
> nobled (10):
>  mesa: split error handling into its own file
>  glapi: add ARB_debug_output.xml
>  mesa: add infrastructure for GL_ARB_debug_output
>  mesa: add some GL_ARB_debug_output functions
>  mesa: add message-toggle booleans for GL_ARB_debug_output
>  mesa: add glDebugMessageControlARB
>  mesa: add yet more context fields for GL_ARB_debug_output
>  mesa: add control for categories of application-provided messages
>  mesa: add struct for managing client debug namespaces
>  mesa: implement the last of GL_ARB_debug_output

Hi Marek,

I did a quick read of these patches and they look OK to me.  Were you
planning on pushing them soon?

-Brian
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [PATCH 00/13] ARB_debug_output

2012-03-07 Thread Marek Olšák
On Wed, Mar 7, 2012 at 6:01 PM, Brian Paul  wrote:
> On Wed, Feb 15, 2012 at 6:28 AM, Marek Olšák  wrote:
>> Hi everyone,
>>
>> this series adds the ARB_debug_output extension. It implements the minimum 
>> feature set required by the spec, which is GL API error logging.
>>
>> I've added a new piglit test for this: arb_debug_output-api_error. I'd like 
>> to push this series in a week if there are no concerns.
>>
>> The whole series is also available here:
>>  git://people.freedesktop.org/~mareko/mesa arb-debug-output
>>
>> Marek Olšák (3):
>>  mesa: print GL errors via debug_output
>>  mesa: display list dispatch for ARB_debug_output
>>  mesa: expose ARB_debug_output
>>
>> nobled (10):
>>  mesa: split error handling into its own file
>>  glapi: add ARB_debug_output.xml
>>  mesa: add infrastructure for GL_ARB_debug_output
>>  mesa: add some GL_ARB_debug_output functions
>>  mesa: add message-toggle booleans for GL_ARB_debug_output
>>  mesa: add glDebugMessageControlARB
>>  mesa: add yet more context fields for GL_ARB_debug_output
>>  mesa: add control for categories of application-provided messages
>>  mesa: add struct for managing client debug namespaces
>>  mesa: implement the last of GL_ARB_debug_output
>
> Hi Marek,
>
> I did a quick read of these patches and they look OK to me.  Were you
> planning on pushing them soon?

Ian told me that he wanted to take a look at the patches and that I
should wait, therefore I didn't push them as I had planned. Ian?

Marek
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [PATCH 00/13] ARB_debug_output

2012-03-09 Thread Eric Anholt
On Wed, 7 Mar 2012 23:26:18 +0100, Marek Olšák  wrote:
> On Wed, Mar 7, 2012 at 6:01 PM, Brian Paul  wrote:
> > On Wed, Feb 15, 2012 at 6:28 AM, Marek Olšák  wrote:
> >> Hi everyone,
> >>
> >> this series adds the ARB_debug_output extension. It implements the minimum 
> >> feature set required by the spec, which is GL API error logging.
> >>
> >> I've added a new piglit test for this: arb_debug_output-api_error. I'd 
> >> like to push this series in a week if there are no concerns.
> >>
> >> The whole series is also available here:
> >>  git://people.freedesktop.org/~mareko/mesa arb-debug-output
> >>
> >> Marek Olšák (3):
> >>  mesa: print GL errors via debug_output
> >>  mesa: display list dispatch for ARB_debug_output
> >>  mesa: expose ARB_debug_output
> >>
> >> nobled (10):
> >>  mesa: split error handling into its own file
> >>  glapi: add ARB_debug_output.xml
> >>  mesa: add infrastructure for GL_ARB_debug_output
> >>  mesa: add some GL_ARB_debug_output functions
> >>  mesa: add message-toggle booleans for GL_ARB_debug_output
> >>  mesa: add glDebugMessageControlARB
> >>  mesa: add yet more context fields for GL_ARB_debug_output
> >>  mesa: add control for categories of application-provided messages
> >>  mesa: add struct for managing client debug namespaces
> >>  mesa: implement the last of GL_ARB_debug_output
> >
> > Hi Marek,
> >
> > I did a quick read of these patches and they look OK to me.  Were you
> > planning on pushing them soon?
> 
> Ian told me that he wanted to take a look at the patches and that I
> should wait, therefore I didn't push them as I had planned. Ian?

I took a look at what was tested relative to the patch series.  Testing
I see missing:

- display lists
- MessageInsert
- DEBUG_OUTPUT_SYNCHRONOUS_ARB
- glGetPointerv() enums
- API errors generated by glDebugMessageControlARB and friends.
- minmax values.

However, it still looks like enough testing that I'm happy to see the
series go in.  I don't think Ian's review should stall it at this point.


pgpDZNrDtwZMg.pgp
Description: PGP signature
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [PATCH 00/13] ARB_debug_output

2012-03-11 Thread Mike Lothian
On 15 February 2012 13:28, Marek Olšák  wrote:
> Hi everyone,
>
> this series adds the ARB_debug_output extension. It implements the minimum 
> feature set required by the spec, which is GL API error logging.
>
> I've added a new piglit test for this: arb_debug_output-api_error. I'd like 
> to push this series in a week if there are no concerns.
>
> The whole series is also available here:
>  git://people.freedesktop.org/~mareko/mesa arb-debug-output
>
> Marek Olšák (3):
>  mesa: print GL errors via debug_output
>  mesa: display list dispatch for ARB_debug_output
>  mesa: expose ARB_debug_output
>
> nobled (10):
>  mesa: split error handling into its own file
>  glapi: add ARB_debug_output.xml
>  mesa: add infrastructure for GL_ARB_debug_output
>  mesa: add some GL_ARB_debug_output functions
>  mesa: add message-toggle booleans for GL_ARB_debug_output
>  mesa: add glDebugMessageControlARB
>  mesa: add yet more context fields for GL_ARB_debug_output
>  mesa: add control for categories of application-provided messages
>  mesa: add struct for managing client debug namespaces
>  mesa: implement the last of GL_ARB_debug_output
>
> Marek
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Hi Marek

Ever since these patches went in by 32bit build broke with the
following error (the 64bit one is fine):

gmake[2]: Entering directory
`/var/tmp/portage/media-libs/mesa-32bit-/work/Mesa-32bit-/src/mapi/glapi'
x86_64-pc-linux-gnu-gcc -c -I../../../include -I../../../src/mapi
-I../../../src/mesa -DMAPI_MODE_BRIDGE
-DMAPI_ABI_HEADER=\"glapi/glapi_mapi_tmp.h\" -O2 -march=native -pipe
-m32 -Wall -std=c99 -Werror=implicit-function-declaration
-Werror=missing-prototypes -fno-strict-aliasing -fno-builtin-memcmp
-O2 -march=native -pipe -m32  -fPIC   -D_GNU_SOURCE -DPTHREADS
-DTEXTURE_FLOAT_ENABLED -DHAVE_POSIX_MEMALIGN -DIN_DRI_DRIVER
-DUSE_XCB -DGLX_INDIRECT_RENDERING -DGLX_DIRECT_RENDERING
-DGLX_USE_TLS -DPTHREADS -DUSE_EXTERNAL_DXTN_LIB=1 -DIN_DRI_DRIVER
-DHAVE_ALIAS -DHAVE_MINCORE -DHAVE_LIBUDEV
-DXCB_DRI2_CONNECT_DEVICE_NAME_BROKEN -D__STDC_CONSTANT_MACROS
-DHAVE_LLVM=0x0300 -fvisibility=hidden ../../../src/mapi/mapi/entry.c
-o entry.o
In file included from ../../../src/mapi/mapi/mapi_tmp.h:48:0,
 from ../../../src/mapi/mapi/entry.c:69:
../../../src/mapi/glapi/glapi_mapi_tmp.h:883:1: error: conflicting
types for 'glDebugMessageCallbackARB'
../../../include/GL/glext.h:7576:21: note: previous declaration of
'glDebugMessageCallbackARB' was here
In file included from ../../../src/mapi/mapi/mapi_tmp.h:48:0,
 from ../../../src/mapi/mapi/entry.c:69:
../../../src/mapi/glapi/glapi_mapi_tmp.h:7330:1: error: conflicting
types for 'glDebugMessageCallbackARB'
../../../include/GL/glext.h:7576:21: note: previous declaration of
'glDebugMessageCallbackARB' was here
gmake[2]: *** [entry.o] Error 1
gmake[2]: Leaving directory
`/var/tmp/portage/media-libs/mesa-32bit-/work/Mesa-32bit-/src/mapi/glapi'
make[1]: *** [subdirs] Error 1
make[1]: Leaving directory
`/var/tmp/portage/media-libs/mesa-32bit-/work/Mesa-32bit-/src'
make: *** [default] Error

Can you let me know if I'm doing something wrong

Cheers

Mike
 * Package:media-libs/mesa-32bit-
 * Repository: FireBurn
 * Maintainer: x...@gentoo.org
 * USE:amd64 classic consolekit egl elibc_glibc g3dvl gallium gbm gles2 kernel_linux llvm multilib nptl openvg pic policykit shared-glapi userland_GNU video_cards_i965 video_cards_r600 xvmc
 * FEATURES:   sandbox
>>> Unpacking source...
GIT update -->
   repository:   git://anongit.freedesktop.org/mesa/mesa
   at the commit:bb1d126b2369bc7608c43738693b6f26cd8f4eeb
   branch:   master
   storage directory:"/usr/portage/distfiles/egit-src/mesa"
   checkout type:bare repository
Cloning into '/var/tmp/portage/media-libs/mesa-32bit-/work/Mesa-32bit-'...
done.
Branch branch-master set up to track remote branch master from origin.
Switched to a new branch 'branch-master'
>>> Unpacked to /var/tmp/portage/media-libs/mesa-32bit-/work/Mesa-32bit-
>>> Source unpacked in /var/tmp/portage/media-libs/mesa-32bit-/work
>>> Preparing source in /var/tmp/portage/media-libs/mesa-32bit-/work/Mesa-32bit- ...
 * Running eautoreconf in '/var/tmp/portage/media-libs/mesa-32bit-/work/Mesa-32bit-' ...
 * Running aclocal ...
 [ ok ]
 * Running libtoolize --copy --force --install ...
 [ ok ]
 * Running aclocal ...
 [ ok ]
 * Running autoconf ...
 [ ok ]
 * Running automake --add-missing --copy --for

Re: [Mesa-dev] [PATCH 00/13] ARB_debug_output

2012-03-11 Thread Jose Fonseca
I saw the same w/ MinGW. I think my latest patch series should fix it.

Jose

- Original Message -
> On 15 February 2012 13:28, Marek Olšák  wrote:
> > Hi everyone,
> >
> > this series adds the ARB_debug_output extension. It implements the
> > minimum feature set required by the spec, which is GL API error
> > logging.
> >
> > I've added a new piglit test for this: arb_debug_output-api_error.
> > I'd like to push this series in a week if there are no concerns.
> >
> > The whole series is also available here:
> >  git://people.freedesktop.org/~mareko/mesa arb-debug-output
> >
> > Marek Olšák (3):
> >  mesa: print GL errors via debug_output
> >  mesa: display list dispatch for ARB_debug_output
> >  mesa: expose ARB_debug_output
> >
> > nobled (10):
> >  mesa: split error handling into its own file
> >  glapi: add ARB_debug_output.xml
> >  mesa: add infrastructure for GL_ARB_debug_output
> >  mesa: add some GL_ARB_debug_output functions
> >  mesa: add message-toggle booleans for GL_ARB_debug_output
> >  mesa: add glDebugMessageControlARB
> >  mesa: add yet more context fields for GL_ARB_debug_output
> >  mesa: add control for categories of application-provided messages
> >  mesa: add struct for managing client debug namespaces
> >  mesa: implement the last of GL_ARB_debug_output
> >
> > Marek
> > ___
> > mesa-dev mailing list
> > mesa-dev@lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/mesa-dev
> 
> Hi Marek
> 
> Ever since these patches went in by 32bit build broke with the
> following error (the 64bit one is fine):
> 
> gmake[2]: Entering directory
> `/var/tmp/portage/media-libs/mesa-32bit-/work/Mesa-32bit-/src/mapi/glapi'
> x86_64-pc-linux-gnu-gcc -c -I../../../include -I../../../src/mapi
> -I../../../src/mesa -DMAPI_MODE_BRIDGE
> -DMAPI_ABI_HEADER=\"glapi/glapi_mapi_tmp.h\" -O2 -march=native -pipe
> -m32 -Wall -std=c99 -Werror=implicit-function-declaration
> -Werror=missing-prototypes -fno-strict-aliasing -fno-builtin-memcmp
> -O2 -march=native -pipe -m32  -fPIC   -D_GNU_SOURCE -DPTHREADS
> -DTEXTURE_FLOAT_ENABLED -DHAVE_POSIX_MEMALIGN -DIN_DRI_DRIVER
> -DUSE_XCB -DGLX_INDIRECT_RENDERING -DGLX_DIRECT_RENDERING
> -DGLX_USE_TLS -DPTHREADS -DUSE_EXTERNAL_DXTN_LIB=1 -DIN_DRI_DRIVER
> -DHAVE_ALIAS -DHAVE_MINCORE -DHAVE_LIBUDEV
> -DXCB_DRI2_CONNECT_DEVICE_NAME_BROKEN -D__STDC_CONSTANT_MACROS
> -DHAVE_LLVM=0x0300 -fvisibility=hidden ../../../src/mapi/mapi/entry.c
> -o entry.o
> In file included from ../../../src/mapi/mapi/mapi_tmp.h:48:0,
>  from ../../../src/mapi/mapi/entry.c:69:
> ../../../src/mapi/glapi/glapi_mapi_tmp.h:883:1: error: conflicting
> types for 'glDebugMessageCallbackARB'
> ../../../include/GL/glext.h:7576:21: note: previous declaration of
> 'glDebugMessageCallbackARB' was here
> In file included from ../../../src/mapi/mapi/mapi_tmp.h:48:0,
>  from ../../../src/mapi/mapi/entry.c:69:
> ../../../src/mapi/glapi/glapi_mapi_tmp.h:7330:1: error: conflicting
> types for 'glDebugMessageCallbackARB'
> ../../../include/GL/glext.h:7576:21: note: previous declaration of
> 'glDebugMessageCallbackARB' was here
> gmake[2]: *** [entry.o] Error 1
> gmake[2]: Leaving directory
> `/var/tmp/portage/media-libs/mesa-32bit-/work/Mesa-32bit-/src/mapi/glapi'
> make[1]: *** [subdirs] Error 1
> make[1]: Leaving directory
> `/var/tmp/portage/media-libs/mesa-32bit-/work/Mesa-32bit-/src'
> make: *** [default] Error
> 
> Can you let me know if I'm doing something wrong
> 
> Cheers
> 
> Mike
> 
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
> 
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [PATCH 00/13] ARB_debug_output

2012-03-11 Thread Mike Lothian
Fantastic - here's hoping they get applied soon

On 11 March 2012 11:19, Jose Fonseca  wrote:
> I saw the same w/ MinGW. I think my latest patch series should fix it.
>
> Jose
>
> - Original Message -
>> On 15 February 2012 13:28, Marek Olšák  wrote:
>> > Hi everyone,
>> >
>> > this series adds the ARB_debug_output extension. It implements the
>> > minimum feature set required by the spec, which is GL API error
>> > logging.
>> >
>> > I've added a new piglit test for this: arb_debug_output-api_error.
>> > I'd like to push this series in a week if there are no concerns.
>> >
>> > The whole series is also available here:
>> >  git://people.freedesktop.org/~mareko/mesa arb-debug-output
>> >
>> > Marek Olšák (3):
>> >  mesa: print GL errors via debug_output
>> >  mesa: display list dispatch for ARB_debug_output
>> >  mesa: expose ARB_debug_output
>> >
>> > nobled (10):
>> >  mesa: split error handling into its own file
>> >  glapi: add ARB_debug_output.xml
>> >  mesa: add infrastructure for GL_ARB_debug_output
>> >  mesa: add some GL_ARB_debug_output functions
>> >  mesa: add message-toggle booleans for GL_ARB_debug_output
>> >  mesa: add glDebugMessageControlARB
>> >  mesa: add yet more context fields for GL_ARB_debug_output
>> >  mesa: add control for categories of application-provided messages
>> >  mesa: add struct for managing client debug namespaces
>> >  mesa: implement the last of GL_ARB_debug_output
>> >
>> > Marek
>> > ___
>> > mesa-dev mailing list
>> > mesa-dev@lists.freedesktop.org
>> > http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>>
>> Hi Marek
>>
>> Ever since these patches went in by 32bit build broke with the
>> following error (the 64bit one is fine):
>>
>> gmake[2]: Entering directory
>> `/var/tmp/portage/media-libs/mesa-32bit-/work/Mesa-32bit-/src/mapi/glapi'
>> x86_64-pc-linux-gnu-gcc -c -I../../../include -I../../../src/mapi
>> -I../../../src/mesa -DMAPI_MODE_BRIDGE
>> -DMAPI_ABI_HEADER=\"glapi/glapi_mapi_tmp.h\" -O2 -march=native -pipe
>> -m32 -Wall -std=c99 -Werror=implicit-function-declaration
>> -Werror=missing-prototypes -fno-strict-aliasing -fno-builtin-memcmp
>> -O2 -march=native -pipe -m32  -fPIC   -D_GNU_SOURCE -DPTHREADS
>> -DTEXTURE_FLOAT_ENABLED -DHAVE_POSIX_MEMALIGN -DIN_DRI_DRIVER
>> -DUSE_XCB -DGLX_INDIRECT_RENDERING -DGLX_DIRECT_RENDERING
>> -DGLX_USE_TLS -DPTHREADS -DUSE_EXTERNAL_DXTN_LIB=1 -DIN_DRI_DRIVER
>> -DHAVE_ALIAS -DHAVE_MINCORE -DHAVE_LIBUDEV
>> -DXCB_DRI2_CONNECT_DEVICE_NAME_BROKEN -D__STDC_CONSTANT_MACROS
>> -DHAVE_LLVM=0x0300 -fvisibility=hidden ../../../src/mapi/mapi/entry.c
>> -o entry.o
>> In file included from ../../../src/mapi/mapi/mapi_tmp.h:48:0,
>>                  from ../../../src/mapi/mapi/entry.c:69:
>> ../../../src/mapi/glapi/glapi_mapi_tmp.h:883:1: error: conflicting
>> types for 'glDebugMessageCallbackARB'
>> ../../../include/GL/glext.h:7576:21: note: previous declaration of
>> 'glDebugMessageCallbackARB' was here
>> In file included from ../../../src/mapi/mapi/mapi_tmp.h:48:0,
>>                  from ../../../src/mapi/mapi/entry.c:69:
>> ../../../src/mapi/glapi/glapi_mapi_tmp.h:7330:1: error: conflicting
>> types for 'glDebugMessageCallbackARB'
>> ../../../include/GL/glext.h:7576:21: note: previous declaration of
>> 'glDebugMessageCallbackARB' was here
>> gmake[2]: *** [entry.o] Error 1
>> gmake[2]: Leaving directory
>> `/var/tmp/portage/media-libs/mesa-32bit-/work/Mesa-32bit-/src/mapi/glapi'
>> make[1]: *** [subdirs] Error 1
>> make[1]: Leaving directory
>> `/var/tmp/portage/media-libs/mesa-32bit-/work/Mesa-32bit-/src'
>> make: *** [default] Error
>>
>> Can you let me know if I'm doing something wrong
>>
>> Cheers
>>
>> Mike
>>
>> ___
>> mesa-dev mailing list
>> mesa-dev@lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>>
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev