lkml.org 
[lkml]   [2007]   [Oct]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH RFC] paravirt: cleanup lazy mode handling
    Rusty Russell wrote:
    > That's good, but this code does lose on native because we no longer
    > simply replace the entire thing with noops.
    >
    > Perhaps inverting this and having (inline) helpers is the way to go?
    >

    I'm thinking that the overhead will be unmeasurably small, and its not
    really worth any more complexity. That's almost certainly true for lazy
    mmu mode, but lazy cpu is used in the middle of a context switch, so
    it's probably worth a bit more attention.

    > I'm thinking something like:
    >
    > static inline void paravirt_enter_lazy(enum paravirt_lazy_mode mode)
    > {
    > BUG_ON(x86_read_percpu(paravirt_lazy_mode) != PARAVIRT_LAZY_NONE);
    > BUG_ON(preemptible());
    >
    > x86_write_percpu(paravirt_lazy_mode, mode);
    > }
    >
    > static inline void paravirt_exit_lazy(enum paravirt_lazy_mode mode)
    > {
    > BUG_ON(x86_read_percpu(paravirt_lazy_mode) != mode);
    > BUG_ON(preemptible());
    >
    > x86_write_percpu(paravirt_lazy_mode, PARAVIRT_LAZY_NONE);
    > }
    >

    Er, they should probably call something to make the switch actually
    happen, no?

    > The only trick would be that the flushes are so rarely required it's
    > probably worth putting the unlikely() in the top level:
    >

    Sure, I guess. Would it make any difference? (I've never personally
    noticed likely/unlikely change the generated code in any seriously
    positive way.)

    J
    -
    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-10-02 08:33    [W:0.049 / U:0.272 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site