Re: [Mesa-dev] [PATCH v2 5/6] radeonsi: use slot indexes for bindless handles

2017-08-03 Thread Samuel Pitoiset
On 08/01/2017 09:43 PM, Marek Olšák wrote: There is an issue independent of this patch: si_update_all_resident_texture_descriptors only updates resident descriptors. Non-resident descriptors aren't updated, and si_make_texture_handle_resident doesn't update them either. I think it should be

Re: [Mesa-dev] [PATCH v2 5/6] radeonsi: use slot indexes for bindless handles

2017-08-01 Thread Marek Olšák
There is an issue independent of this patch: si_update_all_resident_texture_descriptors only updates resident descriptors. Non-resident descriptors aren't updated, and si_make_texture_handle_resident doesn't update them either. I think it should be fixed in si_make_texture_handle_resident. Same

[Mesa-dev] [PATCH v2 5/6] radeonsi: use slot indexes for bindless handles

2017-07-26 Thread Samuel Pitoiset
Using VRAM address as bindless handles is not a good idea because we have to use LLVMIntToPTr and the LLVM CSE pass can't optimize because it has no information about the pointer. Instead, use slots indexes like the existing descriptors. Note that we use fixed 16-dword slots for both samplers and