From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH net-next v3] netlink: Rightsize IFLA_AF_SPEC size calculation Date: Thu, 15 Oct 2015 06:02:39 -0700 (PDT) Message-ID: <20151015.060239.165215853752372323.davem@davemloft.net> References: <1444837888-32656-1-git-send-email-ronen.arad@intel.com> <1444888204-1954-1-git-send-email-ronen.arad@intel.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org To: ronen.arad@intel.com Return-path: Received: from shards.monkeyblade.net ([149.20.54.216]:43095 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752624AbbJOMqd (ORCPT ); Thu, 15 Oct 2015 08:46:33 -0400 In-Reply-To: <1444888204-1954-1-git-send-email-ronen.arad@intel.com> Sender: netdev-owner@vger.kernel.org List-ID: From: Ronen Arad Date: Wed, 14 Oct 2015 22:50:04 -0700 > if_nlmsg_size() overestimates the minimum allocation size of netlink > dump request (when called from rtnl_calcit()) or the size of the > message (when called from rtnl_getlink()). This is because > ext_filter_mask is not supported by rtnl_link_get_af_size() and > rtnl_link_get_size(). > > The over-estimation is significant when at least one netdev has many > VLANs configured (8 bytes for each configured VLAN). > > This patch-set "rightsizes" the protocol specific attribute size > calculation by propagating ext_filter_mask to rtnl_link_get_af_size() > and adding this a argument to get_link_af_size op in rtnl_af_ops. > > Bridge module already used filtering aware sizing for notifications. > br_get_link_af_size_filtered() is consistent with the modified > get_link_af_size op so it replaces br_get_link_af_size() in br_af_ops. > br_get_link_af_size() becomes unused and thus removed. This is missing a proper Signoff