<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> <style type="text/css" style="display:none;"></style> </head> <body dir="ltr"> <div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Arial,Helvetica,sans-serif;" dir="ltr"> <meta content="text/html; charset=UTF-8"> <div dir="ltr"> <div id="x_divtagdefaultwrapper" dir="ltr" style="font-size:12pt; color:#000000; font-family:Calibri,Arial,Helvetica,sans-serif"> Hi Christian, the CSA VA update should be correct, we pin it after it allocated per device during init stage, and we map it upon each process creates their vm page tables, since this CSA is pined, I assume there is no need to validate it prior to submission please correct me if I missed something BR Monk </div> <hr tabindex="-1" style="display:inline-block; width:98%"> <div id="x_divRplyFwdMsg" dir="ltr">发件人: amd-gfx <amd-gfx-bounces@lists.freedesktop.org> 代表 Christian König <deathsimple@vodafone.de> 发送时间: 2016年12月19日 19:17:38 收件人: Liu, Monk; amd-gfx@lists.freedesktop.org 主题: Re: 转发: [PATCH 02/23] drm/amdgpu: add kiq into compiling <div> </div> </div> </div> <div class="PlainText">Yeah, agree totally with Monk here. If you find that you need some KIQ helpers which are independent of the hardware generation you can put them into amdgpu_kiq.c. Monk, David or maybe even Michel and Nicolai could you guys take a closer look at the patches. I'm currently to busy with private issues, so I just briefly skimmed over them, but still found some issues. The VA update stuff seems incorrectly handled to me for example. You only update the page tables once and then assume that everything stays as it is. Regards, Christian. Am 19.12.2016 um 08:13 schrieb Liu, Monk: > Sorry, NAK for the KIQ implement patches. > > KIQ is just another compute queue, and each generation of compute queue initialization is different, so there is no need of a amdgpu_kiq.c file to cover the function of kiq, > > Kiq is good enough to be in each gfx_x_x.c file .e.g : > KIQ for VI is in gfx_v8_0.c > And KIQ for AI is in gfx_v9_0.c > > Most important is that the "queue_init" routine of above two chips on KIQ is different (MQD descriptor as well as registers need programing ... ), please just align with the previous patch style in amd-sriov-4.6 for KIQ part > > BR Monk > > -----邮件原件----- > 发件人: Yu, Xiangliang > 发送时间: Sunday, December 18, 2016 9:47 AM > 收件人: dl.SRDC_SW_GPUVirtualization > 主题: FW: [PATCH 02/23] drm/amdgpu: add kiq into compiling > > > > -----Original Message----- > From: Xiangliang Yu [<a href="mailto:Xiangliang.Yu@amd.com">mailto:Xiangliang.Yu@amd.com</a>] > Sent: Sunday, December 18, 2016 12:16 AM > To: amd-gfx@lists.freedesktop.org; dl.SRDC_SW_GPUVirtualization <dl.SRDC_SW_GPUVirtualization@amd.com> > Cc: Yu, Xiangliang <Xiangliang.Yu@amd.com> > Subject: [PATCH 02/23] drm/amdgpu: add kiq into compiling > > Add support KIQ in Makefile to make it into module. > > Signed-off-by: Xiangliang Yu <Xiangliang.Yu@amd.com> > --- > drivers/gpu/drm/amd/amdgpu/Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/Makefile b/drivers/gpu/drm/amd/amdgpu/Makefile > index dd0b6db..635ff94 100644 > --- a/drivers/gpu/drm/amd/amdgpu/Makefile > +++ b/drivers/gpu/drm/amd/amdgpu/Makefile > @@ -30,7 +30,7 @@ amdgpu-y += amdgpu_device.o amdgpu_kms.o \ > atombios_encoders.o amdgpu_sa.o atombios_i2c.o \ > amdgpu_prime.o amdgpu_vm.o amdgpu_ib.o amdgpu_pll.o \ > amdgpu_ucode.o amdgpu_bo_list.o amdgpu_ctx.o amdgpu_sync.o \ > - amdgpu_gtt_mgr.o amdgpu_vram_mgr.o > + amdgpu_gtt_mgr.o amdgpu_vram_mgr.o amdgpu_kiq.o > > # add asic specific block > amdgpu-$(CONFIG_DRM_AMDGPU_CIK)+= cik.o cik_ih.o kv_smc.o kv_dpm.o \ _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org <a href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a> </div> </div> </body> </html>