[PATCH v2 0/9] user-mode: Prune build dependencies (part 3)

Philippe Mathieu-Daudé posted 9 patches 4 years, 1 month ago
Test docker-mingw@fedora passed
Test docker-quick@centos7 passed
Test checkpatch passed
Test FreeBSD passed
Test asan passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200316001111.31004-1-philmd@redhat.com
Maintainers: Markus Armbruster <armbru@redhat.com>, Michael Roth <mdroth@linux.vnet.ibm.com>, "Daniel P. Berrangé" <berrange@redhat.com>, Eduardo Habkost <ehabkost@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>
There is a newer version of this series
hw/core/qdev-prop-internal.h     |  28 ++
include/hw/qdev-properties.h     |   1 +
hw/core/qdev-properties-system.c | 602 ++++++++++++++++++++++++++++-
hw/core/qdev-properties.c        | 643 ++-----------------------------
hw/core/qdev-stubs.c             |  20 +
hw/core/Makefile.objs            |   3 +
qapi/Makefile.objs               |  37 +-
7 files changed, 714 insertions(+), 620 deletions(-)
create mode 100644 hw/core/qdev-prop-internal.h
create mode 100644 hw/core/qdev-stubs.c
[PATCH v2 0/9] user-mode: Prune build dependencies (part 3)
Posted by Philippe Mathieu-Daudé 4 years, 1 month ago
This is the second part of a series reducing user-mode
dependencies. By stripping out unused code, the build
and testing time is reduced (as is space used by objects).

Part 3:
- Extract code not related to user-mode from hw/core/qdev-properties.c
- Reduce user-mode QAPI generated files

Since v1:
- Addressed Richard and Paolo review comments

Patch missing review: #2 'Use qemu_strtoul() in set_pci_host_devaddr()'

v1: https://www.mail-archive.com/qemu-devel@nongnu.org/msg688486.html

full series available at:
repo: https://github.com/philmd/qemu/
branch: usermode_pruning-v2

Based-on: <20200316000348.29692-1-philmd@redhat.com>

Philippe Mathieu-Daudé (9):
  hw/core/qdev-properties: Use qemu_strtol() in set_mac() handler
  hw/core/qdev-properties: Use qemu_strtoul() in set_pci_host_devaddr()
  hw/core/qdev-properties: Fix code style
  hw/core/qdev-properties: Export enum-related functions
  hw/core/qdev-properties: Export qdev_prop_enum
  hw/core/qdev-properties: Export some integer-related functions
  hw/core/qdev-properties: Extract system-mode specific properties
  hw/core: Add qdev stub for user-mode
  qapi: Restrict code generated for user-mode

 hw/core/qdev-prop-internal.h     |  28 ++
 include/hw/qdev-properties.h     |   1 +
 hw/core/qdev-properties-system.c | 602 ++++++++++++++++++++++++++++-
 hw/core/qdev-properties.c        | 643 ++-----------------------------
 hw/core/qdev-stubs.c             |  20 +
 hw/core/Makefile.objs            |   3 +
 qapi/Makefile.objs               |  37 +-
 7 files changed, 714 insertions(+), 620 deletions(-)
 create mode 100644 hw/core/qdev-prop-internal.h
 create mode 100644 hw/core/qdev-stubs.c

-- 
2.21.1


Re: [PATCH v2 0/9] user-mode: Prune build dependencies (part 3)
Posted by Philippe Mathieu-Daudé 4 years, 1 month ago
Hi Eduardo,

On 3/16/20 1:11 AM, Philippe Mathieu-Daudé wrote:
> This is the second part of a series reducing user-mode
> dependencies. By stripping out unused code, the build
> and testing time is reduced (as is space used by objects).
> 
> Part 3:
> - Extract code not related to user-mode from hw/core/qdev-properties.c
> - Reduce user-mode QAPI generated files
> 
> Since v1:
> - Addressed Richard and Paolo review comments
> 
> Patch missing review: #2 'Use qemu_strtoul() in set_pci_host_devaddr()'
> 
> v1: https://www.mail-archive.com/qemu-devel@nongnu.org/msg688486.html
> 
> full series available at:
> repo: https://github.com/philmd/qemu/
> branch: usermode_pruning-v2
> 
> Based-on: <20200316000348.29692-1-philmd@redhat.com>
> 
> Philippe Mathieu-Daudé (9):
>    hw/core/qdev-properties: Use qemu_strtol() in set_mac() handler
>    hw/core/qdev-properties: Use qemu_strtoul() in set_pci_host_devaddr()
>    hw/core/qdev-properties: Fix code style
>    hw/core/qdev-properties: Export enum-related functions
>    hw/core/qdev-properties: Export qdev_prop_enum
>    hw/core/qdev-properties: Export some integer-related functions
>    hw/core/qdev-properties: Extract system-mode specific properties

If you have to respin "x86 and machine queue" please consider including 
reviewed patches 1-7 :)

>    hw/core: Add qdev stub for user-mode
>    qapi: Restrict code generated for user-mode
> 
>   hw/core/qdev-prop-internal.h     |  28 ++
>   include/hw/qdev-properties.h     |   1 +
>   hw/core/qdev-properties-system.c | 602 ++++++++++++++++++++++++++++-
>   hw/core/qdev-properties.c        | 643 ++-----------------------------
>   hw/core/qdev-stubs.c             |  20 +
>   hw/core/Makefile.objs            |   3 +
>   qapi/Makefile.objs               |  37 +-
>   7 files changed, 714 insertions(+), 620 deletions(-)
>   create mode 100644 hw/core/qdev-prop-internal.h
>   create mode 100644 hw/core/qdev-stubs.c
>