Message ID | 20200317100423.622643-1-david@gibson.dropbear.id.au (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On 17/03/20 11:03, David Gibson wrote: > pseries: Update SLOF firmware image > ppc/spapr: Move GPRs setup to one place > pseries: Update SLOF firmware image > spapr/rtas: Reserve space for RTAS blob and log > pseries: Update SLOF firmware image Oh, no fake-OF patches? Paolo > Cédric Le Goater (1): > spapr/xive: use SPAPR_IRQ_IPI to define IPI ranges exposed to the guest > > David Gibson (19): > ppc: Remove stub support for 32-bit hypervisor mode > ppc: Remove stub of PPC970 HID4 implementation > target/ppc: Correct handling of real mode accesses with vhyp on hash MMU > target/ppc: Introduce ppc_hash64_use_vrma() helper > spapr, ppc: Remove VPM0/RMLS hacks for POWER9 > target/ppc: Remove RMOR register from POWER9 & POWER10 > target/ppc: Use class fields to simplify LPCR masking > target/ppc: Streamline calculation of RMA limit from LPCR[RMLS] > target/ppc: Correct RMLS table > target/ppc: Only calculate RMLS derived RMA limit on demand > target/ppc: Don't store VRMA SLBE persistently > spapr: Don't use weird units for MIN_RMA_SLOF > spapr,ppc: Simplify signature of kvmppc_rma_size() > spapr: Don't attempt to clamp RMA to VRMA constraint > spapr: Don't clamp RMA to 16GiB on new machine types > spapr: Clean up RMA size calculation > spapr: Move creation of ibm,dynamic-reconfiguration-memory dt node > spapr: Move creation of ibm,architecture-vec-5 property > spapr: Rename DT functions to newer naming convention > > Greg Kurz (2): > spapr: Handle pending hot plug/unplug requests at CAS > ppc: Officially deprecate the CPU "compat" property > > Nicholas Piggin (8): > ppc/spapr: Fix FWNMI machine check failure handling > ppc/spapr: Change FWNMI names > ppc/spapr: Add FWNMI System Reset state > ppc/spapr: Fix FWNMI machine check interrupt delivery > ppc/spapr: Allow FWNMI on TCG > target/ppc: allow ppc_cpu_do_system_reset to take an alternate vector > ppc/spapr: Implement FWNMI System Reset delivery > ppc/spapr: Ignore common "ibm,nmi-interlock" Linux bug > > Philippe Mathieu-Daudé (8): > hw/ppc/pnv: Fix typo in comment > hw/scsi/viosrp: Add missing 'hw/scsi/srp.h' include > hw/scsi/spapr_vscsi: Use SRP_MAX_IU_LEN instead of sizeof flexible array > hw/scsi/spapr_vscsi: Simplify a bit > hw/scsi/spapr_vscsi: Introduce req_iu() helper > hw/scsi/spapr_vscsi: Do not mix SRP IU size with DMA buffer size > hw/scsi/spapr_vscsi: Prevent buffer overflow > hw/scsi/spapr_vscsi: Convert debug fprintf() to trace event > > Shivaprasad G Bhat (1): > spapr: Fix Coverity warning while validating nvdimm options > > Vitaly Chikunov (1): > target/ppc: Fix rlwinm on ppc64 > > docs/system/deprecated.rst | 7 + > hw/intc/spapr_xive.c | 4 +- > hw/ppc/pnv_lpc.c | 2 +- > hw/ppc/spapr.c | 980 ++++++++++++++++++++------------------ > hw/ppc/spapr_caps.c | 19 +- > hw/ppc/spapr_cpu_core.c | 16 +- > hw/ppc/spapr_events.c | 51 +- > hw/ppc/spapr_hcall.c | 15 +- > hw/ppc/spapr_nvdimm.c | 7 +- > hw/ppc/spapr_ovec.c | 4 +- > hw/ppc/spapr_rtas.c | 45 +- > hw/scsi/spapr_vscsi.c | 72 +-- > hw/scsi/trace-events | 1 + > hw/scsi/viosrp.h | 3 +- > include/hw/ppc/spapr.h | 34 +- > include/hw/ppc/spapr_cpu_core.h | 4 +- > include/hw/ppc/spapr_ovec.h | 4 +- > pc-bios/README | 2 +- > pc-bios/slof.bin | Bin 931032 -> 965008 bytes > roms/SLOF | 2 +- > target/ppc/cpu-qom.h | 1 + > target/ppc/cpu.h | 28 +- > target/ppc/excp_helper.c | 79 ++- > target/ppc/kvm.c | 5 +- > target/ppc/kvm_ppc.h | 7 +- > target/ppc/mmu-hash64.c | 319 +++++-------- > target/ppc/translate.c | 20 +- > target/ppc/translate_init.inc.c | 116 ++--- > tests/qtest/libqos/libqos-spapr.h | 3 +- > 29 files changed, 930 insertions(+), 920 deletions(-) >
Patchew URL: https://patchew.org/QEMU/20200317100423.622643-1-david@gibson.dropbear.id.au/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [PULL 00/45] ppc-for-5.0 queue 20200317 Message-id: 20200317100423.622643-1-david@gibson.dropbear.id.au 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 === Switched to a new branch 'test' 1689eda pseries: Update SLOF firmware image 72735a9 ppc/spapr: Ignore common "ibm,nmi-interlock" Linux bug 47df6f1 ppc/spapr: Implement FWNMI System Reset delivery f1fef1f target/ppc: allow ppc_cpu_do_system_reset to take an alternate vector 3bd2159 ppc/spapr: Allow FWNMI on TCG 91debae ppc/spapr: Fix FWNMI machine check interrupt delivery c9df752 ppc/spapr: Add FWNMI System Reset state bed131f ppc/spapr: Change FWNMI names b4a37dd ppc/spapr: Fix FWNMI machine check failure handling 6a32b19 spapr: Rename DT functions to newer naming convention 4e79406 spapr: Move creation of ibm,architecture-vec-5 property 21a3ae2 spapr: Move creation of ibm, dynamic-reconfiguration-memory dt node bf5265e spapr/rtas: Reserve space for RTAS blob and log 446dace pseries: Update SLOF firmware image f5bb445 ppc/spapr: Move GPRs setup to one place b824828 target/ppc: Fix rlwinm on ppc64 0e75479 spapr/xive: use SPAPR_IRQ_IPI to define IPI ranges exposed to the guest 397ec17 hw/scsi/spapr_vscsi: Convert debug fprintf() to trace event e8ff332 hw/scsi/spapr_vscsi: Prevent buffer overflow 7bde608 hw/scsi/spapr_vscsi: Do not mix SRP IU size with DMA buffer size 10ef238 hw/scsi/spapr_vscsi: Introduce req_iu() helper 06d1d9a hw/scsi/spapr_vscsi: Simplify a bit 89514b4 hw/scsi/spapr_vscsi: Use SRP_MAX_IU_LEN instead of sizeof flexible array 8cad997 hw/scsi/viosrp: Add missing 'hw/scsi/srp.h' include 684e72f spapr: Clean up RMA size calculation 4015194 spapr: Don't clamp RMA to 16GiB on new machine types 6ba055d spapr: Don't attempt to clamp RMA to VRMA constraint 37120d6 spapr,ppc: Simplify signature of kvmppc_rma_size() ac722f8 spapr: Don't use weird units for MIN_RMA_SLOF 2c86473 target/ppc: Don't store VRMA SLBE persistently 2a4565f target/ppc: Only calculate RMLS derived RMA limit on demand e6944cb target/ppc: Correct RMLS table 27bd1f5 target/ppc: Streamline calculation of RMA limit from LPCR[RMLS] 1d5b686 target/ppc: Use class fields to simplify LPCR masking 41f0aef target/ppc: Remove RMOR register from POWER9 & POWER10 f0bb56d spapr, ppc: Remove VPM0/RMLS hacks for POWER9 49d09fc target/ppc: Introduce ppc_hash64_use_vrma() helper a604105 target/ppc: Correct handling of real mode accesses with vhyp on hash MMU 6fbdf8f ppc: Remove stub of PPC970 HID4 implementation 689d28f ppc: Remove stub support for 32-bit hypervisor mode 405c024 hw/ppc/pnv: Fix typo in comment 7532fbf spapr: Fix Coverity warning while validating nvdimm options 645463a ppc: Officially deprecate the CPU "compat" property 63c40f1 spapr: Handle pending hot plug/unplug requests at CAS 7d74e4f pseries: Update SLOF firmware image === OUTPUT BEGIN === 1/45 Checking commit 7d74e4fe6d74 (pseries: Update SLOF firmware image) 2/45 Checking commit 63c40f1e53c0 (spapr: Handle pending hot plug/unplug requests at CAS) 3/45 Checking commit 645463ab9a98 (ppc: Officially deprecate the CPU "compat" property) 4/45 Checking commit 7532fbf9018e (spapr: Fix Coverity warning while validating nvdimm options) 5/45 Checking commit 405c024f407a (hw/ppc/pnv: Fix typo in comment) 6/45 Checking commit 689d28f975cf (ppc: Remove stub support for 32-bit hypervisor mode) 7/45 Checking commit 6fbdf8f49b17 (ppc: Remove stub of PPC970 HID4 implementation) 8/45 Checking commit a604105404be (target/ppc: Correct handling of real mode accesses with vhyp on hash MMU) 9/45 Checking commit 49d09fc795a5 (target/ppc: Introduce ppc_hash64_use_vrma() helper) 10/45 Checking commit f0bb56d30022 (spapr, ppc: Remove VPM0/RMLS hacks for POWER9) 11/45 Checking commit 41f0aefbf75e (target/ppc: Remove RMOR register from POWER9 & POWER10) 12/45 Checking commit 1d5b6860959d (target/ppc: Use class fields to simplify LPCR masking) 13/45 Checking commit 27bd1f5741a5 (target/ppc: Streamline calculation of RMA limit from LPCR[RMLS]) 14/45 Checking commit e6944cbf3ed9 (target/ppc: Correct RMLS table) 15/45 Checking commit 2a4565fd1852 (target/ppc: Only calculate RMLS derived RMA limit on demand) 16/45 Checking commit 2c864739b507 (target/ppc: Don't store VRMA SLBE persistently) 17/45 Checking commit ac722f832dad (spapr: Don't use weird units for MIN_RMA_SLOF) 18/45 Checking commit 37120d691e31 (spapr,ppc: Simplify signature of kvmppc_rma_size()) 19/45 Checking commit 6ba055dea62c (spapr: Don't attempt to clamp RMA to VRMA constraint) 20/45 Checking commit 40151945633e (spapr: Don't clamp RMA to 16GiB on new machine types) 21/45 Checking commit 684e72f277f7 (spapr: Clean up RMA size calculation) 22/45 Checking commit 8cad997dd4dd (hw/scsi/viosrp: Add missing 'hw/scsi/srp.h' include) 23/45 Checking commit 89514b40c11f (hw/scsi/spapr_vscsi: Use SRP_MAX_IU_LEN instead of sizeof flexible array) 24/45 Checking commit 06d1d9a3c12f (hw/scsi/spapr_vscsi: Simplify a bit) 25/45 Checking commit 10ef2385fe33 (hw/scsi/spapr_vscsi: Introduce req_iu() helper) 26/45 Checking commit 7bde6080932a (hw/scsi/spapr_vscsi: Do not mix SRP IU size with DMA buffer size) 27/45 Checking commit e8ff332d14ad (hw/scsi/spapr_vscsi: Prevent buffer overflow) 28/45 Checking commit 397ec17b6ad5 (hw/scsi/spapr_vscsi: Convert debug fprintf() to trace event) 29/45 Checking commit 0e75479201cd (spapr/xive: use SPAPR_IRQ_IPI to define IPI ranges exposed to the guest) 30/45 Checking commit b824828a0f91 (target/ppc: Fix rlwinm on ppc64) 31/45 Checking commit f5bb44507880 (ppc/spapr: Move GPRs setup to one place) 32/45 Checking commit 446daceb3250 (pseries: Update SLOF firmware image) 33/45 Checking commit bf5265eafdbe (spapr/rtas: Reserve space for RTAS blob and log) 34/45 Checking commit 21a3ae278891 (spapr: Move creation of ibm, dynamic-reconfiguration-memory dt node) WARNING: Block comments use a leading /* on a separate line #310: FILE: hw/ppc/spapr.c:650: + /* spapr_machine_init() checks for rma_size <= node0_size WARNING: Block comments use a trailing */ on a separate line #311: FILE: hw/ppc/spapr.c:651: + * already */ WARNING: line over 80 characters #403: FILE: hw/ppc/spapr.c:742: + _FDT((fdt_setprop_cell(fdt, offset, "slb-size", cpu->hash64_opts->slb_size))); WARNING: line over 80 characters #404: FILE: hw/ppc/spapr.c:743: + _FDT((fdt_setprop_cell(fdt, offset, "ibm,slb-size", cpu->hash64_opts->slb_size))); WARNING: Block comments use a leading /* on a separate line #420: FILE: hw/ppc/spapr.c:759: + /* Advertise VSX (vector extensions) if available WARNING: Block comments use a trailing */ on a separate line #425: FILE: hw/ppc/spapr.c:764: + * are possible, and all of those have VMX */ WARNING: Block comments use a leading /* on a separate line #432: FILE: hw/ppc/spapr.c:771: + /* Advertise DFP (Decimal Floating Point) if available WARNING: Block comments use a trailing */ on a separate line #434: FILE: hw/ppc/spapr.c:773: + * 1 == DFP available */ total: 0 errors, 8 warnings, 534 lines checked Patch 34/45 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 35/45 Checking commit 4e794060fae1 (spapr: Move creation of ibm,architecture-vec-5 property) WARNING: line over 80 characters #123: FILE: hw/ppc/spapr.c:1069: + _FDT(fdt_setprop_cell(fdt, chosen, "qemu,graphic-width", graphic_width)); WARNING: line over 80 characters #124: FILE: hw/ppc/spapr.c:1070: + _FDT(fdt_setprop_cell(fdt, chosen, "qemu,graphic-height", graphic_height)); WARNING: line over 80 characters #125: FILE: hw/ppc/spapr.c:1071: + _FDT(fdt_setprop_cell(fdt, chosen, "qemu,graphic-depth", graphic_depth)); WARNING: line over 80 characters #142: FILE: hw/ppc/spapr.c:1085: + _FDT(fdt_setprop_string(fdt, chosen, "qemu,boot-device", boot_device)); WARNING: line over 80 characters #153: FILE: hw/ppc/spapr.c:1096: + _FDT(fdt_setprop_string(fdt, chosen, "linux,stdout-path", stdout_path)); total: 0 errors, 5 warnings, 193 lines checked Patch 35/45 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 36/45 Checking commit 6a32b195209c (spapr: Rename DT functions to newer naming convention) 37/45 Checking commit b4a37ddeedb7 (ppc/spapr: Fix FWNMI machine check failure handling) 38/45 Checking commit bed131fc204c (ppc/spapr: Change FWNMI names) WARNING: Block comments use a leading /* on a separate line #305: FILE: include/hw/ppc/spapr.h:197: + /* Machine Check Notification Routine address WARNING: Block comments use a leading /* on a separate line #312: FILE: include/hw/ppc/spapr.h:202: + /* Machine Check FWNMI synchronization, fwnmi_machine_check_interlock is total: 0 errors, 2 warnings, 270 lines checked Patch 38/45 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 39/45 Checking commit c9df752a19d8 (ppc/spapr: Add FWNMI System Reset state) WARNING: Block comments use a leading /* on a separate line #81: FILE: include/hw/ppc/spapr.h:197: + /* System Reset and Machine Check Notification Routine addresses total: 0 errors, 1 warnings, 51 lines checked Patch 39/45 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 40/45 Checking commit 91debae337f6 (ppc/spapr: Fix FWNMI machine check interrupt delivery) 41/45 Checking commit 3bd2159a5093 (ppc/spapr: Allow FWNMI on TCG) 42/45 Checking commit f1fef1fa71da (target/ppc: allow ppc_cpu_do_system_reset to take an alternate vector) 43/45 Checking commit 47df6f1786f7 (ppc/spapr: Implement FWNMI System Reset delivery) WARNING: line over 80 characters #50: FILE: hw/ppc/spapr.c:976: + ms->smp.max_cpus * sizeof(uint64_t)*2 + sizeof(uint64_t))); ERROR: code indent should never use tabs #50: FILE: hw/ppc/spapr.c:976: +^I^I^I ms->smp.max_cpus * sizeof(uint64_t)*2 + sizeof(uint64_t)));$ ERROR: spaces required around that '*' (ctx:VxV) #50: FILE: hw/ppc/spapr.c:976: + ms->smp.max_cpus * sizeof(uint64_t)*2 + sizeof(uint64_t))); ^ WARNING: line over 80 characters #75: FILE: hw/ppc/spapr.c:3425: + addr = rtas_addr + RTAS_ERROR_LOG_MAX + cs->cpu_index * sizeof(uint64_t)*2; ERROR: spaces required around that '*' (ctx:VxV) #75: FILE: hw/ppc/spapr.c:3425: + addr = rtas_addr + RTAS_ERROR_LOG_MAX + cs->cpu_index * sizeof(uint64_t)*2; ^ total: 3 errors, 2 warnings, 59 lines checked Patch 43/45 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 44/45 Checking commit 72735a9393da (ppc/spapr: Ignore common "ibm,nmi-interlock" Linux bug) ERROR: code indent should never use tabs #28: FILE: hw/ppc/spapr_rtas.c:465: +^I * The vCPU that hit the NMI should invoke "ibm,nmi-interlock"$ ERROR: code indent should never use tabs #30: FILE: hw/ppc/spapr_rtas.c:467: +^I * for system reset interrupts, despite them not being interlocked.$ ERROR: code indent should never use tabs #31: FILE: hw/ppc/spapr_rtas.c:468: +^I * PowerVM silently ignores this and returns success here. Returning$ ERROR: code indent should never use tabs #32: FILE: hw/ppc/spapr_rtas.c:469: +^I * failure causes Linux to print the error "FWNMI: nmi-interlock$ ERROR: code indent should never use tabs #33: FILE: hw/ppc/spapr_rtas.c:470: +^I * failed: -3", although no other apparent ill effects, this is a$ ERROR: code indent should never use tabs #34: FILE: hw/ppc/spapr_rtas.c:471: +^I * regression for the user when enabling FWNMI. So for now, match$ ERROR: code indent should never use tabs #35: FILE: hw/ppc/spapr_rtas.c:472: +^I * PowerVM. When most Linux clients are fixed, this could be$ ERROR: code indent should never use tabs #36: FILE: hw/ppc/spapr_rtas.c:473: +^I * changed.$ ERROR: code indent should never use tabs #37: FILE: hw/ppc/spapr_rtas.c:474: +^I */$ total: 9 errors, 0 warnings, 20 lines checked Patch 44/45 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 45/45 Checking commit 1689eda27bca (pseries: Update SLOF firmware image) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20200317100423.622643-1-david@gibson.dropbear.id.au/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-devel@redhat.com
On Tue, Mar 17, 2020 at 11:30:31AM +0100, Paolo Bonzini wrote: > On 17/03/20 11:03, David Gibson wrote: > > pseries: Update SLOF firmware image > > ppc/spapr: Move GPRs setup to one place > > pseries: Update SLOF firmware image > > spapr/rtas: Reserve space for RTAS blob and log > > pseries: Update SLOF firmware image > > Oh, no fake-OF patches? Apart from some prelims that make sense on their own, no. Not quite ready to go ahead with that, I'm afraid. > > Paolo > > > Cédric Le Goater (1): > > spapr/xive: use SPAPR_IRQ_IPI to define IPI ranges exposed to the guest > > > > David Gibson (19): > > ppc: Remove stub support for 32-bit hypervisor mode > > ppc: Remove stub of PPC970 HID4 implementation > > target/ppc: Correct handling of real mode accesses with vhyp on hash MMU > > target/ppc: Introduce ppc_hash64_use_vrma() helper > > spapr, ppc: Remove VPM0/RMLS hacks for POWER9 > > target/ppc: Remove RMOR register from POWER9 & POWER10 > > target/ppc: Use class fields to simplify LPCR masking > > target/ppc: Streamline calculation of RMA limit from LPCR[RMLS] > > target/ppc: Correct RMLS table > > target/ppc: Only calculate RMLS derived RMA limit on demand > > target/ppc: Don't store VRMA SLBE persistently > > spapr: Don't use weird units for MIN_RMA_SLOF > > spapr,ppc: Simplify signature of kvmppc_rma_size() > > spapr: Don't attempt to clamp RMA to VRMA constraint > > spapr: Don't clamp RMA to 16GiB on new machine types > > spapr: Clean up RMA size calculation > > spapr: Move creation of ibm,dynamic-reconfiguration-memory dt node > > spapr: Move creation of ibm,architecture-vec-5 property > > spapr: Rename DT functions to newer naming convention > > > > Greg Kurz (2): > > spapr: Handle pending hot plug/unplug requests at CAS > > ppc: Officially deprecate the CPU "compat" property > > > > Nicholas Piggin (8): > > ppc/spapr: Fix FWNMI machine check failure handling > > ppc/spapr: Change FWNMI names > > ppc/spapr: Add FWNMI System Reset state > > ppc/spapr: Fix FWNMI machine check interrupt delivery > > ppc/spapr: Allow FWNMI on TCG > > target/ppc: allow ppc_cpu_do_system_reset to take an alternate vector > > ppc/spapr: Implement FWNMI System Reset delivery > > ppc/spapr: Ignore common "ibm,nmi-interlock" Linux bug > > > > Philippe Mathieu-Daudé (8): > > hw/ppc/pnv: Fix typo in comment > > hw/scsi/viosrp: Add missing 'hw/scsi/srp.h' include > > hw/scsi/spapr_vscsi: Use SRP_MAX_IU_LEN instead of sizeof flexible array > > hw/scsi/spapr_vscsi: Simplify a bit > > hw/scsi/spapr_vscsi: Introduce req_iu() helper > > hw/scsi/spapr_vscsi: Do not mix SRP IU size with DMA buffer size > > hw/scsi/spapr_vscsi: Prevent buffer overflow > > hw/scsi/spapr_vscsi: Convert debug fprintf() to trace event > > > > Shivaprasad G Bhat (1): > > spapr: Fix Coverity warning while validating nvdimm options > > > > Vitaly Chikunov (1): > > target/ppc: Fix rlwinm on ppc64 > > > > docs/system/deprecated.rst | 7 + > > hw/intc/spapr_xive.c | 4 +- > > hw/ppc/pnv_lpc.c | 2 +- > > hw/ppc/spapr.c | 980 ++++++++++++++++++++------------------ > > hw/ppc/spapr_caps.c | 19 +- > > hw/ppc/spapr_cpu_core.c | 16 +- > > hw/ppc/spapr_events.c | 51 +- > > hw/ppc/spapr_hcall.c | 15 +- > > hw/ppc/spapr_nvdimm.c | 7 +- > > hw/ppc/spapr_ovec.c | 4 +- > > hw/ppc/spapr_rtas.c | 45 +- > > hw/scsi/spapr_vscsi.c | 72 +-- > > hw/scsi/trace-events | 1 + > > hw/scsi/viosrp.h | 3 +- > > include/hw/ppc/spapr.h | 34 +- > > include/hw/ppc/spapr_cpu_core.h | 4 +- > > include/hw/ppc/spapr_ovec.h | 4 +- > > pc-bios/README | 2 +- > > pc-bios/slof.bin | Bin 931032 -> 965008 bytes > > roms/SLOF | 2 +- > > target/ppc/cpu-qom.h | 1 + > > target/ppc/cpu.h | 28 +- > > target/ppc/excp_helper.c | 79 ++- > > target/ppc/kvm.c | 5 +- > > target/ppc/kvm_ppc.h | 7 +- > > target/ppc/mmu-hash64.c | 319 +++++-------- > > target/ppc/translate.c | 20 +- > > target/ppc/translate_init.inc.c | 116 ++--- > > tests/qtest/libqos/libqos-spapr.h | 3 +- > > 29 files changed, 930 insertions(+), 920 deletions(-) > > >
On 18/03/2020 09:33, David Gibson wrote: > On Tue, Mar 17, 2020 at 11:30:31AM +0100, Paolo Bonzini wrote: >> On 17/03/20 11:03, David Gibson wrote: >>> pseries: Update SLOF firmware image >>> ppc/spapr: Move GPRs setup to one place >>> pseries: Update SLOF firmware image >>> spapr/rtas: Reserve space for RTAS blob and log >>> pseries: Update SLOF firmware image >> >> Oh, no fake-OF patches? > > Apart from some prelims that make sense on their own, no. > > Not quite ready to go ahead with that, I'm afraid. You reviewed and found problems or you did not have time for that? Either is fine, just curious.
On Wed, Mar 18, 2020 at 10:58:46AM +1100, Alexey Kardashevskiy wrote: > > > On 18/03/2020 09:33, David Gibson wrote: > > On Tue, Mar 17, 2020 at 11:30:31AM +0100, Paolo Bonzini wrote: > >> On 17/03/20 11:03, David Gibson wrote: > >>> pseries: Update SLOF firmware image > >>> ppc/spapr: Move GPRs setup to one place > >>> pseries: Update SLOF firmware image > >>> spapr/rtas: Reserve space for RTAS blob and log > >>> pseries: Update SLOF firmware image > >> > >> Oh, no fake-OF patches? > > > > Apart from some prelims that make sense on their own, no. > > > > Not quite ready to go ahead with that, I'm afraid. > > You reviewed and found problems or you did not have time for that? > Either is fine, just curious. Mostly the second. It's not just the time to actually review the patches, either. Paolo made some good points about what it's safe to move into qemu proper. That means I need to thoroughly rethink where I really want to go with these SLOF removal ideas, and I haven't had time to do that.
On Tue, 17 Mar 2020 at 10:04, David Gibson <david@gibson.dropbear.id.au> wrote: > > The following changes since commit a98135f727595382e200d04c2996e868b7925a01: > > Merge remote-tracking branch 'remotes/kraxel/tags/vga-20200316-pull-request' into staging (2020-03-16 14:55:59 +0000) > > are available in the Git repository at: > > git://github.com/dgibson/qemu.git tags/ppc-for-5.0-20200317 > > for you to fetch changes up to 6961eae79f58385482775dc0a6c3d553f633662d: > > pseries: Update SLOF firmware image (2020-03-17 17:00:22 +1100) > > ---------------------------------------------------------------- > ppc patch queue 2020-03-17 > > Here's my final pull request for the qemu-5.0 soft freeze. Sorry this > is just under the wire - I hit some last minute problems that took a > while to fix up and retest. > > Highlights are: > * Numerous fixes for the FWNMI feature > * A handful of cleanups to the device tree construction code > * Numerous fixes for the spapr-vscsi device > * A number of fixes and cleanups for real mode (MMU off) softmmu > handling > * Fixes for handling of the PAPR RMA > * Better handling of hotplug/unplug events during boot > * Assorted other fixes > Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/5.0 for any user-visible changes. -- PMM