From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH net v5] net/ipv6: Add anycast addresses to a global hashtable Date: Mon, 29 Oct 2018 20:32:11 -0700 (PDT) Message-ID: <20181029.203211.604037421868394185.davem@davemloft.net> References: <20181028015159.10636-1-0xeffeff@gmail.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, kuznet@ms2.inr.ac.ru, yoshfuji@linux-ipv6.org To: 0xeffeff@gmail.com Return-path: Received: from shards.monkeyblade.net ([23.128.96.9]:45202 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725988AbeJ3MXx (ORCPT ); Tue, 30 Oct 2018 08:23:53 -0400 In-Reply-To: <20181028015159.10636-1-0xeffeff@gmail.com> Sender: netdev-owner@vger.kernel.org List-ID: From: Jeff Barnhill <0xeffeff@gmail.com> Date: Sun, 28 Oct 2018 01:51:59 +0000 > +struct ipv6_ac_addrlist { > + struct in6_addr acal_addr; > + possible_net_t acal_pnet; > + refcount_t acal_users; > + struct hlist_node acal_lst; /* inet6_acaddr_lst */ > + struct rcu_head rcu; > +}; Please just add the hlist to ifcaddr6 instead of duplicating so much information and reference counters here. This seems to waste a lot of memory unnecessary and add lots of unnecessary object allocate/setup/destroy logic.