[Qemu-devel] [PATCH v2 0/4] Clean up the ppc configs

Thomas Huth posted 4 patches 7 years, 9 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1516104958-3881-1-git-send-email-thuth@redhat.com
Test checkpatch passed
Test docker-build@min-glib passed
Test docker-mingw@fedora passed
Test docker-quick@centos6 passed
Test ppc passed
Test s390x passed
default-configs/ppc-softmmu.mak   | 59 ++++++++++++++++++++-----------------
default-configs/ppc64-softmmu.mak | 61 +++++----------------------------------
hw/ppc/Makefile.objs              |  4 +--
qemu-doc.texi                     |  6 ++++
target/ppc/translate_init.c       |  5 ++++
5 files changed, 53 insertions(+), 82 deletions(-)
[Qemu-devel] [PATCH v2 0/4] Clean up the ppc configs
Posted by Thomas Huth 7 years, 9 months ago
ppc64-softmmu is a superset of ppc-softmmu which in turn is a superset
of ppcemb-softmmu. But since the config files are currently independent
from each other, we missed to define some CONFIG switches in the super-
sets: CONFIG_SUNGEM is missing from the ppc64-softmmu config, and the
CONFIG_IDE_SII3112 switch is only defined in ppcemb-softmmu.

This patch series fixes these problems, together with cleaning up
the config- and make-files for better usability. qemu-system-ppcemb
is finally marked as deprecated since there is hardly any use case
for this separate build nowadays anymore (and if somebody really
still needs page sizes < 4k, we should likely rather use the dynamic
approach that is already done on the ARM boards instead).

v2:
 - Added patch to disable the PPC4xx boards in the hw/ppc/Makefile
 - Added patch to deprecate qemu-system-ppcemb

Thomas Huth (4):
  default-configs/ppc64-softmmu: Include 32-bit configs instead of
    copying them
  default-configs/ppc-softmmu: Restructure the switches according to the
    machines
  hw/ppc/Makefile: Add a way to disable the PPC4xx boards
  ppc: Deprecate qemu-system-ppcemb

 default-configs/ppc-softmmu.mak   | 59 ++++++++++++++++++++-----------------
 default-configs/ppc64-softmmu.mak | 61 +++++----------------------------------
 hw/ppc/Makefile.objs              |  4 +--
 qemu-doc.texi                     |  6 ++++
 target/ppc/translate_init.c       |  5 ++++
 5 files changed, 53 insertions(+), 82 deletions(-)

-- 
1.8.3.1


Re: [Qemu-devel] [PATCH v2 0/4] Clean up the ppc configs
Posted by David Gibson 7 years, 9 months ago
On Tue, Jan 16, 2018 at 01:15:54PM +0100, Thomas Huth wrote:
> ppc64-softmmu is a superset of ppc-softmmu which in turn is a superset
> of ppcemb-softmmu. But since the config files are currently independent
> from each other, we missed to define some CONFIG switches in the super-
> sets: CONFIG_SUNGEM is missing from the ppc64-softmmu config, and the
> CONFIG_IDE_SII3112 switch is only defined in ppcemb-softmmu.
> 
> This patch series fixes these problems, together with cleaning up
> the config- and make-files for better usability. qemu-system-ppcemb
> is finally marked as deprecated since there is hardly any use case
> for this separate build nowadays anymore (and if somebody really
> still needs page sizes < 4k, we should likely rather use the dynamic
> approach that is already done on the ARM boards instead).

Applied to ppc-for-2.12, thanks.


> 
> v2:
>  - Added patch to disable the PPC4xx boards in the hw/ppc/Makefile
>  - Added patch to deprecate qemu-system-ppcemb
> 
> Thomas Huth (4):
>   default-configs/ppc64-softmmu: Include 32-bit configs instead of
>     copying them
>   default-configs/ppc-softmmu: Restructure the switches according to the
>     machines
>   hw/ppc/Makefile: Add a way to disable the PPC4xx boards
>   ppc: Deprecate qemu-system-ppcemb
> 
>  default-configs/ppc-softmmu.mak   | 59 ++++++++++++++++++++-----------------
>  default-configs/ppc64-softmmu.mak | 61 +++++----------------------------------
>  hw/ppc/Makefile.objs              |  4 +--
>  qemu-doc.texi                     |  6 ++++
>  target/ppc/translate_init.c       |  5 ++++
>  5 files changed, 53 insertions(+), 82 deletions(-)
> 

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson