[libvirt] [PATCH v1 0/3] Loadparm support

Farhan Ali posted 3 patches 6 years, 11 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1494442869.git.alifm@linux.vnet.ibm.com
There is a newer version of this series
docs/formatdomain.html.in                          |  8 ++-
docs/news.xml                                      |  9 +++
docs/schemas/domaincommon.rng                      |  7 +++
src/conf/device_conf.h                             |  1 +
src/conf/domain_conf.c                             | 69 +++++++++++++++++++++-
src/qemu/qemu_capabilities.c                       |  3 +
src/qemu/qemu_capabilities.h                       |  3 +
src/qemu/qemu_command.c                            | 37 ++++++++++++
...-machine-loadparm-multiple-disks-nets-s390.args | 28 +++++++++
...v-machine-loadparm-multiple-disks-nets-s390.xml | 43 ++++++++++++++
.../qemuxml2argv-machine-loadparm-net-s390.args    | 20 +++++++
.../qemuxml2argv-machine-loadparm-net-s390.xml     | 26 ++++++++
...xml2argv-machine-loadparm-s390-char-invalid.xml | 26 ++++++++
...uxml2argv-machine-loadparm-s390-len-invalid.xml | 26 ++++++++
.../qemuxml2argv-machine-loadparm-s390.args        | 20 +++++++
.../qemuxml2argv-machine-loadparm-s390.xml         | 26 ++++++++
tests/qemuxml2argvtest.c                           | 19 ++++++
17 files changed, 367 insertions(+), 4 deletions(-)
create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-multiple-disks-nets-s390.args
create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-multiple-disks-nets-s390.xml
create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-net-s390.args
create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-net-s390.xml
create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390-char-invalid.xml
create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390-len-invalid.xml
create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390.args
create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390.xml
[libvirt] [PATCH v1 0/3] Loadparm support
Posted by Farhan Ali 6 years, 11 months ago
This patch series introduces the support for new s390x 'loadparm'
feature. The 'loadparm' can be used to select the boot entry to
boot from, for a boot device.

Here is a link to the QEMU patches:
https://lists.nongnu.org/archive/html/qemu-devel/2017-05/msg00192.html

Thanks
Farhan Ali


Farhan Ali (3):
  conf : Add loadparm boot option for a boot device
  qemu : Add loadparm to qemu command line string
  tests : Testcases for loadparm

 docs/formatdomain.html.in                          |  8 ++-
 docs/news.xml                                      |  9 +++
 docs/schemas/domaincommon.rng                      |  7 +++
 src/conf/device_conf.h                             |  1 +
 src/conf/domain_conf.c                             | 69 +++++++++++++++++++++-
 src/qemu/qemu_capabilities.c                       |  3 +
 src/qemu/qemu_capabilities.h                       |  3 +
 src/qemu/qemu_command.c                            | 37 ++++++++++++
 ...-machine-loadparm-multiple-disks-nets-s390.args | 28 +++++++++
 ...v-machine-loadparm-multiple-disks-nets-s390.xml | 43 ++++++++++++++
 .../qemuxml2argv-machine-loadparm-net-s390.args    | 20 +++++++
 .../qemuxml2argv-machine-loadparm-net-s390.xml     | 26 ++++++++
 ...xml2argv-machine-loadparm-s390-char-invalid.xml | 26 ++++++++
 ...uxml2argv-machine-loadparm-s390-len-invalid.xml | 26 ++++++++
 .../qemuxml2argv-machine-loadparm-s390.args        | 20 +++++++
 .../qemuxml2argv-machine-loadparm-s390.xml         | 26 ++++++++
 tests/qemuxml2argvtest.c                           | 19 ++++++
 17 files changed, 367 insertions(+), 4 deletions(-)
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-multiple-disks-nets-s390.args
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-multiple-disks-nets-s390.xml
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-net-s390.args
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-net-s390.xml
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390-char-invalid.xml
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390-len-invalid.xml
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390.args
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390.xml

-- 
1.9.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH v1 0/3] Loadparm support
Posted by Farhan Ali 6 years, 11 months ago
Polite ping :)

