Date: Mon, 26 Aug 2013 15:22:36 +0200 From: Andre Oppermann <andre@freebsd.org> To: Roger Pau Monne <roger.pau@citrix.com> Cc: freebsd-current@freebsd.org Subject: Re: [PATCH] xen-netback: fix compile errors introduced by r254804 and r254807 Message-ID: <521B569C.4080906@freebsd.org> In-Reply-To: <1377514921-2132-1-git-send-email-roger.pau@citrix.com> References: <1377514921-2132-1-git-send-email-roger.pau@citrix.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 26.08.2013 13:02, Roger Pau Monne wrote: > r254804 and r254807 changed the types of some of the members of the > mbuf struct, and introduced some compile time errors in netback > debug messages that prevented compiling a XENHVM kernel. Thanks, I fixed the printf's with r254910 in a slightly different way just before I saw your email. There's a dedicated m_print() function upcoming that can/should replace all those hand-grown variants. -- Andre > Cc: andre@freebsd.org > --- > sys/dev/xen/netback/netback.c | 8 ++++---- > 1 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/sys/dev/xen/netback/netback.c b/sys/dev/xen/netback/netback.c > index f18b791..16a54d9 100644 > --- a/sys/dev/xen/netback/netback.c > +++ b/sys/dev/xen/netback/netback.c > @@ -585,16 +585,16 @@ xnb_dump_mbuf(const struct mbuf *m) > > printf("xnb_dump_mbuf:\n"); > if (m->m_flags & M_PKTHDR) { > - printf(" flowid=%10d, csum_flags=%#8x, csum_data=%#8x, " > + printf(" flowid=%10d, csum_flags=%b, csum_data=%#8x, " > "tso_segsz=%5hd\n", > - m->m_pkthdr.flowid, m->m_pkthdr.csum_flags, > + m->m_pkthdr.flowid, (int)m->m_pkthdr.csum_flags, CSUM_BITS, > m->m_pkthdr.csum_data, m->m_pkthdr.tso_segsz); > printf(" rcvif=%16p, header=%18p, len=%19d\n", > - m->m_pkthdr.rcvif, m->m_pkthdr.header, m->m_pkthdr.len); > + m->m_pkthdr.rcvif, m->m_pkthdr.PH_loc.ptr, m->m_pkthdr.len); > } > printf(" m_next=%16p, m_nextpk=%16p, m_data=%16p\n", > m->m_next, m->m_nextpkt, m->m_data); > - printf(" m_len=%17d, m_flags=%#15x, m_type=%18hd\n", > + printf(" m_len=%17d, m_flags=%#15x, m_type=%u\n", > m->m_len, m->m_flags, m->m_type); > > len = m->m_len; >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?521B569C.4080906>