From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from shards.monkeyblade.net ([184.105.139.130]:53312 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933502AbeCGPoq (ORCPT ); Wed, 7 Mar 2018 10:44:46 -0500 Date: Wed, 07 Mar 2018 10:44:43 -0500 (EST) Message-Id: <20180307.104443.336957406656900568.davem@davemloft.net> To: paulb@mellanox.com Cc: tgraf@suug.ch, herbert@gondor.apana.org.au, netdev@vger.kernel.org, kliteyn@mellanox.com, roid@mellanox.com, shahark@mellanox.com, markb@mellanox.com, jiri@mellanox.com, ogerlitz@mellanox.com, matanb@mellanox.com Subject: Re: [PATCH net v3 0/2] rhashtable: Fix rhltable duplicates insertion From: David Miller In-Reply-To: <1520177389-8128-1-git-send-email-paulb@mellanox.com> References: <1520177389-8128-1-git-send-email-paulb@mellanox.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Sender: netdev-owner@vger.kernel.org List-ID: From: Paul Blakey Date: Sun, 4 Mar 2018 17:29:47 +0200 > On our mlx5 driver fs_core.c, we use the rhltable interface to store > flow groups. We noticed that sometimes we get a warning that flow group isn't > found at removal. This rare case was caused when a specific scenario happened, > insertion of a flow group with a similar match criteria (a duplicate), > but only where the flow group rhash_head was second (or not first) > on the relevant rhashtable bucket list. > > The first patch fixes it, and the second one adds a test that show > it is now working. Series applied, thank you.