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 D7FA7A3168 for ; Wed, 16 Oct 2019 13:38:02 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 91FE01E940; Wed, 16 Oct 2019 13:38:02 +0200 (CEST) Received: from mail-io1-f68.google.com (mail-io1-f68.google.com [209.85.166.68]) by dpdk.org (Postfix) with ESMTP id CB9151E934 for ; Wed, 16 Oct 2019 13:38:00 +0200 (CEST) Received: by mail-io1-f68.google.com with SMTP id a1so53408619ioc.6 for ; Wed, 16 Oct 2019 04:38:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=yEQQ3CzNhuHdegMNFGP3Tfz7McKyUFQRV9xHGegQpaE=; b=ITAeYaeltFkz334Yjf42FNx3sG+9FOg12LIEtE1TEpIIRMLb9QRRTVkyyWBL2slU9c uapmjTNJ9x8gH44wvF5kQ3PSvrivnGA/8G4uzmG9t2Ewzm8/aby1adrx6XKMpCb9ou9p 7ksqLmDkIeL+DWmtta0QV9D/m8hy20WjVsasJBVY0VhC1myDyNEsF6mh7V4UnOqWmAYs UbhluADKYlrel3DvBwfdmnR2tsxpMP3xV/0NK0txXWl023B+7qp7xws1USZrFm0Vm7Uy VkLXcHEz/hUsX5qG42EdTrQ1T9v3Yoz1T3NXatc+DemQm/S0xb7BKHCHgXBuYOrop5U1 DK3w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=yEQQ3CzNhuHdegMNFGP3Tfz7McKyUFQRV9xHGegQpaE=; b=VryM3G+C6bOUcy+uvTlWVdx/sKUwJw4xgt8DxE/KtVh2QFw4draPt+7UelED2wNfy4 22/BLqDBonzedvSkjMdOgUIwaEmeopV59UpFuW8u5/DmBauULNjONdBHeMHm4RuM4qdD 9dN/pWkWaHbo1YLqrqISCMMp+ln3DcMfomOXYtNI7PdlbxJbeUrSmrslv8lzn1OUDQKa 75orBBeN9Jrnb8UhdTkM4H8Ub/20O8Cm8ru5siScmkIUrEy5EfIYorBuMyyivZ60RdDE mMSIpCiJpCm7w/qLXIXyX8Ml5UlV05yuzh8oEzct9v8DehH8D8u/KRwnW7MJQ6A4LbyB yMbg== X-Gm-Message-State: APjAAAX7PvkMKwREEDoPEJRmdNa6DI0TzlyyLqFYZ4WefAuAA1hJQFT2 hXlC8LfTr96EXecdadZO0lHcpX75l5GJIBsLVbE= X-Google-Smtp-Source: APXvYqzm2/CJM1lGpZFQ60+ce4MJppkpIjKcHKAkRNmlX8EtXM3c3Hx+7sKKH/d0Uaw3bxsmVC1s4iKPIbk5hfuJAV0= X-Received: by 2002:a5d:8598:: with SMTP id f24mr3621368ioj.60.1571225879970; Wed, 16 Oct 2019 04:37:59 -0700 (PDT) MIME-Version: 1.0 References: <20190906091230.13923-1-vattunuru@marvell.com> <20191008081244.425551a0@hermes.lan> <20191009162831.0c1cf204@hermes.lan> In-Reply-To: From: Jerin Jacob Date: Wed, 16 Oct 2019 17:07:49 +0530 Message-ID: To: Stephen Hemminger Cc: Vamsi Attunuru , dpdk-dev , Thomas Monjalon , Jerin Jacob Content-Type: text/plain; charset="UTF-8" Subject: Re: [dpdk-dev] [PATCH v1 1/1] kernel/linux: introduce vfio_pf kernel module 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" > > > > > > no-iommu case is different where we cannot screw Linux netdev driver, you can create a damage to your self that's an acceptable compromise. > > > > In this case, when DPDK PF bound application dies then it will impact netdev VF driver as gets stalled and there is a security issues to VF netdev driver that DPDK PF can intersect the netdev VF mailbox message. > > > > So this case is different where from Kernel PoV there is damange to netdev VF so this can not be accepted in Linux. > > > > One option is to add this piece of code igb-uio instead to adding new driver. What do you say? > > > > It is the problem for all non bifurcated drivers in DPDK not specific to Marvell. > > > > The workaround is to use igb-uio with non VFIO. > > We can not support UIO through performance reason hence we need a solution that works for VFIO due to HW accelerated mempool architecture (applicable for all NPU) > > > > We can not have VFIO vs UIO specific features disparity in DPDK. Please have same treatment. Either remove igb-uio hacks from Dpdk or enable non upsteamble features through igb-uio.So that there is no disparity for a specific use case / vendor. > > Andrew Rybchenko already acked this patch and I provided enough data > on the need for this patch. > I hope there is no more confusion about the need for this patch. I assume no more issue to merge this patch for RC1. If yes, please discuss.