Tue, Mar 12, 2019 at 02:22:04PM CET, jtluka(a)redhat.com wrote:
Tue, Mar 05, 2019 at 05:09:11PM CET, csfakian(a)redhat.com wrote:
> for perf_test in self.params.perf_tests:
>+ offload_pairs = sub_config.offload_settings.items()
>+ if ((perf_test == 'udp_stream' and ('gro',
'off') in offload_pairs)
I believe that you have reverted the second condition compared to
the original code (master branch). There it is:
if nperf_protocols.find("udp") > -1 and ("gro",
"off") not in setting:
For UDP we don't care about any combination where gro is turned off as
it does not have any impact.
Scratch that. Christos explained offline that I did not notice the
'continue' statement below.
>>+ or
>>+ (perf_test == 'sctp_stream' and not list(val[0] for
val in
>>+ offload_pairs if val[1] == 'off') in [[],
['gso']])):
>>+ continue
>>+
>> flow = PerfFlow(
>> type = perf_test,
>> generator = client_netns,