Cleanup #define maze leftover from pftop compatibility.
authorcanacar <canacar@openbsd.org>
Wed, 16 Jul 2008 10:23:39 +0000 (10:23 +0000)
committercanacar <canacar@openbsd.org>
Wed, 16 Jul 2008 10:23:39 +0000 (10:23 +0000)
Pointed out by mcbride@, makes sense henning@

usr.bin/systat/cache.c
usr.bin/systat/cache.h
usr.bin/systat/config.h [deleted file]
usr.bin/systat/pftop.c

index 136f887..38c9319 100644 (file)
@@ -1,4 +1,4 @@
-/* $Id: cache.c,v 1.1 2008/06/12 22:26:01 canacar Exp $ */
+/* $Id: cache.c,v 1.2 2008/07/16 10:23:39 canacar Exp $ */
 /*
  * Copyright (c) 2001, 2007 Can Erkin Acar <canacar@openbsd.org>
  *
@@ -38,8 +38,8 @@
 #include "cache.h"
 
 /* prototypes */
-void update_state(struct sc_ent *, pf_state_t *, double);
-struct sc_ent *cache_state(pf_state_t *);
+void update_state(struct sc_ent *, struct pfsync_state *, double);
+struct sc_ent *cache_state(struct pfsync_state *);
 static __inline int sc_cmp(struct sc_ent *s1, struct sc_ent *s2);
 
 /* initialize the tree and queue */
@@ -91,22 +91,18 @@ cache_init(int max)
 }
 
 void
-update_state(struct sc_ent *prev, pf_state_t *new, double rate)
+update_state(struct sc_ent *prev, struct pfsync_state *new, double rate)
 {
        assert (prev != NULL && new != NULL);
        prev->t = time(NULL);
        prev->rate = rate;
-#ifdef HAVE_INOUT_COUNT
        prev->bytes = COUNTER(new->bytes[0]) + COUNTER(new->bytes[1]);
-#else
-       prev->bytes = COUNTER(new->bytes);
-#endif
        if (prev->peak < rate)
                prev->peak = rate;
 }
 
 void
