[ovs-git] [openvswitch/ovs] 08288c: datapath-windows: move OVS_MESSAGE to Netlink.h

GitHub noreply at github.com
Mon Sep 29 16:04:09 UTC 2014


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 08288ce0bd23d5fe87cac47e777ce34d52e74efe
      https://github.com/openvswitch/ovs/commit/08288ce0bd23d5fe87cac47e777ce34d52e74efe
  Author: Ankur Sharma <ankursharma at vmware.com>
  Date:   2014-09-29 (Mon, 29 Sep 2014)

  Changed paths:
    M datapath-windows/ovsext/Datapath.h
    M datapath-windows/ovsext/Netlink/Netlink.h

  Log Message:
  -----------
  datapath-windows: move OVS_MESSAGE to Netlink.h

Moved the structure OVS_MESSAGE to Netlink.h.
This change is done for following reasons.

a. Patch 2 in this series provides a generic API in Netlink.c
for creating netlink message. That API needs OVS_MESSAGE.
Including Datapath.h in Netlink.c/h gives compilation error.

b. OVS_MESSAGE defines netlink messages hence moving it to
Netlink.h looked fine to me.

Signed-off-by: Ankur Sharma <ankursharma at vmware.com>
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Acked-by: Nithin Raju <nithin at vmware.com>
Acked-by: Samuel Ghinet <sghinet at cloudbasesolutions.com>
Tested-by: Ankur Sharma <ankursharma at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>


  Commit: d5f1e533c3f0cda6ef07fc8b01c0049c391212b6
      https://github.com/openvswitch/ovs/commit/d5f1e533c3f0cda6ef07fc8b01c0049c391212b6
  Author: Ankur Sharma <ankursharma at vmware.com>
  Date:   2014-09-29 (Mon, 29 Sep 2014)

  Changed paths:
    M datapath-windows/ovsext/Netlink/Netlink.c
    M datapath-windows/ovsext/Netlink/Netlink.h

  Log Message:
  -----------
  datapath-windows/Netlink: Add NlFillOvsMsg API for creating Netlink message headers.

Added NlFillOvsMsg API in Netlink.c This API will be used to populate
netlink message headers.

Signed-off-by: Ankur Sharma <ankursharma at vmware.com>
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Acked-by: Nithin Raju <nithin at vmware.com>
Acked-by: Samuel Ghinet <sghinet at cloudbasesolutions.com>
Tested-by: Ankur Sharma <ankursharma at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>


  Commit: dac95740efd7c0b434714773424362c244120f93
      https://github.com/openvswitch/ovs/commit/dac95740efd7c0b434714773424362c244120f93
  Author: Ankur Sharma <ankursharma at vmware.com>
  Date:   2014-09-29 (Mon, 29 Sep 2014)

  Changed paths:
    M datapath-windows/ovsext/Netlink/Netlink.c
    M datapath-windows/ovsext/Netlink/Netlink.h
    M datapath-windows/ovsext/Types.h

  Log Message:
  -----------
  datapath-windows/Netlink: Add NlAttrLen API

Added an API to retrieve the attribute length.
Added 2 more API for BE16 and BE8 attribute parsing.
Fixed a trailing whitespace issue.

Signed-off-by: Ankur Sharma <ankursharma at vmware.com>
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Acked-by: Nithin Raju <nithin at vmware.com>
Acked-by: Samuel Ghinet <sghinet at cloudbasesolutions.com>
Tested-by: Ankur Sharma <ankursharma at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>


  Commit: 91b95f8b930f34d344a60396e59f7e96e04c2888
      https://github.com/openvswitch/ovs/commit/91b95f8b930f34d344a60396e59f7e96e04c2888
  Author: Ankur Sharma <ankursharma at vmware.com>
  Date:   2014-09-29 (Mon, 29 Sep 2014)

  Changed paths:
    M datapath-windows/ovsext/Netlink/Netlink.c

  Log Message:
  -----------
  datapath-windows/Netlink: Allow support for NESTED Attributes in NlAttrValidate

Signed-off-by: Ankur Sharma <ankursharma at vmware.com>
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Acked-by: Nithin Raju <nithin at vmware.com>
Acked-by: Samuel Ghinet <sghinet at cloudbasesolutions.com>
Tested-by: Ankur Sharma <ankursharma at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>


  Commit: 5b2249547f91337ca1e5150ed9dbd2d9af58165b
      https://github.com/openvswitch/ovs/commit/5b2249547f91337ca1e5150ed9dbd2d9af58165b
  Author: Ankur Sharma <ankursharma at vmware.com>
  Date:   2014-09-29 (Mon, 29 Sep 2014)

  Changed paths:
    M datapath-windows/ovsext/Datapath.c
    M datapath-windows/ovsext/Netlink/Netlink.c
    M datapath-windows/ovsext/Netlink/Netlink.h

  Log Message:
  -----------
  datapath-windows/Netlink: Fixed NlAttrParseNested

NlAttrParseNested was using the whole netlink payload for iteration.
This is not correct, as it would lead to exceeding the
nested attribute boundries. Fixed the same in this patch.

Signed-off-by: Ankur Sharma <ankursharma at vmware.com>
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Acked-by: Nithin Raju <nithin at vmware.com>
Acked-by: Samuel Ghinet <sghinet at cloudbasesolutions.com>
Tested-by: Ankur Sharma <ankursharma at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>


  Commit: 0d9bd68b7b30a83d601cca2040702894a95b2b5a
      https://github.com/openvswitch/ovs/commit/0d9bd68b7b30a83d601cca2040702894a95b2b5a
  Author: Ankur Sharma <ankursharma at vmware.com>
  Date:   2014-09-29 (Mon, 29 Sep 2014)

  Changed paths:
    M datapath-windows/ovsext/Datapath.c
    M datapath-windows/ovsext/Flow.c
    M datapath-windows/ovsext/Flow.h

  Log Message:
  -----------
  datapath-windows/Flow.c : Basic support for add-flow.

