Bleep bloop. Greetings Felix Huettner, I am a robot and I have tried out your
patch.
Thanks for your contribution.
I encountered some error that I wasn't expecting. See the details below.
checkpatch:
WARNING: The subject summary should start with a capital.
WARNING: The subject summary should end with a dot.
Subject: test-conntrack: add per zone benchmark tool
WARNING: Line lacks whitespace around operator
#46 FILE: tests/test-conntrack.c:43:
struct dp_packet *pkt = dp_packet_new(sizeof payload/2);
WARNING: Line is 84 characters long (recommended limit is 79)
#196 FILE: tests/test-conntrack.c:269:
stopwatch_get_stats(STOPWATCH_CT_EXECUTE_NO_COMMIT,
&stats_ct_execute_nocommit);
WARNING: Line has trailing whitespace
#207 FILE: tests/test-conntrack.c:280:
printf("| Min | %16llu us | %19llu us | %12llu us | %13llu us |\n",
WARNING: Line has trailing whitespace
#210 FILE: tests/test-conntrack.c:283:
printf("| Max | %16llu us | %19llu us | %12llu us | %13llu us |\n",
WARNING: Line has trailing whitespace
#213 FILE: tests/test-conntrack.c:286:
printf("| 95%%ile | %16.2f us | %19.2f us | %12.2f us | %13.2f us |\n",
WARNING: Line has trailing whitespace
#216 FILE: tests/test-conntrack.c:289:
printf("| Avg | %16.2f us | %19.2f us | %12.2f us | %13.2f us |\n",
Lines checked: 246, Warnings: 8, Errors: 0
Please check this out. If you feel there has been an error, please email
[email protected]
Thanks,
0-day Robot
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev