| Message ID | 20190321104745.28068-1-shameerali.kolothum.thodi@huawei.com |
|---|---|
| Headers | show |
| Series | ARM virt: ACPI memory hotplug support | expand |
Patchew URL: https://patchew.org/QEMU/20190321104745.28068-1-shameerali.kolothum.thodi@huawei.com/ Hi, This series seems to have some coding style problems. See output below for more information: Message-id: 20190321104745.28068-1-shameerali.kolothum.thodi@huawei.com Subject: [Qemu-devel] [PATCH v3 00/10] ARM virt: ACPI memory hotplug support Type: series === TEST SCRIPT BEGIN === #!/bin/bash git rev-parse base > /dev/null || exit 0 git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram ./scripts/checkpatch.pl --mailback base.. === TEST SCRIPT END === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 From https://github.com/patchew-project/qemu 62a172e6a7..6532dcebb6 master -> master t [tag update] patchew/1553090847-11300-1-git-send-email-lizhengui@huawei.com -> patchew/1553090847-11300-1-git-send-email-lizhengui@huawei.com t [tag update] patchew/20190313162812.8885-1-armbru@redhat.com -> patchew/20190313162812.8885-1-armbru@redhat.com t [tag update] patchew/20190314131049.23175-1-marcandre.lureau@redhat.com -> patchew/20190314131049.23175-1-marcandre.lureau@redhat.com t [tag update] patchew/20190319163551.32499-1-armbru@redhat.com -> patchew/20190319163551.32499-1-armbru@redhat.com * [new tag] patchew/20190321085212.10796-1-lvivier@redhat.com -> patchew/20190321085212.10796-1-lvivier@redhat.com * [new tag] patchew/20190321094012.36541-1-sgarzare@redhat.com -> patchew/20190321094012.36541-1-sgarzare@redhat.com * [new tag] patchew/20190321104745.28068-1-shameerali.kolothum.thodi@huawei.com -> patchew/20190321104745.28068-1-shameerali.kolothum.thodi@huawei.com Switched to a new branch 'test' 125f46770b hw/arm/virt: Init GED device and enable memory hotplug 8b3a542021 hw/acpi: Add ACPI Generic Event Device Support bbb60b29a5 hw/arm/boot: Expose the PC-DIMM nodes in the DT 8617e42573 hw/arm/virt: Introduce opt-in feature "fdt" 9cd5da671c hw/arm/virt-acpi-build: Add PC-DIMM in SRAT b25387b9d3 hw/arm/virt: Add ACPI support for device memory cold-plug ab6d232c4d hw/arm/virt: Add memory hotplug framework ea71a92426 hw/arm/virt: Add virtual ACPI device f093e1aaf2 hw/acpi: Do not create memory hotplug method when handler is not defined 37eb6a4ab4 hw/acpi: Make ACPI IO address space configurable === OUTPUT BEGIN === 1/10 Checking commit 37eb6a4ab493 (hw/acpi: Make ACPI IO address space configurable) 2/10 Checking commit f093e1aaf21d (hw/acpi: Do not create memory hotplug method when handler is not defined) WARNING: line over 80 characters #31: FILE: hw/acpi/memory_hotplug.c:726: + aml_call0(MEMORY_DEVICES_CONTAINER "." MEMORY_SLOT_SCAN_METHOD)); total: 0 errors, 1 warnings, 16 lines checked Patch 2/10 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 3/10 Checking commit ea71a92426cc (hw/arm/virt: Add virtual ACPI device) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #42: new file mode 100644 total: 0 errors, 1 warnings, 115 lines checked Patch 3/10 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 4/10 Checking commit ab6d232c4d16 (hw/arm/virt: Add memory hotplug framework) 5/10 Checking commit b25387b9d3e5 (hw/arm/virt: Add ACPI support for device memory cold-plug) 6/10 Checking commit 9cd5da671c49 (hw/arm/virt-acpi-build: Add PC-DIMM in SRAT) 7/10 Checking commit 8617e4257385 (hw/arm/virt: Introduce opt-in feature "fdt") 8/10 Checking commit bbb60b29a575 (hw/arm/boot: Expose the PC-DIMM nodes in the DT) 9/10 Checking commit 8b3a54202156 (hw/acpi: Add ACPI Generic Event Device Support) ERROR: line over 90 characters #124: FILE: hw/acpi/generic_event_device.c:117: + if_ctx = aml_if(aml_equal(aml_and(irq_sel, aml_int(ged_events[i].selector), NULL), aml_int(ged_events[i].selector))); total: 1 errors, 0 warnings, 269 lines checked Patch 9/10 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 10/10 Checking commit 125f46770b37 (hw/arm/virt: Init GED device and enable memory hotplug) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20190321104745.28068-1-shameerali.kolothum.thodi@huawei.com/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-devel@redhat.com
This series is an attempt to provide device memory hotplug support on ARM virt platform. This is based on Eric's recent works here[1] and carries some of the pc-dimm related patches dropped from his series. The kernel support for arm64 memory hot add was added recently by Robin and hence the guest kernel should be => 5.0-rc1. NVDIM support is not included currently as we still have an unresolved issue while hot adding NVDIMM[2]. However NVDIMM cold plug patches can be included, but not done for now, for keeping it simple. This makes use of GED device to sent hotplug ACPI events to the Guest. GED code is based on Nemu. Thanks to the efforts of Samuel and Sebastien to add the hardware-reduced support to Nemu using GED device[3]. (Please shout if I got the author/signed-off wrong for those patches or missed any names). Thanks, Shameer [1] https://patchwork.kernel.org/cover/10837565/ [2] https://patchwork.kernel.org/cover/10783589/ [3] https://github.com/intel/nemu/blob/topic/virt-x86/hw/acpi/ged.c v2 --> v2 Addressed comments from Igor and Eric, -Made virt acpi device platform independent and moved to hw/acpi/generic_event_device.c -Moved ged specific code into hw/acpi/generic_event_device.c -Introduced an opt-in feature "fdt" to resolve device-memory being treated as early boot memory. -Dropped patch #1 from v2. RFC --> v2 -Use GED device instead of GPIO for ACPI hotplug events. -Removed NVDIMM support for now. -Includes dropped patches from Eric's v9 series. Eric Auger (1): hw/arm/virt: Add memory hotplug framework Samuel Ortiz (3): hw/acpi: Do not create memory hotplug method when handler is not defined hw/arm/virt: Add virtual ACPI device hw/acpi: Add ACPI Generic Event Device Support Shameer Kolothum (6): hw/acpi: Make ACPI IO address space configurable hw/arm/virt: Add ACPI support for device memory cold-plug hw/arm/virt-acpi-build: Add PC-DIMM in SRAT hw/arm/virt: Introduce opt-in feature "fdt" hw/arm/boot: Expose the PC-DIMM nodes in the DT hw/arm/virt: Init GED device and enable memory hotplug default-configs/arm-softmmu.mak | 5 + hw/acpi/Kconfig | 4 + hw/acpi/Makefile.objs | 1 + hw/acpi/generic_event_device.c | 313 +++++++++++++++++++++++++++++++++ hw/acpi/memory_hotplug.c | 34 ++-- hw/arm/boot.c | 44 +++++ hw/arm/virt-acpi-build.c | 27 +++ hw/arm/virt.c | 121 ++++++++++++- hw/i386/acpi-build.c | 3 +- include/hw/acpi/generic_event_device.h | 68 +++++++ include/hw/acpi/memory_hotplug.h | 8 +- include/hw/arm/virt.h | 5 + 12 files changed, 614 insertions(+), 19 deletions(-) create mode 100644 hw/acpi/generic_event_device.c create mode 100644 include/hw/acpi/generic_event_device.h -- 2.7.4