nfsd: simplify fh_verify access checks
[safe/jmp/linux-2.6] / net / core / pktgen.c
index 3045dd1..6eb8d47 100644 (file)
 #include <asm/byteorder.h>
 #include <linux/rcupdate.h>
 #include <linux/bitops.h>
-#include <asm/io.h>
+#include <linux/io.h>
+#include <linux/timex.h>
+#include <linux/uaccess.h>
 #include <asm/dma.h>
-#include <asm/uaccess.h>
 #include <asm/div64.h>         /* do_div */
-#include <asm/timex.h>
-
-#define VERSION  "pktgen v2.70: Packet Generator for packet performance testing.\n"
 
+#define VERSION        "2.72"
 #define IP_NAME_SZ 32
 #define MAX_MPLS_LABELS 16 /* This is the max label stack depth */
 #define MPLS_STACK_BOTTOM htonl(0x00000100)
 #define F_QUEUE_MAP_CPU (1<<14)        /* queue map mirrors smp_processor_id() */
 
 /* Thread control flag bits */
-#define T_TERMINATE   (1<<0)
-#define T_STOP        (1<<1)   /* Stop run */
-#define T_RUN         (1<<2)   /* Start run */
-#define T_REMDEVALL   (1<<3)   /* Remove all devs */
-#define T_REMDEV      (1<<4)   /* Remove one dev */
+#define T_STOP        (1<<0)   /* Stop run */
+#define T_RUN         (1<<1)   /* Start run */
+#define T_REMDEVALL   (1<<2)   /* Remove all devs */
+#define T_REMDEV      (1<<3)   /* Remove one dev */
 
 /* If lock -- can be removed after some work */
 #define   if_lock(t)           spin_lock(&(t->if_lock));
 #define PKTGEN_MAGIC 0xbe9be955
 #define PG_PROC_DIR "pktgen"
 #define PGCTRL     "pgctrl"
-static struct proc_dir_entry *pg_proc_dir = NULL;
+static struct proc_dir_entry *pg_proc_dir;
 
 #define MAX_CFLOWS  65536
 
