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

2012-03-11 Thread Mike Lothian
On 15 February 2012 13:28, Marek Olšák mar...@gmail.com 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 --foreign ...
 [ 

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 mar...@gmail.com 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 jfons...@vmware.com 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 mar...@gmail.com 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-09 Thread Eric Anholt
On Wed, 7 Mar 2012 23:26:18 +0100, Marek Olšák mar...@gmail.com wrote:
 On Wed, Mar 7, 2012 at 6:01 PM, Brian Paul brian.e.p...@gmail.com wrote:
  On Wed, Feb 15, 2012 at 6:28 AM, Marek Olšák mar...@gmail.com 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-07 Thread Brian Paul
On Wed, Feb 15, 2012 at 6:28 AM, Marek Olšák mar...@gmail.com 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 brian.e.p...@gmail.com wrote:
 On Wed, Feb 15, 2012 at 6:28 AM, Marek Olšák mar...@gmail.com 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-02-15 Thread Matt Turner
On Wed, Feb 15, 2012 at 8:28 AM, Marek Olšák mar...@gmail.com 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