Date: Sun, 25 Aug 2013 02:30:54 -0400 From: Glen Barber <gjb@FreeBSD.org> To: Andre Oppermann <andre@FreeBSD.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r254799 - in head/sys: dev/cas dev/hatm dev/iscsi_initiator dev/lge dev/mwl kern sys Message-ID: <20130825063054.GA65644@glenbarber.us> In-Reply-To: <201308241657.r7OGvie8033186@svn.freebsd.org> References: <201308241657.r7OGvie8033186@svn.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--FCuugMFkClbJLl1L Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sat, Aug 24, 2013 at 04:57:44PM +0000, Andre Oppermann wrote: > Author: andre > Date: Sat Aug 24 16:57:44 2013 > New Revision: 254799 > URL: http://svnweb.freebsd.org/changeset/base/254799 >=20 > Log: > Add an mbuf pointer parameter to (*ext_free) to give the external > free function access to the mbuf the external memory was attached > to. > =20 > Mechanically adjust all users to include the mbuf parameter. > =20 > This fixes a long standing annoyance for external free functions. > Before one had to sacrifice one of the argument pointers for this. > =20 > Modified: head/sys/kern/uipc_mbuf.c > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > --- head/sys/kern/uipc_mbuf.c Sat Aug 24 16:55:53 2013 (r254798) > +++ head/sys/kern/uipc_mbuf.c Sat Aug 24 16:57:44 2013 (r254799) > @@ -247,8 +247,8 @@ m_freem(struct mbuf *mb) > */ > int > m_extadd(struct mbuf *mb, caddr_t buf, u_int size, > - void (*freef)(void *, void *), void *arg1, void *arg2, int flags, in= t type, > - int wait) > + void (*freef)(struct mbuf *, void *, void *), void *arg1, void *arg2, > + int flags, int type, int wait) > { > KASSERT(type !=3D EXT_CLUSTER, ("%s: EXT_CLUSTER not allowed", __func__= )); > =20 I think this breaks head/. cc -c -O2 -pipe -fno-strict-aliasing -std=3Dc99 -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual -Wundef -Wno-pointer-sign -fformat-extensions -Wmissing-include-dirs -fdiagnostics-show-option -Wno-error-tautological-compare -Wno-error-empty-body -Wno-error-parentheses-equality -nostdinc -I. -I/src/sys -I/src/sys/contrib/altq -I/src/sys/contrib/libfdt -D_KERNEL -DHAVE_KERNEL_OPTION_HEADERS -include opt_global.h -DGPROF -DGPROF4 -DGUPROF -fno-builtin -mno-aes -mno-avx -mno-mmx -mno-sse -msoft-float -ffreestanding -fstack-protector -Werror -pg /src/sys/kern/uipc_cow.c /src/sys/kern/uipc_cow.c:164:2: error: incompatible pointer types passing 'void (void *, void *)' to parameter of type 'void (*)(struct mbuf *, void *, void *)' [-Werror,-Wincompatible-pointer-types] MEXTADD(m0, sf_buf_kva(sf), PAGE_SIZE, socow_iodone, ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ /src/sys/sys/mbuf.h:760:50: note: expanded from macro 'MEXTADD' (void )m_extadd((m), (caddr_t)(buf), (size), (free), (arg1), (arg2),\ ^~~~~~ /src/sys/sys/mbuf.h:898:14: note: passing argument to parameter here void (*)(struct mbuf *, void *, void *), void *, void *, ^ 1 error generated. *** Error code 1 Stop. bmake[1]: stopped in /obj/i386.i386/src/sys/LINT *** Error code 1 http://tinderbox.freebsd.org/tinderbox-head-build-HEAD-i386-i386.full Glen --FCuugMFkClbJLl1L Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.20 (FreeBSD) iQEcBAEBCAAGBQJSGaSeAAoJEFJPDDeguUaj1D4H/0uYt6ibBP30Clg1JmlkCTpF i1uOKg+kJU4p92BAp8GMbr6Vnv6Id0BBvJeV+ttvXTlNGmgp3i9m7sG5cOHFj8j/ gAiFAQ766L90MDg6cWS1XTecYRM7+EJUOQTD4xUJb6CyWGUoIExtTczPjeZKTR0y tHSFB3uYIOyBGKQtzIYLCjg/ofCpTYKvMb3+jZgpi23bSndLBTb/ppqynueTYRG9 5niUI+JDRJQXNX16q0YETYW7wYqbeQ9R+qZ6IVkVxtYZCOroET262sJodvLTWWzM kqLL1KP24jdvQJ/OacL0b0NqcVf96ZQl0BTTvpSj2LB4KLpVM65HK4NgtCNhGNQ= =C60/ -----END PGP SIGNATURE----- --FCuugMFkClbJLl1L--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20130825063054.GA65644>