On Fri, Dec 6, 2013 at 10:59 AM, Alex Deucher <alexdeucher at gmail.com> wrote: > On Fri, Dec 6, 2013 at 8:58 AM, Kleber Sacilotto de Souza > <klebers at linux.vnet.ibm.com> wrote: >> On 12/05/2013 12:42 PM, Alex Deucher wrote: >>> >>> Well, we'd need to start swapping indirect buffers and the ring as >>> well then which would get tricky as the CP at least does not have >>> separate swapping controls for different things. Probably easier to >>> fix up as appropriate for different asic families. We have function >>> pointers for the rtpr and wptr fetchers now, so we can do this pretty >>> cleanly. >>> >>> Alex >> >> >> Alex, >> >> Are you going to send a patch to fix this? >> >> If not, I don't have the knowledge of which asic families will need this >> fix, but if you inform me what needs to be done I can write the patch. > > I'll try and send a patch out in the next few days.
Patch attached. Compile tested only at the moment. Alex > > Alex > >> >> >> Thanks, >> >> >> -- >> Kleber Sacilotto de Souza >> IBM Linux Technology Center >> -------------- next part -------------- A non-text attachment was scrubbed... Name: 0001-drm-radeon-remove-generic-rptr-wptr-functions.patch Type: text/x-diff Size: 33710 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20131209/8e8c8fda/attachment-0001.patch>