> -----Original Message-----
> From: Ilya Maximets <[email protected]>
> Sent: Saturday, November 30, 2019 2:07 PM
> To: Ophir Munk <[email protected]>; Ilya Maximets
> <[email protected]>; [email protected]
> Cc: Roni Bar Yanai <[email protected]>; Eli Britstein
> <[email protected]>; Ian Stokes <[email protected]>; Ameer
> Mahagneh <[email protected]>
> Subject: Re: [PATCH] dpif-netdev: Use netdev-offload API for port lookup
> while offloading.
>
> On 29.11.2019 15:46, Ophir Munk wrote:
> > Hi Ilya,
> > The first two RFC series are confirmed. They require the third RFC series to
> enable user space vport offloading.
> >
> > 1.
> >
> https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc
> >
> hwork.ozlabs.org%2Fproject%2Fopenvswitch%2Flist%2F%3Fseries%3D14320
> 9&a
> >
> mp;data=02%7C01%7Cophirmu%40mellanox.com%7C0d4c9b5abcef4d0796b5
> 08d7758
> >
> ddfc7%7Ca652971c7d2e4d9ba6a4d149256f461b%7C0%7C0%7C6371071245400
> 95619&
> >
> amp;sdata=OyBtcrlldUxUPZTNy6V3zArvjas1cIAA6EzBjkcXJsY%3D&reser
> ved=
> > 0
> > ("netdev-offload: Prerequisites of vport offloading via DPDK")
> >
> > 2.
> >
> https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc
> >
> hwork.ozlabs.org%2Fproject%2Fopenvswitch%2Flist%2F%3Fseries%3D14455
> 9&a
> >
> mp;data=02%7C01%7Cophirmu%40mellanox.com%7C0d4c9b5abcef4d0796b5
> 08d7758
> >
> ddfc7%7Ca652971c7d2e4d9ba6a4d149256f461b%7C0%7C0%7C6371071245400
> 95619&
> >
> amp;sdata=PBp1gVHPJoRWhTVDGzLDsi4EPRcSlbQRZeVS9K99APE%3D&
> reserved=
> > 0
> > ("dpif-netdev: Use netdev-offload API for port lookup while
> > offloading.")
> >
> > 3.
> >
> https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc
> >
> hwork.ozlabs.org%2Fproject%2Fopenvswitch%2Flist%2F%3Fseries%3D14482
> 6&a
> >
> mp;data=02%7C01%7Cophirmu%40mellanox.com%7C0d4c9b5abcef4d0796b5
> 08d7758
> >
> ddfc7%7Ca652971c7d2e4d9ba6a4d149256f461b%7C0%7C0%7C6371071245400
> 95619&
> >
> amp;sdata=OoZ0OfQunKaPqjKzOhIScIfAJwR%2B1g36qZcQITvAX60%3D&
> ;reserve
> > d=0 ("[ovs-dev,RFC,v1] netdev-dpdk: Add flow_api support for netdev
> > vports")
> >
> > Can you please merge all the three series into one patch set?
> > Then we will be able to test it and send our feedback.
>
> Hi.
>
> This patch (series #2) is more like a bug fix that doesn't relate to vport
> offloading. I'd like it to be reviewed/ACKed as a standalone fix, so I could
> push it directly to master. After that I could rebase series #1.
Patch (series #2) tested successfully.
Acked-by: Ophir Munk <[email protected]>
Looking forward to the rebase of series #1
>
> I don't see the reason for changes in series #3, the doesn't make sense for
> me without looking at your vport offloading code, so I'd suggest to make it a
> part of your vport offlloading patch-set. However, I'm suspecting that you're
> going to use this similarly to what happens in there:
> https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fmail.
> openvswitch.org%2Fpipermail%2Fovs-dev%2F2019-
> November%2F364931.html&data=02%7C01%7Cophirmu%40mellanox.c
> om%7C0d4c9b5abcef4d0796b508d7758ddfc7%7Ca652971c7d2e4d9ba6a4d149
> 256f461b%7C0%7C0%7C637107124540095619&sdata=atkVKubOzY0SeeZ
> Vm0inL9Tsv1UVu4pWC1k1J%2FoX1Es%3D&reserved=0
> So, you may take the same suggestion into account:
> https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fmail.
> openvswitch.org%2Fpipermail%2Fovs-dev%2F2019-
> November%2F365294.html&data=02%7C01%7Cophirmu%40mellanox.c
> om%7C0d4c9b5abcef4d0796b508d7758ddfc7%7Ca652971c7d2e4d9ba6a4d149
> 256f461b%7C0%7C0%7C637107124540095619&sdata=vsVJlVPcxzoRTWA
> OSCNudKzAjzI9cRbmXeHyMGrxDC8%3D&reserved=0
>
> Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
