Re: [PATCH stable v4.9 v2] arm64: entry: Place an SB sequence following an ERET instruction

2020-07-10 Thread Sasha Levin
On Thu, Jul 09, 2020 at 12:50:23PM -0700, Florian Fainelli wrote: From: Will Deacon commit 679db70801da9fda91d26caf13bf5b5ccc74e8e8 upstream Some CPUs can speculate past an ERET instruction and potentially perform speculative accesses to memory before processing the exception return. Since the

Re: [PATCH v6 0/5] clean up redundant 'kvm_run' parameters

2020-07-10 Thread Tianjia Zhang
Hi Paolo, Any opinion on this series patches? Can I help with this patchset ? Thanks and best, Tianjia On 2020/6/23 21:14, Tianjia Zhang wrote: In the current kvm version, 'kvm_run' has been included in the 'kvm_vcpu' structure. For historical reasons, many kvm-related function parameters reta

[PATCH 2/2] KVM: arm64: Leave vcpu FPSIMD synchronization in host

2020-07-10 Thread Andrew Scull
The task state can be checked by the host and the vcpu flags updated before calling into hyp. This more neatly separates the concerns and removes the need to map the task flags to EL2. Hyp acts on the state provided to it by the host and updates it when switching to the vcpu state. No functional

[PATCH 0/2] Manage vcpu flags from the host

2020-07-10 Thread Andrew Scull
This is a clean up and no functional change is intended. The aim is to keep management of the flags in the host and out of hyp. I find this makes it easier to understand how the flags are used. This is the result of a previous thread with Dave where it was explained that there is an intent to syn

[PATCH 1/2] KVM: arm64: Leave KVM_ARM64_DEBUG_DIRTY updates to the host

2020-07-10 Thread Andrew Scull
Move the clearing of KVM_ARM64_DEBUG_DIRTY from being one of the last things hyp does before exiting to the host to being one of the first things the host does after hyp exits. This means the host always manages the state of the bit and hyp simply respects that in the context switch. No functiona

Re: [PATCH v6 0/5] clean up redundant 'kvm_run' parameters

2020-07-10 Thread Paolo Bonzini
On 10/07/20 09:32, Tianjia Zhang wrote: > Hi Paolo, > > Any opinion on this series patches? Can I help with this patchset ? I was hoping to have some Tested-by, for now I'm queuing patches 1 and 2. Thanks, Paolo > Thanks and best, > Tianjia > > On 2020/6/23 21:14, Tianjia Zhang wrote: >> In t

Re: [PATCH v6 3/5] KVM: PPC: clean up redundant kvm_run parameters in assembly

2020-07-10 Thread Paolo Bonzini
On 23/06/20 15:14, Tianjia Zhang wrote: > > /* Load non-volatile guest state from the vcpu */ > - VCPU_LOAD_NVGPRS(r4) > + VCPU_LOAD_NVGPRS(r3) > > kvm_start_lightweight: > /* Copy registers into shadow vcpu so we can access them in real mode */ > - mr r3, r4 >

Re: [PATCH v6 1/5] KVM: s390: clean up redundant 'kvm_run' parameters

2020-07-10 Thread Paolo Bonzini
On 23/06/20 17:31, Christian Borntraeger wrote: > > I have trouble seeing value in this particular patch. We add LOCs > without providing any noticable benefit. All other patches in this series at > least reduce the amount of code. So I would defer this to Paolo if he prefers > to have this way ac