[Starlingx-discuss] Patch to support different address witdhs in DPDK
Hi all, This bug[1] was failing (simplex in the baremetal case) because the /etc/init.d/compute_config was unable to configure the compute causing a "degraded" state and nova-compute to be disabled. While running the compute_config script a lot of kernel messages like this appeared: DMAR: intel_iommu_map: iommu width (39) is not sufficient for the mapped address (7fbdc0000000) It turns out that this is a known issue in DPDK[2] and a patch is available[3]. I applied this patch into the openvswitch package and then we were able to solve the problem. My question is, what would be the best way to handle this patch? - Should we patch our openvswitch package? : we get openvswitch from centos and that version doesn't include this patch. We can apply this patch here and wait for an upgrade in upstream to remove it. - Should be look for a newer openvswitch version? We could look for a newer version but this one won't be a centos standard version. What do you think? Thanks in advance -Erich [1] https://bugs.launchpad.net/starlingx/+bug/1796420 [2] http://mails.dpdk.org/archives/stable/2018-January/004109.html [3] https://github.com/DPDK/dpdk/commit/54a328f552ff2e0098c3f96f9e32302 675f2bcf4
On Wed, Oct 10, 2018 at 3:18 PM Cordoba Malibran, Erich <erich.cordoba.malibran@intel.com> wrote:
- Should be look for a newer openvswitch version? We could look for a newer version but this one won't be a centos standard version.
We already keep more current versions of libvirt and qemu than are supplied with CentOS because there are certain advantages, including performance. openvswitch may fall in to that category too, particularly with the reliance we will have on DPDK. I'm not sure what the criteria should be for making that kind of decision, although I do expect we should have a high bar to meet to do that.. I would like to hear the thoughts from those who know these bits well. The last time I build an openvswitch package was at the Bexar summit... dt -- Dean Troyer dtroyer@gmail.com
The networking team should weigh in here with the recommended path forward. Brent -----Original Message----- From: Dean Troyer [mailto:dtroyer@gmail.com] Sent: Wednesday, October 10, 2018 7:40 PM To: Cordoba Malibran, Erich <erich.cordoba.malibran@intel.com> Cc: starlingx-discuss@lists.starlingx.io Subject: Re: [Starlingx-discuss] Patch to support different address witdhs in DPDK On Wed, Oct 10, 2018 at 3:18 PM Cordoba Malibran, Erich <erich.cordoba.malibran@intel.com> wrote:
- Should be look for a newer openvswitch version? We could look for a newer version but this one won't be a centos standard version.
We already keep more current versions of libvirt and qemu than are supplied with CentOS because there are certain advantages, including performance. openvswitch may fall in to that category too, particularly with the reliance we will have on DPDK. I'm not sure what the criteria should be for making that kind of decision, although I do expect we should have a high bar to meet to do that.. I would like to hear the thoughts from those who know these bits well. The last time I build an openvswitch package was at the Bexar summit... dt -- Dean Troyer dtroyer@gmail.com _______________________________________________ Starlingx-discuss mailing list Starlingx-discuss@lists.starlingx.io http://lists.starlingx.io/cgi-bin/mailman/listinfo/starlingx-discuss
The patch is not for openvswitch itself and is only for the bundled DPDK distribution that is included in the CentOS specific SRPM (currently based on 17.11.0 - old snapshot). Currently CentOS is not a distributing a newer version of the openvswitch package that includes a newer version of DPDK. I don't think we want to up version the DPDK distribution in this package within StarlingX since this would deviate from a supported lineup of Centos. However, that being said, the recommended version from OVS is DPDK 17.11.3. I think the short term solution is we should include this patch in StarlingX to address the immediate issue. Longer term, we would want to push to get the CentOS openvswitch SRPM package updated with a newer version of DPDK 17.11 (currently at 17.11.4) to have all available bug fixes in the dpdk-stable branch. Regards, Matt
-----Original Message----- From: Rowsell, Brent [mailto:Brent.Rowsell@windriver.com] Sent: Wednesday, October 10, 2018 8:49 PM To: Dean Troyer; Cordoba Malibran, Erich Cc: starlingx-discuss@lists.starlingx.io Subject: Re: [Starlingx-discuss] Patch to support different address witdhs in DPDK
The networking team should weigh in here with the recommended path forward.
Brent
-----Original Message----- From: Dean Troyer [mailto:dtroyer@gmail.com] Sent: Wednesday, October 10, 2018 7:40 PM To: Cordoba Malibran, Erich <erich.cordoba.malibran@intel.com> Cc: starlingx-discuss@lists.starlingx.io Subject: Re: [Starlingx-discuss] Patch to support different address witdhs in DPDK
On Wed, Oct 10, 2018 at 3:18 PM Cordoba Malibran, Erich <erich.cordoba.malibran@intel.com> wrote:
- Should be look for a newer openvswitch version? We could look for a newer version but this one won't be a centos standard version.
We already keep more current versions of libvirt and qemu than are supplied with CentOS because there are certain advantages, including performance. openvswitch may fall in to that category too, particularly with the reliance we will have on DPDK. I'm not sure what the criteria should be for making that kind of decision, although I do expect we should have a high bar to meet to do that..
I would like to hear the thoughts from those who know these bits well. The last time I build an openvswitch package was at the Bexar summit...
dt
--
Dean Troyer dtroyer@gmail.com
_______________________________________________ Starlingx-discuss mailing list Starlingx-discuss@lists.starlingx.io http://lists.starlingx.io/cgi-bin/mailman/listinfo/starlingx-discuss _______________________________________________ Starlingx-discuss mailing list Starlingx-discuss@lists.starlingx.io http://lists.starlingx.io/cgi-bin/mailman/listinfo/starlingx-discuss
participants (4)
-
Cordoba Malibran, Erich
-
Dean Troyer
-
Peters, Matt
-
Rowsell, Brent