Bug #7638

Refactor spa_load_impl into several functions

Added by Pavel Zakharov about 2 years ago. Updated 10 months ago.

Status:ClosedStart date:2016-12-02
Priority:NormalDue date:
Assignee:Pavel Zakharov% Done:


Category:zfs - Zettabyte File System
Target version:-
Difficulty:Medium Tags:needs-triage


spa_load_impl has grown out of proportions.
It is currently over 700 lines long and makes it very hard to follow or debug the import process even for experienced ZFS developers.
The objective is to split it up in a series of well commented functions.

A PR is ready on openzfs.


#1 Updated by Electric Monk 10 months ago

  • Status changed from New to Closed

git commit 1fd3785ff6601d3e391378c2dcbf4c5f27e1fe32

commit  1fd3785ff6601d3e391378c2dcbf4c5f27e1fe32
Author: Pavel Zakharov <pavel.zakharov@delphix.com>
Date:   2018-02-06T17:36:44.000Z

    7638 Refactor spa_load_impl into several functions
    Reviewed by: Paul Dagnelie <pcd@delphix.com>
    Reviewed by: George Wilson <george.wilson@delphix.com>
    Reviewed by: Matthew Ahrens <mahrens@delphix.com>
    Reviewed by: Andrew Stormont <andyjstormont@gmail.com>
    Approved by: Dan McDonald <danmcd@joyent.com>

Also available in: Atom