Bug 1485793 - [RFE] Add default display resolution xres and yres for qxl and virtio gpu
Summary: [RFE] Add default display resolution xres and yres for qxl and virtio gpu
Keywords:
Status: POST
Alias: None
Product: Virtualization Tools
Classification: Community
Component: libvirt
Version: unspecified
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: yafu
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-08-28 06:13 UTC by Han Han
Modified: 2019-10-19 04:33 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:


Attachments (Terms of Use)

Description Han Han 2017-08-28 06:13:45 UTC
Description of problem:
This feature is from qemu:
virtio-gpu: add xres and yres properties:
https://lists.gnu.org/archive/html/qemu-devel/2017-03/msg01769.html

qxl: add xres and yres properties:
http://lists.gnu.org/archive/html/qemu-devel/2017-04/msg04158.html

After this feature realized, the xml should be like:
<video>
      <model type='qxl' ram='65536' vram='65536' vgamem='16384' heads='1' primary='yes' xres='' yres=''/>
      <address type='pci' domain='0x0000' bus='0x00' slot='0x02' function='0x0'/>
    </video>

Comment 2 Julio Faracco 2019-10-19 04:33:15 UTC
It was included into upstream by:

commit 7286279797a34b3083d85bc4556432b5e7ad9fff
Author:     Julio Faracco <jcfaracco@gmail.com>
AuthorDate: Thu Oct 17 01:30:31 2019 -0300
Commit:     Cole Robinson <crobinso@redhat.com>
CommitDate: Thu Oct 17 16:18:34 2019 -0400

    conf: Add 'x' and 'y' resolution into video XML definition
    
    This commit adds resolution element with parameters 'x' and 'y' into video
    XML domain group definition. Both, properties were added into an element
    called 'resolution' and it was added inside 'model' element. They are set
    as optional. This element does not follow QEMU properties 'xres' and
    'yres' format. Both HTML documentation and schema were changed too. This
    commit includes a simple test case to cover resolution for QEMU video
    models. The new XML format for resolution looks like:
    
        <model ...>
          <resolution x='800' y='600'/>
        </model>
    
    Reviewed-by: Cole Robinson <crobinso@redhat.com>
    Signed-off-by: Julio Faracco <jcfaracco@gmail.com>


Note You need to log in before you can comment on or make changes to this bug.