all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Lars-Dominik Braun <ldb@leibniz-psychology.org>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: 41720@debbugs.gnu.org
Subject: [bug#41720] [PATCH] store: Use buffered I/O for all protocol writes
Date: Mon, 8 Jun 2020 08:42:28 +0200	[thread overview]
Message-ID: <20200608064228.GB3166@zpidnp36> (raw)
In-Reply-To: <875zc5xkoh.fsf@gnu.org>

[-- Attachment #1: Type: text/plain, Size: 2275 bytes --]

Hi Ludo,

> Nice, though the slowdown on Unix-domain sockets is annoying given that
> it’s the primary transport.  Are the 10% significant and stable over
> multiple runs?
I don’t want to blind your eyes with meaningless statistics here, but let’s do
this properly then. So for `guix environment guix -- true` with 10 trials these
are the results:

---snip---
$ ministat -A -c 99 before_ssh after_ssh
x before_ssh
+ after_ssh
    N           Min           Max        Median           Avg        Stddev
x  10          7.63        10.635       10.1585        9.9382    0.85517585
+  10         5.986         6.424        6.1755        6.1917    0.15398416
Difference at 99.0% confidence
        -3.7465 +/- 0.790815
        -37.698% +/- 5.0788%
        (Student's t, pooled s = 0.614425)

$ ministat -A -c 99 before_unix after_unix
x before_unix
+ after_unix
    N           Min           Max        Median           Avg        Stddev
x  10         1.528         1.624        1.5445        1.5636   0.035842867
+  10         1.576         1.701        1.6145        1.6229   0.037869513
Difference at 99.0% confidence
        0.0593 +/- 0.0474548
        3.79253% +/- 3.08989%
        (Student's t, pooled s = 0.0368701)
---snap---

And the same for `guix environment --ad-hoc r-learnr`:

---snip---
$ ministat -A -c 99 before_ssh after_ssh
x before_ssh
+ after_ssh
    N           Min           Max        Median           Avg        Stddev
x  10        20.104        21.629        21.082       21.0115    0.45142472
+  10        16.082        16.713        16.292       16.3416    0.25018224
Difference at 99.0% confidence
        -4.6699 +/- 0.469719
        -22.2254% +/- 1.86734%
        (Student's t, pooled s = 0.364949)

$ ministat -A -c 99 before_unix after_unix
x before_unix
+ after_unix
    N           Min           Max        Median           Avg        Stddev
x  10        11.322        11.529        11.401       11.4127   0.062467858
+  10        11.249        11.469       11.3515        11.368   0.078689262
No difference proven at 99.0% confidence
---snap---

> Otherwise the patch LGTM as long as it doesn’t introduce any test suite
> regressions.
Yes, it passes the test suite.

Lars


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

  reply	other threads:[~2020-06-08  6:43 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-05  9:11 [bug#41720] [PATCH] store: Use buffered I/O for all protocol writes Lars-Dominik Braun
2020-06-05 21:33 ` Ludovic Courtès
2020-06-08  6:42   ` Lars-Dominik Braun [this message]
2020-06-09  7:49     ` bug#41720: " Ludovic Courtès
2020-06-09  8:21       ` [bug#41720] " Lars-Dominik Braun

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200608064228.GB3166@zpidnp36 \
    --to=ldb@leibniz-psychology.org \
    --cc=41720@debbugs.gnu.org \
    --cc=ludo@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.