Hi Ben,

    Thank you for the patch :)

    I have two concerns about it:

         1) It does not appear to be R300 specific - why doesn't similar
            Radeon ioctl work ? Also, I would imagine that this would
            require a change in r300 driver to work, wouldn't it ?


2) I was able to play Quake for somewhat prolonged periods, I don't think this would have really worked if aging was truly broken, though, maybe, I am wrong on this one.

            Would you have a test app that shows brokenness ? Perhaps
            something that uses a lot of textures once.

Also, if aging does not work on your setup, it might have to do with amount of system RAM and memory controller settings. I *was* wondering why
fbLocation is 0 for R300. If so, this needs to be fixed in the 2d driver.


                        best

                          Vladimir Dergachev

On Sun, 6 Feb 2005, Ben Skeggs wrote:

Hello Vladimir,

I've attached a patch which implements the RADOEN_CMD_DMA_DISCARD ioctl from
the radeon/r200 drm. I thought I'd post here before commiting to cvs in case I've
done something bad.


Without this, eventually r300AllocDmaRegion will get stuck in a loop continually calling
drmDMA (r300_ioctl.c::r300RefillCurrentDmaRegion).


It seems that the drm buffer management code depends on having a scratch register
containing the "age" of a buffer. I'm not sure of the details, I just know that it stops
the infinite drmDMA loop.


Is this the correct way of fixing this? Or have I completely missed something?

Regards,
Ben Skeggs.




-------------------------------------------------------
This SF.Net email is sponsored by: IntelliVIEW -- Interactive Reporting
Tool for open source databases. Create drag-&-drop reports. Save time
by over 75%! Publish reports on the web. Export to DOC, XLS, RTF, etc.
Download a FREE copy at http://www.intelliview.com/go/osdn_nl
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to