From patchwork Wed May 9 12:47:01 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Michal Krawczyk X-Patchwork-Id: 39584 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 216831B62F; Wed, 9 May 2018 14:47:31 +0200 (CEST) Received: from mail-lf0-f66.google.com (mail-lf0-f66.google.com [209.85.215.66]) by dpdk.org (Postfix) with ESMTP id 70C311B3D9 for ; Wed, 9 May 2018 14:47:25 +0200 (CEST) Received: by mail-lf0-f66.google.com with SMTP id r25-v6so6993196lfd.1 for ; Wed, 09 May 2018 05:47:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=semihalf-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=cLM9VV9NWj8dagpVyhfZRNRjmjcYeGZkUaq/SFzlW7M=; b=I2grHInCp8fUNeQr98MGezq0L0Xp0Bpcxb2210zM/sIA/r/rGWLcc+K3/HXVMZeNYd o+w8fnOBMBq6aDSKeDhV07jHuxsuItQ9xHx7feaNF3EVRhgJs25bGHDOiufwaY6+xBAl N7NGo4vVIkrGCuv6OSGgjO2F/oRHiZFswC/ztBARujLz0MBzkytTqS6lkITnITCvi1l5 SHpqxVlNOd2mI/Iow6hOOP7zHcrntyvKRz+3gbDL7k7U4iPhPgBAJ2NRbEze9U3/UExM DKAKWluZxb64azaNBnGwIebVlDm95UufnGVs2K50fz1/FJcR+WfLnjru5CArGDULQTr5 Y2CA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=cLM9VV9NWj8dagpVyhfZRNRjmjcYeGZkUaq/SFzlW7M=; b=l5xGf6e3IPps+TkCDJhM3veN+h0xx/QR8FA7mYICZY/bjeIi/Yn6jF+/BqgAlnUoXx aQ9Q/rHWaer7nB7d+1kRClaK2S3uPdgT8G7VeeueifGbC//2Qm8Ed9Aef6lCI4G+ntvq RMWYL9wvOgfNpn2BphqKQQG5CFckZfz2TzK/9Cq6+ifhFyjv4Y9Fdvmt9KMGMQ8dGhXL LPSLi2OzVIr8qY/M3z2ThTsY7MinnWKGIk3LCevw+dw5W7YX4JFxGyFtTR0yhIz6Njn7 JGDgo6EPpuNuK9X/5gCINd6WLtDAMdFRydqp5JfQ3M2ZZwj0HdYXQsmfVlkw/V+icjrc TS9Q== X-Gm-Message-State: ALQs6tCtNfzOSHeo/rbUDlmPuKVfK8Ne2IKuhs9C6PUdh1599BbMXita 3SxBEAR2vohL3vjzaBVr0g7c7w== X-Google-Smtp-Source: AB8JxZoqQurkffvllmTXcIq6uU/Rswu0ZG/aas3f05Kky024WuME+ogVvhkySFAfKyIzRb/MbHZ5lw== X-Received: by 2002:a2e:9616:: with SMTP id v22-v6mr30126462ljh.130.1525870045096; Wed, 09 May 2018 05:47:25 -0700 (PDT) Received: from mkPC.semihalf.local (31-172-191-173.noc.fibertech.net.pl. [31.172.191.173]) by smtp.gmail.com with ESMTPSA id l10-v6sm5149258lja.62.2018.05.09.05.47.24 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 09 May 2018 05:47:24 -0700 (PDT) From: Michal Krawczyk To: Marcin Wojtas , Michal Krawczyk , Guy Tzalik , Evgeny Schemeilin Cc: dev@dpdk.org, matua@amazon.com Date: Wed, 9 May 2018 14:47:01 +0200 Message-Id: <20180509124714.23305-2-mk@semihalf.com> X-Mailer: git-send-email 2.14.1 In-Reply-To: <20180509124714.23305-1-mk@semihalf.com> References: <20180509124714.23305-1-mk@semihalf.com> Subject: [dpdk-dev] [PATCH v1 11/24] net/ena: add checking for admin queue state X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" The admin queue can stop responding or became inactive due to unexpected behaviour of the device. In that case, the whole device should be restarted. Signed-off-by: Michal Krawczyk --- drivers/net/ena/ena_ethdev.c | 37 +++++++++++++++++++++++++++++-------- drivers/net/ena/ena_ethdev.h | 2 ++ 2 files changed, 31 insertions(+), 8 deletions(-) -- 2.14.1 diff --git a/drivers/net/ena/ena_ethdev.c b/drivers/net/ena/ena_ethdev.c index 3956ec379..2284a8c50 100644 --- a/drivers/net/ena/ena_ethdev.c +++ b/drivers/net/ena/ena_ethdev.c @@ -531,6 +531,8 @@ ena_dev_reset(struct rte_eth_dev *dev) for (i = 0; i < nb_queues; ++i) ena_tx_queue_setup(eth_dev, i, adapter->tx_ring_size, 0, NULL); + adapter->trigger_reset = false; + return 0; } @@ -1419,21 +1421,40 @@ static void ena_interrupt_handler_rte(void *cb_arg) ena_com_aenq_intr_handler(ena_dev, adapter); } +static void check_for_missing_keep_alive(struct ena_adapter *adapter) +{ + if (adapter->keep_alive_timeout == ENA_HW_HINTS_NO_TIMEOUT) + return; + + if (unlikely((rte_get_timer_cycles() - adapter->timestamp_wd) >= + adapter->keep_alive_timeout)) { + RTE_LOG(ERR, PMD, "Keep alive timeout\n"); + adapter->reset_reason = ENA_REGS_RESET_KEEP_ALIVE_TO; + adapter->trigger_reset = true; + } +} + +/* Check if admin queue is enabled */ +static void check_for_admin_com_state(struct ena_adapter *adapter) +{ + if (unlikely(!ena_com_get_admin_running_state(&adapter->ena_dev))) { + RTE_LOG(ERR, PMD, "ENA admin queue is not in running state!\n"); + adapter->reset_reason = ENA_REGS_RESET_ADMIN_TO; + adapter->trigger_reset = true; + } +} + static void ena_timer_wd_callback(__rte_unused struct rte_timer *timer, void *arg) { struct ena_adapter *adapter = (struct ena_adapter *)arg; struct rte_eth_dev *dev = adapter->rte_dev; - if (adapter->keep_alive_timeout == ENA_HW_HINTS_NO_TIMEOUT) - return; + check_for_missing_keep_alive(adapter); + check_for_admin_com_state(adapter); - /* Within reasonable timing range no memory barriers are needed */ - if ((rte_get_timer_cycles() - adapter->timestamp_wd) >= - adapter->keep_alive_timeout) { - RTE_LOG(ERR, PMD, "The ENA device is not responding - " - "performing device reset..."); - adapter->reset_reason = ENA_REGS_RESET_KEEP_ALIVE_TO; + if (unlikely(adapter->trigger_reset)) { + RTE_LOG(ERR, PMD, "Trigger reset is on\n"); _rte_eth_dev_callback_process(dev, RTE_ETH_EVENT_INTR_RESET, NULL); } diff --git a/drivers/net/ena/ena_ethdev.h b/drivers/net/ena/ena_ethdev.h index b44cca23e..1f6a7062f 100644 --- a/drivers/net/ena/ena_ethdev.h +++ b/drivers/net/ena/ena_ethdev.h @@ -194,6 +194,8 @@ struct ena_adapter { struct rte_timer timer_wd; uint64_t timestamp_wd; uint64_t keep_alive_timeout; + + bool trigger_reset; }; #endif /* _ENA_ETHDEV_H_ */