[Nouveau] [bug report] drm/nouveau: stop using dma_resv_excl_fence

Dan Carpenter dan.carpenter at oracle.com
Mon Apr 11 06:39:00 UTC 2022


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


More information about the Nouveau mailing list