drm/amdgpu: Move initialization of xcp before kfd
After partition switch, fill all relevant xcp information before kfd starts initialization. Signed-off-by: Lijo Lazar <lijo.lazar@amd.com> Reviewed-by: Le Ma <le.ma@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
This commit is contained in:
parent
15e3eee8d3
commit
e47947abb9
|
@ -113,12 +113,17 @@ static void __amdgpu_xcp_add_block(struct amdgpu_xcp_mgr *xcp_mgr, int xcp_id,
|
|||
xcp->valid = true;
|
||||
}
|
||||
|
||||
static int __amdgpu_xcp_init(struct amdgpu_xcp_mgr *xcp_mgr, int num_xcps)
|
||||
int amdgpu_xcp_init(struct amdgpu_xcp_mgr *xcp_mgr, int num_xcps, int mode)
|
||||
{
|
||||
struct amdgpu_xcp_ip ip;
|
||||
uint8_t mem_id;
|
||||
int i, j, ret;
|
||||
|
||||
if (!num_xcps || num_xcps > MAX_XCP)
|
||||
return -EINVAL;
|
||||
|
||||
xcp_mgr->mode = mode;
|
||||
|
||||
for (i = 0; i < MAX_XCP; ++i)
|
||||
xcp_mgr->xcp[i].valid = false;
|
||||
|
||||
|
@ -181,13 +186,6 @@ int amdgpu_xcp_switch_partition_mode(struct amdgpu_xcp_mgr *xcp_mgr, int mode)
|
|||
goto out;
|
||||
}
|
||||
|
||||
if (!num_xcps || num_xcps > MAX_XCP) {
|
||||
ret = -EINVAL;
|
||||
goto out;
|
||||
}
|
||||
|
||||
xcp_mgr->mode = mode;
|
||||
__amdgpu_xcp_init(xcp_mgr, num_xcps);
|
||||
out:
|
||||
mutex_unlock(&xcp_mgr->xcp_lock);
|
||||
|
||||
|
@ -240,7 +238,7 @@ int amdgpu_xcp_mgr_init(struct amdgpu_device *adev, int init_mode,
|
|||
mutex_init(&xcp_mgr->xcp_lock);
|
||||
|
||||
if (init_mode != AMDGPU_XCP_MODE_NONE)
|
||||
__amdgpu_xcp_init(xcp_mgr, init_num_xcps);
|
||||
amdgpu_xcp_init(xcp_mgr, init_num_xcps, init_mode);
|
||||
|
||||
adev->xcp_mgr = xcp_mgr;
|
||||
|
||||
|
|
|
@ -105,6 +105,7 @@ int amdgpu_xcp_resume(struct amdgpu_xcp_mgr *xcp_mgr, int xcp_id);
|
|||
|
||||
int amdgpu_xcp_mgr_init(struct amdgpu_device *adev, int init_mode,
|
||||
int init_xcps, struct amdgpu_xcp_mgr_funcs *xcp_funcs);
|
||||
int amdgpu_xcp_init(struct amdgpu_xcp_mgr *xcp_mgr, int num_xcps, int mode);
|
||||
int amdgpu_xcp_query_partition_mode(struct amdgpu_xcp_mgr *xcp_mgr, u32 flags);
|
||||
int amdgpu_xcp_switch_partition_mode(struct amdgpu_xcp_mgr *xcp_mgr, int mode);
|
||||
int amdgpu_xcp_get_partition(struct amdgpu_xcp_mgr *xcp_mgr,
|
||||
|
|
|
@ -321,9 +321,11 @@ static int aqua_vanjaram_switch_partition_mode(struct amdgpu_xcp_mgr *xcp_mgr,
|
|||
if (adev->nbio.funcs->set_compute_partition_mode)
|
||||
adev->nbio.funcs->set_compute_partition_mode(adev, mode);
|
||||
|
||||
ret = __aqua_vanjaram_post_partition_switch(xcp_mgr, flags);
|
||||
|
||||
/* Init info about new xcps */
|
||||
*num_xcps = num_xcc / num_xcc_per_xcp;
|
||||
amdgpu_xcp_init(xcp_mgr, *num_xcps, mode);
|
||||
|
||||
ret = __aqua_vanjaram_post_partition_switch(xcp_mgr, flags);
|
||||
unlock:
|
||||
if (flags & AMDGPU_XCP_OPS_KFD)
|
||||
amdgpu_amdkfd_unlock_kfd(adev);
|
||||
|
|
Loading…
Reference in New Issue