netdev - Re: [PATCH net-next 2/9] net: bridge: add bitfield for options and convert vlan opts

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: <e5b4f87d-c665-9476-c9d3-a7254394c13f@cumulusnetworks.com>
Date:   Wed, 26 Sep 2018 17:55:47 +0300
From:   Nikolay Aleksandrov <nikolay@...ulusnetworks.com>
To:     Andrew Lunn <andrew@...n.ch>
Cc:     netdev@...r.kernel.org, roopa@...ulusnetworks.com,
        davem@...emloft.net, stephen@...workplumber.org,
        bridge@...ts.linux-foundation.org
Subject: Re: [PATCH net-next 2/9] net: bridge: add bitfield for options and
 convert vlan opts

On 26/09/18 17:48, Andrew Lunn wrote:
> Hi Nikolay
> 
>>  struct net_bridge {
>>  	spinlock_t			lock;
>>  	spinlock_t			hash_lock;
>>  	struct list_head		port_list;
>>  	struct net_device		*dev;
>>  	struct pcpu_sw_netstats		__percpu *stats;
>> +	unsigned long			options;
> 
> Maybe a u32 would be better, so we run out of bits at the same time on
> 32 and 64 bit systems?
> 
>    Andrew
> 

Bitops operate on an unsigned long, I actually had a BUILD_BUG_ON() for
32 bits initially, but checked other places and they seem to be using it
as-is without any checks so I decided to leave it as well (e.g.
sock_flags).