From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id EFCE7A034F for ; Sat, 2 Nov 2019 02:35:10 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id C74EA1E886; Sat, 2 Nov 2019 02:35:10 +0100 (CET) Received: from alln-iport-6.cisco.com (alln-iport-6.cisco.com [173.37.142.93]) by dpdk.org (Postfix) with ESMTP id 47E801E86F; Sat, 2 Nov 2019 02:35:07 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=1890; q=dns/txt; s=iport; t=1572658507; x=1573868107; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=UY2RZMGufLYYUaeOxZkq2z+WYUCB3YzSbVgSCvIKyL4=; b=CTtYzxctxOhx9QxaBZ4nya4PNErW+z6rUz0CJDOKZqEtV2hcj/EFDabU CLCCRWtHc3gW6VZd4PySWrL+iyVvQPjXD9g4Nu8yh8SR9dbZLokKwjwKI gaDoXGdfbQvzN/J+FLeM+vvH9qA9S1vXf1uABtm1iZ1iFFpmC3y5WiltM k=; X-IronPort-AV: E=Sophos;i="5.68,257,1569283200"; d="scan'208";a="370801706" Received: from rcdn-core-10.cisco.com ([173.37.93.146]) by alln-iport-6.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 02 Nov 2019 01:35:05 +0000 Received: from cisco.com (savbu-usnic-a.cisco.com [10.193.184.48]) by rcdn-core-10.cisco.com (8.15.2/8.15.2) with ESMTP id xA21Z5k8010940; Sat, 2 Nov 2019 01:35:05 GMT Received: by cisco.com (Postfix, from userid 508933) id 4F31220F2003; Fri, 1 Nov 2019 18:35:05 -0700 (PDT) From: Hyong Youb Kim To: Ferruh Yigit Cc: dev@dpdk.org, John Daley , Hyong Youb Kim , stable@dpdk.org Date: Fri, 1 Nov 2019 18:35:02 -0700 Message-Id: <20191102013502.17965-1-hyonkim@cisco.com> X-Mailer: git-send-email 2.22.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Outbound-SMTP-Client: 10.193.184.48, savbu-usnic-a.cisco.com X-Outbound-Node: rcdn-core-10.cisco.com Subject: [dpdk-stable] [PATCH] net/enic: re-enable link status change interrupt X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" When INTx is used, the interrupt handler needs to explicitly re-enable interrupt in order to receive another one in future. The LSC interrupt handler currently does not, and the link state never gets updated when INTx is used (e.g. uio_pci_generic). Call rte_intr_ack() at the end of the handler, to re-enable INTx. Fixes: fefed3d1e62c ("enic: new driver") Cc: stable@dpdk.org Signed-off-by: Hyong Youb Kim Reviewed-by: John Daley --- drivers/net/enic/Makefile | 1 + drivers/net/enic/enic_main.c | 2 ++ drivers/net/enic/meson.build | 1 + 3 files changed, 4 insertions(+) diff --git a/drivers/net/enic/Makefile b/drivers/net/enic/Makefile index 10736a567..686efa5af 100644 --- a/drivers/net/enic/Makefile +++ b/drivers/net/enic/Makefile @@ -13,6 +13,7 @@ EXPORT_MAP := rte_pmd_enic_version.map LIBABIVER := 1 +CFLAGS += -DALLOW_EXPERIMENTAL_API CFLAGS += -I$(SRCDIR)/base/ CFLAGS += -I$(SRCDIR) CFLAGS += -O3 diff --git a/drivers/net/enic/enic_main.c b/drivers/net/enic/enic_main.c index f5b9b0dcb..57574a5f7 100644 --- a/drivers/net/enic/enic_main.c +++ b/drivers/net/enic/enic_main.c @@ -448,6 +448,8 @@ enic_intr_handler(void *arg) enic_link_update(dev); _rte_eth_dev_callback_process(dev, RTE_ETH_EVENT_INTR_LSC, NULL); enic_log_q_error(enic); + /* Re-enable irq in case of INTx */ + rte_intr_ack(&enic->pdev->intr_handle); } static int enic_rxq_intr_init(struct enic *enic) diff --git a/drivers/net/enic/meson.build b/drivers/net/enic/meson.build index 1bd7cc7e1..204c97fa0 100644 --- a/drivers/net/enic/meson.build +++ b/drivers/net/enic/meson.build @@ -1,6 +1,7 @@ # SPDX-License-Identifier: BSD-3-Clause # Copyright(c) 2018 Cisco Systems, Inc. +cflags += ['-DALLOW_EXPERIMENTAL_API'] sources = files( 'base/vnic_cq.c', 'base/vnic_dev.c', -- 2.22.0