mbox series

[ovs-dev,RFC,V2,0/2] DPDK v17.11 Support

Message ID 1510142410-169357-1-git-send-email-mark.b.kavanagh@intel.com
Headers show
Series DPDK v17.11 Support | expand

Message

Mark Kavanagh Nov. 8, 2017, noon UTC
This patchset presents a preliminary look at DPDK v17.11 support
for OvS-DPDK, and was generated against the following commits:
- OvS: 3220b8a ("vswitch.xml: Fix nonbreaking spaces.")
- DPDK: v17.11-rc3

I'm particularly interested in comments on patch 2, which adds support
for the vhost IOMMU feature.

Mark Kavanagh (2):
  netdev-dpdk: DPDK v17.11 upgrade
  netdev-dpdk: add support for vhost IOMMU feature

 .travis/linux-build.sh                   |  2 +-
 Documentation/faq/releases.rst           |  1 +
 Documentation/intro/install/dpdk.rst     | 10 ++++-----
 Documentation/topics/dpdk/ring.rst       |  2 +-
 Documentation/topics/dpdk/vhost-user.rst | 29 +++++++++++++++++++++----
 NEWS                                     |  3 +++
 lib/netdev-dpdk.c                        | 36 ++++++++++++++++++++++++++------
 vswitchd/vswitch.xml                     | 10 +++++++++
 8 files changed, 76 insertions(+), 17 deletions(-)

---
v2->v1:
    - add checks for vhost client reconfiguration.
    - in DOcumentation, mention the QEMU version from which IOMMU may be used.
1.9.3

Comments

Maxime Coquelin Nov. 8, 2017, 12:47 p.m. UTC | #1
Hi Mark,

On 11/08/2017 01:00 PM, Mark Kavanagh wrote:
> This patchset presents a preliminary look at DPDK v17.11 support
> for OvS-DPDK, and was generated against the following commits:
> - OvS: 3220b8a ("vswitch.xml: Fix nonbreaking spaces.")
> - DPDK: v17.11-rc3
> 
> I'm particularly interested in comments on patch 2, which adds support
> for the vhost IOMMU feature.
> 
> Mark Kavanagh (2):
>    netdev-dpdk: DPDK v17.11 upgrade
>    netdev-dpdk: add support for vhost IOMMU feature
> 
>   .travis/linux-build.sh                   |  2 +-
>   Documentation/faq/releases.rst           |  1 +
>   Documentation/intro/install/dpdk.rst     | 10 ++++-----
>   Documentation/topics/dpdk/ring.rst       |  2 +-
>   Documentation/topics/dpdk/vhost-user.rst | 29 +++++++++++++++++++++----
>   NEWS                                     |  3 +++
>   lib/netdev-dpdk.c                        | 36 ++++++++++++++++++++++++++------
>   vswitchd/vswitch.xml                     | 10 +++++++++
>   8 files changed, 76 insertions(+), 17 deletions(-)

For the series:
Acked-by: Maxime Coquelin <maxime.coquelin@redhat.com>

Thanks!
Maxime
> 
> ---
> v2->v1:
>      - add checks for vhost client reconfiguration.
>      - in DOcumentation, mention the QEMU version from which IOMMU may be used.
> 1.9.3
>
Mark Kavanagh Nov. 8, 2017, 1:24 p.m. UTC | #2
>From: Maxime Coquelin [mailto:maxime.coquelin@redhat.com]
>Sent: Wednesday, November 8, 2017 12:47 PM
>To: Kavanagh, Mark B <mark.b.kavanagh@intel.com>; dev@openvswitch.org
>Cc: Loftus, Ciara <ciara.loftus@intel.com>; ktraynor@redhat.com
>Subject: Re: [ovs-dev][RFC PATCH V2 0/2] DPDK v17.11 Support
>
>Hi Mark,
>
>On 11/08/2017 01:00 PM, Mark Kavanagh wrote:
>> This patchset presents a preliminary look at DPDK v17.11 support
>> for OvS-DPDK, and was generated against the following commits:
>> - OvS: 3220b8a ("vswitch.xml: Fix nonbreaking spaces.")
>> - DPDK: v17.11-rc3
>>
>> I'm particularly interested in comments on patch 2, which adds support
>> for the vhost IOMMU feature.
>>
>> Mark Kavanagh (2):
>>    netdev-dpdk: DPDK v17.11 upgrade
>>    netdev-dpdk: add support for vhost IOMMU feature
>>
>>   .travis/linux-build.sh                   |  2 +-
>>   Documentation/faq/releases.rst           |  1 +
>>   Documentation/intro/install/dpdk.rst     | 10 ++++-----
>>   Documentation/topics/dpdk/ring.rst       |  2 +-
>>   Documentation/topics/dpdk/vhost-user.rst | 29 +++++++++++++++++++++----
>>   NEWS                                     |  3 +++
>>   lib/netdev-dpdk.c                        | 36 ++++++++++++++++++++++++++--
>----
>>   vswitchd/vswitch.xml                     | 10 +++++++++
>>   8 files changed, 76 insertions(+), 17 deletions(-)
>
>For the series:
>Acked-by: Maxime Coquelin <maxime.coquelin@redhat.com>

Thanks for the quick review Maxime, and indeed for all of your work on accommodating older versions of QEMU with DPDK v17.11!
- Mark


>
>Thanks!
>Maxime
>>
>> ---
>> v2->v1:
>>      - add checks for vhost client reconfiguration.
>>      - in DOcumentation, mention the QEMU version from which IOMMU may be
>used.
>> 1.9.3
>>