[Nouveau] low memory

Fri Jan 8 06:55:23 PST 2010

On Fri, Jan 8, 2010 at 1:50 AM, Xavier <shiningxc at gmail.com> wrote:
>
> Strange, I am 99% sure I did update libdrm , and before updating ddx.
> I will double-check tomorrow.
>
> And I indeed found this commit :
> http://cgit.freedesktop.org/nouveau/xf86-video-nouveau/commit/?id=bb1947831d9a4e080b8d1e9dba086af6527ef479
> So I assume the libdrm one is this one on the same day :
> http://cgit.freedesktop.org/mesa/drm/commit/?id=f1660c249198b5cc14ebbb75107da7bcb6972033
>
> So by playing with these two commits, I should also be able to provide
> more information. Thanks !
>

So it more looks like a drm/ttm problem.
With a recent drm/ttm (after 98ba9c1322), firefox just displays a
black box instead of the picture, the system lags a bit, and I get the
following in dmesg :
[   75.800986] [drm] nouveau 0000:03:00.0: validate: -12
[   75.801365] [drm] nouveau 0000:03:00.0: validate: -12
[   80.772360] reserve_ram_pages_type failed 0xc403000-0xc404000,
track 0x8, req 0x10

If I go before the drm/ttm merge, it still works fine. commit b5d1f52b
or earlier.

If I try to go somewhere between b5d1f52b and 98ba9c1322, I get the
attached bug. But that might be because some important bugfix has been
made at the end of that range, like "drm/nouveau: Fix "general
protection fault" in the flipd/flips eviction path."

Here is the commit range I mentioned :

