lkml.org 
[lkml]   [2012]   [Feb]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [Qemu-devel] [RFC] Next gen kvm api
    On 02/06/2012 07:54 AM, Avi Kivity wrote:
    > On 02/06/2012 03:33 PM, Anthony Liguori wrote:
    >>> Look at arch/x86/kvm/i8254.c:pit_ioport_read() for a counterexample.
    >>> There are also interactions with other devices (for example the
    >>> apic/ioapic interaction via the apic bus).
    >>
    >>
    >> Hrm, maybe I'm missing it, but the path that would be hot is:
    >>
    >> if (!status_latched&& !count_latched) {
    >> value = kpit_elapsed()
    >> // manipulate count based on mode
    >> // mask value depending on read_state
    >> }
    >>
    >> This path is side-effect free, and applies relatively simple math to a
    >> time counter.
    >
    > Do guests always read an unlatched counter? Doesn't seem reasonable
    > since they can't get a stable count this way.

    Perhaps. You could have the latching done by writing to persisted scratch
    memory but then locking becomes an issue.

    >> The idea would be to allow the filter to not handle an I/O request
    >> depending on existing state. Anything that's modifies state (like
    >> reading the latch counter) would drop to userspace.
    >
    > This restricts us to a subset of the device which is at the mercy of the
    > guest.

    Yes, but it provides an elegant solution to having a flexible way to do things
    in the fast path in a generic way without presenting additional security concerns.

    A similar, albeit more complex and less elegant, approach would be to make use
    of something like the vtpm optimization to reflect certain exits back into
    injected code into the guest. But this has the disadvantage of being very
    x86-centric and it's not clear if you can avoid double exits which would hurt
    the slow paths.

    > We could define mmio registers for muldiv64, and for communicating over
    > the APIC bus. But then the device model for BPF ends up more
    > complicated than the kernel devices we have put together.

    Maybe what we really need is NaCL for kernel space :-D

    Regards,

    Anthony Liguori




    \
     
     \ /
      Last update: 2012-02-06 15:03    [W:2.193 / U:0.280 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site