netmap: fix all the cdev prototypes
authorFranco Fichtner <franco@lastsummer.de>
Wed, 1 Jan 2014 20:08:08 +0000 (21:08 +0100)
committerFranco Fichtner <franco@lastsummer.de>
Wed, 1 Jan 2014 20:25:43 +0000 (21:25 +0100)
While there, reduce diff noise against vanilla sources.

sys/net/netmap/netmap.c
sys/net/netmap/netmap_freebsd.c
sys/net/netmap/netmap_generic.c
sys/net/netmap/netmap_kern.h

index 5eda96d..84ed662 100644 (file)
@@ -138,19 +138,17 @@ ports attached to the switch)
 #include <sys/sockio.h>
 #include <sys/socketvar.h>     /* struct socket */
 #include <sys/malloc.h>
-#include <sys/kernel.h>
-#include <sys/queue.h>
-#include <sys/event.h>
 #include <sys/poll.h>
 #include <sys/lock.h>
 #include <sys/socket.h> /* sockaddrs */
+#include <sys/event.h>
 #include <sys/sysctl.h>
-#include <sys/bus.h>   /* bus_dmamap_* */
-#include <sys/endian.h>
-#include <sys/refcount.h>
 #include <net/if.h>
 #include <net/if_var.h>
 #include <net/bpf.h>           /* BIOCIMMEDIATE */
+#include <sys/bus.h>   /* bus_dmamap_* */
+#include <sys/endian.h>
+#include <sys/refcount.h>
 
 /* reduce conditional code */
 #define init_waitqueue_head(x) // only needed in linux
@@ -1283,20 +1281,17 @@ out:
  * Return 0 on success, errno otherwise.
  */
 int
