All of lore.kernel.org
 help / color / mirror / Atom feed
* [Nouveau] [bug report] drm/nouveau: stop using dma_resv_excl_fence
@ 2022-04-11  6:39 Dan Carpenter
  0 siblings, 0 replies; only message in thread
From: Dan Carpenter @ 2022-04-11  6:39 UTC (permalink / raw)
  To: christian.koenig; +Cc: nouveau

Hello Christian König,

The patch 756cc94d15fa: "drm/nouveau: stop using dma_resv_excl_fence"
from Nov 3, 2021, leads to the following Smatch static checker
warning:

	drivers/gpu/drm/nouveau/nouveau_bo.c:971 nouveau_bo_vm_cleanup()
	error: uninitialized symbol 'fence'.

drivers/gpu/drm/nouveau/nouveau_bo.c
    955 static void
    956 nouveau_bo_vm_cleanup(struct ttm_buffer_object *bo,
    957                       struct nouveau_drm_tile *new_tile,
    958                       struct nouveau_drm_tile **old_tile)
    959 {
    960         struct nouveau_drm *drm = nouveau_bdev(bo->bdev);
    961         struct drm_device *dev = drm->dev;
    962         struct dma_fence *fence;
    963         int ret;
    964 
    965         /* TODO: This is actually a memory management dependency */
    966         ret = dma_resv_get_singleton(bo->base.resv, false, &fence);
    967         if (ret)
    968                 dma_resv_wait_timeout(bo->base.resv, false, false,
    969                                       MAX_SCHEDULE_TIMEOUT);

"fence" is uninitialized of dma_resv_get_singleton() fails.

    970 
--> 971         nv10_bo_put_tile_region(dev, *old_tile, fence);
    972         *old_tile = new_tile;
    973 }

regards,
dan carpenter

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2022-04-11  6:39 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-04-11  6:39 [Nouveau] [bug report] drm/nouveau: stop using dma_resv_excl_fence Dan Carpenter

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.