This patch covers basic changes in registering add flow handler.
And declaring FLOW related attribute parsing policies.

Signed-off-by: Ankur Sharma <ankursharma at vmware.com>
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Acked-by: Nithin Raju <nithin at vmware.com>
Acked-by: Samuel Ghinet <sghinet at cloudbasesolutions.com>
Tested-by: Ankur Sharma <ankursharma at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>


  Commit: 8a79c2d43150e9ae3fd88370a64458a9c423cf91
      https://github.com/openvswitch/ovs/commit/8a79c2d43150e9ae3fd88370a64458a9c423cf91
  Author: Ankur Sharma <ankursharma at vmware.com>
  Date:   2014-09-29 (Mon, 29 Sep 2014)

  Changed paths:
    M datapath-windows/include/OvsPub.h
    M datapath-windows/ovsext/Datapath.c
    M datapath-windows/ovsext/Flow.c
    M datapath-windows/ovsext/Flow.h

  Log Message:
  -----------
  datapath-windows/Flow.c: FLOW_NEW command handler.

This patch covers the changes needed to support FLOW_NEW command.
API _OvsFlowMapNlToFlowPutFlags has a bug, which will be fixed
with the patches for FLOW_DEL.

Signed-off-by: Ankur Sharma <ankursharma at vmware.com>
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Acked-by: Nithin Raju <nithin at vmware.com>
Acked-by: Samuel Ghinet <sghinet at cloudbasesolutions.com>
Tested-by: Ankur Sharma <ankursharma at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>


  Commit: 22b6623ad4c9ebb61fad852721d7e9ffbd6519d9
      https://github.com/openvswitch/ovs/commit/22b6623ad4c9ebb61fad852721d7e9ffbd6519d9
  Author: Ankur Sharma <ankursharma at vmware.com>
  Date:   2014-09-29 (Mon, 29 Sep 2014)

  Changed paths:
    M datapath-windows/ovsext/Datapath.c

  Log Message:
  -----------
  datapath-windows/Flow.c: FLOW_SET command handler.

Registered FLOW_SET command handler. The same command
handler as FLOW_ADD is good enough to handle FLOW_SET
case as well.

Signed-off-by: Ankur Sharma <ankursharma at vmware.com>
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Acked-by: Nithin Raju <nithin at vmware.com>
Acked-by: Samuel Ghinet <sghinet at cloudbasesolutions.com>
Tested-by: Ankur Sharma <ankursharma at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>


  Commit: 6247bec1331d678bb843827a3851a1d914a766c8
      https://github.com/openvswitch/ovs/commit/6247bec1331d678bb843827a3851a1d914a766c8
  Author: Ankur Sharma <ankursharma at vmware.com>
  Date:   2014-09-29 (Mon, 29 Sep 2014)

  Changed paths:
    M datapath-windows/ovsext/Datapath.c
    M datapath-windows/ovsext/Flow.c

  Log Message:
  -----------
  datapath-windows/Flow.c: FLOW_DEL command handler.

Registered FLOW_DEL command handler. The same command
handler as FLOW_ADD is good enough to handle FLOW_DEL
case as well with minor changes for checking to action
attribute.

Signed-off-by: Ankur Sharma <ankursharma at vmware.com>
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Acked-by: Nithin Raju <nithin at vmware.com>
Acked-by: Samuel Ghinet <sghinet at cloudbasesolutions.com>
Tested-by: Ankur Sharma <ankursharma at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>


  Commit: 3d72793fb2ac1a22c2c3bb98cd995b9e812a27c6
      https://github.com/openvswitch/ovs/commit/3d72793fb2ac1a22c2c3bb98cd995b9e812a27c6
  Author: Ankur Sharma <ankursharma at vmware.com>
  Date:   2014-09-29 (Mon, 29 Sep 2014)

  Changed paths:
    M datapath-windows/ovsext/Flow.c
    M datapath-windows/ovsext/Flow.h

  Log Message:
  -----------
  datapath-windows/Flow.c: DEL_FLOWS command handler.

Added changes to handle DEL_FLOWS (FLUSH) scenario.

Signed-off-by: Ankur Sharma <ankursharma at vmware.com>
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Acked-by: Nithin Raju <nithin at vmware.com>
Acked-by: Samuel Ghinet <sghinet at cloudbasesolutions.com>
Tested-by: Ankur Sharma <ankursharma at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>


  Commit: 0679122d0680c1b2bb8d9b674d91d38e34fb290c
      https://github.com/openvswitch/ovs/commit/0679122d0680c1b2bb8d9b674d91d38e34fb290c
  Author: Ankur Sharma <ankursharma at vmware.com>
  Date:   2014-09-29 (Mon, 29 Sep 2014)

  Changed paths:
    M datapath-windows/ovsext/Datapath.c
    M datapath-windows/ovsext/Flow.c
    M datapath-windows/ovsext/Flow.h

  Log Message:
  -----------
  datapath-windows: Rename OvsFlowNlNewCmdHandler to OvsFlowNlCmdHandler

OvsFlowNlNewCmdHandler NEW/DEL/SET/FLUSH commands hence renaming it to
OvsFlowNlCmdHandler.

Signed-off-by: Ankur Sharma <ankursharma at vmware.com>
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Acked-by: Nithin Raju <nithin at vmware.com>
Acked-by: Samuel Ghinet <sghinet at cloudbasesolutions.com>
Tested-by: Ankur Sharma <ankursharma at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>


Compare: https://github.com/openvswitch/ovs/compare/4464b71eb45c...0679122d0680


More information about the git mailing list