-netmap_ioctl(struct cdev *dev, u_long cmd, caddr_t data,
-       int fflag, struct thread *td)
+netmap_ioctl(struct dev_ioctl_args *ap)
 {
        struct netmap_priv_d *priv = NULL;
        struct ifnet *ifp = NULL;
-       struct nmreq *nmr = (struct nmreq *) data;
+       struct nmreq *nmr = (struct nmreq *) ap->a_data;
        struct netmap_adapter *na = NULL;
        int error;
        u_int i, lim;
        struct netmap_if *nifp;
        struct netmap_kring *krings;
-
-       (void)dev;      /* UNUSED */
-       (void)fflag;    /* UNUSED */
+       u_long cmd = ap->a_cmd;
 
 #if 0
        error = devfs_get_cdevpriv((void **)&priv);
@@ -1507,7 +1502,7 @@ netmap_ioctl(struct cdev *dev, u_long cmd, caddr_t data,
                }
                ifp = na->ifp;
                // so->so_proto not null.
-               error = ifioctl(&so, cmd, data, td);
+               error = ifioctl(&so, cmd, ap->a_data, ap->a_cred);
                netmap_adapter_put(na);
                NMG_UNLOCK();
                break;
index 11b5cd0..3fbe002 100644 (file)
 #include <sys/types.h>
 #include <sys/module.h>
 #include <sys/errno.h>
-#include <sys/devfs.h>
 #include <sys/param.h>  /* defines used in kernel.h */
 #include <sys/kernel.h> /* types used in module initialization */
 #include <sys/conf.h>  /* DEV_MODULE */
-#include <sys/bus.h>   /* bus_dmamap_* */
-#include <sys/malloc.h>
-#include <sys/socket.h> /* sockaddrs */
+#include <sys/devfs.h>
 
 #include <vm/vm.h>      /* vtophys */
 #include <vm/pmap.h>    /* vtophys */
 #include <vm/vm_page2.h>
 #include <vm/vm_pager.h>
 
+
+#include <sys/malloc.h>
+#include <sys/socket.h> /* sockaddrs */
+#include <sys/event.h>
 #include <net/if.h>
 #include <net/if_var.h>
-#include <net/netmap.h>
+#include <sys/bus.h>   /* bus_dmamap_* */
 
+#include <net/netmap.h>
 #include "netmap_kern.h"
 #include "netmap_mem2.h"
 
@@ -278,12 +280,16 @@ static struct cdev_pager_ops netmap_cdev_pager_ops = {
 
 
 static int
-netmap_mmap_single(struct cdev *cdev, vm_ooffset_t *foff,
-       vm_size_t objsize,  vm_object_t *objp, int prot)
+netmap_mmap_single(struct dev_mmap_single_args *ap)
 {
        int error;
+       struct cdev *cdev = ap->a_head.a_dev;
+       vm_ooffset_t *foff = ap->a_offset;
+       vm_object_t *objp = ap->a_object;
+       vm_size_t objsize = ap->a_size;
        struct netmap_vm_handle_t *vmh;
        struct netmap_priv_d *priv;
+       int prot = ap->a_nprot;
        vm_object_t obj;
 
        D("cdev %p foff %jd size %jd objp %p prot %d", cdev,
@@ -336,27 +342,24 @@ err_unlock:
 
 // XXX can we remove this ?
 static int
-netmap_close(struct cdev *dev, int fflag, int devtype, struct thread *td)
+netmap_close(struct dev_close_args *ap)
 {
        if (netmap_verbose)
-               D("dev %p fflag 0x%x devtype %d td %p",
-                       dev, fflag, devtype, td);
+               D("dev %p fflag 0x%x devtype %d",
+                       ap->a_head.a_dev, ap->a_fflag, ap->a_devtype);
        return 0;
 }
 
 
 static int
-netmap_open(struct cdev *dev, int oflags, int devtype, struct thread *td)
+netmap_open(struct dev_open_args *ap)
 {
        struct netmap_priv_d *priv;
 #if 0
        int error;
 #endif
 
-       (void)dev;
-       (void)oflags;
-       (void)devtype;
-       (void)td;
+       (void)ap;
 
        // XXX wait or nowait ?
        priv = kmalloc(sizeof(struct netmap_priv_d), M_DEVBUF,
@@ -377,15 +380,14 @@ netmap_open(struct cdev *dev, int oflags, int devtype, struct thread *td)
 
 
 struct dev_ops netmap_cdevsw = {
-#if 0
-       .d_version = D_VERSION,
-       .d_name = "netmap",
+       { "netmap", 0, 0 },
        .d_open = netmap_open,
        .d_mmap_single = netmap_mmap_single,
        .d_ioctl = netmap_ioctl,
+#if 0
        .d_poll = netmap_poll,
-       .d_close = netmap_close,
 #endif
+       .d_close = netmap_close,
 };
 
 
index 5979906..0b95136 100644 (file)
 #include <sys/types.h>
 #include <sys/errno.h>
 #include <sys/malloc.h>
-#include <sys/event.h>
 #include <sys/lock.h>   /* PROT_EXEC */
 #include <sys/socket.h> /* sockaddrs */
+#include <sys/event.h>
 #include <net/if.h>
 #include <net/if_var.h>
 #include <sys/bus.h>        /* bus_dmamap_* in netmap_kern.h */
 
 // XXX temporary - D() defined here
 #include <net/netmap.h>
-
 #include "netmap_kern.h"
 #include "netmap_mem2.h"
 
index aae74da..15dfbd6 100644 (file)
@@ -303,8 +303,7 @@ struct netmap_adapter {
        void *tailroom;                /* space below the rings array */
                                       /* (used for leases) */
 
-       NM_SELINFO_T tx_si;             /* global tx wait queue */
-       NM_SELINFO_T rx_si;             /* global rx wait queue */
+       NM_SELINFO_T tx_si, rx_si;      /* global wait queues */
 
        /* copy of if_qflush and if_transmit pointers, to intercept
         * packets from the network stack when netmap is active.
@@ -674,7 +673,8 @@ int netmap_get_memory(struct netmap_priv_d* p);
 void netmap_dtor(void *data);
 int netmap_dtor_locked(struct netmap_priv_d *priv);
 
-int netmap_ioctl(struct cdev *dev, u_long cmd, caddr_t data, int fflag, struct thread *td);
+struct dev_ioctl_args; /* XXX this shouldn't be here */
+int netmap_ioctl(struct dev_ioctl_args *ap);
 
 /* netmap_adapter creation/destruction */
 #define NM_IFPNAME(ifp) ((ifp) ? (ifp)->if_xname : "zombie")