From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nikolay Aleksandrov Subject: Re: [PATCH net-next v2 0/9] net: bridge: convert bool options to bits Date: Wed, 26 Sep 2018 17:04:03 +0300 Message-ID: <2ab2aeb3-7667-816c-a1cd-97dbe723db33@cumulusnetworks.com> References: <20180926140107.10218-1-nikolay@cumulusnetworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: roopa@cumulusnetworks.com, davem@davemloft.net, stephen@networkplumber.org, bridge@lists.linux-foundation.org, idosch@idosch.org To: netdev@vger.kernel.org Return-path: Received: from mail-wr1-f65.google.com ([209.85.221.65]:33221 "EHLO mail-wr1-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726768AbeIZURN (ORCPT ); Wed, 26 Sep 2018 16:17:13 -0400 Received: by mail-wr1-f65.google.com with SMTP id f10-v6so6137926wrs.0 for ; Wed, 26 Sep 2018 07:04:05 -0700 (PDT) In-Reply-To: <20180926140107.10218-1-nikolay@cumulusnetworks.com> Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: On 26/09/18 17:00, Nikolay Aleksandrov wrote: > Hi, > A lot of boolean bridge options have been added around the net_bridge > structure resulting in holes and more importantly different cache lines > that need to be fetched in the fast path. This set moves all of those > to bits in a bitfield which resides in a hot cache line thus reducing > the size of net_bridge, the number of holes and the number of cache > lines needed for the fast path. > The set is also sent in preparation for new boolean options to avoid > spreading them in the structure and making new holes. > One nice side-effect is that we avoid potential race conditions by using > the bitops since some of the options were bits being directly set in > parallel risking hard to debug issues (has_ipv6_addr). > > Before: > size: 1184, holes: 8, sum holes: 30 > After: > size: 1160, holes: 3, sum holes: 7 > > Patch 01 is a trivial style fix > Patch 02 adds the new options bitfield and converts the vlan boolean > options to bits > Patches 03-08 convert the rest of the boolean options to bits > Patch 09 re-arranges a few fields in net_bridge to further reduce size > > v2: patch 09: remove the comment about offload_fwd_mark in net_bridge and > leave it in the last 4 bytes, thanks to Ido for spotting it And obviously it's not in the last 4 bytes, but that's fine. That must've said just: "leave it where it is now". > > Stephen, I've added your reviewed-by in patches 01-08, only removed it > from patch 09 as the patch has changed. > > Thanks, > Nik >