c/mirror_zfs
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_zfs.git synced 2024-12-25 18:59:33 +03:00
Code Issues Packages Projects Releases Wiki Activity

Change ASSERT(!"...") to cmn_err(CE_PANIC, ...)

Browse Source 
There are a handful of ASSERT(!"...")'s throughout the code base for
cases which should be impossible.  This patch converts them to use
cmn_err(CE_PANIC, ...) to ensure they are always enabled and so that
additional debugging is logged if they were to occur.

Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Issue #1445
This commit is contained in:
Brian Behlendorf 2015-02-27 12:53:35 -08:00
parent 8c45def24a
commit 989fd514b1
5 changed files with 11 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
module/zcommon/zfs_deleg.c
View File

@ -227,7 +227,7 @@ zfs_deleg_whokey(char *attr, zfs_deleg_who_type_t type,
ZFS_DELEG_FIELD_SEP_CHR); ZFS_DELEG_FIELD_SEP_CHR);
break; break;
default: default:
ASSERT(!"bad zfs_deleg_who_type_t"); cmn_err(CE_PANIC, "bad zfs_deleg_who_type_t %d", type);
} }
} }

2
module/zfs/arc.c
View File

@ -2944,7 +2944,7 @@ arc_access(arc_buf_hdr_t *buf, kmutex_t *hash_lock)
DTRACE_PROBE1(new_state__mfu, arc_buf_hdr_t *, buf); DTRACE_PROBE1(new_state__mfu, arc_buf_hdr_t *, buf);
arc_change_state(arc_mfu, buf, hash_lock); arc_change_state(arc_mfu, buf, hash_lock);
} else { } else {
ASSERT(!"invalid arc state"); cmn_err(CE_PANIC, "invalid arc state 0x%p", buf->b_state);
} }
} }

3
module/zfs/dmu_tx.c
View File

@ -925,7 +925,8 @@ dmu_tx_dirty_buf(dmu_tx_t *tx, dmu_buf_impl_t *db)
match_object = TRUE; match_object = TRUE;
break; break;
default: default:
ASSERT(!"bad txh_type"); cmn_err(CE_PANIC, "bad txh_type %d",
txh->txh_type);
} }
} }
if (match_object && match_offset) { if (match_object && match_offset) {

9
module/zfs/zap_leaf.c
View File

@ -79,8 +79,9 @@ stv(int len, void *addr, uint64_t value)
case 8: case 8:
*(uint64_t *)addr = value; *(uint64_t *)addr = value;
return; return;
default:
cmn_err(CE_PANIC, "bad int len %d", len);
} }
ASSERT(!"bad int len");
} }
static uint64_t static uint64_t
@ -95,8 +96,9 @@ ldv(int len, const void *addr)
return (*(uint32_t *)addr); return (*(uint32_t *)addr);
case 8: case 8:
return (*(uint64_t *)addr); return (*(uint64_t *)addr);
default:
cmn_err(CE_PANIC, "bad int len %d", len);
} }
ASSERT(!"bad int len");
return (0xFEEDFACEDEADBEEFULL); return (0xFEEDFACEDEADBEEFULL);
} }
@ -147,7 +149,8 @@ zap_leaf_byteswap(zap_leaf_phys_t *buf, int size)
/* la_array doesn't need swapping */ /* la_array doesn't need swapping */
break; break;
default: default:
ASSERT(!"bad leaf type"); cmn_err(CE_PANIC, "bad leaf type %d",
lc->l_free.lf_type);
} }
} }
} }

2
module/zfs/zap_micro.c
View File

@ -965,7 +965,7 @@ again:
start = 0; start = 0;
goto again; goto again;
} }
ASSERT(!"out of entries!"); cmn_err(CE_PANIC, "out of entries!");
} }
int int