diff options
| author | Prakash Surya <[email protected]> | 2014-07-11 13:03:29 -0700 |
|---|---|---|
| committer | Brian Behlendorf <[email protected]> | 2014-07-22 09:38:16 -0700 |
| commit | 1be627f5c28a355bcd49e4e097114c13fae7731b (patch) | |
| tree | 1f838b8d08b5054f6e753aa389a51a54e25642d1 | |
| parent | 1e8db7710220332808920a582e5794d6fc37b109 (diff) | |
Move metaslab_group_alloc_update() call
This changes moves the called to metaslab_group_alloc_update() to the
metaslab_sync_reassess() function. The original placement of the call
within metaslab_sync_done() appears to have been a simple mistake,
introduced by ac72fac3eaa569902cad88053167f7d74e7fe7e4.
This aligns us more closely to the upstream illumos code base.
Signed-off-by: Prakash Surya <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
| -rw-r--r-- | module/zfs/metaslab.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/module/zfs/metaslab.c b/module/zfs/metaslab.c index a8b4b0a0e..fe2428ac0 100644 --- a/module/zfs/metaslab.c +++ b/module/zfs/metaslab.c @@ -1398,8 +1398,6 @@ metaslab_sync_done(metaslab_t *msp, uint64_t txg) vdev_dirty(vd, VDD_METASLAB, msp, txg + 1); } - metaslab_group_alloc_update(mg); - /* * If the map is loaded but no longer active, evict it as soon as all * future allocations have synced. (If we unloaded it now and then @@ -1428,6 +1426,8 @@ metaslab_sync_reassess(metaslab_group_t *mg) int64_t failures = mg->mg_alloc_failures; int m; + metaslab_group_alloc_update(mg); + /* * Re-evaluate all metaslabs which have lower offsets than the * bonus area. |