Am 28.12.2016 um 03:59 schrieb Yu, Xiangliang: >> -----Original Message----- >> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf >> Of zhoucm1 >> Sent: Wednesday, December 28, 2016 10:32 AM >> To: Yu, Xiangliang <Xiangliang.Yu at amd.com>; amd- >> gfx at lists.freedesktop.org >> Cc: Liu, Monk <Monk.Liu at amd.com> >> Subject: Re: [V2 1/1] drm/amdgpu/gfx8: add support kernel interface >> queue(KIQ) >> >> >> >> On 2016å¹´12æ??28æ?¥ 10:27, Yu, Xiangliang wrote: >>> + >>>> + amdgpu_bo_kunmap(kiq->eop_obj); >>>> + >> I see it is in many places, which is don't need I think, >> amdgpu_bo_free_kernel will handle it if I'm correct. > Memory kmap should be free ASAP from memory management point. No, that is completely unnecessary as long as you don't run a 32bit system with tight address space. Regards, Christian. > >> Regards, >> David zhou >> _______________________________________________ >> amd-gfx mailing list >> amd-gfx at lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/amd-gfx > _______________________________________________ > amd-gfx mailing list > amd-gfx at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx