]> git.ipfire.org Git - thirdparty/ipset.git/commitdiff
Backport patch: netfilter: ipset: Convert timers to use timer_setup()
authorJozsef Kadlecsik <kadlec@blackhole.kfki.hu>
Wed, 3 Jan 2018 09:57:31 +0000 (10:57 +0100)
committerJozsef Kadlecsik <kadlec@blackhole.kfki.hu>
Wed, 3 Jan 2018 09:57:31 +0000 (10:57 +0100)
configure.ac
kernel/include/linux/netfilter/ipset/ip_set_compat.h.in
kernel/net/netfilter/ipset/ip_set_bitmap_gen.h
kernel/net/netfilter/ipset/ip_set_bitmap_ip.c
kernel/net/netfilter/ipset/ip_set_bitmap_ipmac.c
kernel/net/netfilter/ipset/ip_set_bitmap_port.c
kernel/net/netfilter/ipset/ip_set_hash_gen.h
kernel/net/netfilter/ipset/ip_set_list_set.c

index 0c01680c68e0f7cc03beef0303faa527dca2d12b..9b6c907ddb216d8904894b7b717c4833905fef4f 100644 (file)
@@ -589,6 +589,16 @@ else
        AC_SUBST(HAVE_TYPEDEF_SCTP_SCTPHDR_T, undef)
 fi
 
+AC_MSG_CHECKING([kernel source for timer_setup in timer.h])
+if test -f $ksourcedir/include/linux/timer.h && \
+   $GREP -q ' timer_setup' $ksourcedir/include/linux/timer.h; then
+       AC_MSG_RESULT(yes)
+       AC_SUBST(HAVE_TIMER_SETUP, define)
+else
+       AC_MSG_RESULT(no)
+       AC_SUBST(HAVE_TIMER_SETUP, undef)
+fi
+
 AC_MSG_CHECKING([kernel source for struct net_generic])
 if test -f $ksourcedir/include/net/netns/generic.h && \
    $GREP -q 'struct net_generic' $ksourcedir/include/net/netns/generic.h; then
index 2630600e3d7a773a7e41c4c9e0a309d7c482e5db..e1caa68cd772e7ce20b3aaeb146acb68f8a98eeb 100644 (file)
@@ -43,6 +43,7 @@
 #@HAVE_PASSING_EXTENDED_ACK_TO_PARSERS@ HAVE_PASSING_EXTENDED_ACK_TO_PARSERS
 #@HAVE_PASSING_EXTENDED_ACK_TO_CALLBACKS@ HAVE_PASSING_EXTENDED_ACK_TO_CALLBACKS
 #@HAVE_TYPEDEF_SCTP_SCTPHDR_T@ HAVE_TYPEDEF_SCTP_SCTPHDR_T
+#@HAVE_TIMER_SETUP@ HAVE_TIMER_SETUP
 
 #ifdef HAVE_EXPORT_SYMBOL_GPL_IN_MODULE_H
 #include <linux/module.h>
@@ -337,6 +338,20 @@ static inline u16 nfnl_msg_type(u8 subsys, u8 msg_type)
 #define XT_FAMILY(par)         (par)->family
 #endif
 
+#ifdef HAVE_TIMER_SETUP
+#define GC_ARG                 struct timer_list *t
+#define TIMER_SETUP(a, b)      timer_setup(a, b, 0)
+#define INIT_GC_VARS(type, var)                                \
+       struct type *var = from_timer(var, t, gc);      \
+       struct ip_set *set = var->set
+#else
+#define GC_ARG                 unsigned long ul_set
+#define TIMER_SETUP(a, b)      setup_timer(a, b, (unsigned long)set)
+#define INIT_GC_VARS(type, var)                                \
+       struct ip_set *set = (struct ip_set *)ul_set;   \
+       struct type *var = set->data
+#endif
+
 #ifndef smp_mb__before_atomic
 #define smp_mb__before_atomic()        smp_mb()
 #define smp_mb__after_atomic() smp_mb()
