Nithin Raju | 20 Aug 21:03 2014

OVS on HyperV: meeting minutes for 8/20 IRC meeting

hi,
Just sending out the meeting minutes for future reference.

Attendees: Alin Serdean, Samuel Ghinet, Ankur Sharma, Saurabh Shah, Nithin Raju

1. Review http://openvswitch.org/pipermail/dev/2014-August/044315.html
----------------------------------------------------------------------
- VMware folks committed to reviewing this. Thanks cloudbase folks for working on this criticial patch. It
was clarified that, VIF <-> VIF testing via VXLAN was tested, and we did see NBLs with multiple NBs each.
VLAN based ping, which is much simpler, was not testing (and it is OK). This should be tested before checkin though.

2. Netlink attributes
---------------------
- Discussion was around adding more validation code in the kernel netlink code. Everybody agreed that
we'll add more validation code as we go.
- Cloudbase folks said they'll point out any validation code missing in the basic patches committed so far.

3. Status of netlink implementation: Status of userspace and kernel patches, Events and Packet receive infrastructure.
---------------------------------------------------------------------------------------------------
- #2 continued into #3.
- Ankur has posted a patch for the netlink parsing code. With this, the infrastructure pieces should be done
(big ones at least).
- It was decided that once the infrastructure pieces are done, we'll fan out the work among multiple
developers in VMware and Cloudbase.
- #Action-item: Nithin to create an issue for this, and fan out the commands to developers.
- Ankur's netlink parsing patch to be reviewed ASAP and enable the commands to be done.
- The advantages of ReadFile() v/s OVS_READ_IOCTL were discussed and it was concluded that, there's no
immediate benefit of using ReadFile(). Same goes for WriteFile().
- Nithin clarified the recv_set(), recv_wait(), and recv() workflow and how it maps to overlapped events.
Alin and Sam were satisfied with the explanation, and decided to go ahead with this model.
(Continue reading)

Cathy Collins | 20 Aug 19:52 2014

VMware install base accounts

Hi,

Would you be interested in acquiring leads from "VMware install base
accounts"?

We are pioneers in providing verified Technology user lists. We also have
the VMware products install accounts users - VMware vSphere, VMware vCenter,
VMware VCloud etc.

You can choose from the data fields like SIC, Industry, Geography and Title
etc. 

Let me know if you're interested in reviewing a sample file.

Best Regards,

Cathy Collins

Cathy Collins| Business Coordinator

E-Data List

If you do not wish to receive, any more email please reply to this email
with "Leave out" in subject line

DISCLAIMER:

  _____  

The information contained in this electronic message may be legally
(Continue reading)

Cathy Collins | 20 Aug 19:11 2014

VMware install base accounts

Hi,

Would you be interested in acquiring leads from "VMware install base
accounts"?

We are pioneers in providing verified Technology user lists. We also have
the VMware products install accounts users - VMware vSphere, VMware vCenter,
VMware VCloud etc.

You can choose from the data fields like SIC, Industry, Geography and Title
etc. 

Let me know if you're interested in reviewing a sample file.

Best Regards,

Cathy Collins

Cathy Collins| Business Coordinator

E-Data List

If you do not wish to receive, any more email please reply to this email
with "Leave out" in subject line

DISCLAIMER:

  _____  

The information contained in this electronic message may be legally
(Continue reading)

Ankur Sharma | 20 Aug 18:46 2014

[PATCH 3/3] datapath-windows: remove reference to OvsNetlink.h

In this patch we remove reference to OvsNetlink.h.
Since we do not refer to lib/netlink-protocol.h anymore,
hence removed the WIN_DP based check.

Change-Id: I281a0c6478e3de2e9b04c988bea57b06822a504e
---
 datapath-windows/automake.mk          |   5 +-
 datapath-windows/include/OvsNetlink.h | 174 ----------------------------------
 lib/netlink-protocol.h                |   6 --
 3 files changed, 2 insertions(+), 183 deletions(-)
 delete mode 100644 datapath-windows/include/OvsNetlink.h

