Robert, On Fri, Sep 05, 2014 at 04:46:28PM +0000, Robert Watson wrote: R> Author: rwatson R> Date: Fri Sep 5 16:46:28 2014 R> New Revision: 271174 R> URL: http://svnweb.freebsd.org/changeset/base/271174 R> R> Log: R> Clarify a diagnostic printf() in the mbuf code: M_EXT doesn't necessarily R> imply a cluster is attached; it could also refer to some other sort of R> external storage (e.g., an sf_buf). R> R> MFC after: 3 days R> Sponsored by: EMC / Isilon Storage Division R> R> Modified: R> head/sys/sys/mbuf.h R> R> Modified: head/sys/sys/mbuf.h R> ============================================================================== R> --- head/sys/sys/mbuf.h Fri Sep 5 16:40:47 2014 (r271173) R> +++ head/sys/sys/mbuf.h Fri Sep 5 16:46:28 2014 (r271174) R> @@ -672,7 +672,7 @@ m_clget(struct mbuf *m, int how) R> { R> R> if (m->m_flags & M_EXT) R> - printf("%s: %p mbuf already has cluster\n", __func__, m); R> + printf("%s: %p mbuf already has external storage\n", __func__, m); R> m->m_ext.ext_buf = (char *)NULL; R> uma_zalloc_arg(zone_clust, m, how); R> /* R> @@ -698,7 +698,7 @@ m_cljget(struct mbuf *m, int how, int si R> uma_zone_t zone; R> R> if (m && m->m_flags & M_EXT) R> - printf("%s: %p mbuf already has cluster\n", __func__, m); R> + printf("%s: %p mbuf already has external storage\n", __func__, m); R> if (m != NULL) R> m->m_ext.ext_buf = NULL;
Shouldn't these be KASSERTs? I'm pretty sure that Andre put it as printf(9) because he was not sure it is correct. But since that time we never seen these prints in the wild, so they can be safely converted to KASSERTs. -- Totus tuus, Glebius. _______________________________________________ svn-src-head@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
