<div dir="ltr">looks it's not that easy as I thought.<div>This change is for <a href="https://bugs.freedesktop.org/show_bug.cgi?id=102508">https://bugs.freedesktop.org/show_bug.cgi?id=102508</a> , and the hardware configuration is</div><div><div><br></div><div>* 00:02.0 VGA compatible controller [0300]: Intel Corporation Device [8086:591d] (rev 04)</div></div><div>* Dell Thunderbolt BME dock : <a href="http://www.dell.com/support/article/us/en/04/sln301105/dell-thunderbolt-dock--tb16--information-and-specifications?lang=en">http://www.dell.com/support/article/us/en/04/sln301105/dell-thunderbolt-dock--tb16--information-and-specifications?lang=en</a></div><div>* internal native monitor is 4K, another 2 4K monitors are plugged behind BME dock</div><div><br></div><div>This patch could let this setting work, but looks it not the correct way.</div><div>Not see the the relevant math in original code of commit <span style="color:rgb(0,0,0);white-space:pre-wrap">b321803dfb and ac484963f9 .</span></div><div>Does someone know how the original value 8192 be calculated out?</div><div>So that the way might could be followed for this new change.</div><div><br></div><div>Thanks.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 7, 2017 at 11:32 PM, Ville Syrjälä <span dir="ltr"><<a href="mailto:ville.syrjala@linux.intel.com" target="_blank">ville.syrjala@linux.intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, Dec 07, 2017 at 05:26:00PM +0800, Alex Tu wrote:<br>
> Rrefer to another patch [1] on mesa to extend width/height to 16384.<br>
> For issue :<br>
>  - <a href="https://bugs.freedesktop.org/show_bug.cgi?id=102508" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/<wbr>show_bug.cgi?id=102508</a><br>
>  - LP: #1714178 Triple monitor display failed with Dell Dock (HiDPI)<br>
><br>
> [1] <a href="https://patchwork.freedesktop.org/patch/124918/" rel="noreferrer" target="_blank">https://patchwork.freedesktop.<wbr>org/patch/124918/</a><br>
><br>
> Signed-off-by: Alex Tu <<a href="mailto:alex.tu@canonical.com">alex.tu@canonical.com</a>><br>
> ---<br>
>  drivers/gpu/drm/i915/intel_<wbr>display.c | 6 +++---<br>
>  1 file changed, 3 insertions(+), 3 deletions(-)<br>
><br>
> diff --git a/drivers/gpu/drm/i915/intel_<wbr>display.c b/drivers/gpu/drm/i915/intel_<wbr>display.c<br>
> index 47a2f6acee50..556fa57b18b8 100644<br>
> --- a/drivers/gpu/drm/i915/intel_<wbr>display.c<br>
> +++ b/drivers/gpu/drm/i915/intel_<wbr>display.c<br>
> @@ -13905,7 +13905,7 @@ u32 intel_fb_pitch_limit(struct drm_i915_private *dev_priv,<br>
>               /* "The stride in bytes must not exceed the of the size of 8K<br>
>                *  pixels and 32K bytes."<br>
<br>
</span>The spec queote clearly says the patch is wrong. Either that or the spec<br>
quote is outdated.<br>
<span class=""><br>
>                */<br>
> -             return min(8192 * cpp, 32768);<br>
> +             return min(16384 * cpp, 65536);<br>
>       } else if (gen >= 5 && !HAS_GMCH_DISPLAY(dev_priv)) {<br>
>               return 32*1024;<br>
>       } else if (gen >= 4) {<br>
> @@ -14604,8 +14604,8 @@ int intel_modeset_init(struct drm_device *dev)<br>
>               dev->mode_config.max_width = 4096;<br>
>               dev->mode_config.max_height = 4096;<br>
>       } else {<br>
> -             dev->mode_config.max_width = 8192;<br>
> -             dev->mode_config.max_height = 8192;<br>
> +             dev->mode_config.max_width = 16384;<br>
> +             dev->mode_config.max_height = 16384;<br>
<br>
</span>Even if this would be correct (which it's not for most platforms at<br>
least, not quite sure about the very latest hardware), anyone doing<br>
this should at least do a cursory check of the relevant math in the<br>
driver to make seure we don't end up with integer overflows.<br>
<span class="im HOEnZb"><br>
>       }<br>
><br>
>       if (IS_I845G(dev_priv) || IS_I865G(dev_priv)) {<br>
> --<br>
> 2.11.0<br>
><br>
</span><div class="HOEnZb"><div class="h5">> ______________________________<wbr>_________________<br>
> Intel-gfx mailing list<br>
> <a href="mailto:Intel-gfx@lists.freedesktop.org">Intel-gfx@lists.freedesktop.<wbr>org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/intel-gfx" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/intel-gfx</a><br>
<br>
--<br>
</div></div><span class="HOEnZb"><font color="#888888">Ville Syrjälä<br>
Intel OTC<br>
</font></span></blockquote></div><br></div>