[Ubuntu-x-swat] [Bug 459961] Re: radeon_dma.c:210: radeonRefillCurrentDmaRegion: Assertion `dma_bo->bo->cref == 1' failed.

2009-11-29 Thread ybkosher
Tormod, If I understand this, that while loop was put in because of possible memory leaks, but in bo_allocate, the new bo is made part of a linked list. radeon_bo_legacy_dtor destroys all the bo_legacys in the list. I don't think it is possble to leak the memory. A very unscientific test, but I

[Ubuntu-x-swat] [Bug 459961] Re: radeon_dma.c:210: radeonRefillCurrentDmaRegion: Assertion `dma_bo->bo->cref == 1' failed.

2009-11-27 Thread ybkosher
What I meant to say about the code being reused for all radeon drivers: Just because there is a bug in one that keeps the ref count artificially high, does not mean it is a bug for all of the drivers. Whatever was keeping it high should be fixed. This while loop is too drastic. -- radeon_dma.c:

[Ubuntu-x-swat] [Bug 459961] Re: radeon_dma.c:210: radeonRefillCurrentDmaRegion: Assertion `dma_bo->bo->cref == 1' failed.

2009-11-27 Thread ybkosher
I was getting this crash in UrbanTerror. It only happens WITHOUT KMS on (radeon.modeset=0). I made this change and it works for me: diff -r mesa-7.7.0~git20091125+mesa-7-7-branch.7fac8ce7/src/mesa/drivers/dri/radeon/radeon_dma.c mesa-7.7.0~git20091125+mesa-7-7-branch.7fac8ce7-mine/src/mesa/dr