@@ -232,9 +230,9 @@ struct pktgen_dev {
         */
        struct proc_dir_entry *entry;   /* proc file */
        struct pktgen_thread *pg_thread;/* the owner */
-       struct list_head list;          /* Used for chaining in the thread's run-queue */
+       struct list_head list;          /* chaining in the thread's run-queue */
 
-       int running;            /* if this changes to false, the test will stop */
+       int running;            /* if false, the test will stop */
 
        /* If min != max, then we will either do a linear iteration, or
         * we will do a random selection from within the range.
@@ -252,15 +250,16 @@ struct pktgen_dev {
        __u64 count;            /* Default No packets to send */
        __u64 sofar;            /* How many pkts we've sent so far */
        __u64 tx_bytes;         /* How many bytes we've transmitted */
-       __u64 errors;           /* Errors when trying to transmit, pkts will be re-sent */
+       __u64 errors;           /* Errors when trying to transmit,
+                                  pkts will be re-sent */
 
        /* runtime counters relating to clone_skb */
 
        __u64 allocated_skbs;
        __u32 clone_count;
        int last_ok;            /* Was last skb sent?
-                                * Or a failed transmit of some sort?  This will keep
-                                * sequence numbers in order, for example.
+                                * Or a failed transmit of some sort?
+                                * This will keep sequence numbers in order
                                 */
        ktime_t next_tx;
        ktime_t started_at;
@@ -269,11 +268,14 @@ struct pktgen_dev {
 
        __u32 seq_num;
 
-       int clone_skb;          /* Use multiple SKBs during packet gen.  If this number
-                                * is greater than 1, then that many copies of the same
-                                * packet will be sent before a new packet is allocated.
-                                * For instance, if you want to send 1024 identical packets
-                                * before creating a new packet, set clone_skb to 1024.
+       int clone_skb;          /*
+                                * Use multiple SKBs during packet gen.
+                                * If this number is greater than 1, then
+                                * that many copies of the same packet will be
+                                * sent before a new packet is allocated.
+                                * If you want to send 1024 identical packets
+                                * before creating a new packet,
+                                * set clone_skb to 1024.
                                 */
 
        char dst_min[IP_NAME_SZ];       /* IP, ie 1.2.3.4 */
@@ -305,8 +307,10 @@ struct pktgen_dev {
        __u16 udp_dst_max;      /* exclusive, dest UDP port */
 
        /* DSCP + ECN */
-       __u8 tos;            /* six most significant bits of (former) IPv4 TOS are for dscp codepoint */
-       __u8 traffic_class;  /* ditto for the (former) Traffic Class in IPv6 (see RFC 3260, sec. 4) */
+       __u8 tos;            /* six MSB of (former) IPv4 TOS
+                               are for dscp codepoint */
+       __u8 traffic_class;  /* ditto for the (former) Traffic Class in IPv6
+                               (see RFC 3260, sec. 4) */
 
        /* MPLS */
        unsigned nr_labels;     /* Depth of stack, 0 = no MPLS */
@@ -331,6 +335,7 @@ struct pktgen_dev {
        __u32 cur_src_mac_offset;
        __be32 cur_saddr;
        __be32 cur_daddr;
+       __u16 ip_id;
        __u16 cur_udp_dst;
        __u16 cur_udp_src;
        __u16 cur_queue_map;
@@ -347,15 +352,17 @@ struct pktgen_dev {
         */
        __u16 pad;              /* pad out the hh struct to an even 16 bytes */
 
-       struct sk_buff *skb;    /* skb we are to transmit next, mainly used for when we
+       struct sk_buff *skb;    /* skb we are to transmit next, used for when we
                                 * are transmitting the same one multiple times
                                 */
-       struct net_device *odev;        /* The out-going device.  Note that the device should
-                                        * have it's pg_info pointer pointing back to this
-                                        * device.  This will be set when the user specifies
-                                        * the out-going device name (not when the inject is
-                                        * started as it used to do.)
-                                        */
+       struct net_device *odev; /* The out-going device.
+                                 * Note that the device should have it's
+                                 * pg_info pointer pointing back to this
+                                 * device.
+                                 * Set when the user specifies the out-going
+                                 * device name (not when the inject is
+                                 * started as it used to do.)
+                                 */
        struct flow_state *flows;
        unsigned cflows;        /* Concurrent flows (config) */
        unsigned lflow;         /* Flow length  (config) */
@@ -380,13 +387,14 @@ struct pktgen_hdr {
 };
 
 struct pktgen_thread {
-       spinlock_t if_lock;
+       spinlock_t if_lock;             /* for list of devices */
        struct list_head if_list;       /* All device here */
        struct list_head th_list;
        struct task_struct *tsk;
        char result[512];
 
-       /* Field for thread to receive "posted" events terminate, stop ifs etc. */
+       /* Field for thread to receive "posted" events terminate,
+          stop ifs etc. */
 
        u32 control;
        int cpu;
@@ -412,7 +420,8 @@ static inline int ktime_lt(const ktime_t cmp1, const ktime_t cmp2)
        return cmp1.tv64 < cmp2.tv64;
 }
 
-static const char version[] __initconst = VERSION;
+static const char version[] =
+       "pktgen " VERSION ": Packet Generator for packet performance testing.\n";
 
 static int pktgen_remove_device(struct pktgen_thread *t, struct pktgen_dev *i);
 static int pktgen_add_device(struct pktgen_thread *t, const char *ifname);
@@ -449,12 +458,12 @@ static struct notifier_block pktgen_notifier_block = {
 
 static int pgctrl_show(struct seq_file *seq, void *v)
 {
-       seq_puts(seq, VERSION);
+       seq_puts(seq, version);
        return 0;
 }
 
-static ssize_t pgctrl_write(struct file *file, const char __user * buf,
-                           size_t count, loff_t * ppos)
+static ssize_t pgctrl_write(struct file *file, const char __user *buf,
+                           size_t count, loff_t *ppos)
 {
        int err = 0;
        char data[128];
@@ -545,11 +554,14 @@ static int pktgen_if_show(struct seq_file *seq, void *v)
                           "     daddr: %s  min_daddr: %s  max_daddr: %s\n", b1,
                           b2, b3);
 
-       } else
+       } else {
+               seq_printf(seq,
+                          "     dst_min: %s  dst_max: %s\n",
+                          pkt_dev->dst_min, pkt_dev->dst_max);
                seq_printf(seq,
-                          "     dst_min: %s  dst_max: %s\n     src_min: %s  src_max: %s\n",
-                          pkt_dev->dst_min, pkt_dev->dst_max, pkt_dev->src_min,
-                          pkt_dev->src_max);
+                          "        src_min: %s  src_max: %s\n",
+                          pkt_dev->src_min, pkt_dev->src_max);
+       }
 
        seq_puts(seq, "     src_mac: ");
 
@@ -561,7 +573,8 @@ static int pktgen_if_show(struct seq_file *seq, void *v)
        seq_printf(seq, "%pM\n", pkt_dev->dst_mac);
 
        seq_printf(seq,
-                  "     udp_src_min: %d  udp_src_max: %d  udp_dst_min: %d  udp_dst_max: %d\n",
+                  "     udp_src_min: %d  udp_src_max: %d"
+                  "  udp_dst_min: %d  udp_dst_max: %d\n",
                   pkt_dev->udp_src_min, pkt_dev->udp_src_max,
                   pkt_dev->udp_dst_min, pkt_dev->udp_dst_max);
 
@@ -577,23 +590,21 @@ static int pktgen_if_show(struct seq_file *seq, void *v)
                                   i == pkt_dev->nr_labels-1 ? "\n" : ", ");
        }
 
-       if (pkt_dev->vlan_id != 0xffff) {
+       if (pkt_dev->vlan_id != 0xffff)
                seq_printf(seq, "     vlan_id: %u  vlan_p: %u  vlan_cfi: %u\n",
-                          pkt_dev->vlan_id, pkt_dev->vlan_p, pkt_dev->vlan_cfi);
-       }
+                          pkt_dev->vlan_id, pkt_dev->vlan_p,
+                          pkt_dev->vlan_cfi);
 
-       if (pkt_dev->svlan_id != 0xffff) {
+       if (pkt_dev->svlan_id != 0xffff)
                seq_printf(seq, "     svlan_id: %u  vlan_p: %u  vlan_cfi: %u\n",
-                          pkt_dev->svlan_id, pkt_dev->svlan_p, pkt_dev->svlan_cfi);
-       }
+                          pkt_dev->svlan_id, pkt_dev->svlan_p,
+                          pkt_dev->svlan_cfi);
 
-       if (pkt_dev->tos) {
+       if (pkt_dev->tos)
                seq_printf(seq, "     tos: 0x%02x\n", pkt_dev->tos);
-       }
 
-       if (pkt_dev->traffic_class) {
+       if (pkt_dev->traffic_class)
                seq_printf(seq, "     traffic_class: 0x%02x\n", pkt_dev->traffic_class);
-       }
 
        seq_printf(seq, "     Flags: ");
 
@@ -696,7 +707,8 @@ static int pktgen_if_show(struct seq_file *seq, void *v)
 }
 
 
