[PATCH umr 0/4] gfx9: read VGPRs of halted waves

Tom St Denis tom.stdenis at amd.com
Sat Sep 9 12:23:50 UTC 2017


Hi Nicolai,

Patches look good I'll give them a whirl tomorrow when I have some free 
time.  I'll have to dig out your headless piglit instructions since I 
still can't bring X up on my Raven board.

Also I noticed the use of "(options.bitfields)" (etc, in print_waves) 
which is not your bug (since I did it first hehehe) but is a bug ... 
they should all be "asic->options.bitfields" which I'll fix in a 
follow-up patch after.

(provided they apply and combined with the kernel patch doesn't oops on 
me I'll R-b + push them out tomorrow or Monday morning).

Thanks,
Tom


On 09/09/17 06:55 AM, Nicolai Hähnle wrote:
> Hi all,
> 
> it seems that reading wave VGPRs actually works now with gfx9, at least
> for halted waves, and that can be pretty powerful for debugging.
> 
> Due to the volume of data it prints out, it's only enabled with -O bits.
> 
> In order for it to work properly, you'll also need the kernel patch
> I just sent out (drm/amdgpu/gfx9: implement wave VGPR reading).
> 
> Please review!
> 
> Thanks,
> Nicolai
> 
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
> 



More information about the amd-gfx mailing list