[ovs-dev] [PATCH] don't bother to ask dpif class delete ODPP_NONE.

[YAMAMOTO Takashi]

> On Tue, Aug 27, 2013 at 04:10:03PM +0900, YAMAMOTO Takashi wrote:
>> this fixes ofp_port_status delivery on a patch port removal.
> 
> Can you explain further?  I don't see any problems with patch port
> removal.  Example:
> 
> blp at sigsegv:~/ovs/_clang$ make sandbox
> ...
> ----------------------------------------------------------------------
> You are running in a dummy Open vSwitch environment.  You can use
> ovs-vsctl, ovs-ofctl, ovs-appctl, and other tools to work with the
> dummy switch.  
> 
> Log files, pidfiles, and the configuration database are in the
> "sandbox" subdirectory.
> 
> Exit the shell to kill the running daemons.
> blp at sigsegv:~/ovs/tutorial$ ovs-vsctl add-br br0

i'm using datapath_type=netdev.

kuma% ovs-vsctl add-br br0 -- set bridge br0 datapath_type=netdev
kuma% ovs-ofctl monitor br0 128&
[1] 17027
kuma% ovs-vsctl add-port br0 patch1 -- set interface patch1 type=patch options:peer=patch1 -- add-port br0 patch2 -- set interface patch2 type=patch options:peer=patch2
OFPT_PORT_STATUS (xid=0x0): ADD: 1(patch1): addr:aa:55:aa:55:00:02
     config:     PORT_DOWN
     state:      LINK_DOWN
     speed: 0 Mbps now, 0 Mbps max
OFPT_PORT_STATUS (xid=0x0): ADD: 2(patch2): addr:aa:55:aa:55:00:03
     config:     PORT_DOWN
     state:      LINK_DOWN
     speed: 0 Mbps now, 0 Mbps max
kuma% 

YAMAMOTO Takashi

> blp at sigsegv:~/ovs/tutorial$ ovs-ofctl monitor br0 128&
> [1] 24092
> blp at sigsegv:~/ovs/tutorial$ ovs-vsctl add-port br0 patch1 -- set interface patch1 type=patch options:peer=patch1 -- add-port br0 patch2 -- set interface patch2 type=patch options:peer=patch2
> OFPT_PORT_STATUS (xid=0x0): ADD: 1(patch1): addr:aa:55:aa:55:00:02
>      config:     PORT_DOWN
>      state:      LINK_DOWN
>      speed: 0 Mbps now, 0 Mbps max
> OFPT_PORT_STATUS (xid=0x0): ADD: 2(patch2): addr:aa:55:aa:55:00:03
>      config:     PORT_DOWN
>      state:      LINK_DOWN
>      speed: 0 Mbps now, 0 Mbps max
> blp at sigsegv:~/ovs/tutorial$ ovs-vsctl del-port patch1
> OFPT_PORT_STATUS (xid=0x0): DEL: 1(patch1): addr:aa:55:aa:55:00:02
>      config:     PORT_DOWN
>      state:      LINK_DOWN
>      speed: 0 Mbps now, 0 Mbps max
> blp at sigsegv:~/ovs/tutorial$ ovs-vsctl del-port patch2
> OFPT_PORT_STATUS (xid=0x0): DEL: 2(patch2): addr:aa:55:aa:55:00:03
>      config:     PORT_DOWN
>      state:      LINK_DOWN
>      speed: 0 Mbps now, 0 Mbps max
> blp at sigsegv:~/ovs/tutorial$ ovs-vsctl del-br br0
> ovs-ofctl: vconn_recv (End of file)
> [1]+  Exit 1                  ovs-ofctl monitor br0 128
> blp at sigsegv:~/ovs/tutorial$ exit
> ----------------------------------------------------------------------
> 
> 
> 
> make[1]: Leaving directory `/home/blp/ovs/_clang'
> blp at sigsegv:~/ovs/_clang$ 
> 
> Thanks,
> 
> Ben.
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev