Date: Sun, 18 Apr 2021 18:58:56 +0000
From: Andrew Parker <[email protected]>
Message-ID: <[email protected]>
| Does anyone else have a working L2ARC?
Sorry, don't even know what that is, and don't (currently anyway) use zfs,
but:
| - interval = hz * l2arc_feed_secs;
| + interval = mstohz(l2arc_feed_secs);
Are you sure about that part of the change (the earlier fragment looked
reasonable) ?
mstohz() when starting with seconds (which the name of that var suggests)
looks like it would be much smaller than intended, whereas simply multiplying
seconds by hz gives ticks, which looks to be the objective in all of that.
Alternatively multiply secs by 1000 to generate ms, and mstohz() that.
Watch out for potential overflow in all of this though.
kre