diff --git a/datapath-windows/automake.mk b/datapath-windows/automake.mk
index 45348b7..73bc58e 100644
--- a/datapath-windows/automake.mk
+++ b/datapath-windows/automake.mk
 <at>  <at>  -4,10 +4,9  <at>  <at>  EXTRA_DIST += \
 	datapath-windows/Package/package.VcxProj \
 	datapath-windows/Package/package.VcxProj.user \
 	datapath-windows/include/OvsDpInterfaceExt.h \
-	datapath-windows/include/OvsNetlink.h \
-	datapath-windows/include/OvsPub.h
-    datapath-windows/ovsext/Netlink.c \
     datapath-windows/ovsext/Netlink.h \
+    datapath-windows/ovsext/Netlink.c \
+	datapath-windows/include/OvsPub.h
     datapath-windows/include/NetlinkProto.h \
 	datapath-windows/misc/install.cmd \
 	datapath-windows/misc/uninstall.cmd \
diff --git a/datapath-windows/include/OvsNetlink.h b/datapath-windows/include/OvsNetlink.h
(Continue reading)

Ankur Sharma | 20 Aug 18:46 2014

[PATCH 2/3] datapath-windows: Using windows kernel netlink parsing APIs.

In this patch we incorporate the usage of netlink message
and the parsing APIs that were added in previous commit.

Change-Id: If627ba5a10d78b93668afd32da77807394cd06be
---
 datapath-windows/include/OvsPub.h     | 13 +++-----
 datapath-windows/ovsext/Datapath.c    |  8 ++---
 datapath-windows/ovsext/Datapath.h    |  4 +--
 datapath-windows/ovsext/OvsActions.c  | 57 ++++++++++++++++++-----------------
 datapath-windows/ovsext/OvsDebug.h    |  1 +
 datapath-windows/ovsext/OvsFlow.h     |  2 +-
 datapath-windows/ovsext/OvsPacketIO.h |  2 +-
 datapath-windows/ovsext/OvsUser.c     |  4 +--
 8 files changed, 44 insertions(+), 47 deletions(-)

diff --git a/datapath-windows/include/OvsPub.h b/datapath-windows/include/OvsPub.h
index 1282996..0446309 100644
--- a/datapath-windows/include/OvsPub.h
+++ b/datapath-windows/include/OvsPub.h
 <at>  <at>  -17,12 +17,7  <at>  <at> 
 #ifndef __OVS_PUB_H_
 #define __OVS_PUB_H_ 1

-/* Needed by netlink-protocol.h */
-#define BUILD_ASSERT(EXPR) \
-      typedef char AssertOnCompileFailed[(EXPR) ? 1: -1]
-#define BUILD_ASSERT_DECL(EXPR) BUILD_ASSERT(EXPR)
-
-#include "OvsNetlink.h"
+#include "../ovsext/Netlink.h"
(Continue reading)

Ankur Sharma | 20 Aug 18:46 2014

[PATCH 1/3] datapath-windows: add netlink message parsing APIs

In this change we introduce Netlink.c, Netlink.h and NetlinkProto.h
in datapath-windows. These files will provide netlink message
data structures and parsing APIs.

Changes are on similar lines to userspace netlink code.

Change-Id: Ic225504eff3a25c0619ce3b27d8f54155d8af409
Signed-off-by: Ankur Sharma <ankursharma@...>
Reported-at: https://github.com/openvswitch/ovs-issues/issues/18
---
 datapath-windows/automake.mk           |   5 +-
 datapath-windows/ovsext/Netlink.c      | 339 +++++++++++++++++++++++++++++++++
 datapath-windows/ovsext/Netlink.h      | 104 ++++++++++
 datapath-windows/ovsext/NetlinkProto.h | 118 ++++++++++++
 datapath-windows/ovsext/OvsTypes.h     |   2 +
 datapath-windows/ovsext/ovsext.vcxproj |   3 +
 datapath-windows/ovsext/precomp.h      |   2 +
 7 files changed, 572 insertions(+), 1 deletion(-)
 create mode 100644 datapath-windows/ovsext/Netlink.c
 create mode 100644 datapath-windows/ovsext/Netlink.h
 create mode 100644 datapath-windows/ovsext/NetlinkProto.h