-add_state(pf_state_t *st)
+add_state(struct pfsync_state *st)
 {
        struct sc_ent *ent;
        assert(st != NULL);
@@ -122,22 +118,9 @@ add_state(pf_state_t *st)
 
        cache_size--;
 
-#ifdef HAVE_PFSYNC_STATE
        ent->id[0] = st->id[0];
        ent->id[1] = st->id[1];
-#else
-       ent->addr[0] = st->lan.addr;
-       ent->port[0] = st->lan.port;
-       ent->addr[1] = st->ext.addr;
-       ent->port[1] = st->ext.port;
-       ent->af = st->af;
-       ent->proto = st->proto;
-#endif
-#ifdef HAVE_INOUT_COUNT
        ent->bytes = COUNTER(st->bytes[0]) + COUNTER(st->bytes[1]);
-#else
-       ent->bytes = st->bytes;
-#endif
        ent->peak = 0;
        ent->rate = 0;
        ent->t = time(NULL);
@@ -148,7 +131,7 @@ add_state(pf_state_t *st)
 
 /* must be called only once for each state before cache_endupdate */
 struct sc_ent *
-cache_state(pf_state_t *st)
+cache_state(struct pfsync_state *st)
 {
        struct sc_ent ent, *old;
        double sd, td, r;
@@ -156,17 +139,8 @@ cache_state(pf_state_t *st)
        if (cache_max == 0)
                return (NULL);
 
-#ifdef HAVE_PFSYNC_STATE
        ent.id[0] = st->id[0];
        ent.id[1] = st->id[1];
-#else
-       ent.addr[0] = st->lan.addr;
-       ent.port[0] = st->lan.port;
-       ent.addr[1] = st->ext.addr;
-       ent.port[1] = st->ext.port;
-       ent.af = st->af;
-       ent.proto = st->proto;
-#endif
        old = RB_FIND(sc_tree, &sctree, &ent);
 
        if (old == NULL) {
@@ -174,18 +148,10 @@ cache_state(pf_state_t *st)
                return (NULL);
        }
 
-#ifdef HAVE_INOUT_COUNT
        if (COUNTER(st->bytes[0]) + COUNTER(st->bytes[1]) < old->bytes)
                return (NULL);
 
        sd = COUNTER(st->bytes[0]) + COUNTER(st->bytes[1]) - old->bytes;
-#else
-       if (st->bytes < old->bytes)
-               return (NULL);
-
-       sd = st->bytes - old->bytes;
-#endif
-
        td = time(NULL) - old->t;
 
        if (td > 0) {
@@ -223,7 +189,6 @@ cache_endupdate(void)
 static __inline int
 sc_cmp(struct sc_ent *a, struct sc_ent *b)
 {
-#ifdef HAVE_PFSYNC_STATE
        if (a->id[0] > b->id[0])
                return (1);
        if (a->id[0] < b->id[0])
@@ -232,66 +197,5 @@ sc_cmp(struct sc_ent *a, struct sc_ent *b)
                return (1);
        if (a->id[1] < b->id[1])
                return (-1);
-#else  
-               int diff;
-
-       if ((diff = a->proto - b->proto) != 0)
-               return (diff);
-       if ((diff = a->af - b->af) != 0)
-               return (diff);
-       switch (a->af) {
-       case AF_INET:
-               if (a->addr[0].addr32[0] > b->addr[0].addr32[0])
-                       return (1);
-               if (a->addr[0].addr32[0] < b->addr[0].addr32[0])
-                       return (-1);
-               if (a->addr[1].addr32[0] > b->addr[1].addr32[0])
-                       return (1);
-               if (a->addr[1].addr32[0] < b->addr[1].addr32[0])
-                       return (-1);
-               break;
-       case AF_INET6:
-               if (a->addr[0].addr32[0] > b->addr[0].addr32[0])
-                       return (1);
-               if (a->addr[0].addr32[0] < b->addr[0].addr32[0])
-                       return (-1);
-               if (a->addr[0].addr32[1] > b->addr[0].addr32[1])
-                       return (1);
-               if (a->addr[0].addr32[1] < b->addr[0].addr32[1])
-                       return (-1);
-               if (a->addr[0].addr32[2] > b->addr[0].addr32[2])
-                       return (1);
-               if (a->addr[0].addr32[2] < b->addr[0].addr32[2])
-                       return (-1);
-               if (a->addr[0].addr32[3] > b->addr[0].addr32[3])
-                       return (1);
-               if (a->addr[0].addr32[3] < b->addr[0].addr32[3])
-                       return (-1);
-               if (a->addr[1].addr32[0] > b->addr[1].addr32[0])
-                       return (1);
-               if (a->addr[1].addr32[0] < b->addr[1].addr32[0])
-                       return (-1);
-               if (a->addr[1].addr32[1] > b->addr[1].addr32[1])
-                       return (1);
-               if (a->addr[1].addr32[1] < b->addr[1].addr32[1])
-                       return (-1);
-               if (a->addr[1].addr32[2] > b->addr[1].addr32[2])
-                       return (1);
-               if (a->addr[1].addr32[2] < b->addr[1].addr32[2])
-                       return (-1);
-               if (a->addr[1].addr32[3] > b->addr[1].addr32[3])
-                       return (1);
-               if (a->addr[1].addr32[3] < b->addr[1].addr32[3])
-                       return (-1);
-               break;
-               default:
-                       return 1;
-       }
-
-       if ((diff = a->port[0] - b->port[0]) != 0)
-               return (diff);
-       if ((diff = a->port[1] - b->port[1]) != 0)
-               return (diff);
-#endif
        return (0);
 }
index 5e499b1..d09620d 100644 (file)
@@ -1,4 +1,4 @@
-/* $Id: cache.h,v 1.1 2008/06/12 22:26:01 canacar Exp $ */
+/* $Id: cache.h,v 1.2 2008/07/16 10:23:39 canacar Exp $ */
 /*
  * Copyright (c) 2001, 2007 Can Erkin Acar <canacar@openbsd.org>
  *
 #ifndef _CACHE_H_
 #define _CACHE_H_
 
-#include "config.h"
-
 #include <sys/queue.h>
-#ifdef HAVE_TREE_H
 #include <sys/tree.h>
-#else
-#include "tree.h"
-#endif
-
+#include <net/pfvar.h>
 
 struct sc_ent {
         RB_ENTRY(sc_ent)    tlink;
        TAILQ_ENTRY(sc_ent) qlink;
-#ifdef HAVE_PFSYNC_STATE
        u_int32_t           id[2];
-#else
-       struct pf_addr      addr[2];
-#endif
        double              peak;
        double              rate;
        time_t              t;
        u_int32_t           bytes;
-#ifndef HAVE_PFSYNC_STATE
-        u_int16_t           port[2];
-        u_int8_t            af;
-        u_int8_t            proto;
-#endif
 };
 
 int cache_init(int);
 void cache_endupdate(void);
-struct sc_ent *cache_state(pf_state_t *);
+struct sc_ent *cache_state(struct pfsync_state *);
 extern int cache_max, cache_size;
 
+#define COUNTER(c) ((((u_int64_t) ntohl(c[0]))<<32) + ntohl(c[1]))
+
 
 #endif
diff --git a/usr.bin/systat/config.h b/usr.bin/systat/config.h
deleted file mode 100644 (file)
index 6ea6b2f..0000000
+++ /dev/null
@@ -1,85 +0,0 @@
-/* $Id: config.h,v 1.2 2008/06/29 08:42:15 mcbride Exp $ */
-/*
- * Copyright (c) 2001, 2007 Can Erkin Acar <canacar@openbsd.org>
- *
- * Permission to use, copy, modify, and distribute this software for any
- * purpose with or without fee is hereby granted, provided that the above
- * copyright notice and this permission notice appear in all copies.
- *
- * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
- * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
- * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
- * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
- * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
- * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
- * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
- */
-
-#ifndef _CONFIG_H_
-#define _CONFIG_H_
-
-
-/* OS_LEVEL > 30 */
-#define HAVE_STATE_NOROUTE
-#define HAVE_DEVICE_RO
-#define HAVE_TREE_H
-#define HAVE_QUEUE_H
-#define HAVE_PF_ROUTE
-#define HAVE_RULE_LABELS
-
-/* OS_LEVEL > 31 */
-#define HAVE_RULE_NUMBER
-#define HAVE_ADDR_WRAP
-#define HAVE_RULE_STATES
-#define HAVE_RULE_IFNOT
-#define HAVE_PROTO_NAMES
-#define HAVE_MAX_STATES
-#define HAVE_MAX_MSS
-#define HAVE_RULE_UGID
-
-/* OS_LEVEL > 32 */
-#define HAVE_ADDR_MASK
-#define HAVE_ADDR_TYPE
-#define HAVE_ALTQ
-#define HAVE_RULE_TOS
-#define HAVE_OP_RRG
-
-/* OS_LEVEL > 33 */
-#define HAVE_INOUT_COUNT
-#define HAVE_TAGS
-#define HAVE_RULE_NATPASS
-
-/* OS_LEVEL > 34 */
-#define HAVE_STATE_IFNAME
-
-/* OS_LEVEL > 35 */
-#define HAVE_NEG
-#define HAVE_RULESETS
-
-/* OS_LEVEL > 37 */
-#define HAVE_INOUT_COUNT_RULES
-
-/* OS_LEVEL > 38 */
-#define HAVE_STATE_COUNT_64
-
-/* OS_LEVEL > 41 */
-#define HAVE_PFSYNC_STATE
-
-/* OS_LEVEL > 43 */
-#define HAVE_PFSYNC_KEY
-
-#ifdef HAVE_PFSYNC_STATE
-typedef struct pfsync_state pf_state_t;
-typedef struct pfsync_state_host pf_state_host_t;
-typedef struct pfsync_state_peer pf_state_peer_t;
-#define COUNTER(c) ((((u_int64_t) ntohl(c[0]))<<32) + ntohl(c[1]))
-#define pfs_ifname ifname
-#else
-typedef struct pf_state pf_state_t;
-typedef struct pf_state_host pf_state_host_t;
-typedef struct pf_state_peer pf_state_peer_t;
-#define COUNTER(c) (c)
-#define pfs_ifname u.ifname
-#endif
-
-#endif
index 3d80018..e902baf 100644 (file)
@@ -1,4 +1,4 @@
-/* $Id: pftop.c,v 1.3 2008/06/29 08:42:15 mcbride Exp $         */
+/* $Id: pftop.c,v 1.4 2008/07/16 10:23:39 canacar Exp $         */
 /*
  * Copyright (c) 2001, 2007 Can Erkin Acar
  * Copyright (c) 2001 Daniel Hartmeier
@@ -30,8 +30,6 @@
  *
  */
 
-#include "config.h"
-
 #include <sys/types.h>
 #include <sys/ioctl.h>
 #include <sys/socket.h>
 #include <net/pfvar.h>
 #include <arpa/inet.h>
 
-#ifdef HAVE_ALTQ
 #include <altq/altq.h>
 #include <altq/altq_cbq.h>
 #include <altq/altq_priq.h>
 #include <altq/altq_hfsc.h>
-#endif
 
 #include <ctype.h>
 #include <curses.h>
@@ -72,59 +68,13 @@ extern const char *tcpstates[];
 
 #define DEFAULT_CACHE_SIZE 10000
 
-#ifndef HAVE_PROTO_NAMES
-/* UDP state enumeration */
-#define PFUDPS_NSTATES         3       /* number of state levels */
-
-#define PFUDPS_NAMES { \
-       "NO TRAFFIC", \
-       "SINGLE", \
-       "MULTIPLE", \
-       NULL \
-}
-
-/* Other protocol state enumeration */
-#define PFOTHERS_NSTATES       3       /* number of state levels */
-
-#define PFOTHERS_NAMES { \
-       "NO TRAFFIC", \
-       "SINGLE", \
-       "MULTIPLE", \
-       NULL \
-}
-#endif
-
-#ifdef HAVE_ADDR_WRAP
-#ifdef HAVE_ADDR_TYPE
 /* XXX must also check type before use */
 #define PT_ADDR(x) (&(x)->addr.v.a.addr)
-#else
-#define PT_ADDR(x) (&(x)->addr.addr)
-#endif
-#else
-#define PT_ADDR(x) (&(x)->addr)
-#endif
 
-#ifdef HAVE_ADDR_MASK
-#ifdef HAVE_ADDR_TYPE
 /* XXX must also check type before use */
 #define PT_MASK(x) (&(x)->addr.v.a.mask)
-#else
-#define PT_MASK(x) (&(x)->addr.mask)
-#endif
-#else
-#define PT_MASK(x) (&(x)->mask)
-#endif
 
-#ifdef HAVE_STATE_NOROUTE
-#ifdef HAVE_ADDR_TYPE
 #define PT_NOROUTE(x) ((x)->addr.type == PF_ADDR_NOROUTE)
-#else
-#define PT_NOROUTE(x) ((x)->noroute)
-#endif
-#else
-#define PT_NOROUTE(x) (0)
-#endif
 
 /* view management */
 int select_states(void);
@@ -139,11 +89,9 @@ void print_rules(void);
 int print_header(void);
 int keyboard_callback(int ch);
 
-#ifdef HAVE_ALTQ
 int select_queues(void);
 int read_queues(void);
 void print_queues(void);
-#endif
 
 /* qsort callbacks */
 int sort_size_callback(const void *s1, const void *s2);
@@ -159,7 +107,7 @@ int sort_peak_callback(const void *s1, const void *s2);
 int pf_dev = -1;
 
 struct sc_ent **state_cache = NULL;
-pf_state_t *state_buf = NULL;
+struct pfsync_state *state_buf = NULL;
 int state_buf_len = 0;
 u_int32_t *state_ord = NULL;
 u_int32_t num_states = 0;
@@ -171,10 +119,6 @@ int cachestates = 0;
 char *filter_string = NULL;
 int dumpfilter = 0;
 
-#ifndef HAVE_RULE_LABELS
-#define PF_RULE_LABEL_SIZE 20
-#endif
-
 #define MIN_LABEL_SIZE 5
 #define ANCHOR_FLD_SIZE 12
 
@@ -200,11 +144,11 @@ field_def fields[] = {
        {"KS", 1, 1, 1, FLD_ALIGN_LEFT, -1, 0, 0, 0},
        {"IF", 4, 6, 1, FLD_ALIGN_LEFT, -1, 0, 0, 0},
        {"INFO", 40, 80, 1, FLD_ALIGN_LEFT, -1, 0, 0, 0},
-       {"MAX", 3, 5, 2, FLD_ALIGN_RIGHT, -1, 0, FLD_FLAG_HIDDEN, 0},
+       {"MAX", 3, 5, 2, FLD_ALIGN_RIGHT, -1, 0, 0},
        {"RATE", 5, 8, 1, FLD_ALIGN_RIGHT, -1, 0, 0, 0},
        {"AVG", 5, 8, 1, FLD_ALIGN_RIGHT, -1, 0, 0, 0},
        {"PEAK", 5, 8, 1, FLD_ALIGN_RIGHT, -1, 0, 0, 0},
-       {"ANCHOR", 6, 16, 1, FLD_ALIGN_LEFT, -1, 0, FLD_FLAG_HIDDEN, 0},
+       {"ANCHOR", 6, 16, 1, FLD_ALIGN_LEFT, -1, 0, 0},
        {"QUEUE", 15, 30, 1, FLD_ALIGN_LEFT, -1, 0, 0, 0},
        {"BW", 4, 5, 1, FLD_ALIGN_RIGHT, -1, 0, 0, 0},
        {"SCH", 3, 4, 1, FLD_ALIGN_LEFT, -1, 0, 0, 0},
@@ -339,12 +283,10 @@ struct view_manager rule_mgr = {
        print_rules, keyboard_callback, NULL, NULL
 };
 
-#ifdef HAVE_ALTQ
 struct view_manager queue_mgr = {
        "Queues", select_queues, read_queues, NULL, print_header,
        print_queues, keyboard_callback, NULL, NULL
 };
-#endif
 
 field_view views[] = {
        {view2, "states", '8', &state_mgr},
@@ -356,7 +298,6 @@ field_view views[] = {
 
 /* altq structures from pfctl */
 
-#ifdef HAVE_ALTQ
 union class_stats {
        class_stats_t           cbq_stats;
        struct priq_classstats  priq_stats;
@@ -379,7 +320,6 @@ struct pf_altq_node {
        u_int8_t                 depth;
        u_int8_t                 visited;
 };
-#endif /* HAVE_ALTQ */
 
 
 /* ordering functions */
@@ -387,15 +327,10 @@ struct pf_altq_node {
 int
 sort_size_callback(const void *s1, const void *s2)
 {
-#ifdef HAVE_INOUT_COUNT
        u_int64_t b1 = COUNTER(state_buf[* (u_int32_t *) s1].bytes[0]) + 
                COUNTER(state_buf[* (u_int32_t *) s1].bytes[1]);
        u_int64_t b2 = COUNTER(state_buf[* (u_int32_t *) s2].bytes[0]) + 
                COUNTER(state_buf[* (u_int32_t *) s2].bytes[1]);
-#else
-       u_int64_t b1 = COUNTER(state_buf[* (u_int32_t *) s1].bytes);
-       u_int64_t b2 = COUNTER(state_buf[* (u_int32_t *) s2].bytes);
-#endif
        if (b2 > b1)
                return sortdir;
        if (b2 < b1)
@@ -406,15 +341,10 @@ sort_size_callback(const void *s1, const void *s2)
 int
 sort_pkt_callback(const void *s1, const void *s2)
 {
-#ifdef HAVE_INOUT_COUNT
        u_int64_t p1 = COUNTER(state_buf[* (u_int32_t *) s1].packets[0]) + 
                COUNTER(state_buf[* (u_int32_t *) s1].packets[1]);
        u_int64_t p2 = COUNTER(state_buf[* (u_int32_t *) s2].packets[0]) + 
                COUNTER(state_buf[* (u_int32_t *) s2].packets[1]);
-#else
-       u_int64_t p1 = COUNTER(state_buf[* (u_int32_t *) s1].packets);
-       u_int64_t p2 = COUNTER(state_buf[* (u_int32_t *) s2].packets);
-#endif
        if (p2 > p1)
                return sortdir;
        if (p2 < p1)
@@ -515,14 +445,9 @@ compare_addr(int af, const struct pf_addr *a, const struct pf_addr *b)
        return 0;
 }
 
-#ifdef HAVE_PFSYNC_KEY
-
-#ifdef __GNUC__
-__inline__
-#endif
-int
-sort_addr_callback(const pf_state_t *s1,
-                  const pf_state_t *s2, int dir)
+__inline int
+sort_addr_callback(const struct pfsync_state *s1,
+                  const struct pfsync_state *s2, int dir)
 {
        const struct pf_addr *aa, *ab;
        u_int16_t pa, pb;
@@ -567,12 +492,9 @@ sort_addr_callback(const pf_state_t *s1,
        return -sortdir;
 }
 
-#ifdef __GNUC__
-__inline__
-#endif
-int
-sort_port_callback(const pf_state_t *s1,
-                  const pf_state_t *s2, int dir)
+__inline int
+sort_port_callback(const struct pfsync_state *s1,
+                  const struct pfsync_state *s2, int dir)
 {
        const struct pf_addr *aa, *ab;
        u_int16_t pa, pb;
@@ -621,113 +543,35 @@ sort_port_callback(const pf_state_t *s1,
        return -sortdir;
 }
 
-#else  /* HAVE_PFSYNC_KEY */
-
-#ifdef __GNUC__
-__inline__
-#endif
 int
-sort_addr_callback(const pf_state_t *s1,
-                  const pf_state_t *s2, int dir)
+sort_sa_callback(const void *p1, const void *p2)
 {
-       const pf_state_host_t *a, *b;
-       int af, ret;
-
-       af = s1->af;
-
-       if (af > s2->af)
-               return sortdir;
-       if (af < s2->af)
-               return -sortdir;
-       
-       if (s1->direction == dir) {
-               a = &s1->lan;
-       } else {
-               a = &s1->ext;
-       }
-
-       if (s2->direction == dir) {
-               b = &s2->lan;
-       } else {
-               b = &s2->ext;
-       }
-
-       ret = compare_addr(af, &a->addr, &b->addr);
-       if (ret)
-               return ret * sortdir;
-
-       if (ntohs(a->port) > ntohs(b->port))
-               return sortdir;
-       return -sortdir;
-}
-
-#ifdef __GNUC__
-__inline__
-#endif
-int
-sort_port_callback(const pf_state_t *s1,
-                  const pf_state_t *s2, int dir)
-{
-       const pf_state_host_t *a, *b;
-       int af;
-
-       af = s1->af;
-
-       if (af > s2->af)
-               return sortdir;
-       if (af < s2->af)
-               return -sortdir;
-       
-       if (s1->direction == dir) {
-               a = &s1->lan;
-       } else {
-               a = &s1->ext;
-       }
-
-       if (s2->direction == dir) {
-               b = &s2->lan;
-       } else {
-               b = &s2->ext;
-        }
-
-       if (ntohs(a->port) > ntohs(b->port))
-               return sortdir;
-       if (ntohs(a->port) < ntohs(b->port))
-               return -sortdir;
-
-       if (compare_addr(af, &a->addr, &b->addr) > 0)
-               return sortdir;
-       return -sortdir;
-}
-#endif /* HAVE_PFSYNC_KEY */
-
-int sort_sa_callback(const void *p1, const void *p2)
-{
-       pf_state_t *s1 = state_buf + (* (u_int32_t *) p1);
-       pf_state_t *s2 = state_buf + (* (u_int32_t *) p2);
+       struct pfsync_state *s1 = state_buf + (* (u_int32_t *) p1);
+       struct pfsync_state *s2 = state_buf + (* (u_int32_t *) p2);
        return sort_addr_callback(s1, s2, PF_OUT);
 }
 
-int sort_da_callback(const void *p1, const void *p2)
+int
+sort_da_callback(const void *p1, const void *p2)
 {
-       pf_state_t *s1 = state_buf + (* (u_int32_t *) p1);
-       pf_state_t *s2 = state_buf + (* (u_int32_t *) p2);
+       struct pfsync_state *s1 = state_buf + (* (u_int32_t *) p1);
+       struct pfsync_state *s2 = state_buf + (* (u_int32_t *) p2);
        return sort_addr_callback(s1, s2, PF_IN);
 }
 
 int
 sort_sp_callback(const void *p1, const void *p2)
 {
-       pf_state_t *s1 = state_buf + (* (u_int32_t *) p1);
-       pf_state_t *s2 = state_buf + (* (u_int32_t *) p2);
+       struct pfsync_state *s1 = state_buf + (* (u_int32_t *) p1);
+       struct pfsync_state *s2 = state_buf + (* (u_int32_t *) p2);
        return sort_port_callback(s1, s2, PF_OUT);
 }
 
 int
 sort_dp_callback(const void *p1, const void *p2)
 {
-       pf_state_t *s1 = state_buf + (* (u_int32_t *) p1);
-       pf_state_t *s2 = state_buf + (* (u_int32_t *) p2);
+       struct pfsync_state *s1 = state_buf + (* (u_int32_t *) p1);
+       struct pfsync_state *s2 = state_buf + (* (u_int32_t *) p2);
        return sort_port_callback(s1, s2, PF_IN);
 }
 
@@ -767,7 +611,7 @@ alloc_buf(int ns)
 
        if (len >= state_buf_len) {
                len += NUM_STATE_INC;
-               state_buf = realloc(state_buf, len * sizeof(pf_state_t));
+               state_buf = realloc(state_buf, len * sizeof(struct pfsync_state));
                state_ord = realloc(state_ord, len * sizeof(u_int32_t));
                state_cache = realloc(state_cache, 
                                      len * sizeof(struct sc_ent *));
@@ -795,7 +639,7 @@ read_states(void)
                return -1;
 
        for (;;) {
-               int sbytes = state_buf_len * sizeof(pf_state_t);
+               int sbytes = state_buf_len * sizeof(struct pfsync_state);
 
                ps.ps_len = sbytes;
                ps.ps_buf = (char *) state_buf;
@@ -803,7 +647,7 @@ read_states(void)
                if (ioctl(pf_dev, DIOCGETSTATES, &ps) < 0) {
                        error("DIOCGETSTATES");
                }
-               num_states_all = ps.ps_len / sizeof(pf_state_t);
+               num_states_all = ps.ps_len / sizeof(struct pfsync_state);
 
                if (ps.ps_len < sbytes)
                        break;
@@ -871,7 +715,7 @@ tb_print_addr(struct pf_addr * addr, struct pf_addr * mask, int af)
                        tbprintf("/%u", unmask(mask, af));
        }
 }
-#ifdef HAVE_PFSYNC_KEY
+
 void
 print_fld_host2(field_def *fld, struct pfsync_state_key *ks,
                struct pfsync_state_key *kn, int idx, int af)
@@ -912,31 +756,6 @@ print_fld_host2(field_def *fld, struct pfsync_state_key *ks,
        }
 
 }
-#else
-void
-print_fld_host(field_def *fld, pf_state_host_t * h, int af)
-{
-       u_int16_t p = ntohs(h->port);
-
-       if (fld == NULL)
-               return;
-
-       if (fld->width < 3) {
-               print_fld_str(fld, "*");
-               return;
-       }
-
-       tb_start();
-       tb_print_addr(&h->addr, NULL, af);
-
-       if (af == AF_INET)
-               tbprintf(":%u", p);
-       else
-               tbprintf("[%u]", p);
-
-       print_fld_tb(fld);
-}
-#endif
 
 void
 print_fld_state(field_def *fld, unsigned int proto,
@@ -988,10 +807,11 @@ print_fld_state(field_def *fld, unsigned int proto,
 }
 
 int
-print_state(pf_state_t * s, struct sc_ent * ent)
+print_state(struct pfsync_state * s, struct sc_ent * ent)
 {
-       pf_state_peer_t *src, *dst;
+       struct pfsync_state_peer *src, *dst;
        struct protoent *p;
+       u_int64_t sz;
 
        if (s->direction == PF_OUT) {
                src = &s->src;
@@ -1008,7 +828,6 @@ print_state(pf_state_t * s, struct sc_ent * ent)
        else
                print_fld_uint(FLD_PROTO, s->proto);
 
-#ifdef HAVE_PFSYNC_KEY
        if (s->direction == PF_OUT) {
                print_fld_host2(FLD_SRC, &s->key[PF_SK_WIRE],
                    &s->key[PF_SK_STACK], 1, s->af);
@@ -1020,20 +839,6 @@ print_state(pf_state_t * s, struct sc_ent * ent)
                print_fld_host2(FLD_DEST, &s->key[PF_SK_STACK],
                    &s->key[PF_SK_WIRE], 1, s->af);
        }
-#else
-       if (s->direction == PF_OUT) {
-               print_fld_host(FLD_SRC, &s->lan, s->af);
-               print_fld_host(FLD_DEST, &s->ext, s->af);
-       } else {
-               print_fld_host(FLD_SRC, &s->ext, s->af);
-               print_fld_host(FLD_DEST, &s->lan, s->af);
-       }
-
-       if (PF_ANEQ(&s->lan.addr, &s->gwy.addr, s->af) ||
-           (s->lan.port != s->gwy.port)) {
-               print_fld_host(FLD_GW, &s->gwy, s->af);
-       }
-#endif
 
        if (s->direction == PF_OUT)
                print_fld_str(FLD_DIR, "Out");
@@ -1043,30 +848,16 @@ print_state(pf_state_t * s, struct sc_ent * ent)
        print_fld_state(FLD_STATE, s->proto, src->state, dst->state);
        print_fld_age(FLD_AGE, ntohl(s->creation));
        print_fld_age(FLD_EXP, ntohl(s->expire));
-#ifdef HAVE_INOUT_COUNT
-       {
-               u_int64_t sz = COUNTER(s->bytes[0]) + COUNTER(s->bytes[1]);
-
-               print_fld_size(FLD_PKTS, COUNTER(s->packets[0]) +
-                              COUNTER(s->packets[1]));
-               print_fld_size(FLD_BYTES, sz);
-               print_fld_rate(FLD_SA, (s->creation) ?
-                              ((double)sz/ntohl((double)s->creation)) : -1);
-       }
-#else
-       print_fld_size(FLD_PKTS, s->packets);
-       print_fld_size(FLD_BYTES, s->bytes);
+
+       sz = COUNTER(s->bytes[0]) + COUNTER(s->bytes[1]);
+
+       print_fld_size(FLD_PKTS, COUNTER(s->packets[0]) +
+                      COUNTER(s->packets[1]));
+       print_fld_size(FLD_BYTES, sz);
        print_fld_rate(FLD_SA, (s->creation) ?
-                      ((double)s->bytes/ntohl((double)s->creation)) : -1);
+                      ((double)sz/ntohl((double)s->creation)) : -1);
 
-#endif
-#ifdef HAVE_PFSYNC_STATE
        print_fld_uint(FLD_RULE, s->rule);
-#else
-#ifdef HAVE_RULE_NUMBER
-       print_fld_uint(FLD_RULE, s->rule.nr);
-#endif
-#endif
        if (cachestates && ent != NULL) {
                print_fld_rate(FLD_SI, ent->rate);
                print_fld_rate(FLD_SP, ent->peak);
@@ -1123,23 +914,21 @@ add_rule_alloc(u_int32_t nr)
        }
 }
 
-#ifdef HAVE_RULE_LABELS
 int label_length;
-#endif
 
 int
 read_anchor_rules(char *anchor)
 {
        struct pfioc_rule pr;
        u_int32_t nr, num, off;
+       int len;
 
        if (pf_dev < 0)
                return (-1);
 
        memset(&pr, 0, sizeof(pr));
-#ifdef HAVE_RULESETS
        strlcpy(pr.anchor, anchor, sizeof(pr.anchor));
-#endif
+
        if (ioctl(pf_dev, DIOCGETRULES, &pr)) {
                error("anchor %s: %s", anchor, strerror(errno));
                return (-1);
@@ -1155,25 +944,18 @@ read_anchor_rules(char *anchor)
                        error("DIOCGETRULE: %s", strerror(errno));
                        return (-1);
                }
-#ifdef HAVE_RULESETS
                /* XXX overload pr.anchor, to store a pointer to
                 * anchor name */
                pr.rule.anchor = (struct pf_anchor *) anchor;
-#endif
-#ifdef HAVE_RULE_LABELS
-               {
-                       int len = strlen(pr.rule.label);
-                       if (len > label_length)
-                               label_length = len;
-               }
-#endif
+               len = strlen(pr.rule.label);
+               if (len > label_length)
+                       label_length = len;
                rules[off + nr] = pr.rule;
        }
 
        return (num);
 }
 
-#ifdef HAVE_RULESETS
 struct anchor_name {
        char name[MAXPATHLEN];
        struct anchor_name *next;
@@ -1308,55 +1090,41 @@ compute_anchor_field(void)
                need_update = 1;
        }
 }
-#endif
 
 int
 read_rules(void)
 {
-       int ret;
+       int ret, nw, mw;
        num_rules = 0;
 
        if (pf_dev == -1)
                return (-1);
 
-#ifdef HAVE_RULE_LABELS
        label_length = MIN_LABEL_SIZE;
-#endif
 
-#ifdef HAVE_RULESETS
        reset_anchor_names();
        ret = read_rulesets(NULL);
        compute_anchor_field();
-#else
-       ret = read_anchor_rules(NULL);
-#endif
 
-#ifdef HAVE_RULE_LABELS
-       {
-               int nw, mw;
-               nw = mw = label_length;
-               if (nw > 16)
-                       nw = 16;
-
-               if (FLD_LABEL->norm_width != nw || 
-                   FLD_LABEL->max_width != mw) {
-                       FLD_LABEL->norm_width = nw;
-                       FLD_LABEL->max_width = mw;
-                       field_setup();
-                       need_update = 1;
-               }
+       nw = mw = label_length;
+       if (nw > 16)
+               nw = 16;
+
+       if (FLD_LABEL->norm_width != nw || 
+           FLD_LABEL->max_width != mw) {
+               FLD_LABEL->norm_width = nw;
+               FLD_LABEL->max_width = mw;
+               field_setup();
+               need_update = 1;
        }
-#endif
 
        num_disp = num_rules;
        return (ret);
 }
 
-#ifdef HAVE_ADDR_WRAP
 void
 tb_print_addrw(struct pf_addr_wrap *addr, struct pf_addr *mask, u_int8_t af)
 {
-#ifdef HAVE_ADDR_TYPE
        switch (addr->type) {
        case PF_ADDR_ADDRMASK:
                tb_print_addr(&addr->v.a.addr, mask, af);
@@ -1374,14 +1142,7 @@ tb_print_addrw(struct pf_addr_wrap *addr, struct pf_addr *mask, u_int8_t af)
                tbprintf("UNKNOWN");
                break;
        }
-#else
-       if (addr->addr_dyn != NULL)
-               tbprintf("(%s)", addr->addr.pfa.ifname);
-       else
-               tb_print_addr(&addr->addr, mask, af);
-#endif
 }
-#endif
 
 void
 tb_print_op(u_int8_t op, const char *a1, const char *a2)
@@ -1390,10 +1151,8 @@ tb_print_op(u_int8_t op, const char *a1, const char *a2)
                tbprintf("%s >< %s ", a1, a2);
        else if (op == PF_OP_XRG)
                tbprintf("%s <> %s ", a1, a2);
-#ifdef HAVE_OP_RRG
        else if (op == PF_OP_RRG)
                tbprintf("%s:%s ", a1, a2);
-#endif
        else if (op == PF_OP_EQ)
                tbprintf("= %s ", a1);
        else if (op == PF_OP_NE)
@@ -1445,17 +1204,9 @@ tb_print_fromto(struct pf_rule_addr *src, struct pf_rule_addr *dst,
                         PF_AZERO(PT_MASK(src), AF_INET6))
                        tbprintf("any ");
                else {
-#ifdef HAVE_NEG
                        if (src->neg)
-#else
-                       if (src->not)
-#endif
                                tbprintf("! ");
-#ifdef HAVE_ADDR_WRAP
                        tb_print_addrw(&src->addr, PT_MASK(src), af);
-#else
-                       tb_print_addr(&src->addr, PT_MASK(src), af);
-#endif
                        tbprintf(" ");
                }
                if (src->port_op)
@@ -1470,17 +1221,9 @@ tb_print_fromto(struct pf_rule_addr *src, struct pf_rule_addr *dst,
                         PF_AZERO(PT_MASK(dst), AF_INET6))
                        tbprintf("any ");
                else {
-#ifdef HAVE_NEG
                        if (dst->neg)
-#else
-                       if (dst->not)
-#endif
                                tbprintf("! ");
-#ifdef HAVE_ADDR_WRAP
                        tb_print_addrw(&dst->addr, PT_MASK(dst), af);
-#else
-                       tb_print_addr(&dst->addr, PT_MASK(dst), af);
-#endif
                        tbprintf(" ");
                }
                if (dst->port_op)
@@ -1490,7 +1233,6 @@ tb_print_fromto(struct pf_rule_addr *src, struct pf_rule_addr *dst,
        }
 }
 
-#ifdef HAVE_RULE_UGID
 void
 tb_print_ugid(u_int8_t op, unsigned u1, unsigned u2,
              const char *t, unsigned umax)
@@ -1506,7 +1248,6 @@ tb_print_ugid(u_int8_t op, unsigned u1, unsigned u2,
        else
                tb_print_op(op, a1, a2);
 }
-#endif
 
 void
 tb_print_flags(u_int8_t f)
@@ -1526,33 +1267,19 @@ print_rule(struct pf_rule *pr)
            "no Nat", "Binat", "no Binat", "Rdr", "no Rdr" };
        int numact = sizeof(actiontypes) / sizeof(char *);
 
-#ifdef HAVE_PF_ROUTE
        static const char *routetypes[] = { "", "fastroute", "route-to",
            "dup-to", "reply-to" };
 
        int numroute = sizeof(routetypes) / sizeof(char *);
-#endif
 
        if (pr == NULL) return;
 
-#ifdef HAVE_RULE_LABELS
        print_fld_str(FLD_LABEL, pr->label);
-#endif
-#ifdef HAVE_RULE_STATES
-#ifdef HAVE_PFSYNC_KEY
        print_fld_size(FLD_STATS, pr->states_tot);
-#else
-       print_fld_size(FLD_STATS, pr->states);
-#endif
-#endif
 
-#ifdef HAVE_INOUT_COUNT_RULES
        print_fld_size(FLD_PKTS, pr->packets[0] + pr->packets[1]);
        print_fld_size(FLD_BYTES, pr->bytes[0] + pr->bytes[1]);
-#else
-       print_fld_size(FLD_PKTS, pr->packets);
-       print_fld_size(FLD_BYTES, pr->bytes);
-#endif
+
        print_fld_uint(FLD_RULE, pr->nr);
        print_fld_str(FLD_DIR, pr->direction == PF_OUT ? "Out" : "In");
        if (pr->quick)
@@ -1586,25 +1313,21 @@ print_rule(struct pf_rule *pr)
 
        if (pr->ifname[0]) {
                tb_start();
-#ifdef HAVE_RULE_IFNOT
                if (pr->ifnot)
                        tbprintf("!");
-#endif
                tbprintf("%s", pr->ifname);
                print_fld_tb(FLD_IF);
        }
-#ifdef HAVE_MAX_STATES
        if (pr->max_states)
                print_fld_uint(FLD_STMAX, pr->max_states);
-#endif
+
        /* print info field */
 
        tb_start();
 
-#ifdef HAVE_RULE_NATPASS
        if (pr->natpass)
                tbprintf("pass ");
-#endif
+
        if (pr->action == PF_DROP) {
                if (pr->rule_flag & PFRULE_RETURNRST)
                        tbprintf("return-rst ");
@@ -1620,13 +1343,12 @@ print_rule(struct pf_rule *pr)
                        tbprintf("drop ");
        }
 
-#ifdef HAVE_PF_ROUTE
        if (pr->rt > 0 && pr->rt < numroute) {
                tbprintf("%s ", routetypes[pr->rt]);
                if (pr->rt != PF_FASTROUTE)
                        tbprintf("... ");
        }
-#endif
+
        if (pr->af) {
                if (pr->af == AF_INET)
                        tbprintf("inet ");
@@ -1635,14 +1357,13 @@ print_rule(struct pf_rule *pr)
        }
 
        tb_print_fromto(&pr->src, &pr->dst, pr->af, pr->proto);
-#ifdef HAVE_RULE_UGID
+
        if (pr->uid.op)
                tb_print_ugid(pr->uid.op, pr->uid.uid[0], pr->uid.uid[1],
                        "user", UID_MAX);
        if (pr->gid.op)
                tb_print_ugid(pr->gid.op, pr->gid.gid[0], pr->gid.gid[1],
                        "group", GID_MAX);
-#endif
 
        if (pr->flags || pr->flagset) {
                tbprintf(" flags ");
@@ -1653,10 +1374,8 @@ print_rule(struct pf_rule *pr)
 
        tbprintf(" ");
 
-#ifdef HAVE_RULE_TOS
        if (pr->tos)
                tbprintf("tos 0x%2.2x ", pr->tos);
-#endif
 #ifdef PFRULE_FRAGMENT
        if (pr->rule_flag & PFRULE_FRAGMENT)
                tbprintf("fragment ");
@@ -1671,10 +1390,8 @@ print_rule(struct pf_rule *pr)
 #endif
        if (pr->min_ttl)
                tbprintf("min-ttl %d ", pr->min_ttl);
-#ifdef HAVE_MAX_MSS
        if (pr->max_mss)
                tbprintf("max-mss %d ", pr->max_mss);
-#endif
        if (pr->allow_opts)
                tbprintf("allow-opts ");
 
@@ -1696,13 +1413,11 @@ print_rule(struct pf_rule *pr)
                        tbprintf("fragment reassemble ");
        }
 
-#ifdef HAVE_ALTQ       
        if (pr->qname[0] && pr->pqname[0])
                tbprintf("queue(%s, %s) ", pr->qname, pr->pqname);
        else if (pr->qname[0])
                tbprintf("queue %s ", pr->qname);
-#endif
-#ifdef HAVE_TAGS
+
        if (pr->tagname[0])
                tbprintf("tag %s ", pr->tagname);
        if (pr->match_tagname[0]) {
@@ -1710,13 +1425,11 @@ print_rule(struct pf_rule *pr)
                        tbprintf("! ");
                tbprintf("tagged %s ", pr->match_tagname);
        }
-#endif
+
        print_fld_tb(FLD_RINFO);
 
-#ifdef HAVE_RULESETS
        /* XXX anchor field overloaded with anchor name */
        print_fld_str(FLD_ANCHOR, (char *)pr->anchor);
-#endif
        tb_end();
 
        end_line();
@@ -1737,8 +1450,6 @@ print_rules(void)
 
 /* queue display */
 
-#ifdef HAVE_ALTQ
-
 struct pf_altq_node *
 pfctl_find_altq_node(struct pf_altq_node *root, const char *qname,
     const char *ifname)
@@ -2097,8 +1808,6 @@ print_queues(void)
        }
 }
 
-#endif /* HAVE_ALTQ */
-
 /* main program functions */
 
 void
@@ -2149,11 +1858,6 @@ initpftop(void)
 
        update_cache();
 
-#ifdef HAVE_MAX_STATES
        show_field(FLD_STMAX);
-#endif
-#ifdef HAVE_RULESETS
        show_field(FLD_ANCHOR);
-#endif
-
 }