[ovs-git] [openvswitch/ovs] 5fc8ea: datapath-windows: refactor BuildReplyMsgFromMsgIn ...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 5fc8ea2d6f0e604cad53e48164d64f0edcf43204
      https://github.com/openvswitch/ovs/commit/5fc8ea2d6f0e604cad53e48164d64f0edcf43204
  Author: Nithin Raju <nithin at vmware.com>
  Date:   2014-12-11 (Thu, 11 Dec 2014)

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

  Log Message:
  -----------
  datapath-windows: refactor BuildReplyMsgFromMsgIn & BuildErrorMsg

In this patch, we consolidate code in Netlink.c.

Signed-off-by: Nithin Raju <nithin at vmware.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>


  Commit: 1ad44ad455e4f2f96eb98d8b3faf06f3e358c214
      https://github.com/openvswitch/ovs/commit/1ad44ad455e4f2f96eb98d8b3faf06f3e358c214
  Author: Nithin Raju <nithin at vmware.com>
  Date:   2014-12-11 (Thu, 11 Dec 2014)

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

  Log Message:
  -----------
  datapath-windows: return bool from NlFillOvs[Msg/Hdr]

Per review comment, in this patch, we update the return values of
NlFillOvsMsg() and NlFillOvsHdr() from NTSTATUS to BOOLEAN to make them
consistent with the Nl* functions.

Signed-off-by: Nithin Raju <nithin at vmware.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>


Compare: https://github.com/openvswitch/ovs/compare/9ab0fce17987...1ad44ad455e4