From 9cdd80e387d5ee10a367de12a816937812818218 Mon Sep 17 00:00:00 2001 From: Brian Behlendorf Date: Tue, 29 Jun 2010 10:24:14 -0700 Subject: [PATCH] Revert traverse_impl() changes These changes are now taken care of by the fix-stack-traverse_impl topic branch which not only solves the uninit problem but also moves these locals off the stack and on to the heap. --- module/zfs/dmu_traverse.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/module/zfs/dmu_traverse.c b/module/zfs/dmu_traverse.c index 3350c08bc1..429c76ae11 100644 --- a/module/zfs/dmu_traverse.c +++ b/module/zfs/dmu_traverse.c @@ -357,7 +357,7 @@ traverse_impl(spa_t *spa, uint64_t objset, blkptr_t *rootbp, uint64_t txg_start, int flags, blkptr_cb_t func, void *arg) { struct traverse_data td; - struct prefetch_data pd; + struct prefetch_data pd = { 0 }; zbookmark_t czb; int err; @@ -371,10 +371,7 @@ traverse_impl(spa_t *spa, uint64_t objset, blkptr_t *rootbp, td.td_flags = flags; pd.pd_blks_max = 100; - pd.pd_blks_fetched = 0; pd.pd_flags = flags; - pd.pd_cancel = B_FALSE; - pd.pd_exited = B_FALSE; mutex_init(&pd.pd_mtx, NULL, MUTEX_DEFAULT, NULL); cv_init(&pd.pd_cv, NULL, CV_DEFAULT, NULL);