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 <net/if.h>
>  #include <net/bpf.h>         /* BIOCIMMEDIATE */
>  #include <net/vnet.h>
> -#include <net/netmap.h>
> -#include <dev/netmap/netmap_kern.h>
>  #include <machine/bus.h>     /* bus_dmamap_* */
>  
>  MALLOC_DEFINE(M_NETMAP, "netmap", "Network memory map");
>  #endif /* __FreeBSD__ */
>  
> +#include <net/netmap.h>
> +#include <dev/netmap/netmap_kern.h>
> +
>  /*
>   * 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)
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to