X-Git-Url: https://gerrit.fd.io/r/gitweb?a=blobdiff_plain;f=resources%2Flibraries%2Frobot%2Fperformance%2Fperformance_limits.robot;h=82688ac9c6c2e230a820e970e9c129af12578250;hb=e0eecd155727298f72046f325a432c7e7aa264b4;hp=759ee5c28f2620de9f0d104dccc4a76710e10c4b;hpb=5e6145a4260ffce1c302e94b9b241851f90838e1;p=csit.git

diff --git a/resources/libraries/robot/performance/performance_limits.robot b/resources/libraries/robot/performance/performance_limits.robot
index 759ee5c28f..82688ac9c6 100644
--- a/resources/libraries/robot/performance/performance_limits.robot
+++ b/resources/libraries/robot/performance/performance_limits.robot
@@ -21,19 +21,19 @@
 | | [Documentation]
 | | ... | Framesize can be either integer in case of a single packet
 | | ... | in stream, or set of packets in case of IMIX type or simmilar.
-| | ...
+| |
 | | ... | *Arguments:*
 | | ... | - frame_size - Framesize. Type: integer or string
-| | ...
+| |
 | | ... | *Returns:*
 | | ... | Average frame size. Type: float
-| | ...
+| |
 | | ... | *Example:*
-| | ...
+| |
 | | ... | \| Get Average Frame Size \| IMIX_v4_1 \|
-| | ...
+| |
 | | [Arguments] | ${frame_size}
-| | ...
+| |
 | | Return From Keyword If | '${frame_size}' == 'IMIX_v4_1' | ${353.83333}
 | | ${frame_size} = | Convert To Number | ${frame_size}
 | | Return From Keyword | ${frame_size}
@@ -50,33 +50,40 @@
 | | ... | whitelisted NIC names.
 | | ... | The mapping is hardcoded in nic_limits.yaml
 | | ... | TODO: Make the mapping from NIC names case insensistive.
-| | ...
+| |
 | | ... | TODO: Make pps limit also definable per NIC.
-| | ...
+| |
 | | ... | This keyword computes maximal unidirectional transmit rate
 | | ... | and jumbo boolean (some suites need that for configuration decisions).
 | | ... | To streamline suite autogeneration, both input and output values
 | | ... | are communicated as test (or broader scope) variables,
 | | ... | instead of explicit arguments and return values.
-| | ...
+| |
 | | ... | *Test (or broader scope) variables read:*
-| | ... |   - nic_name - Name of bottleneck NIC. Type: string
-| | ... |   - overhead - Overhead in bytes. Default: 0. Type: integer
-| | ... |   - frame_size - L2 Frame Size [B] or IMIX string. Type: int or str
-| | ...
+| | ... | - nic_name - Name of bottleneck NIC. Type: string
+| | ... | - overhead - Overhead in bytes. Default: 0. Type: integer
+| | ... | - frame_size - L2 Frame Size [B] or IMIX string. Type: int or str
+| |
 | | ... | *Test variables set:*
-| | ... |   - max_rate - Calculated unidirectional maximal transmit rate [pps].
-| | ... |     Type: float
-| | ... |   - jumbo - Jumbo boolean, true if jumbo packet support
-| | ... |     has to be enabled. Type: boolean
-| | ...
+| | ... | - max_rate - Calculated unidirectional maximal transmit rate [pps].
+| | ... | Type: float
+| | ... | - jumbo - Jumbo boolean, true if jumbo packet support has to be
+| | ... | enabled. Type: boolean
+| |
 | | ... | *Example:*
-| | ...
+| |
 | | ... | \| Set test Variable \| \${frame_size} \| IMIX_v4_1 \|
 | | ... | \| Set Max Rate And Jumbo \|
-| | ...
-| | ${pps_limit} = | Set Variable | ${18750000.0}
-| | ${bps_limit} = | Get From Dictionary | ${NIC_NAME_TO_LIMIT} | ${nic_name}
+| |
+| | # Negative overhead is possible, if DUT-DUT traffic is less encapsulated
+| | # than TG-DUT traffic.
+| | # TODO: Re-check overhead values in suites with both traffics encapsulated.
+| | # TODO: Improve layered setup to detect encap/decap and update overhead.
+| | ${overhead} = | Set Variable If | ${overhead} >= 0 | ${overhead} | ${0}
+| | ${pps_limit} = | Get From Dictionary
+| | ... | ${NIC_NAME_TO_PPS_LIMIT} | ${nic_name}
+| | ${bps_limit} = | Get From Dictionary
+| | ... | ${NIC_NAME_TO_BPS_LIMIT} | ${nic_name}
 | | ${avg_size} = | Get Average Frame Size | ${frame_size}
 | | ${max_size} = | Set Variable If | '${frame_size}' == 'IMIX_v4_1'
 | | ... | ${1518} | ${frame_size}
@@ -90,28 +97,3 @@
 | | ... | ${pps_limit} | ${rate}
 | | Set Test Variable | \${jumbo}
 | | Set Test Variable | \${max_rate}
-
-| Set Max Rate And Jumbo And Handle Multi Seg
-| | [Documentation]
-| | ... | This keyword starts with Get Max Rate And Jumbo keyword,
-| | ... | then adds correct multi seg VPP configuration.
-| | ...
-| | ... | See Documentation of Set Max Rate And Jumbo for more details.
-| | ...
-| | ... | *Test (or broader scope) variables read:*
-| | ... |   - nic_name - Name of bottleneck NIC. Type: string
-| | ... |   - overhead - Overhead in bytes. Default: 0. Type: integer
-| | ... |   - frame_size - L2 Frame Size [B] or IMIX string. Type: int or str
-| | ...
-| | ... | *Test variables set:*
-| | ... |   - max_rate - Calculated unidirectional maximal transmit rate [pps].
-| | ... |     Type: float
-| | ... |   - jumbo - Jumbo boolean, true if jumbo packet support
-| | ... |     has to be enabled. Type: boolean
-| | ...
-| | ... | *Example:*
-| | ...
-| | ... | \| Set Max Rate And Jumbo And Handle Multi Seg \|
-| | ...
-| | Set Max Rate And Jumbo
-| | Run Keyword If | not ${jumbo} | Add no multi seg to all DUTs