From patchwork Tue Jan 7 02:51:34 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: John Daley X-Patchwork-Id: 13928009 X-Patchwork-Delegate: kuba@kernel.org Received: from alln-iport-3.cisco.com (alln-iport-3.cisco.com [173.37.142.90]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7C238155335 for ; Tue, 7 Jan 2025 02:52:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=173.37.142.90 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736218372; cv=none; b=lRpqwg4TMYOZ7bgyYw0g29nXtZR9hS2ueCD4+H2QgPO0PqfJw9rjF2qosDfQBkPIuXfyDP0oyQf5vrxjhFmfZF+k5Ke3ElvejWoHWkci+g64ulsCJ2iLQUHYxPOwY3L3NQPVyXVME8u9DUcGofYnFbmmAqh3E2qIdD1TYWD19ag= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736218372; c=relaxed/simple; bh=kj3XAeBVudwD3vo/0EEH75kWe1rcErK6eALotHsNWn4=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=b3EODjWcGX+UrN7dMEmLnsksEt/8dhal3+xTelnvudjZmP/kYqOsJ33VthxK4FmCVqHC0xuq8CP32ILOfQgR8SfN82g+2MS/0n8nJQLLAv5Uz1re4C3MnL5G0IR5Rv98BwwBD2ebnR3h/jg57y8Q1v/MoXW+V5SE6gq/vkwqqtM= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=cisco.com; spf=pass smtp.mailfrom=cisco.com; dkim=pass (1024-bit key) header.d=cisco.com header.i=@cisco.com header.b=MXCt+wIU; arc=none smtp.client-ip=173.37.142.90 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=cisco.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=cisco.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=cisco.com header.i=@cisco.com header.b="MXCt+wIU" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cisco.com; i=@cisco.com; l=3131; q=dns/txt; s=iport; t=1736218370; x=1737427970; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=VPokXnowB9WV2FbOa7AooedmIXf8/hgdXZkkQ7irUCc=; b=MXCt+wIUmiiUzca1QvVcawGi0lFp7EDOiqc+TT7fPEl2Ga6yIDCKlhDe J9GXfG8JixREBdHZTfCUXjR435O+PGvGgJalcPGW1poUbFVmVUH7MmlUK fkY0qcf/idr9e2qcJkg4SPsibhLNCVK2GfuUQWN0mwBS7zf8CXct6j8Gm M=; X-CSE-ConnectionGUID: 4+Q3FWGMT3mjMiCzWEe/hA== X-CSE-MsgGUID: BihqDPxgQLabhY16IbKr7g== X-IPAS-Result: A0A6AADjlXxn/5IQJK1aHQEBAQEJARIBBQUBgX8IAQsBgkqBT0NIjHJfpw2BJQNWDwEBAQ9EBAEBhQcCinQCJjQJDgECBAEBAQEDAgMBAQEBAQEBAQEBAQsBAQUBAQECAQcFgQ4ThgiGWwIBAycLAUYQUSsrBxKDAYJlA7A/gXkzgQHeM4FtgUgBhWqHX3CEdycbgUlEgRWBO4ItiwcEgjOFOYZmhx6PcEiBIQNZLAFVEw0KCwcFgTk6AyILCwwLFBwVAhUfEgYVBHREOYJGaUk3Ag0CNYIeJFiCK4RchEeEVoJJVYJIghd8gRqCQS1AAwsYDUgRLDcGDhsGPm4HmmY8g20BgQ58lRWSH4tylRGEJYFjn2MaM4QElAaSSZh8IoI2oW+EZoFnPIFZMxoIGxWDIlIZD44tFrgHJTI8AgcLAQEDCZF0AQE IronPort-Data: A9a23:9o2QUai20wwljWklZntGf5RaX161QxEKZh0ujC45NGQN5FlHY01je htvXDvUafePYmOkL4txO4rjpBsC78KEmtFrGlBorXg1RCxjpJueD7x1DKtf0wB+jyHnZBg6h ynLQoCYdKjYdleF+FH1dOCn9SQkvU2xbuKUIPbePSxsThNTRi4kiBZy88Y0mYcAbeKRW2thg vus5ZSFULOZ82QsaD9Msvrd8EgHUMna4Vv0gHRvPZing3eG/5UlJMp3Db28KXL+Xr5VEoaSL 87fzKu093/u5BwkDNWoiN7TKiXmlZaLYGBiIlIPM0STqkAqSh4ai87XB9JAAatjsAhlqvgqo Dl7WTNcfi9yVkHEsLx1vxC1iEiSN4UekFPMCSDXXcB+UyQqflO0q8iCAn3aMqVEud8sL15h0 MBDLTIobzuRuM+7h7KSH7wEasQLdKEHPasWvnVmiDWcBvE8TNWbEuPB5MRT23E7gcUm8fT2P pVCL2ExKk2eJUQTZD/7C7pm9Ausrnv4cztUoVaYjaE2+GPUigd21dABNfKJIIfaHZgIxBzwS mTu1WX5Gz0iJNOm7TOP8kP12M7FvjmkV9dHfFG/3rsw6LGJ/UQfAQMbUHO3qOe0j0q5Vc4ZL UEIkgIjobU3/V6mUvHyWBq3pHPCtRkZM/JTDuczwAKA0KzZ50CeHGdsZjdHZMYrq4wwSCAm2 0Ghm87vA3pksNW9UXuX+7GVhSm/NSgcMSkJYipsZQ0I/9XuvqktgR/VCNVuCqi4ipvyAz6Y/ tyRhCE6g7NWiYsA0L+2uAiWxTmtvZPOCAUy4207Q16Y0++wX6b9D6TA1LQRxa8owFqxJrVZg EU5pg== IronPort-HdrOrdr: A9a23:QVEvLKHfx5pXz8SypLqE78eALOsnbusQ8zAXPo5KJiC9Ffbo8P xG88576faZslsssTQb6LK90cq7MBfhHOBOgbX5VI3KNGKNhILrFvAG0WKI+VPd8kPFmtK1rZ 0QEJSXzLbLfCFHZQGQ2njfL+od X-Talos-CUID: 9a23:wfKaImOGFmAQkO5DBGper2ArNeUZKXjf6Xb7MXCVBCFpYejA X-Talos-MUID: 9a23:47OZIgbeBur3GOBTnT7snR1aJMhRx62kCHFUz7QMmNPbOnkl X-IronPort-Anti-Spam-Filtered: true X-IronPort-AV: E=Sophos;i="6.12,294,1728950400"; d="scan'208";a="423805522" Received: from alln-l-core-09.cisco.com ([173.36.16.146]) by alln-iport-3.cisco.com with ESMTP/TLS/TLS_AES_256_GCM_SHA384; 07 Jan 2025 02:51:40 +0000 Received: from cisco.com (savbu-usnic-a.cisco.com [10.193.184.48]) by alln-l-core-09.cisco.com (Postfix) with ESMTP id 96BE11800022D; Tue, 7 Jan 2025 02:51:40 +0000 (GMT) Received: by cisco.com (Postfix, from userid 392789) id 6740820F2004; Mon, 6 Jan 2025 18:51:40 -0800 (PST) From: John Daley To: benve@cisco.com, satishkh@cisco.com, andrew+netdev@lunn.ch, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, netdev@vger.kernel.org Cc: John Daley , Nelson Escobar Subject: [PATCH net-next 1/2] enic: Move RX coalescing set function Date: Mon, 6 Jan 2025 18:51:34 -0800 Message-Id: <20250107025135.15167-2-johndale@cisco.com> X-Mailer: git-send-email 2.35.2 In-Reply-To: <20250107025135.15167-1-johndale@cisco.com> References: <20250107025135.15167-1-johndale@cisco.com> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-Outbound-SMTP-Client: 10.193.184.48, savbu-usnic-a.cisco.com X-Outbound-Node: alln-l-core-09.cisco.com X-Patchwork-Delegate: kuba@kernel.org Move the function used for setting the RX coalescing range to before the function that checks the link status. It needs to be called from there instead of from the probe function. There is no functional change. Co-developed-by: Nelson Escobar Signed-off-by: Nelson Escobar Co-developed-by: Satish Kharat Signed-off-by: Satish Kharat Signed-off-by: John Daley Reviewed-by: Michal Swiatkowski Reviewed-by: Andrew Lunn --- drivers/net/ethernet/cisco/enic/enic_main.c | 60 ++++++++++----------- 1 file changed, 30 insertions(+), 30 deletions(-) diff --git a/drivers/net/ethernet/cisco/enic/enic_main.c b/drivers/net/ethernet/cisco/enic/enic_main.c index 9913952ccb42..957efe73e41a 100644 --- a/drivers/net/ethernet/cisco/enic/enic_main.c +++ b/drivers/net/ethernet/cisco/enic/enic_main.c @@ -428,6 +428,36 @@ static void enic_mtu_check(struct enic *enic) } } +static void enic_set_rx_coal_setting(struct enic *enic) +{ + unsigned int speed; + int index = -1; + struct enic_rx_coal *rx_coal = &enic->rx_coalesce_setting; + + /* 1. Read the link speed from fw + * 2. Pick the default range for the speed + * 3. Update it in enic->rx_coalesce_setting + */ + speed = vnic_dev_port_speed(enic->vdev); + if (speed > ENIC_LINK_SPEED_10G) + index = ENIC_LINK_40G_INDEX; + else if (speed > ENIC_LINK_SPEED_4G) + index = ENIC_LINK_10G_INDEX; + else + index = ENIC_LINK_4G_INDEX; + + rx_coal->small_pkt_range_start = mod_range[index].small_pkt_range_start; + rx_coal->large_pkt_range_start = mod_range[index].large_pkt_range_start; + rx_coal->range_end = ENIC_RX_COALESCE_RANGE_END; + + /* Start with the value provided by UCSM */ + for (index = 0; index < enic->rq_count; index++) + enic->cq[index].cur_rx_coal_timeval = + enic->config.intr_timer_usec; + + rx_coal->use_adaptive_rx_coalesce = 1; +} + static void enic_link_check(struct enic *enic) { int link_status = vnic_dev_link_status(enic->vdev); @@ -1901,36 +1931,6 @@ static void enic_synchronize_irqs(struct enic *enic) } } -static void enic_set_rx_coal_setting(struct enic *enic) -{ - unsigned int speed; - int index = -1; - struct enic_rx_coal *rx_coal = &enic->rx_coalesce_setting; - - /* 1. Read the link speed from fw - * 2. Pick the default range for the speed - * 3. Update it in enic->rx_coalesce_setting - */ - speed = vnic_dev_port_speed(enic->vdev); - if (ENIC_LINK_SPEED_10G < speed) - index = ENIC_LINK_40G_INDEX; - else if (ENIC_LINK_SPEED_4G < speed) - index = ENIC_LINK_10G_INDEX; - else - index = ENIC_LINK_4G_INDEX; - - rx_coal->small_pkt_range_start = mod_range[index].small_pkt_range_start; - rx_coal->large_pkt_range_start = mod_range[index].large_pkt_range_start; - rx_coal->range_end = ENIC_RX_COALESCE_RANGE_END; - - /* Start with the value provided by UCSM */ - for (index = 0; index < enic->rq_count; index++) - enic->cq[index].cur_rx_coal_timeval = - enic->config.intr_timer_usec; - - rx_coal->use_adaptive_rx_coalesce = 1; -} - static int enic_dev_notify_set(struct enic *enic) { int err; From patchwork Tue Jan 7 02:51:35 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: John Daley X-Patchwork-Id: 13928007 X-Patchwork-Delegate: kuba@kernel.org Received: from alln-iport-7.cisco.com (alln-iport-7.cisco.com [173.37.142.94]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DE90118C0C for ; Tue, 7 Jan 2025 02:51:41 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=173.37.142.94 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736218305; cv=none; b=IjLD4S2CTIZoGdkA/l4AgzkNFmjCjR7H7GqgoRT8GuOpP21fTVSENxjjx7HLjlSrZ+fs2Z4wExMP/5yD0Qt8tzfqErUra5KSkPuMp8dMBri9ORJH9tNVScZ+d7yCt3oYfYoD1oEFHJahjgQlIILhHCARoVmrl4nYD0oh8cPgdIs= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736218305; c=relaxed/simple; bh=ZQCjTnrC+eQQwsbIRkFxunZ7IDCq4JWg1FkkSwGz6bU=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version; b=knCrqL450YPwDDf+64+ofEZMGHo0JOsUEuiSZ664xFQBcZX/PjiB14+ktD5NXmYqPA3WXZryELDpsQkvTN77nie9YzizDen3W/Ta7d5CR1CP6h2XBfwM1QkL2UwVtlNyD9QpAAEqsXiU588G88s9yZq54n42HJxamAWvDUmBlqY= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=cisco.com; spf=pass smtp.mailfrom=cisco.com; dkim=pass (1024-bit key) header.d=cisco.com header.i=@cisco.com header.b=KkDnJ4rw; arc=none smtp.client-ip=173.37.142.94 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=cisco.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=cisco.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=cisco.com header.i=@cisco.com header.b="KkDnJ4rw" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cisco.com; i=@cisco.com; l=2081; q=dns/txt; s=iport; t=1736218301; x=1737427901; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=5oTBSVgdz/mXP/359jvtRWK9RXTfrLMUwJKW4oneDJM=; b=KkDnJ4rwmr62qBfkCsi7YLHYrhdVeJ86KZdLRtD/LcdeMSX198SlBoXu 9isNOFO37L+QMHuFW6gwnSUSHdveqQtIr8QjllEm3gO798kNcrt7fh2hM M7P7JH1SEKkqzhfwfEb3dAZHS3i6cb3GYYpiw13Js1oKn1Q9WAX9BbKK2 o=; X-CSE-ConnectionGUID: cRIvfhrmQuC5L6H1s0IqJg== X-CSE-MsgGUID: qtGuCkURSIyIv3UJ+NhFig== X-IPAS-Result: A0ANAADjlXxnj5MQJK1aHAEBAQEBAQcBARIBAQQEAQGBfwcBAQsBhBlDSIxyX6cNFIERA1YPAQEBD0QEAQGFBwKKdAImNAkOAQIEAQEBAQMCAwEBAQEBAQEBAQEBCwEBBQEBAQIBBwUUAQEBAQEBOQVJhgiGWwIBAycLATQSEFErKwcSgwGCZQOwP4F5M4EB3jOBbYFIAYVqh19whHcnG4FJRIJQgT5vhCpmhXcEh2yddEiBIQNZLAFVEw0KCwcFgTk6AyILCwwLFBwVAhUeARIGFQR0RDmCRmlJNwINAjWCHiRYgiuEXIRHhFaCSVWCSIIXfIEagm5AAwsYDUgRLDcGDhsGPm4HmmY8g20BgQ41RzWmf6EDhCWBY59jGjOqU5h8IqQlhGaBZzqBWzMaCBsVgyJSGQ+OLQ0JuAclMjwCBwsBAQMJkXQBAQ IronPort-Data: A9a23:Zzbika+sqSbmdNeAASNcDrUDZX6TJUtcMsCJ2f8bNWPcYEJGY0x3z zFKX2iDMq3bajH8KIx2bI+xoB8HucDSz99lTlRp/CtEQiMRo6IpJzg2wmQcns+2BpeeJK6yx 5xGMrEsFOhtEDmE4E/rauW5xZVF/fngbqLmD+LZMTxGSwZhSSMw4TpugOdRbrRA2bBVOCvT/ 4qoyyHjEAX9gWMsazpKs/nrRC5H5ZwehhtJ5jTSWtgT1LPuvyF9JI4SI6i3M0z5TuF8dsamR /zOxa2O5WjQ+REgELuNyt4XpWVTH9Y+lSDX4pZnc/DKbipq/0Te4Y5nXBYoUnq7vh3S9zxHJ HqhgrTrIeshFvWkdO3wyHC0GQkmVUFN0OevzXRSLaV/wmWeG0YAzcmCA2k9B6wG9OZtBFhT3 qE8dhBUdUiinsC5lefTpulE3qzPLeHiOIcZ/3UlxjbDALN/GdbIQr7B4plT2zJYasJmRKmFI ZFHL2MxKk2cPHWjOX9PYH46tOWvhn/zejlVgFmUvqEwpWPUyWSd1ZCxaIqJKoXbHJo9ckCwv 37p4k/6PyshFc2+12G40DGKhsHOknauMG4VPOblrqEx2gL7KnYoIBEfS1a+ifWwlEO7X9VRN woS9zZGhaU+6UmiXNThdxK/p3GAs1gXXN84O+4n4gilyafO5QudQG8eQVZpYdoqrsIpBjony lOEgfvtGDpp9raVIVqF/72ZqzKaIyUZLWYeIyQDSGMt5dT/rIwtpgzAQ8wlE6OviNDxXzbqz Fi3QDMWjrEXi4sPkq68512C2mrqrZnSRQlz7QLSNo640u9nTIWfRpKm6ULe1OtrcaLHbgCa5 FkducfLuYjiEqqxvCCKRewMGpSg6PCELCDQjDZT838JqW3FF5mLI9w43d1uGHqFJProbtMAX aMyhe+zzMILVJdJRfYrC25UNyjM5fKxfTgCfquPBueimrArKGe6ENhGPCZ8JVzFnkk2ir0YM pyGa8uqBntyIf04l2TvF7dGied0lnxWKYbvqXbTkkrPPV22OS/9dFv5GAHVBgzExPre+VyLr 4Y32zWilEUHDLeWjtbrHX47dg1SciNhWvgaWuRcd/WIJUJ9CXo9BvrKibIncMoNokimvrmgw 51JYWcBkACXrSSecW2iMykzAJuxBswXhSxgYkQR0aOAhyNLjXCHsPxHL8NfkHhO3LAL8MOYu NFfIp7aWK0SFGiZk9nfBLGkxLFfmN2QrVrmF0KYjPIXJvaMmyShFgfYQzbS IronPort-HdrOrdr: A9a23:SImdR6PL6aIdMMBcTsajsMiBIKoaSvp037Dk7S9MoHtuA6ulfq +V/cjzuSWYtN9VYgBDpTniAtjlfZqjz/5ICOAqVN/INjUO+lHYSb2KhrGN/9SPIUHDH5ZmpM Rdm2wUMqyIMbC85vyKhjWFLw== X-Talos-CUID: 9a23:RDCtamDb6FJqg/r6EyJd1VAaGtogS2D2kVOOAxajUUdwarLAHA== X-Talos-MUID: 9a23:iPWkeQQbDONJ9MJdRXT1mB5/H8RV2Z+PGWETyqgk+O+EGy1/bmI= X-IronPort-Anti-Spam-Filtered: true X-IronPort-AV: E=Sophos;i="6.12,294,1728950400"; d="scan'208";a="408481917" Received: from alln-l-core-10.cisco.com ([173.36.16.147]) by alln-iport-7.cisco.com with ESMTP/TLS/TLS_AES_256_GCM_SHA384; 07 Jan 2025 02:51:40 +0000 Received: from cisco.com (savbu-usnic-a.cisco.com [10.193.184.48]) by alln-l-core-10.cisco.com (Postfix) with ESMTP id 97BB118000274; Tue, 7 Jan 2025 02:51:40 +0000 (GMT) Received: by cisco.com (Postfix, from userid 392789) id 67A6F20F2005; Mon, 6 Jan 2025 18:51:40 -0800 (PST) From: John Daley To: benve@cisco.com, satishkh@cisco.com, andrew+netdev@lunn.ch, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, netdev@vger.kernel.org Cc: John Daley , Nelson Escobar Subject: [PATCH net-next 2/2] enic: Obtain the Link speed only after the link comes up Date: Mon, 6 Jan 2025 18:51:35 -0800 Message-Id: <20250107025135.15167-3-johndale@cisco.com> X-Mailer: git-send-email 2.35.2 In-Reply-To: <20250107025135.15167-1-johndale@cisco.com> References: <20250107025135.15167-1-johndale@cisco.com> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-Outbound-SMTP-Client: 10.193.184.48, savbu-usnic-a.cisco.com X-Outbound-Node: alln-l-core-10.cisco.com X-Patchwork-Delegate: kuba@kernel.org The link speed is obtained in the RX adaptive coalescing function. It was being called at probe time when the link may not be up. Change the call to run after the Link comes up. The impact of not getting the correct link speed was that the low end of the adaptive interrupt range was always being set to 0 which could have caused a slight increase in the number of RX interrupts. Co-developed-by: Nelson Escobar Signed-off-by: Nelson Escobar Co-developed-by: Satish Kharat Signed-off-by: Satish Kharat Signed-off-by: John Daley Reviewed-by: Michal Swiatkowski --- drivers/net/ethernet/cisco/enic/enic_main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/cisco/enic/enic_main.c b/drivers/net/ethernet/cisco/enic/enic_main.c index 957efe73e41a..49f6cab01ed5 100644 --- a/drivers/net/ethernet/cisco/enic/enic_main.c +++ b/drivers/net/ethernet/cisco/enic/enic_main.c @@ -109,7 +109,7 @@ static struct enic_intr_mod_table mod_table[ENIC_MAX_COALESCE_TIMERS + 1] = { static struct enic_intr_mod_range mod_range[ENIC_MAX_LINK_SPEEDS] = { {0, 0}, /* 0 - 4 Gbps */ {0, 3}, /* 4 - 10 Gbps */ - {3, 6}, /* 10 - 40 Gbps */ + {3, 6}, /* 10+ Gbps */ }; static void enic_init_affinity_hint(struct enic *enic) @@ -466,6 +466,7 @@ static void enic_link_check(struct enic *enic) if (link_status && !carrier_ok) { netdev_info(enic->netdev, "Link UP\n"); netif_carrier_on(enic->netdev); + enic_set_rx_coal_setting(enic); } else if (!link_status && carrier_ok) { netdev_info(enic->netdev, "Link DOWN\n"); netif_carrier_off(enic->netdev); @@ -3063,7 +3064,6 @@ static int enic_probe(struct pci_dev *pdev, const struct pci_device_id *ent) timer_setup(&enic->notify_timer, enic_notify_timer, 0); enic_rfs_flw_tbl_init(enic); - enic_set_rx_coal_setting(enic); INIT_WORK(&enic->reset, enic_reset); INIT_WORK(&enic->tx_hang_reset, enic_tx_hang_reset); INIT_WORK(&enic->change_mtu_work, enic_change_mtu_work);