2014-01-24 18:28:43

by Darren Hart

[permalink] [raw]
Subject: Re: [PATCH 0/3] perf-bench: introduce futex microbenchmarks

On Mon, 2013-12-16 at 06:44 -0800, Davidlohr Bueso wrote:
> On Mon, 2013-12-16 at 11:08 +0100, Ingo Molnar wrote:
> > * Davidlohr Bueso <[email protected]> wrote:
> >
> > > This patchset adds three programs that stress and measure different
> > > futex operations: (i) uaddr hashing, (ii) wakeups and (iii)
> > > requeuing/waiting.
> > >
> > > More details and usage examples in each individual patch, along with
> > > parameter descriptions in the code.
> > >
> > > While the previous effort (https://lkml.org/lkml/2012/5/17/207) to
> > > add futex benchmarks to perf-bench failed, I strongly believe that
> > > perf is an ideal place for these kinds of programs. This patchset is
> > > different from Hitoshi's because it does not try to take over
> > > Darren's futextest suite, and only deals with finer grained aspects
> > > of the kernel's implementation, and thus mostly useful for kernel
> > > hacking. Furthermore, by being part of the kernel tree, it can get
> > > more attention and naturally evolve with time.
> >
> > Looks pretty useful!
> >
> > Could the two approaches be merged?
>
> Unless Darren doesn't want to, I don't see why not. I can resurrect
> Hitoshi's original patch if/after this series is applied.

Apologies, I only am just now seeing this.

I agree that we should take whatever makes sense for perf out of
futex-test and merge it with perf. It will see greater use and receive
more review and improvements than it will in my obscure repository.

With trinity covering the fuzz testing and perf handling performance
tests, I think futex-test can be reduced down to a functional
test-suite, which is perfectly fine with me.

If there is still interest here, I'll support it.

Thanks,

Darren


2014-01-28 04:12:51

by Davidlohr Bueso

[permalink] [raw]
Subject: Re: [PATCH 0/3] perf-bench: introduce futex microbenchmarks

On Fri, 2014-01-24 at 10:13 -0800, Darren Hart wrote:
> On Mon, 2013-12-16 at 06:44 -0800, Davidlohr Bueso wrote:
> > On Mon, 2013-12-16 at 11:08 +0100, Ingo Molnar wrote:
> > > * Davidlohr Bueso <[email protected]> wrote:
> > >
> > > > This patchset adds three programs that stress and measure different
> > > > futex operations: (i) uaddr hashing, (ii) wakeups and (iii)
> > > > requeuing/waiting.
> > > >
> > > > More details and usage examples in each individual patch, along with
> > > > parameter descriptions in the code.
> > > >
> > > > While the previous effort (https://lkml.org/lkml/2012/5/17/207) to
> > > > add futex benchmarks to perf-bench failed, I strongly believe that
> > > > perf is an ideal place for these kinds of programs. This patchset is
> > > > different from Hitoshi's because it does not try to take over
> > > > Darren's futextest suite, and only deals with finer grained aspects
> > > > of the kernel's implementation, and thus mostly useful for kernel
> > > > hacking. Furthermore, by being part of the kernel tree, it can get
> > > > more attention and naturally evolve with time.
> > >
> > > Looks pretty useful!
> > >
> > > Could the two approaches be merged?
> >
> > Unless Darren doesn't want to, I don't see why not. I can resurrect
> > Hitoshi's original patch if/after this series is applied.
>
> Apologies, I only am just now seeing this.
>
> I agree that we should take whatever makes sense for perf out of
> futex-test and merge it with perf. It will see greater use and receive
> more review and improvements than it will in my obscure repository.
>
> With trinity covering the fuzz testing and perf handling performance
> tests, I think futex-test can be reduced down to a functional
> test-suite, which is perfectly fine with me.
>
> If there is still interest here, I'll support it.

Arnaldo, could you consider this for 3.15? I've also got some additional
work for perf-bench but am waiting for this to get settled first.

Thanks,
Davidlohr

2014-01-28 12:12:45

by Arnaldo Carvalho de Melo

[permalink] [raw]
Subject: Re: [PATCH 0/3] perf-bench: introduce futex microbenchmarks

Em Mon, Jan 27, 2014 at 08:12:42PM -0800, Davidlohr Bueso escreveu:
> On Fri, 2014-01-24 at 10:13 -0800, Darren Hart wrote:
> > I agree that we should take whatever makes sense for perf out of
> > futex-test and merge it with perf. It will see greater use and receive
> > more review and improvements than it will in my obscure repository.

> > With trinity covering the fuzz testing and perf handling performance
> > tests, I think futex-test can be reduced down to a functional
> > test-suite, which is perfectly fine with me.

> > If there is still interest here, I'll support it.

> Arnaldo, could you consider this for 3.15? I've also got some additional
> work for perf-bench but am waiting for this to get settled first.

Sure, I'll try applying it later today.

- Arnaldo