On Thu, 5 Jul 2018, Paolo Bonzini wrote: > On 21/06/2018 23:25, Pavel Tatashin wrote: > > KVM clock is initialized later compared to other hypervisor because it has > > dependency on memblock allocator. > > > > Lets bring it inline with other hypervisors by removing this dependency by > > using memory from BSS instead of allocating it. > > > > The benefits: > > - remove ifdef from common code > > - earlier availability of TSC. > > - remove dependency on memblock, and reduce code > > - earlier kvm sched_clock() > > > > Signed-off-by: Pavel Tatashin <pasha.tatashin@xxxxxxxxxx> > > The reason for this is to avoid wasting a lot of BSS memory when KVM is > not in use. Thomas is going to send his take on this! Got it working with per cpu variables, but there is a different subtle issue with that. The pvclock data is mapped into the VDSO as well, i.e. as a full page. Right now with the linear array, which is forced to be page sized at least this only maps pvclock data or zeroed data (after the last CPU) into the VDSO. With PER CPU variables this would map arbitraty other per cpu data which happens to be in the same page into the VDSO. Not really what we want. That means to utilize PER CPU data this requires to allocate page sized pvclock data space for each CPU to prevent leaking arbitrary stuff. As this data is allocated on demand, i.e. only if kvmclock is used, this might be tolerable, but I'm not so sure. Thanks, tglx -- To unsubscribe from this list: send the line "unsubscribe linux-s390" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