On 05/11/2017 09:27 AM, Farhan Ali wrote:
> This patch series introduces the support for new s390x 'loadparm'
> feature. The 'loadparm' can be used to select the boot entry to
> boot from, for a boot device.
>
> Here is a link to the QEMU patches:
> https://lists.nongnu.org/archive/html/qemu-devel/2017-05/msg00192.html
>
> Thanks
> Farhan Ali
>
>
> Farhan Ali (3):
>   conf : Add loadparm boot option for a boot device
>   qemu : Add loadparm to qemu command line string
>   tests : Testcases for loadparm
>
>  docs/formatdomain.html.in                          |  8 ++-
>  docs/news.xml                                      |  9 +++
>  docs/schemas/domaincommon.rng                      |  7 +++
>  src/conf/device_conf.h                             |  1 +
>  src/conf/domain_conf.c                             | 69 +++++++++++++++++++++-
>  src/qemu/qemu_capabilities.c                       |  3 +
>  src/qemu/qemu_capabilities.h                       |  3 +
>  src/qemu/qemu_command.c                            | 37 ++++++++++++
>  ...-machine-loadparm-multiple-disks-nets-s390.args | 28 +++++++++
>  ...v-machine-loadparm-multiple-disks-nets-s390.xml | 43 ++++++++++++++
>  .../qemuxml2argv-machine-loadparm-net-s390.args    | 20 +++++++
>  .../qemuxml2argv-machine-loadparm-net-s390.xml     | 26 ++++++++
>  ...xml2argv-machine-loadparm-s390-char-invalid.xml | 26 ++++++++
>  ...uxml2argv-machine-loadparm-s390-len-invalid.xml | 26 ++++++++
>  .../qemuxml2argv-machine-loadparm-s390.args        | 20 +++++++
>  .../qemuxml2argv-machine-loadparm-s390.xml         | 26 ++++++++
>  tests/qemuxml2argvtest.c                           | 19 ++++++
>  17 files changed, 367 insertions(+), 4 deletions(-)
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-multiple-disks-nets-s390.args
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-multiple-disks-nets-s390.xml
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-net-s390.args
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-net-s390.xml
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390-char-invalid.xml
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390-len-invalid.xml
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390.args
>  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390.xml
>

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH v1 0/3] Loadparm support
Posted by John Ferlan 6 years, 11 months ago

On 05/17/2017 09:48 AM, Farhan Ali wrote:
> Polite ping :)
> 

Could you please update to top of tree and repost... I know for sure
qemu_capabilities have changed.

In general things look good to me though...

Thanks!

John

> On 05/11/2017 09:27 AM, Farhan Ali wrote:
>> This patch series introduces the support for new s390x 'loadparm'
>> feature. The 'loadparm' can be used to select the boot entry to
>> boot from, for a boot device.
>>
>> Here is a link to the QEMU patches:
>> https://lists.nongnu.org/archive/html/qemu-devel/2017-05/msg00192.html
>>
>> Thanks
>> Farhan Ali
>>
>>
>> Farhan Ali (3):
>>   conf : Add loadparm boot option for a boot device
>>   qemu : Add loadparm to qemu command line string
>>   tests : Testcases for loadparm
>>
>>  docs/formatdomain.html.in                          |  8 ++-
>>  docs/news.xml                                      |  9 +++
>>  docs/schemas/domaincommon.rng                      |  7 +++
>>  src/conf/device_conf.h                             |  1 +
>>  src/conf/domain_conf.c                             | 69
>> +++++++++++++++++++++-
>>  src/qemu/qemu_capabilities.c                       |  3 +
>>  src/qemu/qemu_capabilities.h                       |  3 +
>>  src/qemu/qemu_command.c                            | 37 ++++++++++++
>>  ...-machine-loadparm-multiple-disks-nets-s390.args | 28 +++++++++
>>  ...v-machine-loadparm-multiple-disks-nets-s390.xml | 43 ++++++++++++++
>>  .../qemuxml2argv-machine-loadparm-net-s390.args    | 20 +++++++
>>  .../qemuxml2argv-machine-loadparm-net-s390.xml     | 26 ++++++++
>>  ...xml2argv-machine-loadparm-s390-char-invalid.xml | 26 ++++++++
>>  ...uxml2argv-machine-loadparm-s390-len-invalid.xml | 26 ++++++++
>>  .../qemuxml2argv-machine-loadparm-s390.args        | 20 +++++++
>>  .../qemuxml2argv-machine-loadparm-s390.xml         | 26 ++++++++
>>  tests/qemuxml2argvtest.c                           | 19 ++++++
>>  17 files changed, 367 insertions(+), 4 deletions(-)
>>  create mode 100644
>> tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-multiple-disks-nets-s390.args
>>
>>  create mode 100644
>> tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-multiple-disks-nets-s390.xml
>>
>>  create mode 100644
>> tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-net-s390.args
>>  create mode 100644
>> tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-net-s390.xml
>>  create mode 100644
>> tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390-char-invalid.xml
>>
>>  create mode 100644
>> tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390-len-invalid.xml
>>  create mode 100644
>> tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390.args
>>  create mode 100644
>> tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390.xml
>>
> 
> -- 
> libvir-list mailing list
> libvir-list@redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH v1 0/3] Loadparm support
Posted by Farhan Ali 6 years, 11 months ago

