Bleep bloop.  Greetings Ales Musil, 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: Use of hardcoded table=<NUMBER> or resubmit=(,<NUMBER>) is discouraged 
in tests. Consider using MACRO instead.
#29 FILE: tests/checkpatch.at:612:
    +table=12(ls_in_hairpin      ), priority=1000 , match=(reg0[[14]] == 1), 
action=(next(pipeline=ingress, table=??);)

WARNING: Use of hardcoded table=<NUMBER> or resubmit=(,<NUMBER>) is discouraged 
in tests. Consider using MACRO instead.
#33 FILE: tests/checkpatch.at:616:
    table=12(ls_in_hairpin      ), priority=1000 , match=(reg0[[14]] == 1), 
action=(next(pipeline=ingress, table=??);)

WARNING: Use of hardcoded table=<NUMBER> or resubmit=(,<NUMBER>) is discouraged 
in tests. Consider using MACRO instead.
#38 FILE: tests/checkpatch.at:621:
    +table=??(ls_in_hairpin      ), priority=1000 , match=(reg0[[14]] == 1), 
action=(next(pipeline=ingress, table=13);)

WARNING: Use of hardcoded table=<NUMBER> or resubmit=(,<NUMBER>) is discouraged 
in tests. Consider using MACRO instead.
#42 FILE: tests/checkpatch.at:625:
    table=??(ls_in_hairpin      ), priority=1000 , match=(reg0[[14]] == 1), 
action=(next(pipeline=ingress, table=13);)

WARNING: Use of hardcoded table=<NUMBER> or resubmit=(,<NUMBER>) is discouraged 
in tests. Consider using MACRO instead.
#47 FILE: tests/checkpatch.at:630:
    +priority=1100,ip,reg15=0x1,metadata=0x1,nw_src=10.0.0.1 
actions=load:0x1->OXM_OF_PKT_REG4[[49]],resubmit(,47)

WARNING: Use of hardcoded table=<NUMBER> or resubmit=(,<NUMBER>) is discouraged 
in tests. Consider using MACRO instead.
#51 FILE: tests/checkpatch.at:634:
    priority=1100,ip,reg15=0x1,metadata=0x1,nw_src=10.0.0.1 
actions=load:0x1->OXM_OF_PKT_REG4[[49]],resubmit(,47)

WARNING: Use of hardcoded table=<NUMBER> or resubmit=(,<NUMBER>) is discouraged 
in tests. Consider using MACRO instead.
#56 FILE: tests/checkpatch.at:639:
    +C_H_E_C_K([as gw1 ovs-ofctl dump-flows br-int table=42 | grep 
"dl_dst=00:00:02:01:02:04" | wc -l], [0], [[1]])

WARNING: Use of hardcoded table=<NUMBER> or resubmit=(,<NUMBER>) is discouraged 
in tests. Consider using MACRO instead.
#60 FILE: tests/checkpatch.at:643:
    C_H_E_C_K([as gw1 ovs-ofctl dump-flows br-int table=42 | grep 
"dl_dst=00:00:02:01:02:04" | wc -l], [0], [[1]])

Lines checked: 104, Warnings: 8, Errors: 0


Please check this out.  If you feel there has been an error, please email 
acon...@redhat.com

Thanks,
0-day Robot
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to