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 ac72fac3ea.

This aligns us more closely to the upstream illumos code base.

Signed-off-by: Prakash Surya <surya1@llnl.gov>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
This commit is contained in:
Prakash Surya 2014-07-11 13:03:29 -07:00 committed by Brian Behlendorf
parent 1e8db77102
commit 1be627f5c2

View File

@ -1398,8 +1398,6 @@ metaslab_sync_done(metaslab_t *msp, uint64_t txg)
vdev_dirty(vd, VDD_METASLAB, msp, txg + 1); 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 * 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 * 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; int64_t failures = mg->mg_alloc_failures;
int m; int m;
metaslab_group_alloc_update(mg);
/* /*
* Re-evaluate all metaslabs which have lower offsets than the * Re-evaluate all metaslabs which have lower offsets than the
* bonus area. * bonus area.