-static int hex32_arg(const char __user *user_buffer, unsigned long maxlen, __u32 *num)
+static int hex32_arg(const char __user *user_buffer, unsigned long maxlen,
+                    __u32 *num)
 {
        int i = 0;
        *num = 0;
@@ -846,9 +858,9 @@ static ssize_t pktgen_if_write(struct file *file,
        /* Read variable name */
 
        len = strn_len(&user_buffer[i], sizeof(name) - 1);
-       if (len < 0) {
+       if (len < 0)
                return len;
-       }
+
        memset(name, 0, sizeof(name));
        if (copy_from_user(name, &user_buffer[i], len))
                return -EFAULT;
@@ -872,9 +884,9 @@ static ssize_t pktgen_if_write(struct file *file,
 
        if (!strcmp(name, "min_pkt_size")) {
                len = num_arg(&user_buffer[i], 10, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if (value < 14 + 20 + 8)
                        value = 14 + 20 + 8;
@@ -889,9 +901,9 @@ static ssize_t pktgen_if_write(struct file *file,
 
        if (!strcmp(name, "max_pkt_size")) {
                len = num_arg(&user_buffer[i], 10, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if (value < 14 + 20 + 8)
                        value = 14 + 20 + 8;
@@ -908,9 +920,9 @@ static ssize_t pktgen_if_write(struct file *file,
 
        if (!strcmp(name, "pkt_size")) {
                len = num_arg(&user_buffer[i], 10, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if (value < 14 + 20 + 8)
                        value = 14 + 20 + 8;
@@ -925,9 +937,9 @@ static ssize_t pktgen_if_write(struct file *file,
 
        if (!strcmp(name, "debug")) {
                len = num_arg(&user_buffer[i], 10, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                debug = value;
                sprintf(pg_result, "OK: debug=%u", debug);
@@ -936,9 +948,9 @@ static ssize_t pktgen_if_write(struct file *file,
 
        if (!strcmp(name, "frags")) {
                len = num_arg(&user_buffer[i], 10, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                pkt_dev->nfrags = value;
                sprintf(pg_result, "OK: frags=%u", pkt_dev->nfrags);
@@ -946,14 +958,14 @@ static ssize_t pktgen_if_write(struct file *file,
        }
        if (!strcmp(name, "delay")) {
                len = num_arg(&user_buffer[i], 10, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if (value == 0x7FFFFFFF)
                        pkt_dev->delay = ULLONG_MAX;
                else
-                       pkt_dev->delay = (u64)value * NSEC_PER_USEC;
+                       pkt_dev->delay = (u64)value;
 
                sprintf(pg_result, "OK: delay=%llu",
                        (unsigned long long) pkt_dev->delay);
@@ -961,9 +973,9 @@ static ssize_t pktgen_if_write(struct file *file,
        }
        if (!strcmp(name, "udp_src_min")) {
                len = num_arg(&user_buffer[i], 10, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if (value != pkt_dev->udp_src_min) {
                        pkt_dev->udp_src_min = value;
@@ -974,9 +986,9 @@ static ssize_t pktgen_if_write(struct file *file,
        }
        if (!strcmp(name, "udp_dst_min")) {
                len = num_arg(&user_buffer[i], 10, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if (value != pkt_dev->udp_dst_min) {
                        pkt_dev->udp_dst_min = value;
@@ -987,9 +999,9 @@ static ssize_t pktgen_if_write(struct file *file,
        }
        if (!strcmp(name, "udp_src_max")) {
                len = num_arg(&user_buffer[i], 10, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if (value != pkt_dev->udp_src_max) {
                        pkt_dev->udp_src_max = value;
@@ -1000,9 +1012,9 @@ static ssize_t pktgen_if_write(struct file *file,
        }
        if (!strcmp(name, "udp_dst_max")) {
                len = num_arg(&user_buffer[i], 10, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if (value != pkt_dev->udp_dst_max) {
                        pkt_dev->udp_dst_max = value;
@@ -1013,9 +1025,9 @@ static ssize_t pktgen_if_write(struct file *file,
        }
        if (!strcmp(name, "clone_skb")) {
                len = num_arg(&user_buffer[i], 10, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                pkt_dev->clone_skb = value;
 
@@ -1024,9 +1036,9 @@ static ssize_t pktgen_if_write(struct file *file,
        }
        if (!strcmp(name, "count")) {
                len = num_arg(&user_buffer[i], 10, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                pkt_dev->count = value;
                sprintf(pg_result, "OK: count=%llu",
@@ -1035,9 +1047,9 @@ static ssize_t pktgen_if_write(struct file *file,
        }
        if (!strcmp(name, "src_mac_count")) {
                len = num_arg(&user_buffer[i], 10, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if (pkt_dev->src_mac_count != value) {
                        pkt_dev->src_mac_count = value;
@@ -1049,9 +1061,9 @@ static ssize_t pktgen_if_write(struct file *file,
        }
        if (!strcmp(name, "dst_mac_count")) {
                len = num_arg(&user_buffer[i], 10, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if (pkt_dev->dst_mac_count != value) {
                        pkt_dev->dst_mac_count = value;
@@ -1065,9 +1077,9 @@ static ssize_t pktgen_if_write(struct file *file,
                char f[32];
                memset(f, 0, 32);
                len = strn_len(&user_buffer[i], sizeof(f) - 1);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                if (copy_from_user(f, &user_buffer[i], len))
                        return -EFAULT;
                i += len;
@@ -1166,9 +1178,8 @@ static ssize_t pktgen_if_write(struct file *file,
        }
        if (!strcmp(name, "dst_min") || !strcmp(name, "dst")) {
                len = strn_len(&user_buffer[i], sizeof(pkt_dev->dst_min) - 1);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
 
                if (copy_from_user(buf, &user_buffer[i], len))
                        return -EFAULT;
@@ -1188,9 +1199,9 @@ static ssize_t pktgen_if_write(struct file *file,
        }
        if (!strcmp(name, "dst_max")) {
                len = strn_len(&user_buffer[i], sizeof(pkt_dev->dst_max) - 1);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
 
                if (copy_from_user(buf, &user_buffer[i], len))
                        return -EFAULT;
@@ -1301,9 +1312,9 @@ static ssize_t pktgen_if_write(struct file *file,
        }
        if (!strcmp(name, "src_min")) {
                len = strn_len(&user_buffer[i], sizeof(pkt_dev->src_min) - 1);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                if (copy_from_user(buf, &user_buffer[i], len))
                        return -EFAULT;
                buf[len] = 0;
@@ -1322,9 +1333,9 @@ static ssize_t pktgen_if_write(struct file *file,
        }
        if (!strcmp(name, "src_max")) {
                len = strn_len(&user_buffer[i], sizeof(pkt_dev->src_max) - 1);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                if (copy_from_user(buf, &user_buffer[i], len))
                        return -EFAULT;
                buf[len] = 0;
@@ -1348,9 +1359,9 @@ static ssize_t pktgen_if_write(struct file *file,
                memcpy(old_dmac, pkt_dev->dst_mac, ETH_ALEN);
 
                len = strn_len(&user_buffer[i], sizeof(valstr) - 1);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                memset(valstr, 0, sizeof(valstr));
                if (copy_from_user(valstr, &user_buffer[i], len))
                        return -EFAULT;
@@ -1390,9 +1401,9 @@ static ssize_t pktgen_if_write(struct file *file,
                memcpy(old_smac, pkt_dev->src_mac, ETH_ALEN);
 
                len = strn_len(&user_buffer[i], sizeof(valstr) - 1);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                memset(valstr, 0, sizeof(valstr));
                if (copy_from_user(valstr, &user_buffer[i], len))
                        return -EFAULT;
@@ -1433,9 +1444,9 @@ static ssize_t pktgen_if_write(struct file *file,
 
        if (!strcmp(name, "flows")) {
                len = num_arg(&user_buffer[i], 10, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if (value > MAX_CFLOWS)
                        value = MAX_CFLOWS;
@@ -1447,9 +1458,9 @@ static ssize_t pktgen_if_write(struct file *file,
 
        if (!strcmp(name, "flowlen")) {
                len = num_arg(&user_buffer[i], 10, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                pkt_dev->lflow = value;
                sprintf(pg_result, "OK: flowlen=%u", pkt_dev->lflow);
@@ -1458,9 +1469,9 @@ static ssize_t pktgen_if_write(struct file *file,
 
        if (!strcmp(name, "queue_map_min")) {
                len = num_arg(&user_buffer[i], 5, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                pkt_dev->queue_map_min = value;
                sprintf(pg_result, "OK: queue_map_min=%u", pkt_dev->queue_map_min);
@@ -1469,9 +1480,9 @@ static ssize_t pktgen_if_write(struct file *file,
 
        if (!strcmp(name, "queue_map_max")) {
                len = num_arg(&user_buffer[i], 5, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                pkt_dev->queue_map_max = value;
                sprintf(pg_result, "OK: queue_map_max=%u", pkt_dev->queue_map_max);
@@ -1503,9 +1514,9 @@ static ssize_t pktgen_if_write(struct file *file,
 
        if (!strcmp(name, "vlan_id")) {
                len = num_arg(&user_buffer[i], 4, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if (value <= 4095) {
                        pkt_dev->vlan_id = value;  /* turn on VLAN */
@@ -1530,9 +1541,9 @@ static ssize_t pktgen_if_write(struct file *file,
 
        if (!strcmp(name, "vlan_p")) {
                len = num_arg(&user_buffer[i], 1, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if ((value <= 7) && (pkt_dev->vlan_id != 0xffff)) {
                        pkt_dev->vlan_p = value;
@@ -1545,9 +1556,9 @@ static ssize_t pktgen_if_write(struct file *file,
 
        if (!strcmp(name, "vlan_cfi")) {
                len = num_arg(&user_buffer[i], 1, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if ((value <= 1) && (pkt_dev->vlan_id != 0xffff)) {
                        pkt_dev->vlan_cfi = value;
@@ -1560,9 +1571,9 @@ static ssize_t pktgen_if_write(struct file *file,
 
        if (!strcmp(name, "svlan_id")) {
                len = num_arg(&user_buffer[i], 4, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if ((value <= 4095) && ((pkt_dev->vlan_id != 0xffff))) {
                        pkt_dev->svlan_id = value;  /* turn on SVLAN */
@@ -1587,9 +1598,9 @@ static ssize_t pktgen_if_write(struct file *file,
 
        if (!strcmp(name, "svlan_p")) {
                len = num_arg(&user_buffer[i], 1, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if ((value <= 7) && (pkt_dev->svlan_id != 0xffff)) {
                        pkt_dev->svlan_p = value;
@@ -1602,9 +1613,9 @@ static ssize_t pktgen_if_write(struct file *file,
 
        if (!strcmp(name, "svlan_cfi")) {
                len = num_arg(&user_buffer[i], 1, &value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if ((value <= 1) && (pkt_dev->svlan_id != 0xffff)) {
                        pkt_dev->svlan_cfi = value;
@@ -1618,9 +1629,9 @@ static ssize_t pktgen_if_write(struct file *file,
        if (!strcmp(name, "tos")) {
                __u32 tmp_value = 0;
                len = hex32_arg(&user_buffer[i], 2, &tmp_value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if (len == 2) {
                        pkt_dev->tos = tmp_value;
@@ -1634,9 +1645,9 @@ static ssize_t pktgen_if_write(struct file *file,
        if (!strcmp(name, "traffic_class")) {
                __u32 tmp_value = 0;
                len = hex32_arg(&user_buffer[i], 2, &tmp_value);
-               if (len < 0) {
+               if (len < 0)
                        return len;
-               }
+
                i += len;
                if (len == 2) {
                        pkt_dev->traffic_class = tmp_value;
@@ -1871,8 +1882,10 @@ static void pktgen_change_name(struct net_device *dev)
 
                        remove_proc_entry(pkt_dev->entry->name, pg_proc_dir);
 
-                       pkt_dev->entry = create_proc_entry(dev->name, 0600,
-                                                          pg_proc_dir);
+                       pkt_dev->entry = proc_create_data(dev->name, 0600,
+                                                         pg_proc_dir,
+                                                         &pktgen_if_fops,
+                                                         pkt_dev);
                        if (!pkt_dev->entry)
                                printk(KERN_ERR "pktgen: can't move proc "
                                       " entry for '%s'\n", dev->name);
@@ -1906,13 +1919,14 @@ static int pktgen_device_event(struct notifier_block *unused,
        return NOTIFY_DONE;
 }
 
-static struct net_device *pktgen_dev_get_by_name(struct pktgen_dev *pkt_dev, const char *ifname)
+static struct net_device *pktgen_dev_get_by_name(struct pktgen_dev *pkt_dev,
+                                                const char *ifname)
 {
        char b[IFNAMSIZ+5];
        int i = 0;
 
-       for(i=0; ifname[i] != '@'; i++) {
-               if(i == IFNAMSIZ)
+       for (i = 0; ifname[i] != '@'; i++) {
+               if (i == IFNAMSIZ)
                        break;
 
                b[i] = ifname[i];
@@ -1979,7 +1993,7 @@ static void pktgen_setup_inject(struct pktgen_dev *pkt_dev)
                printk(KERN_WARNING "pktgen: WARNING: Requested "
                       "queue_map_min (zero-based) (%d) exceeds valid range "
                       "[0 - %d] for (%d) queues on %s, resetting\n",
-                      pkt_dev->queue_map_min, (ntxq ?: 1)- 1, ntxq,
+                      pkt_dev->queue_map_min, (ntxq ?: 1) - 1, ntxq,
                       pkt_dev->odev->name);
                pkt_dev->queue_map_min = ntxq - 1;
        }
@@ -1987,7 +2001,7 @@ static void pktgen_setup_inject(struct pktgen_dev *pkt_dev)
                printk(KERN_WARNING "pktgen: WARNING: Requested "
                       "queue_map_max (zero-based) (%d) exceeds valid range "
                       "[0 - %d] for (%d) queues on %s, resetting\n",
-                      pkt_dev->queue_map_max, (ntxq ?: 1)- 1, ntxq,
+                      pkt_dev->queue_map_max, (ntxq ?: 1) - 1, ntxq,
                       pkt_dev->odev->name);
                pkt_dev->queue_map_max = ntxq - 1;
        }
@@ -2028,7 +2042,8 @@ static void pktgen_setup_inject(struct pktgen_dev *pkt_dev)
                         */
 
                        rcu_read_lock();
-                       if ((idev = __in6_dev_get(pkt_dev->odev)) != NULL) {
+                       idev = __in6_dev_get(pkt_dev->odev);
+                       if (idev) {
                                struct inet6_ifaddr *ifp;
 
                                read_lock_bh(&idev->lock);
@@ -2090,18 +2105,20 @@ static void pktgen_setup_inject(struct pktgen_dev *pkt_dev)
 
 static void spin(struct pktgen_dev *pkt_dev, ktime_t spin_until)
 {
-       ktime_t start;
-       s32 remaining;
+       ktime_t start_time, end_time;
+       s64 remaining;
        struct hrtimer_sleeper t;
 
        hrtimer_init_on_stack(&t.timer, CLOCK_MONOTONIC, HRTIMER_MODE_ABS);
        hrtimer_set_expires(&t.timer, spin_until);
 
        remaining = ktime_to_us(hrtimer_expires_remaining(&t.timer));
-       if (remaining <= 0)
+       if (remaining <= 0) {
+               pkt_dev->next_tx = ktime_add_ns(spin_until, pkt_dev->delay);
                return;
+       }
 
-       start = ktime_now();
+       start_time = ktime_now();
        if (remaining < 100)
                udelay(remaining);      /* really small just spin */
        else {
@@ -2120,7 +2137,10 @@ static void spin(struct pktgen_dev *pkt_dev, ktime_t spin_until)
                } while (t.task && pkt_dev->running && !signal_pending(current));
                __set_current_state(TASK_RUNNING);
        }
-       pkt_dev->idle_acc += ktime_to_ns(ktime_sub(ktime_now(), start));
+       end_time = ktime_now();
+
+       pkt_dev->idle_acc += ktime_to_ns(ktime_sub(end_time, start_time));
+       pkt_dev->next_tx = ktime_add_ns(end_time, pkt_dev->delay);
 }
 
 static inline void set_pkt_overhead(struct pktgen_dev *pkt_dev)
@@ -2181,7 +2201,7 @@ static void get_ipsec_sa(struct pktgen_dev *pkt_dev, int flow)
                if (x) {
                        pkt_dev->flows[flow].x = x;
                        set_pkt_overhead(pkt_dev);
-                       pkt_dev->pkt_overhead+=x->props.header_len;
+                       pkt_dev->pkt_overhead += x->props.header_len;
                }
 
        }
@@ -2193,7 +2213,7 @@ static void set_cur_queue_map(struct pktgen_dev *pkt_dev)
        if (pkt_dev->flags & F_QUEUE_MAP_CPU)
                pkt_dev->cur_queue_map = smp_processor_id();
 
-       else if (pkt_dev->queue_map_min < pkt_dev->queue_map_max) {
+       else if (pkt_dev->queue_map_min <= pkt_dev->queue_map_max) {
                __u16 t;
                if (pkt_dev->flags & F_QUEUE_MAP_RND) {
                        t = random32() %
@@ -2320,18 +2340,18 @@ static void mod_cur_headers(struct pktgen_dev *pkt_dev)
 
        if (!(pkt_dev->flags & F_IPV6)) {
 
-               if ((imn = ntohl(pkt_dev->saddr_min)) < (imx =
-                                                        ntohl(pkt_dev->
-                                                              saddr_max))) {
+               imn = ntohl(pkt_dev->saddr_min);
+               imx = ntohl(pkt_dev->saddr_max);
+               if (imn < imx) {
                        __u32 t;
                        if (pkt_dev->flags & F_IPSRC_RND)
                                t = random32() % (imx - imn) + imn;
                        else {
                                t = ntohl(pkt_dev->cur_saddr);
                                t++;
-                               if (t > imx) {
+                               if (t > imx)
                                        t = imn;
-                               }
+
                        }
                        pkt_dev->cur_saddr = htonl(t);
                }
@@ -2442,7 +2462,7 @@ static int pktgen_output_ipsec(struct sk_buff *skb, struct pktgen_dev *pkt_dev)
        if (err)
                goto error;
 
-       x->curlft.bytes +=skb->len;
+       x->curlft.bytes += skb->len;
        x->curlft.packets++;
 error:
        spin_unlock(&x->lock);
@@ -2474,11 +2494,11 @@ static int process_ipsec(struct pktgen_dev *pkt_dev,
                        int ret;
                        __u8 *eth;
                        nhead = x->props.header_len - skb_headroom(skb);
-                       if (nhead >0) {
+                       if (nhead > 0) {
                                ret = pskb_expand_head(skb, nhead, 0, GFP_ATOMIC);
                                if (ret < 0) {
                                        printk(KERN_ERR "Error expanding "
-                                              "ipsec packet %d\n",ret);
+                                              "ipsec packet %d\n", ret);
                                        goto err;
                                }
                        }
@@ -2488,13 +2508,13 @@ static int process_ipsec(struct pktgen_dev *pkt_dev,
                        ret = pktgen_output_ipsec(skb, pkt_dev);
                        if (ret) {
                                printk(KERN_ERR "Error creating ipsec "
-                                      "packet %d\n",ret);
+                                      "packet %d\n", ret);
                                goto err;
                        }
                        /* restore ll */
                        eth = (__u8 *) skb_push(skb, ETH_HLEN);
                        memcpy(eth, pkt_dev->hh, 12);
-                       *(u16 *) & eth[12] = protocol;
+                       *(u16 *) &eth[12] = protocol;
                }
        }
        return 1;
@@ -2507,9 +2527,9 @@ err:
 static void mpls_push(__be32 *mpls, struct pktgen_dev *pkt_dev)
 {
        unsigned i;
-       for (i = 0; i < pkt_dev->nr_labels; i++) {
+       for (i = 0; i < pkt_dev->nr_labels; i++)
                *mpls++ = pkt_dev->labels[i] & ~MPLS_STACK_BOTTOM;
-       }
+
        mpls--;
        *mpls |= MPLS_STACK_BOTTOM;
 }
@@ -2611,6 +2631,8 @@ static struct sk_buff *fill_packet_ipv4(struct net_device *odev,
        iph->protocol = IPPROTO_UDP;    /* UDP */
        iph->saddr = pkt_dev->cur_saddr;
        iph->daddr = pkt_dev->cur_daddr;
+       iph->id = htons(pkt_dev->ip_id);
+       pkt_dev->ip_id++;
        iph->frag_off = 0;
        iplen = 20 + 8 + datalen;
        iph->tot_len = htons(iplen);
@@ -2622,24 +2644,26 @@ static struct sk_buff *fill_packet_ipv4(struct net_device *odev,
        skb->dev = odev;
        skb->pkt_type = PACKET_HOST;
 
-       if (pkt_dev->nfrags <= 0)
+       if (pkt_dev->nfrags <= 0) {
                pgh = (struct pktgen_hdr *)skb_put(skb, datalen);
-       else {
+               memset(pgh + 1, 0, datalen - sizeof(struct pktgen_hdr));
+       } else {
                int frags = pkt_dev->nfrags;
-               int i;
+               int i, len;
 
                pgh = (struct pktgen_hdr *)(((char *)(udph)) + 8);
 
                if (frags > MAX_SKB_FRAGS)
                        frags = MAX_SKB_FRAGS;
                if (datalen > frags * PAGE_SIZE) {
-                       skb_put(skb, datalen - frags * PAGE_SIZE);
+                       len = datalen - frags * PAGE_SIZE;
+                       memset(skb_put(skb, len), 0, len);
                        datalen = frags * PAGE_SIZE;
                }
 
                i = 0;
                while (datalen > 0) {
-                       struct page *page = alloc_pages(GFP_KERNEL, 0);
+                       struct page *page = alloc_pages(GFP_KERNEL | __GFP_ZERO, 0);
                        skb_shinfo(skb)->frags[i].page = page;
                        skb_shinfo(skb)->frags[i].page_offset = 0;
                        skb_shinfo(skb)->frags[i].size =
@@ -2676,8 +2700,9 @@ static struct sk_buff *fill_packet_ipv4(struct net_device *odev,
                }
        }
 
-       /* Stamp the time, and sequence number, convert them to network byte order */
-
+       /* Stamp the time, and sequence number,
+        * convert them to network byte order
+        */
        if (pgh) {
                struct timeval timestamp;
 
@@ -2931,7 +2956,7 @@ static struct sk_buff *fill_packet_ipv6(struct net_device *odev,
        udph = udp_hdr(skb);
 
        memcpy(eth, pkt_dev->hh, 12);
-       *(__be16 *) & eth[12] = protocol;
+       *(__be16 *) &eth[12] = protocol;
 
        /* Eth + IPh + UDPh + mpls */
        datalen = pkt_dev->cur_pkt_size - 14 -
@@ -3025,8 +3050,10 @@ static struct sk_buff *fill_packet_ipv6(struct net_device *odev,
                }
        }
 
-       /* Stamp the time, and sequence number, convert them to network byte order */
-       /* should we update cloned packets too ? */
+       /* Stamp the time, and sequence number,
+        * convert them to network byte order
+        * should we update cloned packets too ?
+        */
        if (pgh) {
                struct timeval timestamp;
 
@@ -3174,7 +3201,8 @@ static void pktgen_run_all_threads(void)
 
        mutex_unlock(&pktgen_thread_lock);
 
-       schedule_timeout_interruptible(msecs_to_jiffies(125));  /* Propagate thread->control  */
+       /* Propagate thread->control  */
+       schedule_timeout_interruptible(msecs_to_jiffies(125));
 
        pktgen_wait_all_threads_run();
 }
@@ -3192,7 +3220,8 @@ static void pktgen_reset_all_threads(void)
 
        mutex_unlock(&pktgen_thread_lock);
 
-       schedule_timeout_interruptible(msecs_to_jiffies(125));  /* Propagate thread->control  */
+       /* Propagate thread->control  */
+       schedule_timeout_interruptible(msecs_to_jiffies(125));
 
        pktgen_wait_all_threads_run();
 }
@@ -3345,58 +3374,53 @@ static void pktgen_rem_thread(struct pktgen_thread *t)
        mutex_unlock(&pktgen_thread_lock);
 }
 
-static void idle(struct pktgen_dev *pkt_dev)
+static void pktgen_resched(struct pktgen_dev *pkt_dev)
 {
        ktime_t idle_start = ktime_now();
+       schedule();
+       pkt_dev->idle_acc += ktime_to_ns(ktime_sub(ktime_now(), idle_start));
+}
 
-       if (need_resched())
-               schedule();
-       else
-               cpu_relax();
+static void pktgen_wait_for_skb(struct pktgen_dev *pkt_dev)
+{
+       ktime_t idle_start = ktime_now();
+
+       while (atomic_read(&(pkt_dev->skb->users)) != 1) {
+               if (signal_pending(current))
+                       break;
 
+               if (need_resched())
+                       pktgen_resched(pkt_dev);
+               else
+                       cpu_relax();
+       }
        pkt_dev->idle_acc += ktime_to_ns(ktime_sub(ktime_now(), idle_start));
 }
 
-
 static void pktgen_xmit(struct pktgen_dev *pkt_dev)
 {
        struct net_device *odev = pkt_dev->odev;
-       int (*xmit)(struct sk_buff *, struct net_device *)
+       netdev_tx_t (*xmit)(struct sk_buff *, struct net_device *)
                = odev->netdev_ops->ndo_start_xmit;
        struct netdev_queue *txq;
        u16 queue_map;
        int ret;
 
-       if (pkt_dev->delay) {
-               spin(pkt_dev, pkt_dev->next_tx);
-
-               /* This is max DELAY, this has special meaning of
-                * "never transmit"
-                */
-               if (pkt_dev->delay == ULLONG_MAX) {
-                       pkt_dev->next_tx = ktime_add_ns(ktime_now(), ULONG_MAX);
-                       return;
-               }
-       }
-
-       if (!pkt_dev->skb) {
-               set_cur_queue_map(pkt_dev);
-               queue_map = pkt_dev->cur_queue_map;
-       } else {
-               queue_map = skb_get_queue_mapping(pkt_dev->skb);
+       /* If device is offline, then don't send */
+       if (unlikely(!netif_running(odev) || !netif_carrier_ok(odev))) {
+               pktgen_stop_device(pkt_dev);
+               return;
        }
 
-       txq = netdev_get_tx_queue(odev, queue_map);
-       /* Did we saturate the queue already? */
-       if (netif_tx_queue_stopped(txq) || netif_tx_queue_frozen(txq)) {
-               /* If device is down, then all queues are permnantly frozen */
-               if (netif_running(odev))
-                       idle(pkt_dev);
-               else
-                       pktgen_stop_device(pkt_dev);
+       /* This is max DELAY, this has special meaning of
+        * "never transmit"
+        */
+       if (unlikely(pkt_dev->delay == ULLONG_MAX)) {
+               pkt_dev->next_tx = ktime_add_ns(ktime_now(), ULONG_MAX);
                return;
        }
 
+       /* If no skb or clone count exhausted then get new one */
        if (!pkt_dev->skb || (pkt_dev->last_ok &&
                              ++pkt_dev->clone_count >= pkt_dev->clone_skb)) {
                /* build a new pkt */
@@ -3415,60 +3439,45 @@ static void pktgen_xmit(struct pktgen_dev *pkt_dev)
                pkt_dev->clone_count = 0;       /* reset counter */
        }
 
-       /* fill_packet() might have changed the queue */
+       if (pkt_dev->delay && pkt_dev->last_ok)
+               spin(pkt_dev, pkt_dev->next_tx);
+
        queue_map = skb_get_queue_mapping(pkt_dev->skb);
        txq = netdev_get_tx_queue(odev, queue_map);
 
        __netif_tx_lock_bh(txq);
-       if (unlikely(netif_tx_queue_stopped(txq) || netif_tx_queue_frozen(txq)))
-               pkt_dev->last_ok = 0;
-       else {
-               atomic_inc(&(pkt_dev->skb->users));
+       atomic_inc(&(pkt_dev->skb->users));
 
-       retry_now:
+       if (unlikely(netif_tx_queue_stopped(txq) || netif_tx_queue_frozen(txq)))
+               ret = NETDEV_TX_BUSY;
+       else
                ret = (*xmit)(pkt_dev->skb, odev);
-               switch (ret) {
-               case NETDEV_TX_OK:
-                       txq_trans_update(txq);
-                       pkt_dev->last_ok = 1;
-                       pkt_dev->sofar++;
-                       pkt_dev->seq_num++;
-                       pkt_dev->tx_bytes += pkt_dev->cur_pkt_size;
-                       break;
-               case NETDEV_TX_LOCKED:
-                       cpu_relax();
-                       goto retry_now;
-               default: /* Drivers are not supposed to return other values! */
-                       if (net_ratelimit())
-                               pr_info("pktgen: %s xmit error: %d\n",
-                                       odev->name, ret);
-                       pkt_dev->errors++;
-                       /* fallthru */
-               case NETDEV_TX_BUSY:
-                       /* Retry it next time */
-                       atomic_dec(&(pkt_dev->skb->users));
-                       pkt_dev->last_ok = 0;
-               }
-
-               if (pkt_dev->delay)
-                       pkt_dev->next_tx = ktime_add_ns(ktime_now(),
-                                                       pkt_dev->delay);
+
+       switch (ret) {
+       case NETDEV_TX_OK:
+               txq_trans_update(txq);
+               pkt_dev->last_ok = 1;
+               pkt_dev->sofar++;
+               pkt_dev->seq_num++;
+               pkt_dev->tx_bytes += pkt_dev->cur_pkt_size;
+               break;
+       default: /* Drivers are not supposed to return other values! */
+               if (net_ratelimit())
+                       pr_info("pktgen: %s xmit error: %d\n",
+                               odev->name, ret);
+               pkt_dev->errors++;
+               /* fallthru */
+       case NETDEV_TX_LOCKED:
+       case NETDEV_TX_BUSY:
+               /* Retry it next time */
+               atomic_dec(&(pkt_dev->skb->users));
+               pkt_dev->last_ok = 0;
        }
        __netif_tx_unlock_bh(txq);
 
        /* If pkt_dev->count is zero, then run forever */
        if ((pkt_dev->count != 0) && (pkt_dev->sofar >= pkt_dev->count)) {
-               if (atomic_read(&(pkt_dev->skb->users)) != 1) {
-                       ktime_t idle_start = ktime_now();
-                       while (atomic_read(&(pkt_dev->skb->users)) != 1) {
-                               if (signal_pending(current)) {
-                                       break;
-                               }
-                               schedule();
-                       }
-                       pkt_dev->idle_acc += ktime_to_ns(ktime_sub(ktime_now(),
-                                                                  idle_start));
-               }
+               pktgen_wait_for_skb(pkt_dev);
 
                /* Done with this */
                pktgen_stop_device(pkt_dev);
@@ -3491,7 +3500,8 @@ static int pktgen_thread_worker(void *arg)
        init_waitqueue_head(&t->queue);
        complete(&t->start_done);
 
-       pr_debug("pktgen: starting pktgen/%d:  pid=%d\n", cpu, task_pid_nr(current));
+       pr_debug("pktgen: starting pktgen/%d:  pid=%d\n",
+                cpu, task_pid_nr(current));
 
        set_current_state(TASK_INTERRUPTIBLE);
 
@@ -3500,20 +3510,24 @@ static int pktgen_thread_worker(void *arg)
        while (!kthread_should_stop()) {
                pkt_dev = next_to_run(t);
 
-               if (!pkt_dev &&
-                   (t->control & (T_STOP | T_RUN | T_REMDEVALL | T_REMDEV))
-                   == 0) {
-                       prepare_to_wait(&(t->queue), &wait,
-                                       TASK_INTERRUPTIBLE);
-                       schedule_timeout(HZ / 10);
-                       finish_wait(&(t->queue), &wait);
+               if (unlikely(!pkt_dev && t->control == 0)) {
+                       wait_event_interruptible_timeout(t->queue,
+                                                        t->control != 0,
+                                                        HZ/10);
+                       continue;
                }
 
                __set_current_state(TASK_RUNNING);
 
-               if (pkt_dev)
+               if (likely(pkt_dev)) {
                        pktgen_xmit(pkt_dev);
 
+                       if (need_resched())
+                               pktgen_resched(pkt_dev);
+                       else
+                               cpu_relax();
+               }
+
                if (t->control & T_STOP) {
                        pktgen_stop(t);
                        t->control &= ~(T_STOP);
@@ -3838,10 +3852,15 @@ static void __exit pg_cleanup(void)
 module_init(pg_init);
 module_exit(pg_cleanup);
 
-MODULE_AUTHOR("Robert Olsson <robert.olsson@its.uu.se");
+MODULE_AUTHOR("Robert Olsson <robert.olsson@its.uu.se>");
 MODULE_DESCRIPTION("Packet Generator tool");
 MODULE_LICENSE("GPL");
+MODULE_VERSION(VERSION);
 module_param(pg_count_d, int, 0);
+MODULE_PARM_DESC(pg_count_d, "Default number of packets to inject");
 module_param(pg_delay_d, int, 0);
+MODULE_PARM_DESC(pg_delay_d, "Default delay between packets (nanoseconds)");
 module_param(pg_clone_skb_d, int, 0);
+MODULE_PARM_DESC(pg_clone_skb_d, "Default number of copies of the same packet");
 module_param(debug, int, 0);
+MODULE_PARM_DESC(debug, "Enable debugging of pktgen module");