Re: [Mesa-dev] pfn_notify in clCreateContext()

2017-03-05 Thread Giuseppe Bilotta
On Fri, Feb 24, 2017 at 5:30 PM, Ilia Mirkin wrote: > I'd like to point out that when I hooked up this logic in clover > (after my triumphant addition of KHR_debug in gallium), I had no idea > how OpenCL stuff worked. I still don't. If you think passing this type > of information is inappropriate,

Re: [Mesa-dev] pfn_notify in clCreateContext()

2017-02-24 Thread Ilia Mirkin
I'd like to point out that when I hooked up this logic in clover (after my triumphant addition of KHR_debug in gallium), I had no idea how OpenCL stuff worked. I still don't. If you think passing this type of information is inappropriate, or that some of the message types should be filtered out, yo

[Mesa-dev] pfn_notify in clCreateContext()

2017-02-24 Thread Vedran Miletić
Hello, I noticed that Clover outputs si_shader_dump_stats() via pfn_notify callback registered clCreateContext() (it's not done directly, but that's the result). According to OpenCL standard documentation [1], pfn_notify should be used only to report errors. One of the programs that gets confused