diff --git a/datapath-windows/automake.mk b/datapath-windows/automake.mk
index 47ef225..45348b7 100644
--- a/datapath-windows/automake.mk
+++ b/datapath-windows/automake.mk
 <at>  <at>  -5,7 +5,10  <at>  <at>  EXTRA_DIST += \
 	datapath-windows/Package/package.VcxProj.user \
 	datapath-windows/include/OvsDpInterfaceExt.h \
 	datapath-windows/include/OvsNetlink.h \
(Continue reading)

Debian FTP Masters | 20 Aug 18:04 2014
Picon

openvswitch_2.3.0+git20140819-2_i386.changes ACCEPTED into unstable


Accepted:

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Format: 1.8
Date: Wed, 20 Aug 2014 08:31:11 -0700
Source: openvswitch
Binary: openvswitch-common openvswitch-switch openvswitch-ipsec openvswitch-pki openvswitch-dbg
python-openvswitch openvswitch-test openvswitch-vtep
Architecture: source i386 all
Version: 2.3.0+git20140819-2
Distribution: unstable
Urgency: low
Maintainer: Open vSwitch developers <dev@...>
Changed-By: Ben Pfaff <pfaffben@...>
Description:
 openvswitch-common - Open vSwitch common components
 openvswitch-dbg - Debug symbols for Open vSwitch packages
 openvswitch-ipsec - Open vSwitch GRE-over-IPsec support
 openvswitch-pki - Open vSwitch public key infrastructure dependency package
 openvswitch-switch - Open vSwitch switch implementations
 openvswitch-test - Open vSwitch test package
 openvswitch-vtep - Open vSwitch VTEP utilities
 python-openvswitch - Python bindings for Open vSwitch
Changes:
 openvswitch (2.3.0+git20140819-2) unstable; urgency=low
 .
   * debian/rules: Rerun checks on tests that fail the first time.  Skip
(Continue reading)

Debian FTP Masters | 20 Aug 17:52 2014
Picon

Processing of openvswitch_2.3.0+git20140819-2_i386.changes

openvswitch_2.3.0+git20140819-2_i386.changes uploaded successfully to localhost
along with the files:
  openvswitch-common_2.3.0+git20140819-2_i386.deb
  openvswitch-switch_2.3.0+git20140819-2_i386.deb
  openvswitch-ipsec_2.3.0+git20140819-2_i386.deb
  openvswitch-dbg_2.3.0+git20140819-2_i386.deb
  openvswitch-vtep_2.3.0+git20140819-2_i386.deb
  openvswitch-pki_2.3.0+git20140819-2_all.deb
  python-openvswitch_2.3.0+git20140819-2_all.deb
  openvswitch-test_2.3.0+git20140819-2_all.deb
  openvswitch_2.3.0+git20140819-2.dsc
  openvswitch_2.3.0+git20140819-2.debian.tar.xz

Greetings,

	Your Debian queue daemon (running on host franck.debian.org)
Daniele Di Proietto | 20 Aug 02:11 2014

[PATCH v4 0/2] DPDK performance improvement: exact match cache

This series introduces in dpif-netdev an exact match cache for frequently
used flows. It uses the RSS hash (currently implemented only for netdev-dpdk)
to search the miniflow in an hash table.

While there might be still some tuning left to do, these patch provides
significant performance improvements for the following simple testcase:

