[ovs-dev] [PATCHv5 00/12] Revalidate flows with unique identifiers.

[Joe Stringer]

Bump. Any chance of some feedback?

On 15 September 2014 14:25, Joe Stringer <joestringer at nicira.com> wrote:

> Patches #1-4 have been reviewed and Ack'd by Ben, which I have held out
> from
> pushing upstream as they are only useful with the rest of the series. The
> remaining patches are yet to be reviewed.
>
> This series is also made available here to assist review:
> https://github.com/joestringer/openvswitch/tree/submit/ufid_v5
>
> v5:
> - Rebase
> - Various bugfixes
> - Improve logging
>
> v4:
> - Datapath memory leak fixes
> - Enable UID-based terse dumping and deleting by default
> - Shifted UID generation down to dpif
> - Log flow UIDs in more places
> - Various fixes
>
> RFCv3:
> - Add datapath implementation
> - Minor fixes
> - Rebased
>
> RFCv2:
> - Revised early patches from v1 feedback
> - Add Acks from Ben
> - Rebased
>
> RFCv1:
> http://openvswitch.org/pipermail/dev/2014-August/044383.html
>
> Joe Stringer (12):
>   revalidator: Use 'cmap' for storing ukeys.
>   revalidator: Protect ukeys with a mutex.
>   udpif: Separate udpif_key maps from revalidators.
>   upcall: Rename dump_op -> ukey_op.
>   upcall: Create ukeys in handler threads.
>   upcall: Revalidate using cache of mask, actions.
>   hash: Add 128-bit murmurhash.
>   dpif: Add Unique flow identifiers.
>   datapath: Add support for OVS_FLOW_ATTR_UID.
>   revalidator: Reduce ukey contention.
>   revalidator: Distinguish new and duplicate flows.
>   upcall: Distinguish duplicate upcalls from conflicts.
>
>  datapath/datapath.c                               |  215 ++++--
>  datapath/flow.h                                   |   12 +-
>  datapath/flow_netlink.c                           |   60 ++
>  datapath/flow_netlink.h                           |    2 +
>  datapath/flow_table.c                             |  229 +++++-
>  datapath/flow_table.h                             |    5 +-
>  datapath/linux/compat/include/linux/openvswitch.h |   29 +
>  include/openvswitch/types.h                       |    5 +
>  lib/dpctl.c                                       |   14 +-
>  lib/dpif-linux.c                                  |  133 ++--
>  lib/dpif-netdev.c                                 |  144 ++--
>  lib/dpif-provider.h                               |    9 +-
>  lib/dpif.c                                        |   47 +-
>  lib/dpif.h                                        |   31 +-
>  lib/flow.h                                        |    6 +
>  lib/hash.c                                        |  194 ++++-
>  lib/hash.h                                        |    4 +-
>  lib/odp-util.c                                    |   55 ++
>  lib/odp-util.h                                    |   25 +
>  ofproto/ofproto-dpif-upcall.c                     |  813
> ++++++++++++++-------
>  ofproto/ofproto-dpif-upcall.h                     |    1 +
>  ofproto/ofproto-dpif.c                            |   93 ++-
>  ofproto/ofproto-dpif.h                            |    2 +
>  tests/dpif-netdev.at                              |    5 +
>  tests/ofproto-dpif.at                             |   28 +-
>  tests/ofproto-macros.at                           |    1 +
>  26 files changed, 1632 insertions(+), 530 deletions(-)
>
> --
> 1.7.10.4
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>