diff options
author | Pavel Zakharov <[email protected]> | 2016-03-10 10:16:02 -0500 |
---|---|---|
committer | Brian Behlendorf <[email protected]> | 2018-05-08 21:30:10 -0700 |
commit | 4a0ee12af8967a859c3607530aaef466e21cebc3 (patch) | |
tree | ae5079139035cfa72fdc0e4673775954ab9f8206 /include/sys/vdev_impl.h | |
parent | ca0845d59efe5881d14e0bb90603a5152eb07227 (diff) |
OpenZFS 8961 - SPA load/import should tell us why it failed
Problem
=======
When we fail to open or import a storage pool, we typically don't
get any additional diagnostic information, just "no pool found" or
"can not import".
While there may be no additional user-consumable information, we should
at least make this situation easier to debug/diagnose for developers
and support. For example, we could start by using `zfs_dbgmsg()`
to log each thing that we try when importing, and which things
failed. E.g. "tried uberblock of txg X from label Y of device Z". Also,
we could log each of the stages that we go through in `spa_load_impl()`.
Solution
========
Following the cleanup to `spa_load_impl()`, debug messages have been
added to every point of failure in that function. Additionally,
debug messages have been added to strategic places, such as
`vdev_disk_open()`.
Authored by: Pavel Zakharov <[email protected]>
Reviewed by: George Wilson <[email protected]>
Reviewed by: Matthew Ahrens <[email protected]>
Reviewed by: Andrew Stormont <[email protected]>
Approved by: Dan McDonald <[email protected]>
Ported-by: Tim Chase <[email protected]>
Signed-off-by: Tim Chase <[email protected]>
OpenZFS-issue: https://illumos.org/issues/8961
OpenZFS-commit: https://github.com/openzfs/openzfs/commit/418079e0
Closes #7459
Diffstat (limited to 'include/sys/vdev_impl.h')
0 files changed, 0 insertions, 0 deletions