[libvirt] [PATCH v3 0/9] Enable ramfb parameter for mediated devices

Jonathon Jongsma posted 9 patches 4 years, 6 months ago
Failed in applying to current master (apply log)
docs/formatdomain.html.in                     |  8 ++
docs/schemas/domaincommon.rng                 |  5 ++
src/conf/domain_capabilities.c                | 17 +++-
src/conf/domain_conf.c                        | 11 +++
src/conf/domain_conf.h                        |  1 +
src/qemu/qemu_capabilities.c                  |  3 +
src/qemu/qemu_command.c                       | 17 +++-
src/qemu/qemu_domain.c                        | 61 ++++++-------
src/qemu/qemu_process.c                       | 61 -------------
.../qemu_1.7.0.x86_64.xml                     |  1 +
.../qemu_2.12.0-virt.aarch64.xml              |  1 +
.../qemu_2.12.0.ppc64.xml                     |  1 +
.../qemu_2.12.0.s390x.xml                     |  1 +
.../qemu_2.12.0.x86_64.xml                    |  1 +
.../qemu_2.6.0-virt.aarch64.xml               |  1 +
.../qemu_2.6.0.aarch64.xml                    |  1 +
.../domaincapsschemadata/qemu_2.6.0.ppc64.xml |  1 +
.../qemu_2.6.0.x86_64.xml                     |  1 +
.../domaincapsschemadata/qemu_2.7.0.s390x.xml |  1 +
.../qemu_2.8.0-tcg.x86_64.xml                 |  1 +
.../domaincapsschemadata/qemu_2.8.0.s390x.xml |  1 +
.../qemu_2.8.0.x86_64.xml                     |  1 +
.../qemu_2.9.0-q35.x86_64.xml                 |  1 +
.../qemu_2.9.0-tcg.x86_64.xml                 |  1 +
.../qemu_2.9.0.x86_64.xml                     |  1 +
.../domaincapsschemadata/qemu_3.0.0.s390x.xml |  2 +
.../qemu_3.1.0.x86_64.xml                     |  2 +
.../domaincapsschemadata/qemu_4.0.0.s390x.xml |  1 +
.../qemu_4.0.0.x86_64.xml                     |  2 +
.../qemu_4.1.0.x86_64.xml                     |  2 +
.../qemu_4.2.0.aarch64.xml                    |  2 +
.../domaincapsschemadata/qemu_4.2.0.ppc64.xml |  1 +
.../qemu_4.2.0.x86_64.xml                     |  2 +
tests/qemuhotplugtest.c                       |  3 +
...tdev-mdev-display-ramfb.x86_64-latest.args | 37 ++++++++
.../hostdev-mdev-display-ramfb.xml            | 33 +++++++
tests/qemuxml2argvtest.c                      |  2 +
tests/qemuxml2xmltest.c                       | 89 +++++++++++--------
tests/securityselinuxlabeltest.c              |  9 ++
39 files changed, 254 insertions(+), 133 deletions(-)
create mode 100644 tests/qemuxml2argvdata/hostdev-mdev-display-ramfb.x86_64-latest.args
create mode 100644 tests/qemuxml2argvdata/hostdev-mdev-display-ramfb.xml
[libvirt] [PATCH v3 0/9] Enable ramfb parameter for mediated devices
Posted by Jonathon Jongsma 4 years, 6 months ago
This is the third version of a patch series that adds support for a boot
display for mediated vgpu devices using the 'ramfb' parameter. This version
fixes a bunch of test failures that I had inadvertently introduced in the last
series. It also splits the 4th patch up into several separate patches as
suggested by Cole to make things more readable.