CPU: Intel(R) Xeon(R) CPU E5-2650 0  <at>  2.00GHz
NIC: Intel Corporation Ethernet Controller 10-Gigabit X540-AT2 (rev 01)
     dual port (dpdk0, dpdk1)

    Bridge "br0"
        Port "dpdk1"
            Interface "dpdk1"
                type: dpdk
        Port "br0"
            Interface "br0"
                type: internal
        Port "dpdk0"
            Interface "dpdk0"
                type: dpdk

Source connected to dpdk0, sink to dpdk1

1 flow 64 bytes UDP packets going from source to sink

zero drop throughput:

   master: ~ 5.6 Mpps
new patch: ~13.0 Mpps
(Continue reading)

Terry Wilson | 20 Aug 01:28 2014
Picon

[PATCH] ovsdb Allow comparison on optional scalar types

This allows things like initiating a wait request on an interface
ofport being set.

When the optional field is empty and operation is != or excludes
then the result is true; otherwise it is false. If the field is
set then the field is compared normally for its type.

Signed-off-by: Terry Wilson <twilson@...>
---
 lib/ovsdb-types.h        |  9 ++++++
 ovsdb/condition.c        | 24 +++++++++++---
 tests/ovsdb-condition.at | 81 ++++++++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 109 insertions(+), 5 deletions(-)

diff --git a/lib/ovsdb-types.h b/lib/ovsdb-types.h
index efd83a7..5b6b0f0 100644
--- a/lib/ovsdb-types.h
+++ b/lib/ovsdb-types.h
 <at>  <at>  -152,6 +152,8  <at>  <at>  bool ovsdb_type_is_valid(const struct ovsdb_type *);

 static inline bool ovsdb_type_is_scalar(const struct ovsdb_type *);
 static inline bool ovsdb_type_is_optional(const struct ovsdb_type *);
+static inline bool ovsdb_type_is_optional_scalar(
+    const struct ovsdb_type *);
 static inline bool ovsdb_type_is_composite(const struct ovsdb_type *);
 static inline bool ovsdb_type_is_set(const struct ovsdb_type *);
 static inline bool ovsdb_type_is_map(const struct ovsdb_type *);
 <at>  <at>  -202,6 +204,13  <at>  <at>  static inline bool ovsdb_type_is_optional(const struct ovsdb_type *type)
     return type->n_min == 0;
 }
(Continue reading)

Nithin Raju | 19 Aug 22:51 2014

[PATCH 1/4 v4] Add extentions to the standard datapath interface

The datapath interface defined in odp-netlink.h needs some extensions
that are platform dependent. Some examples are the name of the communication
device on Windows and a set of commands that are specific to Windows.

In this change we define a datapath-windows/include/OvsDpInterfaceExt.h
to include any platform specific interface extensions.

OvsDpInterfaceExt.h is in turn included in odp-netlink.h ONLY for _WIN32.
This approach was chosen to avoid including OvsDpInterfaceExt.h directly
although the latter approach is as good as the former.

Also, we define three ioctls in OvsDpInterfaceExt.h:
read:     provides an output buffer (mimics a recv)
write:    provides an input buffer (mimics a send)
transact: provides an input and optionally an output buffer.
          (mimics a send followed by recv)

Signed-off-by: Nithin Raju <nithin@...>
Co-Authored-by: Ben Pfaff <blp@...>
Acked-by: Alin Serdean <aserdean@...>
Acked-by: Ankur Sharma <ankursharma@...>
Acked-by: Saurabh Shah <ssaurabh@...>
---
 build-aux/extract-odp-netlink-h              |    7 +++
 datapath-windows/automake.mk                 |    1 +
 datapath-windows/include/OvsDpInterfaceExt.h |   68 ++++++++++++++++++++++++++
 datapath-windows/ovsext/ovsext.vcxproj       |    3 +-
 include/automake.mk                          |    3 +
 5 files changed, 81 insertions(+), 1 deletions(-)
 create mode 100644 datapath-windows/include/OvsDpInterfaceExt.h
(Continue reading)


Gmane