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