On 28.11.2019 14:42, Kevin Traynor wrote: > On 27/11/2019 20:33, David Marchand wrote: >> On Thu, Nov 21, 2019 at 3:35 PM Ilya Maximets <i.maxim...@ovn.org> wrote: >>> >>> On 21.11.2019 14:58, Ilya Maximets wrote: >>>> 'ol_flags' of DPDK mbuf could contain bits responsible for external >>>> or indirect buffers which are not actually offload flags in a common >>>> sense. Clearing/copying of these flags could lead to memory leaks of >>>> external memory chunks and crashes due to access to wrong memory. >>>> >>>> OVS should not clear these flags while resetting offloads and also >>>> should not copy them to the newly allocated packets. >>>> >>>> This change is required to support DPDK 19.11, as some drivers may >>>> return mbufs with these flags set. However, it might be good to do >>>> the same for DPDK 18.11, because these flags are present and should >>>> be taken into account. >>>> >>>> Signed-off-by: Ilya Maximets <i.maxim...@ovn.org> >> >> As discussed during the OVS-DPDK public meeting earlier, I am ok with >> your suggestion of going with this patch as a first step. >> We will have to keep an eye on new flags on the dpdk side. >> >> Then once the dpdk ring ports have been dropped, we can revisit this. >> >> Reviewed-by: David Marchand <david.march...@redhat.com> >> > > +1 > > Acked-by: Kevin Traynor <ktray...@redhat.com>
Thanks! Applied to master and branch-2.12. OVS 2.11 uses DPDK 18.11 too, but it seems that we're not clearing flags on that branch. Best regards, Ilya Maximets. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev