From owner-svn-src-all@FreeBSD.ORG Tue Jul 31 03:47:27 2012 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx2.freebsd.org (mx2.freebsd.org [69.147.83.53]) by hub.freebsd.org (Postfix) with ESMTP id 39A61106564A; Tue, 31 Jul 2012 03:47:27 +0000 (UTC) (envelope-from dougb@FreeBSD.org) Received: from opti.dougb.net (hub.freebsd.org [IPv6:2001:4f8:fff6::36]) by mx2.freebsd.org (Postfix) with ESMTP id 7C0F214DE36; Tue, 31 Jul 2012 03:47:26 +0000 (UTC) Message-ID: <5017554E.2020806@FreeBSD.org> Date: Mon, 30 Jul 2012 20:47:26 -0700 From: Doug Barton Organization: http://SupersetSolutions.com/ User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:14.0) Gecko/20120728 Thunderbird/14.0 MIME-Version: 1.0 To: Luigi Rizzo References: <201207301821.q6UILnA8022241@svn.freebsd.org> In-Reply-To: <201207301821.q6UILnA8022241@svn.freebsd.org> X-Enigmail-Version: 1.4.2 OpenPGP: id=1A1ABC84 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r238912 - head/sys/dev/netmap X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 31 Jul 2012 03:47:27 -0000 It seems this broke the build? On 07/30/2012 11:21, Luigi Rizzo wrote: > Author: luigi > Date: Mon Jul 30 18:21:48 2012 > New Revision: 238912 > URL: http://svn.freebsd.org/changeset/base/238912 > > Log: > - move the inclusion of netmap headers to the common part of the code; > - more portable annotations for unused arguments; > > Modified: > head/sys/dev/netmap/netmap.c > head/sys/dev/netmap/netmap_mem2.c > > Modified: head/sys/dev/netmap/netmap.c > ============================================================================== > --- head/sys/dev/netmap/netmap.c Mon Jul 30 15:37:47 2012 (r238911) > +++ head/sys/dev/netmap/netmap.c Mon Jul 30 18:21:48 2012 (r238912) > @@ -90,13 +90,14 @@ __FBSDID("$FreeBSD$"); > #include > #include /* BIOCIMMEDIATE */ > #include > -#include > -#include > #include /* bus_dmamap_* */ > > MALLOC_DEFINE(M_NETMAP, "netmap", "Network memory map"); > #endif /* __FreeBSD__ */ > > +#include > +#include > + > /* > * lock and unlock for the netmap memory allocator > */ > @@ -764,8 +765,8 @@ netmap_set_ringid(struct netmap_priv_d * > * Return 0 on success, errno otherwise. > */ > static int > -netmap_ioctl(__unused struct cdev *dev, u_long cmd, caddr_t data, > - __unused int fflag, struct thread *td) > +netmap_ioctl(struct cdev *dev, u_long cmd, caddr_t data, > + int fflag, struct thread *td) > { > struct netmap_priv_d *priv = NULL; > struct ifnet *ifp; > @@ -775,6 +776,8 @@ netmap_ioctl(__unused struct cdev *dev, > u_int i, lim; > struct netmap_if *nifp; > > + (void)dev; /* UNUSED */ > + (void)fflag; /* UNUSED */ > #ifdef linux > #define devfs_get_cdevpriv(pp) \ > ({ *(struct netmap_priv_d **)pp = ((struct file *)td)->private_data; \ > @@ -1551,7 +1554,7 @@ linux_netmap_poll(struct file * file, st > } > > static int > -netmap_mmap(__unused struct file *f, struct vm_area_struct *vma) > +netmap_mmap(struct file *f, struct vm_area_struct *vma) > { > int lut_skip, i, j; > int user_skip = 0; > @@ -1565,6 +1568,7 @@ netmap_mmap(__unused struct file *f, str > * vma->vm_end: end of the mapping user address space > */ > > + (void)f; /* UNUSED */ > // XXX security checks > > for (i = 0; i < 3; i++) { /* loop through obj_pools */ > @@ -1599,7 +1603,7 @@ netmap_start_linux(struct sk_buff *skb, > } > > > -#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,38) > +#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,37) // XXX was 38 > #define LIN_IOCTL_NAME .ioctl > int > linux_netmap_ioctl(struct inode *inode, struct file *file, u_int cmd, u_long data /* arg */) > @@ -1623,8 +1627,9 @@ linux_netmap_ioctl(struct file *file, u_ > > > static int > -netmap_release(__unused struct inode *inode, struct file *file) > +netmap_release(struct inode *inode, struct file *file) > { > + (void)inode; /* UNUSED */ > if (file->private_data) > netmap_dtor(file->private_data); > return (0); > > Modified: head/sys/dev/netmap/netmap_mem2.c > ============================================================================== > --- head/sys/dev/netmap/netmap_mem2.c Mon Jul 30 15:37:47 2012 (r238911) > +++ head/sys/dev/netmap/netmap_mem2.c Mon Jul 30 18:21:48 2012 (r238912) > @@ -25,7 +25,7 @@ > > /* > * $FreeBSD$ > - * $Id: netmap_mem2.c 10830 2012-03-22 18:06:01Z luigi $ > + * $Id: netmap_mem2.c 11445 2012-07-30 10:49:07Z luigi $ > * > * New memory allocator for netmap > */ > @@ -300,12 +300,13 @@ netmap_obj_free_va(struct netmap_obj_poo > > > static void > -netmap_new_bufs(struct netmap_if *nifp __unused, > +netmap_new_bufs(struct netmap_if *nifp, > struct netmap_slot *slot, u_int n) > { > struct netmap_obj_pool *p = nm_mem->nm_buf_pool; > uint32_t i = 0; /* slot counter */ > > + (void)nifp; /* UNUSED */ > for (i = 0; i < n; i++) { > void *vaddr = netmap_buf_malloc(); > if (vaddr == NULL) { > -- I am only one, but I am one. I cannot do everything, but I can do something. And I will not let what I cannot do interfere with what I can do. -- Edward Everett Hale, (1822 - 1909)