Adding Ian and others
> -----Original Message----- > From: [email protected] <ovs-dev- > [email protected]> On Behalf Of Ophir Munk > Sent: Sunday, March 17, 2019 6:48 PM > To: Ilya Maximets <[email protected]>; [email protected] > Cc: Shahaf Shuler <[email protected]>; Roni Bar Yanai > <[email protected]> > Subject: [Suspected-Phishing]Re: [ovs-dev] [PATCH v6 0/3] Move offloading > code into a new file > > Hi Ian, > Can you please update on the status of this series? > > Regards, > Ophir > > > -----Original Message----- > > From: Ophir Munk > > Sent: Wednesday, March 6, 2019 5:28 PM > > To: Ilya Maximets <[email protected]>; ovs- > [email protected] > > Cc: Asaf Penso <[email protected]>; Ian Stokes > > <[email protected]>; Shahaf Shuler <[email protected]>; Olga > > Shern <[email protected]>; Kevin Traynor <[email protected]>; > Roni > > Bar Yanai <[email protected]> > > Subject: RE: [PATCH v6 0/3] Move offloading code into a new file > > > > > > > > > -----Original Message----- > > > From: Ilya Maximets <[email protected]> > > > Sent: Wednesday, March 6, 2019 5:18 PM > > > To: Ophir Munk <[email protected]>; [email protected] > > > Cc: Asaf Penso <[email protected]>; Ian Stokes > > > <[email protected]>; Shahaf Shuler <[email protected]>; Olga > > > Shern <[email protected]>; Kevin Traynor <[email protected]>; > > Roni > > > Bar Yanai <[email protected]> > > > Subject: Re: [PATCH v6 0/3] Move offloading code into a new file > > > > > > On 05.03.2019 19:49, Ophir Munk wrote: > > > > Hardware offloading code is moved to a new file called > > > > netdev-rte-offloads.c. The original offloading code is copied as > > > > is from the netdev-dpdk.c file to the new file where future > > > > offloading code should be added as well. > > > > The new code is refactored based on code styling. > > > > > > > > This series is essential for offloading code development for the > > > > following > > > > reasons: > > > > 1. This series does not change the existing OVS code flows/logic > > > > on master > > > branch. > > > > OVS functionality is the same before and after this series. > > > > 2. The separation is essential for new offloading code development > > > > without interfering with the rest of OVS development. > > > > 3. Vice versa: it is essential that while developing offloading > > > > code > > > > - to be able to frequently rebase on top of master branch. > > > > 4. The OVS-kernel is practicing the same approach. Please note the > > > > file > > > lib/netdev-tc-offloads.c. > > > > > > > > Based on the points mentioned above we kindly ask that the series > > > > will be applied on top of the master branch. > > > > > > > > Ophir Munk (1): > > > > netdev-rte-offloads: Rename netdev_dpdk_* functions > > > > > > > > Roni Bar Yanai (2): > > > > netdev-dpdk: Expose flow creation/destruction calls > > > > netdev-dpdk: Move offloading code to a new file > > > > > > > > lib/automake.mk | 4 +- > > > > lib/netdev-dpdk.c | 708 > > > > ++------------------------------------------ > > > > lib/netdev-dpdk.h | 17 ++ > > > > lib/netdev-rte-offloads.c | 733 > > > > ++++++++++++++++++++++++++++++++++++++++++++++ > > > > lib/netdev-rte-offloads.h | 40 +++ > > > > 5 files changed, 810 insertions(+), 692 deletions(-) create mode > > > > 100644 lib/netdev-rte-offloads.c create mode 100644 > > > > lib/netdev-rte-offloads.h > > > > > > I had no chance to check this in runtime, but the code looks OK. > > > > > > Acked-by: Ilya Maximets <[email protected]> > > > > > > BTW, Ophir, you probably need to add yourself as a co-author to the > > > first patch. > > > > > > > Ilya- thank you for your reviews. > > Ian - once this series is accepted is it possible that you add me as > > co-author on the first patch? > > > > > > > > > > v1: > > > > Initial release > > > > v2: > > > > Rebase on top of master branch + remove extra spaces, see > > > > > > > > > > https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpat > > > > > > > > > > chwork.ozlabs.org%2Fpatch%2F1044187%2F&data=02%7C01%7Cophirm > > > u%40me > > > > > > > > > > llanox.com%7C1a231322175b4168a92808d6a246f7aa%7Ca652971c7d2e4d9ba > > > 6a4d1 > > > > > > > > > > 49256f461b%7C0%7C0%7C636874823029025522&sdata=faXjfChEFsrS2Klr > > > DQ5F > > > > iEAWT27tavnM7huF1sW%2BZ8w%3D&reserved=0 > > > > v3: > > > > Rebase on top of master branch > > > > Code rafactored based on v2 reviews and coding style > > > > v4: > > > > Rebase on top of master branch > > > > Updates based on v3 reviews > > > > v5: > > > > Updates based on v4 reviews > > > > v6: > > > > Rebase on top of master branch > > > > Remove multiple consecutive blank lines > > > > > _______________________________________________ > dev mailing list > [email protected] > https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fmail. > openvswitch.org%2Fmailman%2Flistinfo%2Fovs- > dev&data=02%7C01%7Cophirmu%40mellanox.com%7Cf7b5df98aa4641 > 588b2a08d6aaf86299%7Ca652971c7d2e4d9ba6a4d149256f461b%7C0%7C0%7 > C636884381119310573&sdata=JQOpjOXMMvdSDemuLXPkunVp5%2B%2 > BGlI74qkBLXwIy4xM%3D&reserved=0 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
