search for: nouveau_bo_invalidate_cach

...+0x58c/0x5c0 [ +0.000015] ttm_bo_validate+0x152/0x190 [ +0.000004] ? ttm_bo_init_reserved+0x3d8/0x490 [ +0.000012] ? mutex_trylock+0xcd/0xe0 [ +0.000004] ? ttm_bo_handle_move_mem+0x58/0x5c0 [ +0.000007] ttm_bo_init_reserved+0x3f4/0x490 [ +0.000010] ttm_bo_init+0x2f/0xa0 [ +0.000009] ? nouveau_bo_invalidate_caches+0x10/0x10 [ +0.000005] nouveau_bo_new+0x416/0x590 [ +0.000007] ? nouveau_bo_invalidate_caches+0x10/0x10 [ +0.000009] ? nouveau_gem_new+0x100/0x100 [ +0.000004] nouveau_gem_new+0x49/0x100 [ +0.000009] nouveau_gem_ioctl_new+0x41/0xc0 [ +0.000009] drm_ioctl_kernel+0x59/0xb0 [ +0.000008]...

...0 >> [ +0.000004] ? ttm_bo_init_reserved+0x3d8/0x490 >> [ +0.000012] ? mutex_trylock+0xcd/0xe0 >> [ +0.000004] ? ttm_bo_handle_move_mem+0x58/0x5c0 >> [ +0.000007] ttm_bo_init_reserved+0x3f4/0x490 >> [ +0.000010] ttm_bo_init+0x2f/0xa0 >> [ +0.000009] ? nouveau_bo_invalidate_caches+0x10/0x10 >> [ +0.000005] nouveau_bo_new+0x416/0x590 >> [ +0.000007] ? nouveau_bo_invalidate_caches+0x10/0x10 >> [ +0.000009] ? nouveau_gem_new+0x100/0x100 >> [ +0.000004] nouveau_gem_new+0x49/0x100 >> [ +0.000009] nouveau_gem_ioctl_new+0x41/0xc0 >> [...

...m_bo_validate+0x152/0x190 > [ +0.000004] ? ttm_bo_init_reserved+0x3d8/0x490 > [ +0.000012] ? mutex_trylock+0xcd/0xe0 > [ +0.000004] ? ttm_bo_handle_move_mem+0x58/0x5c0 > [ +0.000007] ttm_bo_init_reserved+0x3f4/0x490 > [ +0.000010] ttm_bo_init+0x2f/0xa0 > [ +0.000009] ? nouveau_bo_invalidate_caches+0x10/0x10 > [ +0.000005] nouveau_bo_new+0x416/0x590 > [ +0.000007] ? nouveau_bo_invalidate_caches+0x10/0x10 > [ +0.000009] ? nouveau_gem_new+0x100/0x100 > [ +0.000004] nouveau_gem_new+0x49/0x100 > [ +0.000009] nouveau_gem_ioctl_new+0x41/0xc0 > [ +0.000009] drm_ioctl_...

...0x2e/0x60 [ttm] [ 1035.437067] ttm_bo_handle_move_mem+0x4cf/0x550 [ttm] [ 1035.437073] ttm_bo_validate+0x119/0x130 [ttm] [ 1035.437104] ? drm_get_edid_switcheroo+0x16/0x40 [drm] [ 1035.437109] ttm_bo_init_reserved+0x334/0x380 [ttm] [ 1035.437114] ? ttm_bo_init+0x62/0xd0 [ttm] [ 1035.437190] ? nouveau_bo_invalidate_caches+0x10/0x10 [nouveau] [ 1035.437226] ? nouveau_bo_new+0x401/0x580 [nouveau] [ 1035.437262] ? nouveau_bo_invalidate_caches+0x10/0x10 [nouveau] [ 1035.437298] ? nouveau_gem_new+0x120/0x120 [nouveau] [ 1035.437334] ? nouveau_gem_new+0x5d/0x120 [nouveau] [ 1035.437370] ? nouveau_gem_ioctl_new+0x53...

...7067] ttm_bo_handle_move_mem+0x4cf/0x550 [ttm] > [ 1035.437073] ttm_bo_validate+0x119/0x130 [ttm] > [ 1035.437104] ? drm_get_edid_switcheroo+0x16/0x40 [drm] > [ 1035.437109] ttm_bo_init_reserved+0x334/0x380 [ttm] > [ 1035.437114] ? ttm_bo_init+0x62/0xd0 [ttm] > [ 1035.437190] ? nouveau_bo_invalidate_caches+0x10/0x10 [nouveau] > [ 1035.437226] ? nouveau_bo_new+0x401/0x580 [nouveau] > [ 1035.437262] ? nouveau_bo_invalidate_caches+0x10/0x10 [nouveau] > [ 1035.437298] ? nouveau_gem_new+0x120/0x120 [nouveau] > [ 1035.437334] ? nouveau_gem_new+0x5d/0x120 [nouveau] > [ 1035.437370] ? n...

