Actions
Bug #6681
closedzfs list burning lots of time in dodefault() via dsl_prop_*
Start date:
2016-02-27
Due date:
% Done:
100%
Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:
Description
Flamegraphing "zfs list" commands suggests that there is unnecessary time being burnt in dodefault() – see https://us-east.manta.joyent.com/arekinath/public/zfs-flame3.html
dodefault() takes a string argument, the name of a ZFS property, then looks it up to get a zfs_prop_t using zfs_name_to_prop. However, in most of the places dodefault() is called, the zfs_prop_t is already known in the function calling. We can just pass it through and avoid wasting time here.
Updated by Robert Mustacchi about 6 years ago
- Subject changed from zfs list burning lots of time in dodefault() via dsl_prop_* to zfs list burning lots of time in dodefault() via dsl_prop_*
Updated by Electric Monk about 6 years ago
- Status changed from New to Closed
git commit d09e4475f635b6f66ee68d8c17a32bba7be17c96
commit d09e4475f635b6f66ee68d8c17a32bba7be17c96 Author: Alex Wilson <alex.wilson@joyent.com> Date: 2016-03-10T21:46:48.000Z 6681 zfs list burning lots of time in dodefault() via dsl_prop_* Reviewed by: Patrick Mooney <patrick.mooney@joyent.com> Reviewed by: Matthew Ahrens <mahrens@delphix.com> Reviewed by: Dan McDonald <danmcd@omniti.com> Approved by: Matthew Ahrens <mahrens@delphix.com>
Actions