[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#26256: [PATCH 4/6] gnu: fio: Enable rbd support.
From: |
Ludovic Courtès |
Subject: |
bug#26256: [PATCH 4/6] gnu: fio: Enable rbd support. |
Date: |
Tue, 28 Mar 2017 11:45:19 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Marius Bakke <address@hidden> skribis:
> Ludovic Courtès <address@hidden> writes:
>
>> Marius Bakke <address@hidden> skribis:
>>
>>> * gnu/packages/benchmark.scm (fio)[inputs]: Add ceph:lib.
>>
>> Would be worth checking how this affects the closure size. It’s a
>> developer tool though, so I’d argue that it’s less of a problem if the
>> closure size is increased noticeably in this cases.
>
> ceph:lib increases fios closure size from 452.3 MiB to a good 738 MiB.
>
> However, by moving the fio auxiliary scripts to a separate output, I
> managed to get the baseline size down from 452.3 MiB to 75.3 MiB. See
> attached patch. WDYT?
>
> From b942036bef17a4eacfd7406a9a5130fd2255252d Mon Sep 17 00:00:00 2001
> From: Marius Bakke <address@hidden>
> Date: Tue, 28 Mar 2017 00:08:36 +0200
> Subject: [PATCH] gnu: fio: Install python scripts in separate output.
>
> * gnu/packages/benchmark.scm (fio)[outputs]: New field.
> [arguments]: Rename 'wrap-python-scripts' to 'move-outputs'. Keep wrapper.
[...]
> + (add-after 'install 'move-outputs
> + (lambda* (#:key outputs #:allow-other-keys)
> + (let ((out (assoc-ref outputs "out"))
> + (utils (string-append (assoc-ref outputs "utils")
> "/bin")))
Add a comment explaining that this saves X MiB on the closure.
> + (outputs '("out" "utils"))
Maybe s/utils/bin/ if there aren’t any other binaries, otherwise LGTM!
Thank you!
Ludo’.
- bug#26256: [PATCH 0/6] Add ceph + multipath-tools., Marius Bakke, 2017/03/25
- bug#26256: [PATCH 5/6] gnu: Add userspace-rcu., Marius Bakke, 2017/03/25
- bug#26256: [PATCH 5/6] gnu: Add userspace-rcu., Ludovic Courtès, 2017/03/27
- bug#26256: [PATCH 5/6] gnu: Add userspace-rcu., Marius Bakke, 2017/03/27
- bug#26256: [PATCH 5/6] gnu: Add userspace-rcu., Marius Bakke, 2017/03/27
- bug#26256: [PATCH 5/6] gnu: Add userspace-rcu., Marius Bakke, 2017/03/27
- bug#26256: [PATCH 5/6] gnu: Add userspace-rcu., Ludovic Courtès, 2017/03/28
- bug#26256: [PATCH 5/6] gnu: Add userspace-rcu., Ludovic Courtès, 2017/03/28
- bug#26256: [PATCH 5/6] gnu: Add userspace-rcu., Tobias Geerinckx-Rice, 2017/03/27
- bug#26256: [PATCH 5/6] gnu: Add userspace-rcu., Tobias Geerinckx-Rice, 2017/03/27
- bug#26256: [PATCH 5/6] gnu: Add userspace-rcu., Marius Bakke, 2017/03/27