Commit e59c0205 authored by Alex Xie's avatar Alex Xie Committed by Alex Deucher

drm/amdgpu: Move compute vm bug logic to amdgpu_vm.c

  In review, Christian would like to keep the logic
  inside amdgpu_vm.c with a cost of slightly slower.
  The loop is still optimized out with this patch.

v2: remove the if statement. Now it is not slower.
Signed-off-by: default avatarAlex Xie <AlexBin.Xie@amd.com>
Reviewed-by: default avatarChristian König <christian.koeng@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 117a48a7
...@@ -2190,6 +2190,8 @@ int amdgpu_device_init(struct amdgpu_device *adev, ...@@ -2190,6 +2190,8 @@ int amdgpu_device_init(struct amdgpu_device *adev,
adev->accel_working = true; adev->accel_working = true;
amdgpu_vm_check_compute_bug(adev);
/* Initialize the buffer migration limit. */ /* Initialize the buffer migration limit. */
if (amdgpu_moverate >= 0) if (amdgpu_moverate >= 0)
max_MBps = amdgpu_moverate; max_MBps = amdgpu_moverate;
......
...@@ -154,36 +154,6 @@ void amdgpu_ring_undo(struct amdgpu_ring *ring) ...@@ -154,36 +154,6 @@ void amdgpu_ring_undo(struct amdgpu_ring *ring)
ring->funcs->end_use(ring); ring->funcs->end_use(ring);
} }
/**
* amdgpu_ring_check_compute_vm_bug - check whether this ring has compute vm bug
*
* @adev: amdgpu_device pointer
* @ring: amdgpu_ring structure holding ring information
*/
static void amdgpu_ring_check_compute_vm_bug(struct amdgpu_device *adev,
struct amdgpu_ring *ring)
{
const struct amdgpu_ip_block *ip_block;
ring->has_compute_vm_bug = false;
if (ring->funcs->type != AMDGPU_RING_TYPE_COMPUTE)
/* only compute rings */
return;
ip_block = amdgpu_get_ip_block(adev, AMD_IP_BLOCK_TYPE_GFX);
if (!ip_block)
return;
/* Compute ring has a VM bug for GFX version < 7.
And compute ring has a VM bug for GFX 8 MEC firmware version < 673.*/
if (ip_block->version->major <= 7) {
ring->has_compute_vm_bug = true;
} else if (ip_block->version->major == 8)
if (adev->gfx.mec_fw_version < 673)
ring->has_compute_vm_bug = true;
}
/** /**
* amdgpu_ring_init - init driver ring struct. * amdgpu_ring_init - init driver ring struct.
* *
...@@ -292,8 +262,6 @@ int amdgpu_ring_init(struct amdgpu_device *adev, struct amdgpu_ring *ring, ...@@ -292,8 +262,6 @@ int amdgpu_ring_init(struct amdgpu_device *adev, struct amdgpu_ring *ring,
DRM_ERROR("Failed to register debugfs file for rings !\n"); DRM_ERROR("Failed to register debugfs file for rings !\n");
} }
amdgpu_ring_check_compute_vm_bug(adev, ring);
return 0; return 0;
} }
......
...@@ -212,9 +212,4 @@ static inline void amdgpu_ring_clear_ring(struct amdgpu_ring *ring) ...@@ -212,9 +212,4 @@ static inline void amdgpu_ring_clear_ring(struct amdgpu_ring *ring)
} }
static inline bool amdgpu_ring_has_compute_vm_bug(struct amdgpu_ring *ring)
{
return ring->has_compute_vm_bug;
}
#endif #endif
...@@ -656,6 +656,41 @@ static int amdgpu_vm_alloc_reserved_vmid(struct amdgpu_device *adev, ...@@ -656,6 +656,41 @@ static int amdgpu_vm_alloc_reserved_vmid(struct amdgpu_device *adev,
return r; return r;
} }
/**
* amdgpu_vm_check_compute_bug - check whether asic has compute vm bug
*
* @adev: amdgpu_device pointer
*/
void amdgpu_vm_check_compute_bug(struct amdgpu_device *adev)
{
const struct amdgpu_ip_block *ip_block;
bool has_compute_vm_bug;
struct amdgpu_ring *ring;
int i;
has_compute_vm_bug = false;
ip_block = amdgpu_get_ip_block(adev, AMD_IP_BLOCK_TYPE_GFX);
if (ip_block) {
/* Compute has a VM bug for GFX version < 7.
Compute has a VM bug for GFX 8 MEC firmware version < 673.*/
if (ip_block->version->major <= 7)
has_compute_vm_bug = true;
else if (ip_block->version->major == 8)
if (adev->gfx.mec_fw_version < 673)
has_compute_vm_bug = true;
}
for (i = 0; i < adev->num_rings; i++) {
ring = adev->rings[i];
if (ring->funcs->type == AMDGPU_RING_TYPE_COMPUTE)
/* only compute rings */
ring->has_compute_vm_bug = has_compute_vm_bug;
else
ring->has_compute_vm_bug = false;
}
}
bool amdgpu_vm_need_pipeline_sync(struct amdgpu_ring *ring, bool amdgpu_vm_need_pipeline_sync(struct amdgpu_ring *ring,
struct amdgpu_job *job) struct amdgpu_job *job)
{ {
...@@ -664,8 +699,7 @@ bool amdgpu_vm_need_pipeline_sync(struct amdgpu_ring *ring, ...@@ -664,8 +699,7 @@ bool amdgpu_vm_need_pipeline_sync(struct amdgpu_ring *ring,
struct amdgpu_vm_id_manager *id_mgr = &adev->vm_manager.id_mgr[vmhub]; struct amdgpu_vm_id_manager *id_mgr = &adev->vm_manager.id_mgr[vmhub];
struct amdgpu_vm_id *id; struct amdgpu_vm_id *id;
bool gds_switch_needed; bool gds_switch_needed;
bool vm_flush_needed = job->vm_needs_flush || bool vm_flush_needed = job->vm_needs_flush || ring->has_compute_vm_bug;
amdgpu_ring_has_compute_vm_bug(ring);
if (job->vm_id == 0) if (job->vm_id == 0)
return false; return false;
......
...@@ -245,5 +245,6 @@ void amdgpu_vm_adjust_size(struct amdgpu_device *adev, uint64_t vm_size); ...@@ -245,5 +245,6 @@ void amdgpu_vm_adjust_size(struct amdgpu_device *adev, uint64_t vm_size);
int amdgpu_vm_ioctl(struct drm_device *dev, void *data, struct drm_file *filp); int amdgpu_vm_ioctl(struct drm_device *dev, void *data, struct drm_file *filp);
bool amdgpu_vm_need_pipeline_sync(struct amdgpu_ring *ring, bool amdgpu_vm_need_pipeline_sync(struct amdgpu_ring *ring,
struct amdgpu_job *job); struct amdgpu_job *job);
void amdgpu_vm_check_compute_bug(struct amdgpu_device *adev);
#endif #endif
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment