On Wed, Nov 23, 2016 at 6:50 PM, Alex Deucher <alexdeucher at gmail.com> wrote: > On Wed, Nov 23, 2016 at 6:49 PM, Alex Deucher <alexdeucher at gmail.com> wrote: >> On Wed, Nov 23, 2016 at 6:00 PM, Jay Cornwall <jay at jcornwall.me> wrote: >>> On Wed, Nov 23, 2016, at 14:27, Alex Deucher wrote: >>>> It's per queue not per pipe. >>> >>> Are you sure? I was under the impression that EOP queeus were per-pipe >>> on Gfx7 and per-queue on Gfx8 onwards (to support context save/restore). >>> It's also hinted at by the register name (HPD == Hardware Pipe >>> Descriptor, HQD == Hardware Queue Descriptor). >> >> No, I'm not sure. I have a patch set for amdgpu to leave the >> fb_location as programmed by the vbios rather than reprogramming which >> resulted in vram not being at 0 in the GPU's address space. I then >> got ring failures on the compute queues until I made the eops per >> queue. It could be something else, but this seemed to fix it. > > Actually I've only tried this on gfx8, I just assumed gfx7 was also > affected, but it's possible this is not needed for gfx 7. consider this patch withdrawn. Alex
