Re: [Mesa-dev] Using the right context in st_texture_release_all_sampler_views()

2015-07-25 Thread Marek Olšák
FYI, Unreal Engine demos reproduce this issue too: https://wiki.unrealengine.com/Linux_Demos Marek On Thu, Jul 23, 2015 at 6:59 PM, Stéphane Marchesin stephane.marche...@gmail.com wrote: On Thu, Jul 23, 2015 at 9:55 AM, Alex Deucher alexdeuc...@gmail.com wrote: On Wed, Jul 22, 2015 at 6:32

Re: [Mesa-dev] Using the right context in st_texture_release_all_sampler_views()

2015-07-23 Thread Marek Olšák
Hi Brian, Like the commit message says, I have no idea how Firefox does it. My commit only changed the situation from crash on all drivers to undefined behavior on drivers where sampler views are not sharable by contexts. Marek On Thu, Jul 23, 2015 at 12:32 AM, Brian Paul bri...@vmware.com

Re: [Mesa-dev] Using the right context in st_texture_release_all_sampler_views()

2015-07-23 Thread Alex Deucher
On Wed, Jul 22, 2015 at 6:32 PM, Brian Paul bri...@vmware.com wrote: Hi Marek, This is regarding your commit st/mesa: use pipe_sampler_view_release for releasing sampler views from last October. Basically, we have: void st_texture_release_all_sampler_views(struct st_context *st,

Re: [Mesa-dev] Using the right context in st_texture_release_all_sampler_views()

2015-07-23 Thread Stéphane Marchesin
On Thu, Jul 23, 2015 at 9:55 AM, Alex Deucher alexdeuc...@gmail.com wrote: On Wed, Jul 22, 2015 at 6:32 PM, Brian Paul bri...@vmware.com wrote: Hi Marek, This is regarding your commit st/mesa: use pipe_sampler_view_release for releasing sampler views from last October. Basically, we have:

Re: [Mesa-dev] Using the right context in st_texture_release_all_sampler_views()

2015-07-23 Thread Roland Scheidegger
Well they are not sharable by definition (per-context entities), it is not meant to work. So, it definitely should be fixed one way or another. Roland Am 23.07.2015 um 17:44 schrieb Marek Olšák: Hi Brian, Like the commit message says, I have no idea how Firefox does it. My commit only

Re: [Mesa-dev] Using the right context in st_texture_release_all_sampler_views()

2015-07-23 Thread Jose Fonseca
On 23/07/15 01:08, Jose Fonseca wrote: On 23/07/15 01:00, Brian Paul wrote: On 07/22/2015 05:31 PM, Jose Fonseca wrote: On 22/07/15 23:32, Brian Paul wrote: Hi Marek, This is regarding your commit st/mesa: use pipe_sampler_view_release for releasing sampler views from last October.

Re: [Mesa-dev] Using the right context in st_texture_release_all_sampler_views()

2015-07-22 Thread Brian Paul
On 07/22/2015 05:31 PM, Jose Fonseca wrote: On 22/07/15 23:32, Brian Paul wrote: Hi Marek, This is regarding your commit st/mesa: use pipe_sampler_view_release for releasing sampler views from last October. Basically, we have: void st_texture_release_all_sampler_views(struct st_context *st,

Re: [Mesa-dev] Using the right context in st_texture_release_all_sampler_views()

2015-07-22 Thread Jose Fonseca
On 22/07/15 23:32, Brian Paul wrote: Hi Marek, This is regarding your commit st/mesa: use pipe_sampler_view_release for releasing sampler views from last October. Basically, we have: void st_texture_release_all_sampler_views(struct st_context *st, struct

Re: [Mesa-dev] Using the right context in st_texture_release_all_sampler_views()

2015-07-22 Thread Jose Fonseca
On 23/07/15 01:00, Brian Paul wrote: On 07/22/2015 05:31 PM, Jose Fonseca wrote: On 22/07/15 23:32, Brian Paul wrote: Hi Marek, This is regarding your commit st/mesa: use pipe_sampler_view_release for releasing sampler views from last October. Basically, we have: void