From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from out4-smtp.messagingengine.com (out4-smtp.messagingengine.com [66.111.4.28]) by dpdk.org (Postfix) with ESMTP id C95661B4AE for ; Fri, 23 Nov 2018 23:29:42 +0100 (CET) Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.nyi.internal (Postfix) with ESMTP id E6A6222255; Fri, 23 Nov 2018 17:29:41 -0500 (EST) Received: from mailfrontend2 ([10.202.2.163]) by compute1.internal (MEProxy); Fri, 23 Nov 2018 17:29:41 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding:content-type; s=mesmtp; bh=0gdHoW1qhcul0RVewbpBcxaFryNuu8HH7gal7osZ2lc=; b=T0ImhS5KA6m+ zz3T4k9hhaNPP8bKbz1dVDPEad6HggRtJcUmbztT0ao0hedhHVQUzs9eOuibHjEx idfR3FplV3y8u5jQmm2YWlgCegPYHIp7gShedmyMOsj4qYKU2q1F1rvFbfqk+1DA Z6XN+b4IwhiV4+9ZTrV4NKH8stpqFE4= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm1; bh=0gdHoW1qhcul0RVewbpBcxaFryNuu8HH7gal7osZ2 lc=; b=fJ0Q7Xe/js9cffrGA96wLq+ONA9dYTl2WLf+JkgQqHo9mkBkxJ7aw4Z91 sZBCnl8mFvkr+KyRXpXLRdoFBgFj7Xx3jO+H8uZBAHMz4U/Bn/EyDDWC4pMt01os 4/7pQHDRiDsZRHtOA52Qkh6sXa4Oq13cFYW1A66tGsFQQErgspt++DPyix43WSpL pa9SfL/RoNg7G3Rdj0oRoLjlDWA02H3b0gTy1yplFAcdndREAHqbfYreIUEgjQRq 0rekMdKYbFYyFWjAVJlzi6cjOhPaNnuE743runxX775relAdOcoeT1rVEa4F2XwH waHxEpduP6DFgJfF3zRGrYDa0fK3Q== X-ME-Sender: X-ME-Proxy: Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id BD8A2102EE; Fri, 23 Nov 2018 17:29:39 -0500 (EST) From: Thomas Monjalon To: Ilya Maximets Cc: Kevin Traynor , Maxime Coquelin , dev@dpdk.org, David Marchand , Tiwei Bie , Zhihong Wang , Ferruh Yigit , Ian Stokes , Bruce Richardson Date: Fri, 23 Nov 2018 23:29:38 +0100 Message-ID: <2699010.cjLPUA7OeM@xps> In-Reply-To: <2f95ce88-8f61-10c0-9131-49e2a5d10048@redhat.com> References: <20181123143620.10480-1-i.maximets@samsung.com> <2f95ce88-8f61-10c0-9131-49e2a5d10048@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [PATCH v3 0/2] IOPL related fixes 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: , X-List-Received-Date: Fri, 23 Nov 2018 22:29:43 -0000 23/11/2018 21:07, Kevin Traynor: > On 11/23/2018 04:25 PM, Maxime Coquelin wrote: > > > > > > On 11/23/18 4:53 PM, Ilya Maximets wrote: > >> On 23.11.2018 18:41, Maxime Coquelin wrote: > >>> Hi, > >>> > >>> On 11/23/18 4:39 PM, Ilya Maximets wrote: > >>>> Patches primary targeted to fix OVS unit test failures with > >>>> DPDK 18.11 due to following error: > >>>> 'IOPL call failed - cannot use virtio PMD'. > >>> > >>> You mention v18.11, do you mean this is a regression? > >> > >> Kind of. But not really a bug. It's just a message that shows up > >> every time you starting the app as a non-root user. > >> > >> The message itself was introduced long time ago, but it wasn't > >> printed for unclear reasons. It's probably some change in logging > >> subsystem uncovered it. > >> > >> I just checked and found that message appears starting from v18.02. > >> But OVS stuck with 17.11 LTS where this message exists, but not > >> printed for some reason. That's why this wasn't an issue previously. > > > > Thanks for the clarification. > > In that case, we may want it in v18.11 LTS, but I think it can wait or > > v18.11.1. > > It would be better if it can be in 18.11. Upstream OVS will point at a > release tarball, and at present an 18.11.1 release is not due until > mid-January. If it's too late for 18.11, alternatively we could do an > earlier 18.11.1 release. I won't take any risk at this stage. The memory of the headaches we had in the past with iopl is too much strong. The real change of this patch is to register the PMD even without successful iopl. I would like to be sure it won't cause any regression.