[ovs-dev] [PATCHv2 00/11] Backport upstream Linux patches and support 4.12 kernel

Greg Rose gvrose8192 at gmail.com
Mon Jul 24 15:31:07 UTC 2017


On 07/21/2017 04:46 PM, Joe Stringer wrote:
> This series takes Greg's recent proposed series, rolls in a few missed
> upstream commits, and orders them the same as the upstream net-next tree.
>
> Greg, would you mind taking a look at patches #1, #2, #4 and #7? Note that for
> some of the patches it wasn't possible to introduce a function in the compat
> that hides the differences between kernels; for example, when there are
> different fields in a structure, we can't hide differences in this way.
>
> I've added my sign-off to all of the other patches, which I have reviewed.

The patch series looks good to me.  I note that you went ahead and backported
patches for the current release candidate kernel.  I had held off on them (as noted
in my original patch series cover letter) because the kernel wasn't released yet and I
was unsure of policy on that.  As it stands I don't think any of the patches for
the unreleased kernel will actually be used yet unless someone is building against a
4.13-rc* kernel.  I suppose that can happen but even then they would have to update
acinclude.m4 to allow compile against a 4.13 kernel.

In any case, going forward I'll go ahead and backport patches even though the kernel
they are targeted for has not been released yet.

Thank you Joe!

Regards,

- Greg


>
> Greg Rose (7):
>    datapath: introduce nf_conntrack_helper_put function
>    datapath: Fix inconsistent teardown and release of private netdev
>      state.
>    compat: Implement upstream net device free change.
>    datapath: get rid of redundant vxlan_dev.flags
>    datapath: fix mis-ordered comment lines for ovs_skb_cb
>    acinclude.m4: Support Linux kernel 4.12
>    travis: Update test kernels
>
> Joe Stringer (3):
>    datapath: more accurate checksumming in queue_userspace_packet()
>    compat: convert many more places to skb_put_zero().
>    compat: net: store port/representator id in metadata_dst.
>
> Tonghao Zhang (1):
>    datapath: Avoid using stack larger than 1024.
>
>   .travis.yml                                        | 18 ++---
>   acinclude.m4                                       | 14 +++-
>   datapath/conntrack.c                               |  4 +-
>   datapath/datapath.c                                | 85 +++++++++++++++-------
>   datapath/datapath.h                                |  2 +-
>   datapath/flow_netlink.c                            |  4 +-
>   datapath/linux/Modules.mk                          |  1 +
>   datapath/linux/compat/include/linux/netdevice.h    | 15 ++++
>   datapath/linux/compat/include/linux/skbuff.h       | 12 +++
>   datapath/linux/compat/include/net/dst_metadata.h   | 15 ++++
>   .../include/net/netfilter/nf_conntrack_helper.h    | 12 +++
>   datapath/linux/compat/lisp.c                       |  4 +
>   datapath/linux/compat/stt.c                        |  4 +
>   datapath/vport-internal_dev.c                      |  7 ++
>   datapath/vport-vxlan.c                             |  8 ++
>   15 files changed, 166 insertions(+), 39 deletions(-)
>   create mode 100644 datapath/linux/compat/include/net/netfilter/nf_conntrack_helper.h
>



More information about the dev mailing list