From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gregory Etelson Subject: Re: [PATCH v2] igb_uio: issue FLR during open and release of device file Date: Sun, 16 Jul 2017 07:22:04 +0300 Message-ID: <4563824.ntyntO6lBN@polaris> References: <1497260285-27536-1-git-send-email-shijith.thotton@caviumnetworks.com> <1499426031-2664-1-git-send-email-shijith.thotton@caviumnetworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: "Tan, Jianfeng" , "dev@dpdk.org" , "Yigit, Ferruh" , Thomas Monjalon , Stephen Hemminger , "Lu, Wenzhuo" , spdk@lists.01.org To: Shijith Thotton Return-path: Received: from mail-wm0-f46.google.com (mail-wm0-f46.google.com [74.125.82.46]) by dpdk.org (Postfix) with ESMTP id 53C6F1E34 for ; Sun, 16 Jul 2017 06:22:07 +0200 (CEST) Received: by mail-wm0-f46.google.com with SMTP id 62so49794126wmw.1 for ; Sat, 15 Jul 2017 21:22:07 -0700 (PDT) In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hello Shijith, Please add the patch to uio_pci_generic.c file in Linux kernel We experience similar faults with NVMe devices On Wednesday, 12 July 2017 06:40:55 IDT Tan, Jianfeng wrote: > > > -----Original Message----- > > From: Shijith Thotton [mailto:shijith.thotton@caviumnetworks.com] > > Sent: Friday, July 7, 2017 7:14 PM > > To: dev@dpdk.org > > Cc: Yigit, Ferruh; Gregory Etelson; Thomas Monjalon; Stephen Hemminger; > > Tan, Jianfeng; Lu, Wenzhuo > > Subject: [PATCH v2] igb_uio: issue FLR during open and release of device file > > > > Set UIO info device file operations open and release. Call pci reset > > function inside open and release to clear device state at start and end. > > Copied this behaviour from vfio_pci kernel module code. With this patch, > > it is not mandatory to issue FLR by PMD's during init and close. > > > > Bus master enable and disable are added in open and release respectively > > to take care of device DMA. > > > > Signed-off-by: Shijith Thotton > > Reviewed-by: Jianfeng Tan