net: ipmr: always define mroute_reg_vif_num
authorNikolay Aleksandrov <nikolay@cumulusnetworks.com>
Sat, 21 Nov 2015 14:57:25 +0000 (15:57 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 23 Nov 2015 20:06:37 +0000 (15:06 -0500)
Before mroute_reg_vif_num was defined only if any of the CONFIG_PIMSM_
options were set, but that's not really necessary as the size of the
struct is the same in both cases (checked with pahole, both cases size
is 3256 bytes) and we can remove some unnecessary ifdefs to simplify the
code.

Signed-off-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/ipmr.c

index 5271e2eee110ab7ecdd9334cfd58f8ebea4ed841..dd2462f70d3432621d9f278ce02d9299a8ce6f67 100644 (file)
@@ -84,9 +84,7 @@ struct mr_table {
        atomic_t                cache_resolve_queue_len;
        bool                    mroute_do_assert;
        bool                    mroute_do_pim;
-#if defined(CONFIG_IP_PIMSM_V1) || defined(CONFIG_IP_PIMSM_V2)
        int                     mroute_reg_vif_num;
-#endif
 };
 
 struct ipmr_rule {
@@ -347,9 +345,7 @@ static struct mr_table *ipmr_new_table(struct net *net, u32 id)
        setup_timer(&mrt->ipmr_expire_timer, ipmr_expire_process,
                    (unsigned long)mrt);
 
-#ifdef CONFIG_IP_PIMSM
        mrt->mroute_reg_vif_num = -1;
-#endif
 #ifdef CONFIG_IP_MROUTE_MULTIPLE_TABLES
        list_add_tail_rcu(&mrt->list, &net->ipv4.mr_tables);
 #endif
@@ -584,10 +580,8 @@ static int vif_delete(struct mr_table *mrt, int vifi, int notify,
                return -EADDRNOTAVAIL;
        }
 
-#ifdef CONFIG_IP_PIMSM
        if (vifi == mrt->mroute_reg_vif_num)
                mrt->mroute_reg_vif_num = -1;
-#endif
 
        if (vifi + 1 == mrt->maxvif) {
                int tmp;
@@ -824,10 +818,8 @@ static int vif_add(struct net *net, struct mr_table *mrt,
        /* And finish update writing critical data */
        write_lock_bh(&mrt_lock);
        v->dev = dev;
-#ifdef CONFIG_IP_PIMSM
        if (v->flags & VIFF_REGISTER)
                mrt->mroute_reg_vif_num = vifi;
-#endif
        if (vifi+1 > mrt->maxvif)
                mrt->maxvif = vifi+1;
        write_unlock_bh(&mrt_lock);