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

Reply via email to