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


Groups > linux.kernel > #1739157 > unrolled thread

Re: [PATCH v3 00/20] Speculative page faults

Started byAlexei Starovoitov <alexei.starovoitov@gmail.com>
First post2017-09-25 18:30 +0200
Last post2017-09-26 10:40 +0200
Articles 3 — 3 participants

Back to article view | Back to linux.kernel


Contents

  Re: [PATCH v3 00/20] Speculative page faults Alexei Starovoitov <alexei.starovoitov@gmail.com> - 2017-09-25 18:30 +0200
    Re: [PATCH v3 00/20] Speculative page faults Andrew Morton <akpm@linux-foundation.org> - 2017-09-26 01:40 +0200
    Re: [PATCH v3 00/20] Speculative page faults Laurent Dufour <ldufour@linux.vnet.ibm.com> - 2017-09-26 10:40 +0200

#1739157 — Re: [PATCH v3 00/20] Speculative page faults

FromAlexei Starovoitov <alexei.starovoitov@gmail.com>
Date2017-09-25 18:30 +0200
SubjectRe: [PATCH v3 00/20] Speculative page faults
Message-ID<utEqC-mA-23@gated-at.bofh.it>
On Mon, Sep 18, 2017 at 12:15 AM, Laurent Dufour
<ldufour@linux.vnet.ibm.com> wrote:
> Despite the unprovable lockdep warning raised by Sergey, I didn't get any
> feedback on this series.
>
> Is there a chance to get it moved upstream ?

what is the status ?
We're eagerly looking forward for this set to land,
since we have several use cases for tracing that
will build on top of this set as discussed at Plumbers.

[toc] | [next] | [standalone]


#1739362

FromAndrew Morton <akpm@linux-foundation.org>
Date2017-09-26 01:40 +0200
Message-ID<utL8J-4Uf-3@gated-at.bofh.it>
In reply to#1739157
On Mon, 25 Sep 2017 09:27:43 -0700 Alexei Starovoitov <alexei.starovoitov@gmail.com> wrote:

> On Mon, Sep 18, 2017 at 12:15 AM, Laurent Dufour
> <ldufour@linux.vnet.ibm.com> wrote:
> > Despite the unprovable lockdep warning raised by Sergey, I didn't get any
> > feedback on this series.
> >
> > Is there a chance to get it moved upstream ?
> 
> what is the status ?
> We're eagerly looking forward for this set to land,
> since we have several use cases for tracing that
> will build on top of this set as discussed at Plumbers.

There has been sadly little review and testing so far :(

I'll be taking a close look at it all over the next couple of weeks. 

One terribly important thing (especially for a patchset this large and
intrusive) is the rationale for merging it: the justification, usually
in the form of end-user benefit.

Laurent's [0/n] provides some nice-looking performance benefits for
workloads which are chosen to show performance benefits(!) but, alas,
no quantitative testing results for workloads which we may suspect will
be harmed by the changes(?).  Even things as simple as impact upon
single-threaded pagefault-intensive workloads and its effect upon
CONFIG_SMP=n .text size?

If you have additional usecases then please, spell them out for us in
full detail so we can better understand the benefits which this
patchset provides.

[toc] | [prev] | [next] | [standalone]


#1739688

FromLaurent Dufour <ldufour@linux.vnet.ibm.com>
Date2017-09-26 10:40 +0200
Message-ID<utTzm-2jm-75@gated-at.bofh.it>
In reply to#1739157
Hi Alexei,

Le 25/09/2017 à 18:27, Alexei Starovoitov a écrit :
> On Mon, Sep 18, 2017 at 12:15 AM, Laurent Dufour
> <ldufour@linux.vnet.ibm.com> wrote:
>> Despite the unprovable lockdep warning raised by Sergey, I didn't get any
>> feedback on this series.
>>
>> Is there a chance to get it moved upstream ?
> 
> what is the status ?

As mentioned by Andrew this lacks review and test, what about your 
Ack/Review/Tested-By ?

> We're eagerly looking forward for this set to land,
> since we have several use cases for tracing that
> will build on top of this set as discussed at Plumbers.

Unfortunately I was not able to attend Plumbers this year, but I'll be 
pleased, as well as the MM mailing readers, to get your feedback and use 
cases of this series.

Thanks,
Laurent.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web