Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1412782

Re: [PATCH v5 1/6] qspinlock: powerpc support qspinlock

Show all headers | View raw

On Fri, 2016-06-03 at 12:10 +0800, xinhui wrote:
> On 2016年06月03日 09:32, Benjamin Herrenschmidt wrote:
> > On Fri, 2016-06-03 at 11:32 +1000, Benjamin Herrenschmidt wrote:
> >> On Thu, 2016-06-02 at 17:22 +0800, Pan Xinhui wrote:
> >>>
> >>> Base code to enable qspinlock on powerpc. this patch add some
> >>> #ifdef
> >>> here and there. Although there is no paravirt related code, we
> can
> >>> successfully build a qspinlock kernel after apply this patch.
> >> This is missing the IO_SYNC stuff ... It means we'll fail to do a
> >> full
> >> sync to order vs MMIOs.
> >>
> >> You need to add that back in the unlock path.
> >
> > Well, and in the lock path as well...
> >
> Oh, yes. I missed IO_SYNC stuff.
> 
> thank you, Ben :)

Ok couple of other things that would be nice from my perspective (and
Michael's) if you can produce them:

 - Some benchmarks of the qspinlock alone, without the PV stuff,
   so we understand how much of the overhead is inherent to the
   qspinlock and how much is introduced by the PV bits.

 - For the above, can you show (or describe) where the qspinlock
   improves things compared to our current locks. While there's
   theory and to some extent practice on x86, it would be nice to
   validate the effects on POWER.

 - Comparative benchmark with the PV stuff in on a bare metal system
   to understand the overhead there.

 - Comparative benchmark with the PV stuff under pHyp and KVM

Spinlocks are fiddly and a critical piece of infrastructure, it's
important we fully understand the performance implications before we
decide to switch to a new model.

Cheers,
Ben.

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v5 1/6] qspinlock: powerpc support qspinlock Pan Xinhui <xinhui.pan@linux.vnet.ibm.com> - 2016-06-02 11:30 +0200
  Re: [PATCH v5 1/6] qspinlock: powerpc support qspinlock Benjamin Herrenschmidt <benh@kernel.crashing.org> - 2016-06-03 03:40 +0200
    Re: [PATCH v5 1/6] qspinlock: powerpc support qspinlock Benjamin Herrenschmidt <benh@au1.ibm.com> - 2016-06-03 03:40 +0200
      Re: [PATCH v5 1/6] qspinlock: powerpc support qspinlock xinhui <xinhui.pan@linux.vnet.ibm.com> - 2016-06-03 06:20 +0200
        Re: [PATCH v5 1/6] qspinlock: powerpc support qspinlock Benjamin Herrenschmidt <benh@kernel.crashing.org> - 2016-06-03 06:40 +0200
          Re: [PATCH v5 1/6] qspinlock: powerpc support qspinlock xinhui <xinhui.pan@linux.vnet.ibm.com> - 2016-06-03 09:10 +0200
          Re: [PATCH v5 1/6] qspinlock: powerpc support qspinlock Peter Zijlstra <peterz@infradead.org> - 2016-06-06 18:00 +0200
            Re: [PATCH v5 1/6] qspinlock: powerpc support qspinlock Benjamin Herrenschmidt <benh@kernel.crashing.org> - 2016-06-07 00:20 +0200

csiph-web