On Mon, Jun 08, 2020 at 04:42:42PM +0100, Marc Zyngier wrote: > Hi Andrew, > > On 2020-06-08 15:51, Andrew Scull wrote: > > On Mon, Jun 08, 2020 at 09:56:57AM +0100, Marc Zyngier wrote: > > > For very long, we have kept this pointer back to the per-cpu > > > host state, despite having working per-cpu accessors at EL2 > > > for some time now. > > > > > > Recent investigations have shown that this pointer is easy > > > to abuse in preemptible context, which is a sure sign that > > > it would better be gone. Not to mention that a per-cpu > > > pointer is faster to access at all times. > > > > Helps to make the references to `kvm_host_data` clearer with there now > > being just one way to get to it and shows that it is scoped to the > > current CPU. A good change IMO! > > Thanks! Can I take this as a Reviewed-by or Acked-by tag? Just let me know.
Build and booted your kvm-arm64/ptrauth-fixes branch contianing this patch with VHE and nVHE on qemu. Booted a VM within each with kvmtool. Reviewed-by: Andrew Scull <asc...@google.com> _______________________________________________ kvmarm mailing list kvmarm@lists.cs.columbia.edu https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
