[Spice-devel] [PATCH qxl-wddm-dod v2 16/25] Remove minimum size restrict for custom resolution.

Frediano Ziglio fziglio at redhat.com
Fri Sep 9 11:41:02 UTC 2016


> On Wed, Sep 7, 2016 at 4:57 PM, Christophe Fergeau < cfergeau at redhat.com >
> wrote:

> > On Tue, Sep 06, 2016 at 05:29:22PM +0300, Sameeh Jubran wrote:
> 
> > > It is the recommended minimum, however Windows can handle low resolution
> 
> > > just fine. We have tested that,
> 
> > > even the old driver behaves like this. I see no reason for making any
> 
> > > restriction at all.
> 

> > Can you please add this kind of reasoning/explanation to the very terse
> 
> > commit log which is there currently ?
> 

> Sure

> > Thanks,
> 

> > Christophe
> 

This patch is just changing the "Add arbitrary resolution and monitors_config Escape" patch 
which is not in master, so would be better to squash there. 
Also the "Add arbitrary resolution and monitors_config Escape" already has a rationale for 
this patch that is: 

"Windows can handle low resolution just fine, the old (XPDM) 
driver behaves like this too. There is no reason for making 
any restriction at all." 

(actually this rationale apply the the wrong patch!) 

Frediano 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20160909/d2d3625a/attachment.html>


More information about the Spice-devel mailing list