On Tue May 16, 2023 at 11:54 AM AEST, Narayana Murty N wrote: > > On 5/15/23 12:02, Nicholas Piggin wrote: > > On Sat Apr 29, 2023 at 12:30 AM AEST, Fabiano Rosas wrote: > >>>> Could you describe in more detail what is your setup? Specifically > >>>> whether both guests are running TCG or KVM (info kvm) and the state of > >>>> the nested-hv capability in QEMU command line. > >>> Currently the issue is seen with any pseries KVM guest running on a > >>> PowerNV host. > > Okay originally I thought you were talking about a powernv target > > that is running a pseries guest and dumping that. But after re-reading, I > > think you're talking about dumping a pseries target? > > yes, The qemu-memory-dump tested on different combinations like pseries > as well as powernv guests with and without -enable-kvm.
Still not quite sure what you're talking about from that or the v2 changelog. powernv is not a guest, it is a target machine, and it can't use -enable-kvm. The target runs on the host which is your hardware. The machine can emulate Linux/KVM hosting a guest but that's all *inside* the target and not a property of the target QEMU. powernv just knows it is executing with MSR[HV]=0, LPIDR!=0, etc. > > Questions still remain about why that's the best way to go. If the > > target was running a nested-HV guest, is it reasonable that the guest > > can change the endinaness of the target dump on a whim by changing its > > ILE? > > > > Thanks, > > Nick > > But the crash tool expects the endianness of the dump loaded should be same as > supplied debug kernel image endianness. Seems like an important detail to mention in the changelog. > Detailed test cases we can find at V2 of this patch. > https://lore.kernel.org/all/20230509104701.12473-1-nnmli...@linux.ibm.com/ The patch may be okay, but changelog needs work. Please explain the cases using QEMU terminology like host and target, and describe what the target is doing in a way that's clear you aren't talking about the QEMU that is the subject of the patch. Host Target Workload BE powernv BE Linux running LE KVM guest LE powernv ... LE pseries KVM etc LE pseries TCG Don't have to enumerate the entire matrix or put it in that format exactly, but something clear and precise. ILE does not always match vmlinux being run by target either, mind you. Generally in early boot. I guess that is an unavoidable gap without doing more stuff (either make crash work or add a way to set dump endian or something). That's okay, ILE might be a reasonable proxy, but it might be worth mentioning in the changelog too. Thanks, Nick