index 856cc26119f180a787a369243eb20b9ded67776a..aa3dbb139d22d77d7dd4b1421818574549445e83 100644 (file)
 #define get_ext(set, map, id)  ((map)->extensions + ((set)->dsize * (id)))
 
 static void
-mtype_gc_init(struct ip_set *set, void (*gc)(unsigned long ul_set))
+mtype_gc_init(struct ip_set *set, void (*gc)(GC_ARG))
 {
        struct mtype *map = set->data;
 
-       setup_timer(&map->gc, gc, (unsigned long)set);
+       TIMER_SETUP(&map->gc, gc);
        mod_timer(&map->gc, jiffies + IPSET_GC_PERIOD(set->timeout) * HZ);
 }
 
@@ -274,10 +274,9 @@ out:
 }
 
 static void
-mtype_gc(unsigned long ul_set)
+mtype_gc(GC_ARG)
 {
-       struct ip_set *set = (struct ip_set *)ul_set;
-       struct mtype *map = set->data;
+       INIT_GC_VARS(mtype, map);
        void *x;
        u32 id;
 
index 9f4f28d6e714df6a280efdacb8831ccfa6f840f2..d3e017478e027223045bbdd8e33b2cba210f019b 100644 (file)
@@ -48,6 +48,9 @@ struct bitmap_ip {
        size_t memsize;         /* members size */
        u8 netmask;             /* subnet netmask */
        struct timer_list gc;   /* garbage collection */
+#ifdef HAVE_TIMER_SETUP
+       struct ip_set *set;     /* attached to this ip_set */
+#endif
        unsigned char extensions[0]     /* data extensions */
                __aligned(__alignof__(u64));
 };
@@ -232,6 +235,9 @@ init_map_ip(struct ip_set *set, struct bitmap_ip *map,
        map->netmask = netmask;
        set->timeout = IPSET_NO_TIMEOUT;
 
+#ifdef HAVE_TIMER_SETUP
+       map->set = set;
+#endif
        set->data = map;
        set->family = NFPROTO_IPV4;
 
index f7415d41e8928872339cfa02b25970fb91732a85..319600ad85ff18d6b4673167e5017613e873c40d 100644 (file)
@@ -52,6 +52,9 @@ struct bitmap_ipmac {
        u32 elements;           /* number of max elements in the set */
        size_t memsize;         /* members size */
        struct timer_list gc;   /* garbage collector */
+#ifdef HAVE_TIMER_SETUP
+       struct ip_set *set;     /* attached to this ip_set */
+#endif
        unsigned char extensions[0]     /* MAC + data extensions */
                __aligned(__alignof__(u64));
 };
@@ -307,6 +310,9 @@ init_map_ipmac(struct ip_set *set, struct bitmap_ipmac *map,
        map->elements = elements;
        set->timeout = IPSET_NO_TIMEOUT;
 
+#ifdef HAVE_TIMER_SETUP
+       map->set = set;
+#endif
        set->data = map;
        set->family = NFPROTO_IPV4;
 
index c6e1ebf0f351ae817baf3b4b516e3a0a0ecde452..99edaac1c24dacb8a392387805d22c688e6e3ac9 100644 (file)
@@ -40,6 +40,9 @@ struct bitmap_port {
        u32 elements;           /* number of max elements in the set */
        size_t memsize;         /* members size */
        struct timer_list gc;   /* garbage collection */
+#ifdef HAVE_TIMER_SETUP
+       struct ip_set *set;     /* attached to this ip_set */
+#endif
        unsigned char extensions[0]     /* data extensions */
                __aligned(__alignof__(u64));
 };
@@ -214,6 +217,9 @@ init_map_port(struct ip_set *set, struct bitmap_port *map,
        map->last_port = last_port;
        set->timeout = IPSET_NO_TIMEOUT;
 
+#ifdef HAVE_TIMER_SETUP
+       map->set = set;
+#endif
        set->data = map;
        set->family = NFPROTO_UNSPEC;
 
index 7cc0784069baab194fbda7f6533428e623c9674d..d24ee3c87788fb9e23f5ff6d26a090c72053b29c 100644 (file)
@@ -280,6 +280,9 @@ htable_bits(u32 hashsize)
 struct htype {
        struct htable __rcu *table; /* the hash table */
        struct timer_list gc;   /* garbage collection when timeout enabled */
+#ifdef HAVE_TIMER_SETUP
+       struct ip_set *set;     /* attached to this ip_set */
+#endif
        u32 maxelem;            /* max elements in the hash */
        u32 initval;            /* random jhash init value */
 #ifdef IP_SET_HASH_WITH_MARKMASK
@@ -429,11 +432,11 @@ mtype_destroy(struct ip_set *set)
 }
 
 static void
-mtype_gc_init(struct ip_set *set, void (*gc)(unsigned long ul_set))
+mtype_gc_init(struct ip_set *set, void (*gc)(GC_ARG))
 {
        struct htype *h = set->data;
 
-       setup_timer(&h->gc, gc, (unsigned long)set);
+       TIMER_SETUP(&h->gc, gc);
        mod_timer(&h->gc, jiffies + IPSET_GC_PERIOD(set->timeout) * HZ);
        pr_debug("gc initialized, run in every %u\n",
                 IPSET_GC_PERIOD(set->timeout));
@@ -527,10 +530,9 @@ mtype_expire(struct ip_set *set, struct htype *h)
 }
 
 static void
-mtype_gc(unsigned long ul_set)
+mtype_gc(GC_ARG)
 {
-       struct ip_set *set = (struct ip_set *)ul_set;
-       struct htype *h = set->data;
+       INIT_GC_VARS(htype, h);
 
        pr_debug("called\n");
        spin_lock_bh(&set->lock);
@@ -1315,6 +1317,9 @@ IPSET_TOKEN(HTYPE, _create)(struct net *net, struct ip_set *set,
        t->htable_bits = hbits;
        RCU_INIT_POINTER(h->table, t);
 
+#ifdef HAVE_TIMER_SETUP
+       h->set = set;
+#endif
        set->data = h;
 #ifndef IP_SET_PROTO_UNDEF
        if (set->family == NFPROTO_IPV4) {
index 2fff6b5dc6f0bee83bbd464d8a7d09c190be57a0..55ecea2d444a74e9c91ea1c572e04d129fb448f1 100644 (file)
@@ -44,6 +44,9 @@ struct set_adt_elem {
 struct list_set {
        u32 size;               /* size of set list array */
        struct timer_list gc;   /* garbage collection */
+#ifdef HAVE_TIMER_SETUP
+       struct ip_set *set;     /* attached to this ip_set */
+#endif
        struct net *net;        /* namespace */
        struct list_head members; /* the set members */
 };
@@ -568,10 +571,9 @@ static const struct ip_set_type_variant set_variant = {
 };
 
 static void
-list_set_gc(unsigned long ul_set)
+list_set_gc(GC_ARG)
 {
-       struct ip_set *set = (struct ip_set *)ul_set;
-       struct list_set *map = set->data;
+       INIT_GC_VARS(list_set, map);
 
        spin_lock_bh(&set->lock);
        set_cleanup_entries(set);
@@ -582,11 +584,11 @@ list_set_gc(unsigned long ul_set)
 }
 
 static void
-list_set_gc_init(struct ip_set *set, void (*gc)(unsigned long ul_set))
+list_set_gc_init(struct ip_set *set, void (*gc)(GC_ARG))
 {
        struct list_set *map = set->data;
 
-       setup_timer(&map->gc, gc, (unsigned long)set);
+       TIMER_SETUP(&map->gc, gc);
        mod_timer(&map->gc, jiffies + IPSET_GC_PERIOD(set->timeout) * HZ);
 }
 
@@ -603,6 +605,9 @@ init_list_set(struct net *net, struct ip_set *set, u32 size)
 
        map->size = size;
        map->net = net;
+#ifdef HAVE_TIMER_SETUP
+       map->set = set;
+#endif
        INIT_LIST_HEAD(&map->members);
        set->data = map;