Workaround: add dpopts to UserSwitch
Currently, slicing on the user switch breaks bandwidth limits. We don't yet have a good way of using both of them at the same time. I'm inclined to turn off slicing by default, but I have to think about it and also see if any one is using it...
This commit is contained in:
+4
-2
@@ -801,7 +801,7 @@ class UserSwitch( Switch ):
|
|||||||
|
|
||||||
dpidLen = 12
|
dpidLen = 12
|
||||||
|
|
||||||
def __init__( self, name, **kwargs ):
|
def __init__( self, name, dpopts='', **kwargs ):
|
||||||
"""Init.
|
"""Init.
|
||||||
name: name for the switch"""
|
name: name for the switch"""
|
||||||
Switch.__init__( self, name, **kwargs )
|
Switch.__init__( self, name, **kwargs )
|
||||||
@@ -810,6 +810,7 @@ class UserSwitch( Switch ):
|
|||||||
'(openflow.org)' )
|
'(openflow.org)' )
|
||||||
if self.listenPort:
|
if self.listenPort:
|
||||||
self.opts += ' --listen=ptcp:%i ' % self.listenPort
|
self.opts += ' --listen=ptcp:%i ' % self.listenPort
|
||||||
|
self.dpopts = dpopts
|
||||||
|
|
||||||
@classmethod
|
@classmethod
|
||||||
def setup( cls ):
|
def setup( cls ):
|
||||||
@@ -840,7 +841,8 @@ class UserSwitch( Switch ):
|
|||||||
self.cmd( 'ifconfig lo up' )
|
self.cmd( 'ifconfig lo up' )
|
||||||
intfs = [ str( i ) for i in self.intfList() if not i.IP() ]
|
intfs = [ str( i ) for i in self.intfList() if not i.IP() ]
|
||||||
self.cmd( 'ofdatapath -i ' + ','.join( intfs ) +
|
self.cmd( 'ofdatapath -i ' + ','.join( intfs ) +
|
||||||
' punix:/tmp/' + self.name + ' -d ' + self.dpid +
|
' punix:/tmp/' + self.name + ' -d %s ' % self.dpid +
|
||||||
|
self.dpopts +
|
||||||
' 1> ' + ofdlog + ' 2> ' + ofdlog + ' &' )
|
' 1> ' + ofdlog + ' 2> ' + ofdlog + ' &' )
|
||||||
self.cmd( 'ofprotocol unix:/tmp/' + self.name +
|
self.cmd( 'ofprotocol unix:/tmp/' + self.name +
|
||||||
' ' + clist +
|
' ' + clist +
|
||||||
|
|||||||
Reference in New Issue
Block a user