drm/nouveau: Fix ttm move init with multiple GPUs
authorJames Jones <jajones@nvidia.com>
Tue, 17 Dec 2019 00:56:12 +0000 (16:56 -0800)
committerBen Skeggs <bskeggs@redhat.com>
Wed, 15 Jan 2020 00:49:59 +0000 (10:49 +1000)
commit72ecb0a6ce89dd7f9d0f8171a45b0ed814b0f9a8
tree6d7cb4de425f05754984aa698a7d762c5af9e013
parentb4229fc67d9b9d8910ad32c508fd94518bd01194
drm/nouveau: Fix ttm move init with multiple GPUs

The pointer used to walk the table of move ops
and pick the right one for the current GPU was
declared static, meaning its state was carried
over between invocations of the function, and also
made the function non-rentrant and thread-unsafe.
Since the table is ordered such that newer GPU
methods are listed first, the result of this was
that initializing newer GPUs after older GPUs
would result in no suitable ttm move acceleration
operations being found, and ttm would fall back
to CPU blits on the older GPUs.

This change declares the walking pointer
separately from the table and makes it non-static
to fix the logic.

Signed-off-by: James Jones <jajones@nvidia.com>
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
drivers/gpu/drm/nouveau/nouveau_bo.c