Re: [Mesa-dev] [PATCH v2] nv50/ir: do not call textureMask() for surface ops

2015-12-02 Thread Samuel Pitoiset
On 12/02/2015 12:46 AM, Ilia Mirkin wrote: On Tue, Dec 1, 2015 at 6:26 PM, Samuel Pitoiset wrote: While we are it, apply the same change for Tesla. Does tesla even have surface ops? I don't see anything in envydis... Please don't add code that doesn't need to be

Re: [Mesa-dev] [PATCH v2] nv50/ir: do not call textureMask() for surface ops

2015-12-01 Thread Ilia Mirkin
On Tue, Dec 1, 2015 at 6:26 PM, Samuel Pitoiset wrote: > While we are it, apply the same change for Tesla. Does tesla even have surface ops? I don't see anything in envydis... Please don't add code that doesn't need to be there. -ilia

[Mesa-dev] [PATCH v2] nv50/ir: do not call textureMask() for surface ops

2015-12-01 Thread Samuel Pitoiset
That texture mask thing doesn't seem to be needed for surface ops, so just as nve4+, let do that only for texture ops. This fixes a segfault with 'test_surface_st' from gallium/tests/trivial/compute.c on Fermi because this test uses sustp. While we are it, apply the same change for Tesla.