Bleep bloop.  Greetings Yanqin Wei, 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: Line is 84 characters long (recommended limit is 79)
#36 FILE: lib/dpif-netdev.c:192:
/* DEFAULT_EMC_PREFETCH_SIZE can cover majority traffic including TCP/UDP 
protocol*/

ERROR: Improper whitespace around control block
#60 FILE: lib/dpif-netdev.c:6179:
    if(md_is_valid)

ERROR: Improper whitespace around control block
#99 FILE: lib/dpif-netdev.c:6203:
    if(md_is_valid)

ERROR: Inappropriate bracing around statement
#132 FILE: lib/dpif-netdev.c:6451:
            if (OVS_LIKELY(dp_packet_rss_valid(packet)))

WARNING: Line is 87 characters long (recommended limit is 79)
#135 FILE: lib/dpif-netdev.c:6454:
                key->hash = 
dpif_netdev_packet_get_packet_rss_hash(packet,md_is_valid);

WARNING: Line is 96 characters long (recommended limit is 79)
#137 FILE: lib/dpif-netdev.c:6456:
                    ovs_prefetch_range(&cache->emc_cache.entries[key->hash & 
EM_FLOW_HASH_MASK],

ERROR: Inappropriate bracing around statement
#141 FILE: lib/dpif-netdev.c:6460:
            else

ERROR: Inappropriate bracing around statement
#144 FILE: lib/dpif-netdev.c:6463:
        else

WARNING: Line is 90 characters long (recommended limit is 79)
#152 FILE: lib/dpif-netdev.c:6471:
            key->hash = dpif_netdev_packet_get_hash_5tuple(packet, &key->mf, 
md_is_valid);

Lines checked: 159, Warnings: 4, Errors: 5


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