On 05/17/2017 05:59 PM, John Ferlan wrote:
>
>
> On 05/17/2017 09:48 AM, Farhan Ali wrote:
>> Polite ping :)
>>
>
> Could you please update to top of tree and repost... I know for sure
> qemu_capabilities have changed.
>
> In general things look good to me though...
>
> Thanks!
>
> John

Hi John,

Thanks for taking a look, I have submitted a v2 to the mailing list.
Would appreciate it if you could take a look.

Thanks so much
Farhan

>
>> On 05/11/2017 09:27 AM, Farhan Ali wrote:
>>> This patch series introduces the support for new s390x 'loadparm'
>>> feature. The 'loadparm' can be used to select the boot entry to
>>> boot from, for a boot device.
>>>
>>> Here is a link to the QEMU patches:
>>> https://lists.nongnu.org/archive/html/qemu-devel/2017-05/msg00192.html
>>>
>>> Thanks
>>> Farhan Ali
>>>
>>>
>>> Farhan Ali (3):
>>>   conf : Add loadparm boot option for a boot device
>>>   qemu : Add loadparm to qemu command line string
>>>   tests : Testcases for loadparm
>>>
>>>  docs/formatdomain.html.in                          |  8 ++-
>>>  docs/news.xml                                      |  9 +++
>>>  docs/schemas/domaincommon.rng                      |  7 +++
>>>  src/conf/device_conf.h                             |  1 +
>>>  src/conf/domain_conf.c                             | 69
>>> +++++++++++++++++++++-
>>>  src/qemu/qemu_capabilities.c                       |  3 +
>>>  src/qemu/qemu_capabilities.h                       |  3 +
>>>  src/qemu/qemu_command.c                            | 37 ++++++++++++
>>>  ...-machine-loadparm-multiple-disks-nets-s390.args | 28 +++++++++
>>>  ...v-machine-loadparm-multiple-disks-nets-s390.xml | 43 ++++++++++++++
>>>  .../qemuxml2argv-machine-loadparm-net-s390.args    | 20 +++++++
>>>  .../qemuxml2argv-machine-loadparm-net-s390.xml     | 26 ++++++++
>>>  ...xml2argv-machine-loadparm-s390-char-invalid.xml | 26 ++++++++
>>>  ...uxml2argv-machine-loadparm-s390-len-invalid.xml | 26 ++++++++
>>>  .../qemuxml2argv-machine-loadparm-s390.args        | 20 +++++++
>>>  .../qemuxml2argv-machine-loadparm-s390.xml         | 26 ++++++++
>>>  tests/qemuxml2argvtest.c                           | 19 ++++++
>>>  17 files changed, 367 insertions(+), 4 deletions(-)
>>>  create mode 100644
>>> tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-multiple-disks-nets-s390.args
>>>
>>>  create mode 100644
>>> tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-multiple-disks-nets-s390.xml
>>>
>>>  create mode 100644
>>> tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-net-s390.args
>>>  create mode 100644
>>> tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-net-s390.xml
>>>  create mode 100644
>>> tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390-char-invalid.xml
>>>
>>>  create mode 100644
>>> tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390-len-invalid.xml
>>>  create mode 100644
>>> tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390.args
>>>  create mode 100644
>>> tests/qemuxml2argvdata/qemuxml2argv-machine-loadparm-s390.xml
>>>
>>
>> --
>> libvir-list mailing list
>> libvir-list@redhat.com
>> https://www.redhat.com/mailman/listinfo/libvir-list
>

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