...uveau_bo_validate(struct nouveau_bo *nvbo, bool interruptible, > { > int ret; > > + nouveau_bo_sync_for_device(nvbo); > + > ret = ttm_bo_validate(&nvbo->bo, &nvbo->placement, > interruptible, no_wait_gpu); > if (ret) > @@ -487,6 +489,36 @@ nouveau_bo_invalidate_caches(struct ttm_bo_device *bdev, uint32_t flags) > return 0; > } > > +#ifdef NOUVEAU_NEED_CACHE_SYNC I don't like this ifdef at all. I know calling this functions will add a little overhead to x86 where it isn't strictly required, but I think it's negligible. When I look...

...328/0x4a0 [ttm] [ 132.907036] ttm_bo_validate+0x98/0x110 [ttm] [ 132.907047] ? drm_vma_offset_add+0x41/0x60 [drm] [ 132.907057] ? drm_add_edid_modes.part.29+0x193e/0x19b0 [drm] [ 132.907064] ttm_bo_init_reserved+0x382/0x430 [ttm] [ 132.907069] ttm_bo_init+0x52/0xc0 [ttm] [ 132.907093] ? nouveau_bo_invalidate_caches+0x10/0x10 [nouveau] [ 132.907117] ? nouveau_gem_ioctl_pushbuf+0x9bf/0x1600 [nouveau] [ 132.907141] nouveau_bo_new+0x416/0x590 [nouveau] [ 132.907164] ? nouveau_bo_invalidate_caches+0x10/0x10 [nouveau] [ 132.907188] ? nouveau_gem_new+0x100/0x100 [nouveau] [ 132.907210] nouveau_gem_new+0x...

...ttm_mem_evict_first+0x145/0x1a0 [ttm] kernel: [514909.423149] ttm_bo_mem_space+0x312/0x470 [ttm] kernel: [514909.423156] ttm_bo_validate+0xd4/0x160 [ttm] kernel: [514909.423162] ttm_bo_init_reserved+0x3bb/0x450 [ttm] kernel: [514909.423168] ttm_bo_init+0x2d/0x90 [ttm] kernel: [514909.423218] ? nouveau_bo_invalidate_caches+0x10/0x10 [nouveau] kernel: [514909.423262] nouveau_bo_new+0x1bc/0x2f0 [nouveau] kernel: [514909.423303] ? nouveau_bo_invalidate_caches+0x10/0x10 [nouveau] kernel: [514909.423344] nouveau_gem_new+0x64/0x130 [nouveau] kernel: [514909.423384] nouveau_gem_ioctl_new+0x80/0x140 [nouveau] kernel: [...

...m/nouveau/nouveau_bo.c @@ -407,6 +407,8 @@ nouveau_bo_validate(struct nouveau_bo *nvbo, bool interruptible, { int ret; + nouveau_bo_sync_for_device(nvbo); + ret = ttm_bo_validate(&nvbo->bo, &nvbo->placement, interruptible, no_wait_gpu); if (ret) @@ -487,6 +489,36 @@ nouveau_bo_invalidate_caches(struct ttm_bo_device *bdev, uint32_t flags) return 0; } +#ifdef NOUVEAU_NEED_CACHE_SYNC +void +nouveau_bo_sync_for_cpu(struct nouveau_bo *nvbo) +{ + struct nouveau_device *device; + struct ttm_tt *ttm = nvbo->bo.ttm; + + device = nouveau_dev(nouveau_bdev(ttm->bdev)->dev); + + if (nv...

...el: ttm_bo_handle_move_mem+0x3b1/0x590 [ttm] feb 23 17:21:08 dell-2017 kernel: ? drm_vma_offset_add+0x3c/0x60 feb 23 17:21:08 dell-2017 kernel: ttm_bo_evict+0x145/0x320 [ttm] feb 23 17:21:08 dell-2017 kernel: ? gf119_disp_chan_uevent_fini+0x3d/0x60 [nouveau] feb 23 17:21:08 dell-2017 kernel: ? nouveau_bo_invalidate_caches+0x10/0x10 [nouveau] feb 23 17:21:08 dell-2017 kernel: ? drm_vma_offset_add+0x3c/0x60 feb 23 17:21:08 dell-2017 kernel: ? drm_mode_std+0x479/0x4a0 feb 23 17:21:08 dell-2017 kernel: ttm_mem_evict_first+0x18b/0x210 [ttm] feb 23 17:21:08 dell-2017 kernel: ttm_bo_force_list_clean+0x8a/0x150 [ttm]...

This small series introduces TTM helper functions as well as Nouveau hooks that are needed to ensure buffer coherency on ARM. Most of this series is a forward-port of some patches Lucas Stach sent last year and that are also needed for Nouveau GK20A support: http://lists.freedesktop.org/archives/nouveau/2013-August/014026.html Another patch takes care of flushing the CPU write-buffer when

https://bugs.freedesktop.org/show_bug.cgi?id=105117 Bug ID: 105117 Summary: desktop freeze playing video nouveau: DATA_ERROR INVALID_BITFIELD Product: xorg Version: 7.6 (2010.12) Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW Severity: major Priority: medium