From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id B99E21023 for ; Tue, 24 Jan 2017 14:35:27 +0100 (CET) Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga101.fm.intel.com with ESMTP; 24 Jan 2017 05:35:26 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,278,1477983600"; d="scan'208";a="926055846" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.38]) ([10.237.220.38]) by orsmga003.jf.intel.com with ESMTP; 24 Jan 2017 05:35:24 -0800 To: Jianfeng Tan , dev@dpdk.org References: <1485243248-27082-1-git-send-email-jianfeng.tan@intel.com> <1485243248-27082-4-git-send-email-jianfeng.tan@intel.com> Cc: john.mcnamara@intel.com, yuanhan.liu@linux.intel.com, stephen@networkplumber.org, Thomas Monjalon From: Ferruh Yigit Message-ID: <66ee7e72-95a3-1e31-278c-b6823168de6c@intel.com> Date: Tue, 24 Jan 2017 13:35:24 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0 MIME-Version: 1.0 In-Reply-To: <1485243248-27082-4-git-send-email-jianfeng.tan@intel.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH 3/3] doc: remove ABI changes in igb_uio 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: Tue, 24 Jan 2017 13:35:28 -0000 On 1/24/2017 7:34 AM, Jianfeng Tan wrote: > We announced ABI changes to remove iomem and ioport mapping in > igb_uio. But it has potential backward compatibility issue: cannot > run old version DPDK on modified igb_uio. > > The purpose of this changes was to fix a bug: when DPDK app crashes, > those devices by igb_uio are not stopped either DPDK PMD driver or > igb_uio driver. We need to figure out new way to fix this bug. Hi Jianfeng, I believe it would be good to fix this potential defect. Is "remove iomem and ioport" a must for that fix? If so, I suggest re-think about it. If I see correctly, dpdk1.8 and older uses igb_uio iomem files. So backward compatibility is the possible issue for dpdk1.8 and older. Since v1.8 two years old, I would prefer fixing defect instead of keeping that backward compatibility. Jianfeng, Thomas, What do you think postponing this deprecation notice to next release, instead of removing it, and discuss more? And overall, if "remove iomem and ioport" is not a must for this fix, no problem to remove deprecation notice. Thanks, ferruh > > Fixes: 3bac1dbc1ed ("doc: announce iomem and ioport removal from igb_uio") > > Suggested-by: Stephen Hemminger > Signed-off-by: Jianfeng Tan > --- > doc/guides/rel_notes/deprecation.rst | 5 ----- > 1 file changed, 5 deletions(-) > > diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst > index 755dc65..0f039dd 100644 > --- a/doc/guides/rel_notes/deprecation.rst > +++ b/doc/guides/rel_notes/deprecation.rst > @@ -8,11 +8,6 @@ API and ABI deprecation notices are to be posted here. > Deprecation Notices > ------------------- > > -* igb_uio: iomem mapping and sysfs files created for iomem and ioport in > - igb_uio will be removed, because we are able to detect these from what Linux > - has exposed, like the way we have done with uio-pci-generic. This change > - targets release 17.02. > - > * ABI/API changes are planned for 17.02: ``rte_device``, ``rte_driver`` will be > impacted because of introduction of a new ``rte_bus`` hierarchy. This would > also impact the way devices are identified by EAL. A bus-device-driver model >