Jonathon Jongsma (9):
  qemu: fix domain device validation
  qemu: use g_autoptr in qemuDomainDeviceDefValidate()
  qemu: Set capabilities properly for tests
  qemu: set domain capability for ramfb device
  qemu: set domain capability for video type "none"
  qemu: validate vhost-user video backend in qemu_domain.c
  qemu: move validation of video accel to qemu_domain.c
  qemu: use domain caps to validate video device model
  qemu: add 'ramfb' attribute for mediated devices

 docs/formatdomain.html.in                     |  8 ++
 docs/schemas/domaincommon.rng                 |  5 ++
 src/conf/domain_capabilities.c                | 17 +++-
 src/conf/domain_conf.c                        | 11 +++
 src/conf/domain_conf.h                        |  1 +
 src/qemu/qemu_capabilities.c                  |  3 +
 src/qemu/qemu_command.c                       | 17 +++-
 src/qemu/qemu_domain.c                        | 61 ++++++-------
 src/qemu/qemu_process.c                       | 61 -------------
 .../qemu_1.7.0.x86_64.xml                     |  1 +
 .../qemu_2.12.0-virt.aarch64.xml              |  1 +
 .../qemu_2.12.0.ppc64.xml                     |  1 +
 .../qemu_2.12.0.s390x.xml                     |  1 +
 .../qemu_2.12.0.x86_64.xml                    |  1 +
 .../qemu_2.6.0-virt.aarch64.xml               |  1 +
 .../qemu_2.6.0.aarch64.xml                    |  1 +
 .../domaincapsschemadata/qemu_2.6.0.ppc64.xml |  1 +
 .../qemu_2.6.0.x86_64.xml                     |  1 +
 .../domaincapsschemadata/qemu_2.7.0.s390x.xml |  1 +
 .../qemu_2.8.0-tcg.x86_64.xml                 |  1 +
 .../domaincapsschemadata/qemu_2.8.0.s390x.xml |  1 +
 .../qemu_2.8.0.x86_64.xml                     |  1 +
 .../qemu_2.9.0-q35.x86_64.xml                 |  1 +
 .../qemu_2.9.0-tcg.x86_64.xml                 |  1 +
 .../qemu_2.9.0.x86_64.xml                     |  1 +
 .../domaincapsschemadata/qemu_3.0.0.s390x.xml |  2 +
 .../qemu_3.1.0.x86_64.xml                     |  2 +
 .../domaincapsschemadata/qemu_4.0.0.s390x.xml |  1 +
 .../qemu_4.0.0.x86_64.xml                     |  2 +
 .../qemu_4.1.0.x86_64.xml                     |  2 +
 .../qemu_4.2.0.aarch64.xml                    |  2 +
 .../domaincapsschemadata/qemu_4.2.0.ppc64.xml |  1 +
 .../qemu_4.2.0.x86_64.xml                     |  2 +
 tests/qemuhotplugtest.c                       |  3 +
 ...tdev-mdev-display-ramfb.x86_64-latest.args | 37 ++++++++
 .../hostdev-mdev-display-ramfb.xml            | 33 +++++++
 tests/qemuxml2argvtest.c                      |  2 +
 tests/qemuxml2xmltest.c                       | 89 +++++++++++--------
 tests/securityselinuxlabeltest.c              |  9 ++
 39 files changed, 254 insertions(+), 133 deletions(-)
 create mode 100644 tests/qemuxml2argvdata/hostdev-mdev-display-ramfb.x86_64-latest.args
 create mode 100644 tests/qemuxml2argvdata/hostdev-mdev-display-ramfb.xml

-- 
2.21.0

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH v3 0/9] Enable ramfb parameter for mediated devices
Posted by Cole Robinson 4 years, 5 months ago
On 10/18/19 11:30 AM, Jonathon Jongsma wrote:
> This is the third version of a patch series that adds support for a boot
> display for mediated vgpu devices using the 'ramfb' parameter. This version
> fixes a bunch of test failures that I had inadvertently introduced in the last
> series. It also splits the 4th patch up into several separate patches as
> suggested by Cole to make things more readable.
> 
> Jonathon Jongsma (9):
>   qemu: fix domain device validation
>   qemu: use g_autoptr in qemuDomainDeviceDefValidate()
>   qemu: Set capabilities properly for tests
>   qemu: set domain capability for ramfb device
>   qemu: set domain capability for video type "none"
>   qemu: validate vhost-user video backend in qemu_domain.c
>   qemu: move validation of video accel to qemu_domain.c
>   qemu: use domain caps to validate video device model
>   qemu: add 'ramfb' attribute for mediated devices
> 

Pushed with some fixups:

* rebased and resolved conflicts
* fixed function spacing in the last two patches, to preserve existing 2
line spacing (this was a bit ambiguous in the second to last patch)
* adjusted docs version in the last patch to mention 5.10.0

Thanks,
Cole

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH v3 0/9] Enable ramfb parameter for mediated devices
Posted by Cole Robinson 4 years, 5 months ago
On 11/14/19 11:40 AM, Cole Robinson wrote:
> On 10/18/19 11:30 AM, Jonathon Jongsma wrote:
>> This is the third version of a patch series that adds support for a boot
>> display for mediated vgpu devices using the 'ramfb' parameter. This version
>> fixes a bunch of test failures that I had inadvertently introduced in the last
>> series. It also splits the 4th patch up into several separate patches as
>> suggested by Cole to make things more readable.
>>
>> Jonathon Jongsma (9):
>>   qemu: fix domain device validation
>>   qemu: use g_autoptr in qemuDomainDeviceDefValidate()
>>   qemu: Set capabilities properly for tests
>>   qemu: set domain capability for ramfb device
>>   qemu: set domain capability for video type "none"
>>   qemu: validate vhost-user video backend in qemu_domain.c
>>   qemu: move validation of video accel to qemu_domain.c
>>   qemu: use domain caps to validate video device model
>>   qemu: add 'ramfb' attribute for mediated devices
>>
> 
> Pushed with some fixups:
> 
> * rebased and resolved conflicts
> * fixed function spacing in the last two patches, to preserve existing 2
> line spacing (this was a bit ambiguous in the second to last patch)
> * adjusted docs version in the last patch to mention 5.10.0

Also, ramfb addition warrants a news.xml update too

Thanks,
Cole

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list