netdev - Re: [RFC net-next 2/6] net: Preparation for vrf device

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <559D4B91.7020600@cumulusnetworks.com>
Date:	Wed, 08 Jul 2015 10:10:57 -0600
From:	David Ahern <dsa@...ulusnetworks.com>
To:	nicolas.dichtel@...nd.com, netdev@...r.kernel.org
CC:	shm@...ulusnetworks.com, roopa@...ulusnetworks.com,
	gospo@...ulusnetworks.com, jtoppins@...ulusnetworks.com,
	nikolay@...ulusnetworks.com, ddutt@...ulusnetworks.com,
	hannes@...essinduktion.org, stephen@...workplumber.org,
	hadi@...atatu.com, ebiederm@...ssion.com, davem@...emloft.net
Subject: Re: [RFC net-next 2/6] net: Preparation for vrf device


On 7/8/15 2:37 AM, Nicolas Dichtel wrote:
> Le 06/07/2015 17:03, David Ahern a écrit :
>> Add a VRF_MASTER flag for interfaces and helper functions for determining
>> if a device is a VRF_MASTER.
>>
>> Also, add link attribute for passing VRF_TABLE id.
>>
>> Both are used in the following patch that adds a VRF device driver.
>>
>> Signed-off-by: Shrijeet Mukherjee <shm@...ulusnetworks.com>
>> Signed-off-by: David Ahern <dsa@...ulusnetworks.com>
>> ---
>>   include/linux/netdevice.h    | 21 +++++++++++++++++++++
>>   include/uapi/linux/if_link.h |  9 +++++++++
>>   2 files changed, 30 insertions(+)
>>
>> diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h
>> index e20979dfd6a9..142cb64f139c 100644
>> --- a/include/linux/netdevice.h
>> +++ b/include/linux/netdevice.h
>> @@ -1274,6 +1274,7 @@ enum netdev_priv_flags {
>>       IFF_XMIT_DST_RELEASE_PERM    = 1<<22,
>>       IFF_IPVLAN_MASTER        = 1<<23,
>>       IFF_IPVLAN_SLAVE        = 1<<24,
>> +    IFF_VRF_MASTER                = 1<<25,
> nit: use tab instead space here  ^^^^^^^
>
> Also, why calling this '_MASTER', is there a notion of SLAVE?
>
>>   };
>>
>>   #define IFF_802_1Q_VLAN            IFF_802_1Q_VLAN
>> @@ -1301,6 +1302,7 @@ enum netdev_priv_flags {
>>   #define IFF_XMIT_DST_RELEASE_PERM    IFF_XMIT_DST_RELEASE_PERM
>>   #define IFF_IPVLAN_MASTER        IFF_IPVLAN_MASTER
>>   #define IFF_IPVLAN_SLAVE        IFF_IPVLAN_SLAVE
>> +#define IFF_VRF_MASTER                IFF_VRF_MASTER
> nit: use tab instead space here          ^^^^^^^
>
>>
>>   /**
>>    *    struct net_device - The DEVICE structure.
>> @@ -1417,6 +1419,7 @@ enum netdev_priv_flags {
>>    *    @dn_ptr:    DECnet specific data
>>    *    @ip6_ptr:    IPv6 specific data
>>    *    @ax25_ptr:    AX.25 specific data
>> + *    @vrf_ptr:    VRF specific data
>>    *    @ieee80211_ptr:    IEEE 802.11 specific data, assign before
>> registering
>>    *
>>    *    @last_rx:    Time of last Rx
>> @@ -1629,6 +1632,7 @@ struct net_device {
>>       struct dn_dev __rcu     *dn_ptr;
>>       struct inet6_dev __rcu    *ip6_ptr;
>>       void            *ax25_ptr;
>> +    struct net_vrf_dev      *vrf_ptr;
> nit: use tab here         ^^^^^^
>
>>       struct wireless_dev    *ieee80211_ptr;
>>       struct wpan_dev        *ieee802154_ptr;
>>   #if IS_ENABLED(CONFIG_MPLS_ROUTING)
>> @@ -3781,6 +3785,23 @@ static inline bool netif_supports_nofcs(struct
>> net_device *dev)
>>       return dev->priv_flags & IFF_SUPP_NOFCS;
>>   }
>>
>> +static inline bool netif_is_vrf(struct net_device *dev)
>> +{
>> +    return dev->priv_flags & IFF_VRF_MASTER;
>> +}
>> +
>> +static inline bool netif_idx_is_vrf(struct net *net, int idx)
> Usally, the index of an interface is named 'ifindex', it eases code reading
> to keep the same name. Something like:
> netif_index_is_vrf(struct net *net, int ifindex)
>
>> +{
>> +    struct net_device *dev = dev_get_by_index(net, idx);
>> +    bool rc = false;
>> +
>> +    if (dev) {
>> +        rc = netif_is_vrf(dev);
>> +        dev_put(dev);
>> +    }
>> +    return rc;
>> +}
>> +
> [snip]

ack on all comments. Updated patch.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html