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 doesn't at all match with

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 ran in >> >> https://

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 https://public-inbox.org/git/CACBZZX5e58bWuf3NdDYTxu2KyZj29hHONzN=rp-7vxd8nur...

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

2017-06-05 Thread Ben Peart
On 6/2/2017 5: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 I tried Ævar's perf test with that (on a beefy laptop with SSD), and got significantly slower

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 different versions you're testing. This test w

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 merged into the master

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 download that since yesterday, sti

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 comments, >> unfortun

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 co

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

2017-06-02 Thread David Turner
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 I tried Ævar's perf test with that (on a beefy laptop with SSD), and got significantly slower results with bp/fsmonitor: Test