lkml.org 
[lkml]   [2007]   [Apr]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 1/3] Introduce cpuid_on_cpu() and cpuid_eax_on_cpu()
    Andi Kleen wrote:
    > On Monday 02 April 2007 13:38, Alexey Dobriyan wrote:
    >> They will be used by cpuid driver and powernow-k8 cpufreq driver.
    >>
    >> With these changes powernow-k8 driver could run correctly on OpenVZ kernels
    >> with virtual cpus enabled (SCHED_VCPU).
    >
    > This means openvz has multiple virtual CPU levels? One for cpuid/rdmsr and one
    > for the rest of the kernel? Both powernow-k8 and cpuid attempt to schedule
    > to the target CPU so they should already run there. But it is some other CPU,
    > but when they ask your _on_cpu() functions they suddenly get a "real" CPU?
    > Where is the difference between these levels of virtualness?
    >

    The CPUID and MSR drivers do not schedule to the target CPU; instead, on
    hardware, they rely on IPI'ing the target processor if it is not the one
    that's currently running.

    There were a lot of discussion back when about which was the better
    solution. Alan Cox, in particular, really preferred the interrupt
    solution as being less likely to cause implicit deadlock.

    I do want to add that it's been on my list for some time -- in fact, I
    keep implementing it half-way and then having other things to do -- to
    add MSR and CPUID ioctls() that allow the full register file to be set
    and read back, in order to support architecturally broken MSR and CPUID
    levels.

    -hpa
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2007-04-02 18:25    [W:0.023 / U:0.420 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site