From: Igor Mammedov <imammedo@redhat.com>
To: qemu-devel@nongnu.org
Cc: qemu-arm@nongnu.org, peter.maydell@linaro.org, eric.auger@redhat.com
Subject: [Qemu-devel] [PATCH for-2.13 0/4] arm: isolate and clean up dtb generation
Date: Thu, 12 Apr 2018 18:40:17 +0200 [thread overview]
Message-ID: <1523551221-11612-1-git-send-email-imammedo@redhat.com> (raw)
While working on cpu hotplug for mach-virt, I've noticed that DTB
is generated incrementally across whole machvirt_init(). While it's
fine for machines with static DTB, it won't really work in case of
cpu hotplug and followed up reset since machine will load old DTB
that doesn't account for hotplugged CPUs.
So I've set on a quest to consolidate DTB generation and make it
reentrant so that on reset guest would see update DTB.
It's preliminary series which makes possible to call load_dtb()
later outside of arm_load_kernel() and in process of it drops
several machine_done notifiers, that were introduced to make
plaform-bus-devices work. Hopefully it makes code easier to follow.
It replaces machine_done notifiers with device hotplug framework
to allow for dynamic sysbus devices mapping at the moment they
are created instead of waiting for machine_done time and trying to
juggle with notifiers order to do initialization in propper order.
Mostly 'make check' tested +
manually with "ppce500" machine type and eTSEC device
(eTSEC is still initialized with the same IRQs as before series)
CC: qemu-arm@nongnu.org
CC: peter.maydell@linaro.org
CC: eric.auger@redhat.com
Igor Mammedov (4):
arm: reuse arm_boot_address_space() in armv7m_load_kernel()
platform-bus-device: use device plug callback instead of machine_done
notifier
arm: always start from first_cpu when registering loader cpu reset
callback
arm/boot: split load_dtb() from arm_load_kernel()
hw/ppc/e500.h | 3 ++
include/hw/arm/arm.h | 39 ++++++++++++-----
include/hw/arm/sysbus-fdt.h | 37 ++++------------
include/hw/arm/virt.h | 3 ++
include/hw/platform-bus.h | 4 +-
hw/arm/armv7m.c | 10 +----
hw/arm/boot.c | 85 +++++++++++--------------------------
hw/arm/sysbus-fdt.c | 67 +++--------------------------
hw/arm/virt.c | 100 +++++++++++++++++++++++++++++---------------
hw/core/platform-bus.c | 29 +++----------
hw/ppc/e500.c | 37 +++++++++-------
hw/ppc/e500plat.c | 60 +++++++++++++++++++++++++-
12 files changed, 227 insertions(+), 247 deletions(-)
--
2.7.4
next reply other threads:[~2018-04-12 16:41 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-12 16:40 Igor Mammedov [this message]
2018-04-12 16:40 ` [Qemu-devel] [PATCH for-2.13 1/4] arm: reuse arm_boot_address_space() in armv7m_load_kernel() Igor Mammedov
2018-04-12 18:22 ` Peter Maydell
2018-04-13 13:41 ` Igor Mammedov
2018-04-12 16:40 ` [Qemu-devel] [PATCH for-2.13 2/4] platform-bus-device: use device plug callback instead of machine_done notifier Igor Mammedov
2018-04-13 18:00 ` Auger Eric
2018-04-16 8:00 ` Igor Mammedov
2018-04-16 2:43 ` David Gibson
2018-04-16 8:19 ` Igor Mammedov
2018-04-17 1:15 ` David Gibson
2018-04-17 16:34 ` [Qemu-devel] [PATCH] ppc: e500: switch E500 based machines to full machine definition Igor Mammedov
2018-04-18 9:38 ` David Gibson
2018-04-16 17:25 ` [Qemu-devel] [PATCH for-2.13 2/4] platform-bus-device: use device plug callback instead of machine_done notifier Peter Maydell
2018-04-12 16:40 ` [Qemu-devel] [PATCH for-2.13 3/4] arm: always start from first_cpu when registering loader cpu reset callback Igor Mammedov
2018-04-12 18:29 ` [Qemu-devel] [Qemu-arm] " Peter Maydell
2018-04-13 13:59 ` Igor Mammedov
2018-04-16 17:17 ` Peter Maydell
2018-04-17 11:35 ` Igor Mammedov
2018-04-12 16:40 ` [Qemu-devel] [PATCH for-2.13 4/4] arm/boot: split load_dtb() from arm_load_kernel() Igor Mammedov
2018-04-16 17:34 ` Peter Maydell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1523551221-11612-1-git-send-email-imammedo@redhat.com \
--to=imammedo@redhat.com \
--cc=eric.auger@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).