Re: [WIP/PATCH 7/6] perf: add a performance test for core.fsmonitor

2017-06-07 Thread Ben Peart
On 6/7/2017 5:46 PM, Ævar Arnfjörð Bjarmason wrote: On Wed, Jun 7, 2017 at 9:51 PM, Ben Peart wrote: On 6/2/2017 7:06 PM, Ævar Arnfjörð Bjarmason wrote: I don't have time to update the perf test now or dig into it, but most of what you're describing in this mail

Re: [WIP/PATCH 7/6] perf: add a performance test for core.fsmonitor

2017-06-07 Thread Ævar Arnfjörð Bjarmason
On Wed, Jun 7, 2017 at 9:51 PM, Ben Peart wrote: > > > On 6/2/2017 7:06 PM, Ævar Arnfjörð Bjarmason wrote: >> >> >> I don't have time to update the perf test now or dig into it, but most >> of what you're describing in this mail doesn't at all match with the >> ad-hoc tests I

Re: [WIP/PATCH 7/6] perf: add a performance test for core.fsmonitor

2017-06-07 Thread Ben Peart
On 6/2/2017 7:06 PM, Ævar Arnfjörð Bjarmason wrote: I don't have time to update the perf test now or dig into it, but most of what you're describing in this mail doesn't at all match with the ad-hoc tests I ran in

Re: [WIP/PATCH 7/6] perf: add a performance test for core.fsmonitor

2017-06-05 Thread Ben Peart
Ævar Arnfjörð Bjarmason <ava...@gmail.com> Subject: [WIP/PATCH 7/6] perf: add a performance test for core.fsmonitor Add a performance test for the new core.fsmonitor facility using the sample query-fsmonitor hook. This is WIP code for the reasons explained in the setup comments, unfortunate

Re: [WIP/PATCH 7/6] perf: add a performance test for core.fsmonitor

2017-06-04 Thread Jeff King
On Sun, Jun 04, 2017 at 09:46:19AM +0200, Ævar Arnfjörð Bjarmason wrote: > What I'm referring to is not a limitation of git (we'll always be able > to turn off core.fsmonitor), but a limitation of the perf framework. > There's no way to run a test like this: > > ./run master next -- p*.sh >

Re: [WIP/PATCH 7/6] perf: add a performance test for core.fsmonitor

2017-06-04 Thread Ævar Arnfjörð Bjarmason
On Sun, Jun 4, 2017 at 3:59 AM, Junio C Hamano wrote: > Ævar Arnfjörð Bjarmason writes: > >> This is WIP code for the reasons explained in the setup comments, >> unfortunately the perf code doesn't easily allow you to run different >> setup code for

Re: [WIP/PATCH 7/6] perf: add a performance test for core.fsmonitor

2017-06-03 Thread Junio C Hamano
Ævar Arnfjörð Bjarmason writes: > This is WIP code for the reasons explained in the setup comments, > unfortunately the perf code doesn't easily allow you to run different > setup code for different versions you're testing. This test will stop > working if the fsmonitor is

Re: [WIP/PATCH 7/6] perf: add a performance test for core.fsmonitor

2017-06-03 Thread Ævar Arnfjörð Bjarmason
On Fri, Jun 2, 2017 at 11:44 PM, David Turner wrote: > BTW, a medium-sized (~250k files across 40k dirs) synthetic repo is available > over bittorrent at: > http://bitmover.com/2015-04-03-1M-git-bare.tar.bz2.torrent Can you seed this please? I've been trying to

Re: [WIP/PATCH 7/6] perf: add a performance test for core.fsmonitor

2017-06-02 Thread Ævar Arnfjörð Bjarmason
On Sat, Jun 3, 2017 at 12:05 AM, Ben Peart wrote: > > > On 6/2/2017 6:28 AM, Ævar Arnfjörð Bjarmason wrote: >> >> Add a performance test for the new core.fsmonitor facility using the >> sample query-fsmonitor hook. >> >> This is WIP code for the reasons explained in the setup

Re: [WIP/PATCH 7/6] perf: add a performance test for core.fsmonitor

2017-06-02 Thread Ben Peart
On 6/2/2017 6:28 AM, Ævar Arnfjörð Bjarmason wrote: Add a performance test for the new core.fsmonitor facility using the sample query-fsmonitor hook. This is WIP code for the reasons explained in the setup comments, unfortunately the perf code doesn't easily allow you to run different setup

RE: [WIP/PATCH 7/6] perf: add a performance test for core.fsmonitor

2017-06-02 Thread David Turner
ristian > Couder <christian.cou...@gmail.com>; Ævar Arnfjörð Bjarmason > <ava...@gmail.com> > Subject: [WIP/PATCH 7/6] perf: add a performance test for core.fsmonitor > > Add a performance test for the new core.fsmonitor facility using the sample > query-fsmonitor ho

[WIP/PATCH 7/6] perf: add a performance test for core.fsmonitor

2017-06-02 Thread Ævar Arnfjörð Bjarmason
Add a performance test for the new core.fsmonitor facility using the sample query-fsmonitor hook. This is WIP code for the reasons explained in the setup comments, unfortunately the perf code doesn't easily allow you to run different setup code for different versions you're testing. This test