<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>
</head>
<body dir="ltr">
<div id="divtagdefaultwrapper" style="font-size: 12pt; color: rgb(0, 0, 0); font-family: Calibri, Helvetica, sans-serif, "EmojiFont", "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols;" dir="ltr">
<p style="margin-top:0;margin-bottom:0">I think you are getting lucky with the pageflip buffers always ending up in vram. Also, the legacy dce code (e.g., dce_v11_0.c) always pins in vram for modesets.<br>
</p>
<p style="margin-top:0;margin-bottom:0"><br>
</p>
<p style="margin-top:0;margin-bottom:0">Alex<br>
</p>
<p style="margin-top:0;margin-bottom:0"><br>
</p>
<div style="color: rgb(0, 0, 0);">
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font style="font-size:11pt" face="Calibri, sans-serif" color="#000000"><b>From:</b> Li, Samuel<br>
<b>Sent:</b> Friday, January 12, 2018 4:35 PM<br>
<b>To:</b> Koenig, Christian; Alex Deucher; amd-gfx@lists.freedesktop.org<br>
<b>Cc:</b> Deucher, Alexander<br>
<b>Subject:</b> RE: [PATCH] drm/amdgpu: only allow scatter/gather display with DC</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">Somehow my test case seems working although DC not enabled?<br>
</div>
<div class="PlainText"><br>
</div>
<div class="PlainText"><br>
Regards,<br>
Samuel Li<br>
<br>
<br>
> -----Original Message-----<br>
> From: amd-gfx [<a href="mailto:amd-gfx-bounces@lists.freedesktop.org">mailto:amd-gfx-bounces@lists.freedesktop.org</a>] On Behalf<br>
> Of Christian König<br>
> Sent: Friday, January 12, 2018 4:03 PM<br>
> To: Alex Deucher <alexdeucher@gmail.com>; amd-<br>
> gfx@lists.freedesktop.org<br>
> Cc: Deucher, Alexander <Alexander.Deucher@amd.com><br>
> Subject: Re: [PATCH] drm/amdgpu: only allow scatter/gather display with DC<br>
> <br>
> Am 12.01.2018 um 20:58 schrieb Alex Deucher:<br>
> > Check if DC is enabled before allowing scanout buffers to be pinned in<br>
> > system memory.<br>
> ><br>
> > Signed-off-by: Alex Deucher <alexander.deucher@amd.com><br>
> <br>
> Reviewed-by: Christian König <christian.koenig@amd.com><br>
> <br>
> > ---<br>
> > drivers/gpu/drm/amd/amdgpu/amdgpu_display.c | 5 ++++-<br>
> > 1 file changed, 4 insertions(+), 1 deletion(-)<br>
> ><br>
> > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c<br>
> > b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c<br>
> > index 8ede2645a06c..859942552e9f 100644<br>
> > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c<br>
> > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c<br>
> > @@ -507,9 +507,12 @@ uint32_t<br>
> amdgpu_display_framebuffer_domains(struct amdgpu_device *adev)<br>
> > {<br>
> > uint32_t domain = AMDGPU_GEM_DOMAIN_VRAM;<br>
> ><br>
> > +#if defined(CONFIG_DRM_AMD_DC)<br>
> > if (adev->asic_type >= CHIP_CARRIZO && adev->asic_type <<br>
> CHIP_RAVEN &&<br>
> > - adev->flags & AMD_IS_APU)<br>
> > + adev->flags & AMD_IS_APU &&<br>
> > + amdgpu_device_asic_has_dc_support(adev->asic_type))<br>
> > domain |= AMDGPU_GEM_DOMAIN_GTT;<br>
> > +#endif<br>
> ><br>
> > return domain;<br>
> > }<br>
> <br>
> _______________________________________________<br>
> amd-gfx mailing list<br>
> amd-gfx@lists.freedesktop.org<br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a><br>
</div>
</span></font></div>
</div>
</div>
</body>
</html>