From: Shameer Kolothum <shameerali.kolothum.thodi@huawei.com>
To: <qemu-devel@nongnu.org>, <qemu-arm@nongnu.org>,
<eric.auger@redhat.com>, <imammedo@redhat.com>
Cc: peter.maydell@linaro.org, xiaoguangrong.eric@gmail.com,
david@redhat.com, mst@redhat.com, linuxarm@huawei.com,
xuwei5@hisilicon.com, shannon.zhaosl@gmail.com,
lersek@redhat.com
Subject: [PATCH v3 00/10] ARM virt: Add NVDIMM support
Date: Wed, 11 Mar 2020 17:20:04 +0000 [thread overview]
Message-ID: <20200311172014.33052-1-shameerali.kolothum.thodi@huawei.com> (raw)
This series adds NVDIMM support to arm/virt platform.
The series reuses some of the patches posted by Eric
in his earlier attempt here[1].
This also include few fixes to qemu in general which were
discovered while adding nvdimm support to arm/virt.
Patch #2 addresses the issue[2] that, during migration, the
source and destination might end up with an inconsistency
in acpi table memory region sizes.
Patch #3 is to fix the qemu_ram_resize() callback issue[2].
Patch #4 is another fix to the nvdimm aml issue discussed
here[3].
I have done a basic sanity testing of NVDIMM devices
with Guest booting with ACPI. Further testing is always
welcome.
Please let me know your feedback.
Thanks,
Shameer
[1] https://patchwork.kernel.org/cover/10830777/
[2] https://patchwork.kernel.org/patch/11339591/
[3] https://patchwork.kernel.org/cover/11174959/
v2 --> v3
- Added patch #1 and # 2 to fix the inconsistency in acpi
table memory region sizes during migration. Thanks to
David H.
- The fix for qemu_ram_resize() callback was modified to
the one in patch #3. Again thanks to David H.
- Addressed comments from MST and Eric on tests added.
- Addressed comments from Igor/MST on Integer size in patch #4
- Added Eric's R-by to patch #7.
v1 --> v2
-Reworked patch #1 and now fix is inside qemu_ram_resize().
-Added patch #2 to fix the nvdim aml issue.
-Dropped support to DT cold plug.
-Updated test_acpi_virt_tcg_memhp() with pc-dimm and nvdimms(patch #7)
David Hildenbrand (1):
exec: Fix for qemu_ram_resize() callback
Kwangwoo Lee (2):
nvdimm: Use configurable ACPI IO base and size
hw/arm/virt: Add nvdimm hot-plug infrastructure
Shameer Kolothum (7):
acpi: Use macro for table-loader file name
fw_cfg: Migrate ACPI table mr sizes separately
hw/acpi/nvdimm: Fix for NVDIMM incorrect DSM output buffer length
hw/arm/virt: Add nvdimm hotplug support
tests: Update ACPI tables list for upcoming arm/virt test changes
tests/bios-tables-test: Update arm/virt memhp test
tests/acpi: add expected tables for bios-tables-test
docs/specs/acpi_hw_reduced_hotplug.rst | 1 +
exec.c | 14 +++-
hw/acpi/generic_event_device.c | 15 ++++-
hw/acpi/nvdimm.c | 72 +++++++++++++++++----
hw/arm/Kconfig | 1 +
hw/arm/virt-acpi-build.c | 8 ++-
hw/arm/virt.c | 35 ++++++++--
hw/core/machine.c | 1 +
hw/i386/acpi-build.c | 8 ++-
hw/i386/acpi-build.h | 3 +
hw/i386/pc_piix.c | 2 +
hw/i386/pc_q35.c | 2 +
hw/mem/Kconfig | 2 +-
hw/nvram/fw_cfg.c | 86 ++++++++++++++++++++++++-
include/hw/acpi/aml-build.h | 1 +
include/hw/acpi/generic_event_device.h | 1 +
include/hw/arm/virt.h | 1 +
include/hw/mem/nvdimm.h | 3 +
include/hw/nvram/fw_cfg.h | 6 ++
tests/data/acpi/pc/SSDT.dimmpxm | Bin 685 -> 734 bytes
tests/data/acpi/q35/SSDT.dimmpxm | Bin 685 -> 734 bytes
tests/data/acpi/virt/DSDT.memhp | Bin 6644 -> 6668 bytes
tests/data/acpi/virt/NFIT.memhp | Bin 0 -> 224 bytes
tests/data/acpi/virt/SSDT.memhp | Bin 0 -> 736 bytes
tests/qtest/bios-tables-test.c | 9 ++-
25 files changed, 244 insertions(+), 27 deletions(-)
create mode 100644 tests/data/acpi/virt/NFIT.memhp
create mode 100644 tests/data/acpi/virt/SSDT.memhp
--
2.17.1
next reply other threads:[~2020-03-11 17:26 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-11 17:20 Shameer Kolothum [this message]
2020-03-11 17:20 ` [PATCH v3 01/10] acpi: Use macro for table-loader file name Shameer Kolothum
2020-03-23 12:23 ` Igor Mammedov
2020-03-11 17:20 ` [PATCH v3 02/10] fw_cfg: Migrate ACPI table mr sizes separately Shameer Kolothum
2020-03-11 17:48 ` David Hildenbrand
2020-03-11 20:43 ` Michael S. Tsirkin
2020-03-11 21:09 ` Michael S. Tsirkin
2020-03-12 9:27 ` Shameerali Kolothum Thodi
2020-03-19 17:51 ` Michael S. Tsirkin
2020-03-20 11:53 ` Shameerali Kolothum Thodi
2020-03-23 12:34 ` Igor Mammedov
2020-03-23 13:59 ` Shameerali Kolothum Thodi
2020-03-11 17:20 ` [PATCH v3 03/10] exec: Fix for qemu_ram_resize() callback Shameer Kolothum
2020-03-11 17:44 ` David Hildenbrand
2020-03-23 13:03 ` Igor Mammedov
2020-03-11 17:20 ` [PATCH v3 04/10] hw/acpi/nvdimm: Fix for NVDIMM incorrect DSM output buffer length Shameer Kolothum
2020-03-23 14:59 ` Igor Mammedov
2020-03-11 17:20 ` [PATCH v3 05/10] nvdimm: Use configurable ACPI IO base and size Shameer Kolothum
2020-03-23 15:14 ` Igor Mammedov
2020-03-11 17:20 ` [PATCH v3 06/10] hw/arm/virt: Add nvdimm hot-plug infrastructure Shameer Kolothum
2020-03-23 15:22 ` Igor Mammedov
2020-03-11 17:20 ` [PATCH v3 07/10] hw/arm/virt: Add nvdimm hotplug support Shameer Kolothum
2020-03-24 6:16 ` Shannon Zhao
2020-03-11 17:20 ` [PATCH v3 08/10] tests: Update ACPI tables list for upcoming arm/virt test changes Shameer Kolothum
2020-03-11 17:20 ` [PATCH v3 09/10] tests/bios-tables-test: Update arm/virt memhp test Shameer Kolothum
2020-03-23 15:28 ` Igor Mammedov
2020-03-11 17:20 ` [PATCH v3 10/10] tests/acpi: add expected tables for bios-tables-test Shameer Kolothum
2020-03-11 19:30 ` [PATCH v3 00/10] ARM virt: Add NVDIMM support no-reply
2020-03-11 19:32 ` no-reply
2020-03-29 10:45 ` Michael S. Tsirkin
2020-03-30 8:44 ` Shameerali Kolothum Thodi
2020-03-30 8:46 ` David Hildenbrand
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=20200311172014.33052-1-shameerali.kolothum.thodi@huawei.com \
--to=shameerali.kolothum.thodi@huawei.com \
--cc=david@redhat.com \
--cc=eric.auger@redhat.com \
--cc=imammedo@redhat.com \
--cc=lersek@redhat.com \
--cc=linuxarm@huawei.com \
--cc=mst@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=shannon.zhaosl@gmail.com \
--cc=xiaoguangrong.eric@gmail.com \
--cc=xuwei5@hisilicon.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.