diff options
author | glebius <glebius@FreeBSD.org> | 2015-04-06 09:15:18 +0000 |
---|---|---|
committer | glebius <glebius@FreeBSD.org> | 2015-04-06 09:15:18 +0000 |
commit | 0d3620e498ef1fb6f4bff312713b789f8f57e15e (patch) | |
tree | 646c003ea4c6005462063f3b0f3491edf9d1eae4 | |
parent | eadef9904618942bce40e2d7adc4554e408ee6db (diff) | |
download | FreeBSD-src-0d3620e498ef1fb6f4bff312713b789f8f57e15e.zip FreeBSD-src-0d3620e498ef1fb6f4bff312713b789f8f57e15e.tar.gz |
Convert printfs in m_ext allocator to KASSERTs.
Discussed with: rwatson
Sponsored by: Nginx, Inc.
-rw-r--r-- | sys/sys/mbuf.h | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/sys/sys/mbuf.h b/sys/sys/mbuf.h index 6203212..9c9ceea 100644 --- a/sys/sys/mbuf.h +++ b/sys/sys/mbuf.h @@ -686,8 +686,8 @@ static __inline int m_clget(struct mbuf *m, int how) { - if (m->m_flags & M_EXT) - printf("%s: %p mbuf already has external storage\n", __func__, m); + KASSERT((m->m_flags & M_EXT) == 0, ("%s: mbuf %p has M_EXT", + __func__, m)); m->m_ext.ext_buf = (char *)NULL; uma_zalloc_arg(zone_clust, m, how); /* @@ -713,10 +713,11 @@ m_cljget(struct mbuf *m, int how, int size) { uma_zone_t zone; - if (m && m->m_flags & M_EXT) - printf("%s: %p mbuf already has external storage\n", __func__, m); - if (m != NULL) + if (m != NULL) { + KASSERT((m->m_flags & M_EXT) == 0, ("%s: mbuf %p has M_EXT", + __func__, m)); m->m_ext.ext_buf = NULL; + } zone = m_getzone(size); return (uma_zalloc_arg(zone, m, how)); |