[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v3] xen/x86: vpmu: Unmap per-vCPU PMU page when the domain is destroyed





On 04/12/2019 15:59, Boris Ostrovsky wrote:


On Dec 4, 2019, at 10:55 AM, Julien Grall <jgrall@xxxxxxxxxx> wrote:

Hi Boris,

On 28/11/2019 21:50, Boris Ostrovsky wrote:
On 11/28/19 5:23 AM, Jan Beulich wrote:
On 28.11.2019 10:38, Paul Durrant wrote:

--- a/xen/arch/x86/cpu/vpmu.c
+++ b/xen/arch/x86/cpu/vpmu.c
@@ -576,11 +576,36 @@ static void vpmu_arch_destroy(struct vcpu *v)
             vpmu->arch_vpmu_ops->arch_vpmu_destroy(v);
      }
+
+    vpmu_reset(vpmu, VPMU_CONTEXT_ALLOCATED);
  }
Boris, to be on the safe side - are you in agreement with this
change, now that the setting of the flag is being left untouched?
Yes, this is fine.

Can we take this as an ACK?


Yes, of course.

Thank you!

@Andrew, @Jan: this is x86 code, but I am happy to commit it if you prefer.

Cheers,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.