On Wed, Jun 15, 2011 at 01:47:34PM -0700, Peter Phaal wrote:
> How about this for the second diagram?
[...]
> In case formatting doesn't come through clearly, the diagram includes
> a bracket on the right to indicate that the ofproto-dpif, dpif,
> dpif-provider and datapath together form an implementation of an
> ofproto provider.

That's good!  I pushed it to "master", as follows.

--8<--------------------------cut here-------------------------->8--

From: Ben Pfaff <b...@nicira.com>
Date: Wed, 15 Jun 2011 14:08:25 -0700
Subject: [PATCH] PORTING: Improve second diagram.

Suggested-by: Peter Phaal <peter.ph...@inmon.com>
---
 AUTHORS |    1 +
 PORTING |   26 +++++++++++++-------------
 2 files changed, 14 insertions(+), 13 deletions(-)

diff --git a/AUTHORS b/AUTHORS
index 2acef53..cd6832c 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -76,6 +76,7 @@ Mikael Doverhag         mdover...@nicira.com
 Pankaj Thakkar          thak...@nicira.com
 Paulo Cravero           pcrav...@as2594.net
 Peter Balland           pe...@nicira.com
+Peter Phaal             peter.ph...@inmon.com
 Ram Jothikumar          rjothiku...@nicira.com
 Rob Hoes                rob.h...@citrix.com
 Roger Leigh             rle...@codelibre.net
diff --git a/PORTING b/PORTING
index c395d65..3a6faa4 100644
--- a/PORTING
+++ b/PORTING
@@ -192,19 +192,19 @@ into the Open vSwitch architecture:
                |   |    ovs-vswitchd   |<-->ovsdb-server
                |   +-------------------+
                |   |      ofproto      |<-->OpenFlow controllers
-               |   +--------+-+--------+
-               |   | netdev | |ofproto-|
-     userspace |   +--------+ |  dpif  |
-               |   | netdev | +--------+
-               |   |provider| |  dpif  |
-               |   +---||---+ +--------+
-               |       ||     |  dpif  |
-               |       ||     |provider|
-               |_      ||     +---||---+
-                       ||         ||
-                _  +---||-----+---||---+
-               |   |          |datapath|
-        kernel |   |          +--------+
+               |   +--------+-+--------+  _
+               |   | netdev | |ofproto-|   |
+     userspace |   +--------+ |  dpif  |   |
+               |   | netdev | +--------+   |
+               |   |provider| |  dpif  |   |
+               |   +---||---+ +--------+   | 
+               |       ||     |  dpif  |   | implementation of
+               |       ||     |provider|   | ofproto provider
+               |_      ||     +---||---+   |
+                       ||         ||       |
+                _  +---||-----+---||---+   |
+               |   |          |datapath|   |
+        kernel |   |          +--------+  _|
                |   |                   |
                |_  +--------||---------+
                             ||
-- 
1.7.4.4

_______________________________________________
discuss mailing list
discuss@openvswitch.org
http://openvswitch.org/mailman/listinfo/discuss

Reply via email to