Re: [Mesa-dev] [PATCH 00/13] Threaded Gallium for RadeonSI

2017-05-14 Thread Turo Lamminen
On Thu, 11 May 2017 21:02:21 +0200 Nicolai Hähnle wrote: > Violating the "async" promise on debug callbacks is a problem. This > breaks the OpenGL API in a place where it wasn't broken before, and > that's not okay. As an application developer who has occasionally relied on this behavior to debu

Re: [Mesa-dev] [PATCH 00/13] Threaded Gallium for RadeonSI

2017-05-14 Thread Rob Clark
On Wed, May 10, 2017 at 6:45 PM, Marek Olšák wrote: > Hi, > > This series adds an optional module into gallium/util that wraps > around pipe_context and moves execution of all pipe_context calls into > a separate thread. > > It puts a lot of new requirements on the driver, especially on thread- >

Re: [Mesa-dev] [PATCH 00/13] Threaded Gallium for RadeonSI

2017-05-12 Thread Nicolai Hähnle
On 11.05.2017 23:01, Marek Olšák wrote: On Thu, May 11, 2017 at 9:02 PM, Nicolai Hähnle > Some general remarks: Violating the "async" promise on debug callbacks is a problem. This breaks the OpenGL API in a place where it wasn't broken before, and that's not okay. I'm not sure what to do about

Re: [Mesa-dev] [PATCH 00/13] Threaded Gallium for RadeonSI

2017-05-11 Thread Marek Olšák
On Thu, May 11, 2017 at 9:02 PM, Nicolai Hähnle > Some general remarks: > > Violating the "async" promise on debug callbacks is a problem. This breaks > the OpenGL API in a place where it wasn't broken before, and that's not > okay. I'm not sure what to do about this precisely, but the spec is ver

Re: [Mesa-dev] [PATCH 00/13] Threaded Gallium for RadeonSI

2017-05-11 Thread Nicolai Hähnle
On 11.05.2017 00:45, Marek Olšák wrote: Hi, This series adds an optional module into gallium/util that wraps around pipe_context and moves execution of all pipe_context calls into a separate thread. It puts a lot of new requirements on the driver, especially on thread- safetiness of pipe_contex

Re: [Mesa-dev] [PATCH 00/13] Threaded Gallium for RadeonSI

2017-05-11 Thread Marek Olšák
On May 11, 2017 2:30 AM, "Timothy Arceri" wrote: On 11/05/17 08:45, Marek Olšák wrote: > Hi, > > This series adds an optional module into gallium/util that wraps > around pipe_context and moves execution of all pipe_context calls into > a separate thread. > > It puts a lot of new requirements on

Re: [Mesa-dev] [PATCH 00/13] Threaded Gallium for RadeonSI

2017-05-10 Thread Timothy Arceri
On 11/05/17 08:45, Marek Olšák wrote: Hi, This series adds an optional module into gallium/util that wraps around pipe_context and moves execution of all pipe_context calls into a separate thread. It puts a lot of new requirements on the driver, especially on thread- safetiness of pipe_context

[Mesa-dev] [PATCH 00/13] Threaded Gallium for RadeonSI

2017-05-10 Thread Marek Olšák
Hi, This series adds an optional module into gallium/util that wraps around pipe_context and moves execution of all pipe_context calls into a separate thread. It puts a lot of new requirements on the driver, especially on thread- safetiness of pipe_context functions, and even expects different be