Re: [PATCH 00/13] [PATCH RFC] Paravirtualized ticketlocks

2011-09-06 Thread Jeremy Fitzhardinge
On 09/02/2011 04:22 AM, Stefano Stabellini wrote: do you have a git tree somewhere with this series? git://github.com/jsgf/linux-xen.git upstream/pvticketlock-slowflag J -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org

Re: [PATCH 00/13] [PATCH RFC] Paravirtualized ticketlocks

2011-09-02 Thread Stefano Stabellini
On Fri, 2 Sep 2011, Jeremy Fitzhardinge wrote: From: Jeremy Fitzhardinge jeremy.fitzhardi...@citrix.com This series replaces the existing paravirtualized spinlock mechanism with a paravirtualized ticketlock mechanism. Ticket locks have an inherent problem in a virtualized case, because

Re: [PATCH 00/13] [PATCH RFC] Paravirtualized ticketlocks

2011-09-02 Thread Linus Torvalds
On Thu, Sep 1, 2011 at 5:54 PM, Jeremy Fitzhardinge jer...@goop.org wrote: The inner part of ticket lock code becomes:        inc = xadd(lock-tickets, inc);        inc.tail = ~TICKET_SLOWPATH_FLAG;        for (;;) {                unsigned count = SPIN_THRESHOLD;                do {      

Re: [PATCH 00/13] [PATCH RFC] Paravirtualized ticketlocks

2011-09-02 Thread Jeremy Fitzhardinge
On 09/02/2011 08:38 AM, Linus Torvalds wrote: On Thu, Sep 1, 2011 at 5:54 PM, Jeremy Fitzhardinge jer...@goop.org wrote: The inner part of ticket lock code becomes: inc = xadd(lock-tickets, inc); inc.tail = ~TICKET_SLOWPATH_FLAG; for (;;) { unsigned count

Re: [PATCH 00/13] [PATCH RFC] Paravirtualized ticketlocks

2011-09-02 Thread Linus Torvalds
On Fri, Sep 2, 2011 at 1:07 PM, Jeremy Fitzhardinge jer...@goop.org wrote: I don't know whether that fastpath code is small enough to consider inlining everywhere? No. There's no point in inlining something that ends up containing a conditional function call: gcc will have to effectively

Re: [PATCH 00/13] [PATCH RFC] Paravirtualized ticketlocks

2011-09-02 Thread Jeremy Fitzhardinge
On 09/02/2011 01:27 PM, Linus Torvalds wrote: On Fri, Sep 2, 2011 at 1:07 PM, Jeremy Fitzhardinge jer...@goop.org wrote: I don't know whether that fastpath code is small enough to consider inlining everywhere? No. There's no point in inlining something that ends up containing a conditional

[PATCH 00/13] [PATCH RFC] Paravirtualized ticketlocks

2011-09-01 Thread Jeremy Fitzhardinge
From: Jeremy Fitzhardinge jeremy.fitzhardi...@citrix.com This series replaces the existing paravirtualized spinlock mechanism with a paravirtualized ticketlock mechanism. Ticket locks have an inherent problem in a virtualized case, because the vCPUs are scheduled rather than running concurrently