On Thu, Mar 09, 2017 at 10:57:27AM +0100, Christian König wrote:
> Am 09.03.2017 um 04:44 schrieb Alex Deucher:
> > From: Ken Wang <[email protected]>
> > 
> > Newer asics use 64 bit wptrs
> 
> We need a better patch description. Newer asics doesn't use 64bit wptrs, but
> rather need them!
> 
> E.g. if the wptr is now smaller than the rptr that doesn't indicate a wrap
> around any more.

What does it means ? Doesn't make sense, you are allocating 1TB command
buffer and know you will never run out of space ? ;)

Either we are talking about a ring buffer and then wptr < rptr means wrap
around or we are talking about something new that is not a ring buffer.

64 bits or 32 bits doesn't matter from ring buffer point of view. So i
am puzzle about what this new thing is. Anywhere i can read about this
new command buffer thingy ? Because from where i stand it looks like it
would be better to be something else than a ring buffer if there is no
wrap around.

Cheers,
Jérôme 
_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to