From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-yw0-f180.google.com (mail-yw0-f180.google.com [209.85.161.180]) by dpdk.org (Postfix) with ESMTP id A019D95EA for ; Fri, 13 May 2016 12:09:11 +0200 (CEST) Received: by mail-yw0-f180.google.com with SMTP id j74so98224099ywg.1 for ; Fri, 13 May 2016 03:09:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc; bh=zfWsVVXV047i3twwPsmivnvrqZZbUJMt/wh9sEIBmUw=; b=AKf6knTc72Q4lVtt/szTPUWH0LpQLrJyv77wkduUDyZlg2balGZcugnMiwHY+K6Yl1 FxxYaVbUI6ovEfOghdOo+9g51lmFX0Z4ovsMcRRbO20+KBGggZh8+MMr8/mM0JjzFTdr 8wwP+a+18xNYbpcEfrLEYuKLd0bU+C3I4d0Uo= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc; bh=zfWsVVXV047i3twwPsmivnvrqZZbUJMt/wh9sEIBmUw=; b=dHvOxwrKhmfR0Dy6C4UsjV988S2zZfQRoAOlZ/hxlFiEpnRVZC6+90+zYzj1p2fhr5 TEsD65xCBIWSdhMzwpYAqWW3BAkOP0U4uhjVxT0DUu/ulHlWK1N+xeJXfElYWNo9AIo1 A3Z9nk6HeQC5RgcKIgQUyFiD/RorYBXUHGFUqtzWIm10EQCa/bUx5WfBpr1j3ZNph65F ebhO2xY/QR2+H67Cl3VyuluN8j0QUemNyX9ibiZBsCEUVqgQSDQTx8rJl4p81qubmDSU nbRUn8mdNSDhMqWh2Pq7pbQs4LCTAiPuzPlCNTTAnStXZQVafK3b4gV7HLqY+jx5tztv bkeA== X-Gm-Message-State: AOPr4FWggA0MXI3kXqc8McSLrwntDwL6JrB/m7veEAMNLVHmrq2kPLsG/2lZI2LPxckGS3Z4BXj5rvw8jFdm/nhK MIME-Version: 1.0 X-Received: by 10.37.194.198 with SMTP id s189mr6988017ybf.100.1463134150959; Fri, 13 May 2016 03:09:10 -0700 (PDT) Received: by 10.37.223.2 with HTTP; Fri, 13 May 2016 03:09:10 -0700 (PDT) In-Reply-To: <20160513074710.GA4425@localhost.localdomain> References: <20160512031642.GA5855@santosh-Latitude-E5530-non-vPro> <20160512050638.GA7301@santosh-Latitude-E5530-non-vPro> <20160512083342.GA12841@santosh-Latitude-E5530-non-vPro> <20160512103103.GA16360@santosh-Latitude-E5530-non-vPro> <20160513074710.GA4425@localhost.localdomain> Date: Fri, 13 May 2016 18:09:10 +0800 Message-ID: From: Jianbo Liu To: Jerin Jacob Cc: Hemant Agrawal , Santosh Shukla , Stephen Hemminger , "dev@dpdk.org" , Thomas Monjalon Content-Type: text/plain; charset=UTF-8 Subject: Re: [dpdk-dev] [PATCHv3 1/2] config/armv8a: disable igb_uio X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 13 May 2016 10:09:12 -0000 On 13 May 2016 at 15:47, Jerin Jacob wrote: > On Fri, May 13, 2016 at 03:37:01AM +0000, Hemant Agrawal wrote: >> >> >> > -----Original Message----- >> > From: Jianbo Liu [mailto:jianbo.liu@linaro.org] >> > Sent: Friday, May 13, 2016 7:13 AM >> > To: Santosh Shukla >> > Cc: Stephen Hemminger ; Jerin Jacob >> > ; Hemant Agrawal >> > ; dev@dpdk.org; Thomas Monjalon >> > >> > Subject: Re: [dpdk-dev] [PATCHv3 1/2] config/armv8a: disable igb_uio >> > >> > On 12 May 2016 at 18:31, Santosh Shukla >> > wrote: >> > > On Thu, May 12, 2016 at 05:52:54PM +0800, Jianbo Liu wrote: >> > >> On 12 May 2016 at 16:57, Santosh Shukla >> > >> wrote: >> > >> > On Thu, May 12, 2016 at 01:54:13PM +0800, Jianbo Liu wrote: >> > >> >> On 12 May 2016 at 13:06, Santosh Shukla >> > >> >> wrote: >> > >> >> > On Thu, May 12, 2016 at 11:42:26AM +0800, Jianbo Liu wrote: >> > >> >> >> On 12 May 2016 at 11:17, Santosh Shukla >> > >> >> >> wrote: >> > >> >> >> > On Thu, May 12, 2016 at 10:01:05AM +0800, Jianbo Liu wrote: >> > >> >> >> >> On 12 May 2016 at 02:25, Stephen Hemminger >> > wrote: >> > >> >> >> >> > On Wed, 11 May 2016 22:32:16 +0530 Jerin Jacob >> > >> >> >> >> > wrote: >> > >> >> >> >> > >> > >> >> >> >> >> On Wed, May 11, 2016 at 08:22:59AM -0700, Stephen >> > Hemminger wrote: >> > >> >> >> >> >> > On Wed, 11 May 2016 19:17:58 +0530 Hemant Agrawal >> > >> >> >> >> >> > wrote: >> > >> >> >> >> >> > >> > >> >> >> >> >> > > IGB_UIO not supported for arm64 arch in kernel so disable. >> > >> >> >> >> >> > > >> > >> >> >> >> >> > > Signed-off-by: Hemant Agrawal >> > >> >> >> >> >> > > >> > >> >> >> >> >> > > Reviewed-by: Santosh Shukla >> > >> >> >> >> >> > > >> > >> >> >> >> >> > >> > >> >> >> >> >> > Really, I have use IGB_UIO on ARM64 >> > >> >> >> >> >> >> > >> >> >> >> >> May I know what is the technical use case for igb_uio on >> > >> >> >> >> >> arm64 which cannot be addressed through vfio or vfioionommu. >> > >> >> >> >> > >> > >> >> >> >> > I was running on older kernel which did not support vfioionommu >> > mode. >> > >> >> >> >> >> > >> >> >> >> As I said, most of DPDK developers are not kernel >> > >> >> >> >> developers. They may have their own kernel tree, and >> > >> >> >> >> couldn't like to upgrade to latest kernel. >> > >> >> >> >> They can choose to use or not use igb_uio when binding the >> > >> >> >> >> driver. But blindly disabling it in the base config seems unreasonable. >> > >> >> >> > >> > >> >> >> > if user keeping his own kernel so they could also keep >> > >> >> >> > IGB_UIO=y in their local >> > >> >> >> Most likely they don't have local dpdk tree. They write their >> > >> >> >> own applications, complie and link to dpdk lib, then done. >> > >> >> >> >> > >> >> >> > dpdk tree. Why are you imposing user-x custome depedancy on >> > >> >> >> > upstream dpdk base >> > >> >> >> Customer requiremnts is important. I want they can choose the way >> > they like. >> > >> >> >> >> > >> >> > >> > >> >> > so you choose to keep igb_uio option, provided arch doesn't support? >> > >> >> > new user did reported issues with igb_uio for arm64, refer this >> > >> >> > thread [1], as well hemanth too faced issues. we want to avoid that. >> > >> >> > >> > >> >> > If customer maintaing out-of-tree kernel then he can also switch to vfio- >> > way. >> > >> >> > isn;t it? >> > >> >> > >> > >> >> >> > config. Is it not enough for explanation that - Base config >> > >> >> >> > ie.. armv8 doesn;t support pci mmap, so igb_uio is n/a. New >> > >> >> >> > user wont able to build/run dpdk/arm64 in igb_uio-way, He'll >> > >> >> >> > prefer to use upstream stuff. I think, you are not making >> > >> >> >> You are wrong, he can build dpdk. If he like to use upstream >> > >> >> >> without patching, he can use vfio. >> > >> >> > >> > >> >> > I disagree, we want to avoid [1] for new user. >> > >> >> > >> > >> >> >> But you can't ignore the need from old user which is more >> > >> >> >> comfortable with older kernel. >> > >> >> >> >> > >> >> > arm/arm64 dpdk support recently added and I am guessing, most >> > >> >> > likely customer using near latest kernel, switching to vfio won't be so >> > difficult. >> > >> >> > >> > >> >> > Or can you take up responsibility of upstreaming pci mmap patch, >> > >> >> > then we don't need this patch. >> > >> >> > >> > >> >> > [1] http://dpdk.org/ml/archives/dev/2016-January/031313.html >> > >> >> >> > >> >> Can you read carefully about the guide at >> > >> >> http://dpdk.org/doc/guides/linux_gsg/build_dpdk.html? It says to >> > >> >> use uio_pci_generic, igb_uio or vfio-pci. >> > >> > >> > >> > *** applicable and works for x86 only, not for arm64: because pci >> > >> > mmap support not present for arm64, in that case we should update the >> > doc. >> > >> > >> > >> >> Could it be possible that the user in that thread has already read >> > >> >> and tried them all and found that he can't enable vifo with his >> > >> >> kernel, and igb_uio is the easy way for him and asked for help from >> > community? >> > >> >> If so, we have no choice but keeping igb_uio enabled. >> > >> > >> > >> > By then vfionoiommu support was wip progress in dpdk/linux. but now >> > >> > it merged and it works. So no need to retain igb_uio in base config >> > >> > for which to work - user need to use mmap patch at linux side. >> > >> >> > >> We can't decide which kernel user will use. >> > >> >> > > >> > > yes, we can't decide kernel for user but we should be explicit to user >> > > on - what works for dpdk/linux out-of-box vs what could work with use >> > > of out-of-tree patch/RFC's.. example igb_uio. >> > > >> > >> > OK, please persuade Stephen Hemminger and the other guy to use upstream >> > kernel first! >> >> [Hemant] It is just a matter for choice for arm64 maintainers. You only have two choices >> 1. with the default arm64 support in DPDK, you allow it to be used with any kernel seamlessly. If they need any specific function, they can enable/disable it. >> 2. Or, you maintain specific support in the default config, and force all others to manually disable it/or, disable it via their specific configuration. >> >> I was not intending to change the default arm64 config in my original patch, I rather used it to disable it via NXP specific platform config. >> I still agree with Santosh that a new arm64 user should not be worry about searching & patching the kernel. >> >> >> In any case, who is going to take a call here? > > My take is to disable for arm64 to make sync with upstream kernel. > If Jianbo still feel it is better to keep for arm64 then lets disable it for > NXP and ThunderX and keep it enable for arm64 def config. > Thanks, Jerin. The reason why I want to keep base config untouched is that users often update the DPDK more frequently than kernel, and some of them are still using older kernel. For new users, we can encourage them to use vfio-pci. We will remove igb_uio on ARM finally, and hope this transition period is not long.