From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f46.google.com ([74.125.82.46]:34154 "EHLO mail-wm0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750969AbeBZTul (ORCPT ); Mon, 26 Feb 2018 14:50:41 -0500 Received: by mail-wm0-f46.google.com with SMTP id a20so18493442wmd.1 for ; Mon, 26 Feb 2018 11:50:41 -0800 (PST) Subject: Re: [PATCH net-next v2] r8169: improve interrupt handling To: David Miller Cc: nic_swsd@realtek.com, romieu@fr.zoreil.com, netdev@vger.kernel.org References: <7f1cbc71-9d2d-627f-9e0e-b7e7b1f6ce9b@gmail.com> <20180226.135622.1205217981314568094.davem@davemloft.net> From: Heiner Kallweit Message-ID: <83b2824d-187c-de76-f6e3-14084f216d59@gmail.com> Date: Mon, 26 Feb 2018 20:50:32 +0100 MIME-Version: 1.0 In-Reply-To: <20180226.135622.1205217981314568094.davem@davemloft.net> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: netdev-owner@vger.kernel.org List-ID: Am 26.02.2018 um 19:56 schrieb David Miller: > From: Heiner Kallweit > Date: Sat, 24 Feb 2018 16:53:23 +0100 > >> @@ -736,8 +736,7 @@ struct ring_info { >> }; >> >> enum features { >> - RTL_FEATURE_MSI = (1 << 0), >> - RTL_FEATURE_GMII = (1 << 1), >> + RTL_FEATURE_GMII = (1 << 0), >> }; >> >> struct rtl8169_counters { > ... >> + if (tp->mac_version <= RTL_GIGA_MAC_VER_06) { > > Please, if you are going to keep the logic the same for the older > chips, just keep the RTL_FEATURE_MSI flag around instead of adding > new (and potentially regression causing) tests for this condition. > I see your point. In the case here the condition is meant to be true for chip versions: - having the MSIEnable bit - being PCI, not PCIe Both is true for chip versions <= 06 only, as can be seen in different places in the driver, e.g. - where bit MSIEnable is defined comment says: /* 8169 only. Reserved in the 8168. */ - array rtl_chip_infos[] definition shows that only versions <= 06 are named RTL8169xx and are marked as PCI Last but not least condition "chip version <= 06" is used also in other places in the driver when it's about the RTL8169xx PCI chips. At least I'm convinced this gives enough confidence that we can get rid of flag RTL_FEATURE_MSI. > Thank you. >