Re: [Mesa-dev] [PATCH] radeon: Use dup fd as key in drm-winsys hash table to fix ZaphodHeads.

2015-06-29 Thread Marek Olšák
Reviewed-by: Marek Olšák marek.ol...@amd.com The fix for amdgpu should be in libdrm_amdgpu / amdgpu_device.c. Marek On Sun, Jun 28, 2015 at 7:49 AM, Mario Kleiner mario.kleiner...@gmail.com wrote: Thanks for the review Ilia. Just saw that the new amdgpu winsys driver will need the same fix,

[Mesa-dev] [PATCH] radeon: Use dup fd as key in drm-winsys hash table to fix ZaphodHeads.

2015-06-27 Thread Mario Kleiner
Same problem and fix as for nouveau's ZaphodHeads trouble. See patch ... nouveau: Use dup fd as key in drm-winsys hash table to fix ZaphodHeads. ... for reference. Cc: 10.3 10.4 10.5 10.6 mesa-sta...@lists.freedesktop.org Signed-off-by: Mario Kleiner mario.kleiner...@gmail.com Cc: Ilia Mirkin

Re: [Mesa-dev] [PATCH] radeon: Use dup fd as key in drm-winsys hash table to fix ZaphodHeads.

2015-06-27 Thread Ilia Mirkin
Without being intimately familiar with the radeon winsys code, this seems sane (esp after re-reviewing the situation for nouveau). Reviewed-by: Ilia Mirkin imir...@alum.mit.edu. But one of the radeon people definitely needs to look at this before pushing. On Sat, Jun 27, 2015 at 9:02 PM, Mario

Re: [Mesa-dev] [PATCH] radeon: Use dup fd as key in drm-winsys hash table to fix ZaphodHeads.

2015-06-27 Thread Mario Kleiner
Thanks for the review Ilia. Just saw that the new amdgpu winsys driver will need the same fix, once this one is confirmed to be ok. thanks, -mario On 06/28/2015 07:23 AM, Ilia Mirkin wrote: Without being intimately familiar with the radeon winsys code, this seems sane (esp after re-reviewing