* commit 98ba9c1322ed334b223117654cb671ea320919d5
| Author: Francisco Jerez <currojerez at riseup.net>
| Date:   Wed Dec 16 19:03:28 2009 +0100
|
|     drm/nouveau: Drop redundant placement initialization.
|
|     Signed-off-by: Francisco Jerez <currojerez at riseup.net>
|
* commit 4ee7ee3f1e824561a779c068773ed21bd7df5413
| Author: Francisco Jerez <currojerez at riseup.net>
| Date:   Wed Dec 16 19:05:38 2009 +0100
|
|     drm/nouveau: No need to force evict=true when swapping evicted
BOs back in.
|
|     Signed-off-by: Francisco Jerez <currojerez at riseup.net>
|
* commit 81c1019857b70efbba831f4a02b377066b836cbc
| Author: Francisco Jerez <currojerez at riseup.net>
| Date:   Wed Dec 16 19:05:00 2009 +0100
|
|     drm/nouveau: Fix "general protection fault" in the flipd/flips
eviction path.
|
|     Signed-off-by: Francisco Jerez <currojerez at riseup.net>
|
* commit f4e1a1bd530b6e4d2bfd1fbd7daddd5020359f15
| Author: Maarten Maathuis <madman2003 at gmail.com>
| Date:   Wed Dec 16 18:42:35 2009 +0100
|
|     Revert "drm/nouveau: unpin cursor and lut bo when suspending"
|
|     This reverts commit a330aea77289d8d4465ea0d072fb868f9c782577.
|
|     - For some odd reason the cursor gets stuck with the last image,
don't see why.
|     - The clut is already regenerated after resume it seems.
|
|     Signed-off-by: Maarten Maathuis <madman2003 at gmail.com>
|
* commit dd6e87021a45c56acddbf229221543239f12a924
| Author: Francisco Jerez <currojerez at riseup.net>
| Date:   Wed Dec 16 12:27:11 2009 +0100
|
|     drm/i2c/ch7006: Drop build time dependency to nouveau.
|
|     This partially reverts e4b41066, as this driver is intended to be
|     useful with any KMS driver for suitable hardware. The missing build
|     dependency that commit workarounded was DRM_KMS_HELPER.
|
|     Signed-off-by: Francisco Jerez <currojerez at riseup.net>
|
* commit 64aec72df1fa782ec3b060ccbdfa3fc9594f6119
| Author: Francisco Jerez <currojerez at riseup.net>
| Date:   Fri Dec 11 16:51:09 2009 +0100
|
|     drm/nouveau: Make the MM aware of pre-G80 tiling.
|
|     Signed-off-by: Francisco Jerez <currojerez at riseup.net>
|
* commit a49b387671a26e875d07befdc1d65087c557f5a8
| Author: Francisco Jerez <currojerez at riseup.net>
| Date:   Wed Dec 16 12:12:27 2009 +0100
|
|     drm/nouveau: Pre-G80 tiling support.
|
|     Signed-off-by: Francisco Jerez <currojerez at riseup.net>
|
* commit dddd2e625429cfd3c692928bbba3b74aa2d528b2
| Author: Francisco Jerez <currojerez at riseup.net>
| Date:   Sun Dec 13 20:07:42 2009 +0100
|
|     drm/nouveau: Add cache_flush/pull fifo engine functions.
|
|     Signed-off-by: Francisco Jerez <currojerez at riseup.net>
|
*   commit 69c27c09270b6898a06948d61b4b310e80990798
|\  Merge: b5d1f52 5012f50
| | Author: Ben Skeggs <bskeggs at redhat.com>
| | Date:   Wed Dec 16 16:46:17 2009 +1000
| |
| |     Merge branch 'drm-core-next' of
git://git.kernel.org/pub/scm/linux/kernel/git/airlied/drm-2.6 into
nouveau
| |
| * commit 5012f5063f41ca01240e5983c3b2cceb9aafc7a4
| | Author: Jerome Glisse <jglisse at redhat.com>
| | Date:   Thu Dec 10 18:07:26 2009 +0100
| |
| |     drm/ttm: Fix memory type manager debug information printing
| |
| |     System memory type doesn't have a drm_mm manager associated to
| |     it. This patch avoid trying to call drm_mm_debug on unitialized
| |     drm_mm when printing debug info on the system memory manager.
| |
| |     Signed-off-by: Jerome Glisse <jglisse at redhat.com>
| |     Signed-off-by: Dave Airlie <airlied at redhat.com>
| |
| * commit eb6d2c39dbe46ea1c4f3da4eac5728c73f109ea1
| | Author: Jerome Glisse <jglisse at redhat.com>
| | Date:   Thu Dec 10 16:15:52 2009 +0100
| |
| |     drm/ttm: Fix printk format & compute bo->mem.size at bo initialization
| |
| |     Signed-off-by: Jerome Glisse <jglisse at redhat.com>
| |     Signed-off-by: Dave Airlie <airlied at redhat.com>
| |
| * commit 9c51ba1db37cab780f38b2210913959f22d7b830
| | Author: Thomas Hellstrom <thellstrom at vmware.com>
| | Date:   Wed Dec 2 18:33:46 2009 +0100
| |
| |     drm/ttm: Fix potential ttm_mem_evict_first races.
| |
| |     1) The function was previously called with a potentially empty
| |     LRU list which would have lead to an OOPS or servere corruption.
| |     2) In rare cases, after reservation has succeeded, another process may
| |     already have evicted it or even pinned it. We must revalidate the
| |     buffer status after releasing the lru lock.
| |
| |     Signed-off-by: Thomas Hellstrom <thellstrom at vmware.com>
| |     Signed-off-by: Dave Airlie <airlied at redhat.com>
| |
| * commit aaa207369436d04bb85382ddbb688a5b9461fd21
| | Author: Thomas Hellstrom <thellstrom at vmware.com>
| | Date:   Wed Dec 2 18:33:45 2009 +0100
| |
| |     drm/ttm: Delayed delete fixes.
| |
| |     1) Remove from lru before reserving so we avoid competing with
| |     evicting processes.
| |     2) Avoid calling kref_put() on bo::list_kref while spinlocked.
| |     3) Additional refcounting bug-checking.
| |
| |     Signed-off-by: Thomas Hellstrom <thellstrom at vmware.com>
| |     Signed-off-by: Dave Airlie <airlied at redhat.com>
| |
| * commit b663752627e7c6b4bc414684d71c6adce5719fce
| | Author: Dave Airlie <airlied at redhat.com>
| | Date:   Mon Dec 14 14:51:35 2009 +1000
| |
| |     drm/ttm: fix two bugs in new placement routines.
| |
| |     a) the loops were going to <= not <, leading to illegal memory access
| |     b) the busy placement checks were using the placement arrays not the
| |        busy placement ones.
| |
| |     Acked-by: Jerome Glisse <jglisse at redhat.com>
| |     Signed-off-by: Dave Airlie <airlied at redhat.com>
| |
| * commit b8ff7357da45e025c446fe0479612215fe56a249
| | Author: Dave Airlie <airlied at redhat.com>
| | Date:   Tue Dec 15 08:07:12 2009 +1000
| |
| |     drm/ttm: fix incorrect logic in ttm_bo_io path
| |
| |     This path isn't used by radeon yet, but future drivers will want it,
| |     so fix it right.
| |
| |     Reported-by: Luca Barbieri <luca at luca-barbieri.com>
| |     Signed-off-by: Dave Airlie <airlied at redhat.com>
| |
* | commit b5d1f52b73059bd34355e35654abc91015e03bb5
| | Author: Ben Skeggs <bskeggs at redhat.com>
| | Date:   Wed Dec 16 16:22:42 2009 +1000
| |
| |     drm/nouveau: fix bug causing pinned buffers to lose their NO_EVICT flag
| |
| |     Signed-off-by: Ben Skeggs <bskeggs at redhat.com>
| |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bug
Type: application/octet-stream
Size: 3323 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/nouveau/attachments/20100108/725e90f0/attachment.obj 