[dpdk-test-report] |WARNING| pw26539 [PATCH v3] lib/librte_sched: fix update tc_credits

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Jul 6 15:45:06 CEST 2017


Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/26539

_coding style issues_


ERROR:TRAILING_WHITESPACE: trailing whitespace
#56: FILE: lib/librte_sched/rte_sched.c:88:
+^Iuint32_t tc_rate[RTE_SCHED_TRAFFIC_CLASSES_PER_PIPE];  $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#74: FILE: lib/librte_sched/rte_sched.c:573:
+^I$

ERROR:CODE_INDENT: code indent should use tabs where possible
#76: FILE: lib/librte_sched/rte_sched.c:575:
+^I^I        dst->tc_rate[j] = src->tc_rate[j];$

ERROR:TRAILING_WHITESPACE: trailing whitespace
#81: FILE: lib/librte_sched/rte_sched.c:580:
+^I$

ERROR:CODE_INDENT: code indent should use tabs where possible
#89: FILE: lib/librte_sched/rte_sched.c:846:
+^I        s->tc_rate[i] =  params->tc_rate[i];$

WARNING:LONG_LINE: line over 90 characters
#101: FILE: lib/librte_sched/rte_sched.c:1504:
+	        if (likely((subport->tc_credits[0] + subport->tc_credits_per_period[0]) < subport->tc_rate[0])) {

ERROR:CODE_INDENT: code indent should use tabs where possible
#101: FILE: lib/librte_sched/rte_sched.c:1504:
+^I        if (likely((subport->tc_credits[0] + subport->tc_credits_per_period[0]) < subport->tc_rate[0])) {$

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#101: FILE: lib/librte_sched/rte_sched.c:1504:
+	        if (likely((subport->tc_credits[0] + subport->tc_credits_per_period[0]) < subport->tc_rate[0])) {
[...]
+		else {
[...]

ERROR:CODE_INDENT: code indent should use tabs where possible
#102: FILE: lib/librte_sched/rte_sched.c:1505:
+^I^I        subport->tc_credits[0] += subport->tc_credits_per_period[0];$

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#104: FILE: lib/librte_sched/rte_sched.c:1507:
+		}
+		else {

ERROR:CODE_INDENT: code indent should use tabs where possible
#105: FILE: lib/librte_sched/rte_sched.c:1508:
+^I^I        subport->tc_credits[0] = subport->tc_rate[0];$

WARNING:LONG_LINE: line over 90 characters
#107: FILE: lib/librte_sched/rte_sched.c:1510:
+		if (likely((subport->tc_credits[1] + subport->tc_credits_per_period[1]) < subport->tc_rate[1])) {

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#107: FILE: lib/librte_sched/rte_sched.c:1510:
+		if (likely((subport->tc_credits[1] + subport->tc_credits_per_period[1]) < subport->tc_rate[1])) {
[...]
+		else {
[...]

ERROR:CODE_INDENT: code indent should use tabs where possible
#108: FILE: lib/librte_sched/rte_sched.c:1511:
+^I^I        subport->tc_credits[1] += subport->tc_credits_per_period[1];$

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#110: FILE: lib/librte_sched/rte_sched.c:1513:
+		}
+		else {

ERROR:CODE_INDENT: code indent should use tabs where possible
#111: FILE: lib/librte_sched/rte_sched.c:1514:
+^I^I        subport->tc_credits[1] = subport->tc_rate[1];$

WARNING:LONG_LINE: line over 90 characters
#113: FILE: lib/librte_sched/rte_sched.c:1516:
+		if (likely((subport->tc_credits[2] + subport->tc_credits_per_period[2]) < subport->tc_rate[2])) {

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#113: FILE: lib/librte_sched/rte_sched.c:1516:
+		if (likely((subport->tc_credits[2] + subport->tc_credits_per_period[2]) < subport->tc_rate[2])) {
[...]
+		else {
[...]

ERROR:CODE_INDENT: code indent should use tabs where possible
#114: FILE: lib/librte_sched/rte_sched.c:1517:
+^I^I        subport->tc_credits[2] += subport->tc_credits_per_period[2];$

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#116: FILE: lib/librte_sched/rte_sched.c:1519:
+		}
+		else {

ERROR:CODE_INDENT: code indent should use tabs where possible
#117: FILE: lib/librte_sched/rte_sched.c:1520:
+^I^I        subport->tc_credits[2] = subport->tc_rate[2];$

WARNING:LONG_LINE: line over 90 characters
#119: FILE: lib/librte_sched/rte_sched.c:1522:
+		if (likely((subport->tc_credits[3] + subport->tc_credits_per_period[3]) < subport->tc_rate[3])) {

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#119: FILE: lib/librte_sched/rte_sched.c:1522:
+		if (likely((subport->tc_credits[3] + subport->tc_credits_per_period[3]) < subport->tc_rate[3])) {
[...]
+		else {
[...]

ERROR:CODE_INDENT: code indent should use tabs where possible
#120: FILE: lib/librte_sched/rte_sched.c:1523:
+^I^I        subport->tc_credits[3] += subport->tc_credits_per_period[3];$

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#122: FILE: lib/librte_sched/rte_sched.c:1525:
+		}
+		else {

ERROR:CODE_INDENT: code indent should use tabs where possible
#123: FILE: lib/librte_sched/rte_sched.c:1526:
+^I^I        subport->tc_credits[3] = subport->tc_rate[3];$

WARNING:LONG_LINE: line over 90 characters
#134: FILE: lib/librte_sched/rte_sched.c:1533:
+	        if (likely((pipe->tc_credits[0] + params->tc_credits_per_period[0]) < params->tc_rate[0])) {

ERROR:CODE_INDENT: code indent should use tabs where possible
#134: FILE: lib/librte_sched/rte_sched.c:1533:
+^I        if (likely((pipe->tc_credits[0] + params->tc_credits_per_period[0]) < params->tc_rate[0])) {$

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#134: FILE: lib/librte_sched/rte_sched.c:1533:
+	        if (likely((pipe->tc_credits[0] + params->tc_credits_per_period[0]) < params->tc_rate[0])) {
[...]
+		else {
[...]

ERROR:CODE_INDENT: code indent should use tabs where possible
#135: FILE: lib/librte_sched/rte_sched.c:1534:
+^I^I        pipe->tc_credits[0] += params->tc_credits_per_period[0];$

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#137: FILE: lib/librte_sched/rte_sched.c:1536:
+		}
+		else {

ERROR:CODE_INDENT: code indent should use tabs where possible
#138: FILE: lib/librte_sched/rte_sched.c:1537:
+^I^I        pipe->tc_credits[0] = params->tc_rate[0];$

WARNING:LONG_LINE: line over 90 characters
#140: FILE: lib/librte_sched/rte_sched.c:1539:
+	        if (likely((pipe->tc_credits[1] + params->tc_credits_per_period[1]) < params->tc_rate[1])) {

ERROR:CODE_INDENT: code indent should use tabs where possible
#140: FILE: lib/librte_sched/rte_sched.c:1539:
+^I        if (likely((pipe->tc_credits[1] + params->tc_credits_per_period[1]) < params->tc_rate[1])) {$

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#140: FILE: lib/librte_sched/rte_sched.c:1539:
+	        if (likely((pipe->tc_credits[1] + params->tc_credits_per_period[1]) < params->tc_rate[1])) {
[...]
+		else {
[...]

ERROR:CODE_INDENT: code indent should use tabs where possible
#141: FILE: lib/librte_sched/rte_sched.c:1540:
+^I^I        pipe->tc_credits[1] += params->tc_credits_per_period[1];$

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#143: FILE: lib/librte_sched/rte_sched.c:1542:
+		}
+		else {

ERROR:CODE_INDENT: code indent should use tabs where possible
#144: FILE: lib/librte_sched/rte_sched.c:1543:
+^I^I        pipe->tc_credits[1] = params->tc_rate[1];$

WARNING:LONG_LINE: line over 90 characters
#146: FILE: lib/librte_sched/rte_sched.c:1545:
+	        if (likely((pipe->tc_credits[2] + params->tc_credits_per_period[2]) < params->tc_rate[2])) {

ERROR:CODE_INDENT: code indent should use tabs where possible
#146: FILE: lib/librte_sched/rte_sched.c:1545:
+^I        if (likely((pipe->tc_credits[2] + params->tc_credits_per_period[2]) < params->tc_rate[2])) {$

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#146: FILE: lib/librte_sched/rte_sched.c:1545:
+	        if (likely((pipe->tc_credits[2] + params->tc_credits_per_period[2]) < params->tc_rate[2])) {
[...]
+		else {
[...]

ERROR:CODE_INDENT: code indent should use tabs where possible
#147: FILE: lib/librte_sched/rte_sched.c:1546:
+^I^I        pipe->tc_credits[2] += params->tc_credits_per_period[2];$

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#149: FILE: lib/librte_sched/rte_sched.c:1548:
+		}
+		else {

ERROR:CODE_INDENT: code indent should use tabs where possible
#150: FILE: lib/librte_sched/rte_sched.c:1549:
+^I^I        pipe->tc_credits[2] = params->tc_rate[2];$

WARNING:LONG_LINE: line over 90 characters
#152: FILE: lib/librte_sched/rte_sched.c:1551:
+	        if (likely((pipe->tc_credits[3] + params->tc_credits_per_period[3]) < params->tc_rate[3])) {

ERROR:CODE_INDENT: code indent should use tabs where possible
#152: FILE: lib/librte_sched/rte_sched.c:1551:
+^I        if (likely((pipe->tc_credits[3] + params->tc_credits_per_period[3]) < params->tc_rate[3])) {$

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#152: FILE: lib/librte_sched/rte_sched.c:1551:
+	        if (likely((pipe->tc_credits[3] + params->tc_credits_per_period[3]) < params->tc_rate[3])) {
[...]
+		else {
[...]

ERROR:CODE_INDENT: code indent should use tabs where possible
#153: FILE: lib/librte_sched/rte_sched.c:1552:
+^I^I        pipe->tc_credits[3] += params->tc_credits_per_period[3];$

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#155: FILE: lib/librte_sched/rte_sched.c:1554:
+		}
+		else {

ERROR:CODE_INDENT: code indent should use tabs where possible
#156: FILE: lib/librte_sched/rte_sched.c:1555:
+^I^I        pipe->tc_credits[3] = params->tc_rate[3];$

WARNING:LONG_LINE: line over 90 characters
#171: FILE: lib/librte_sched/rte_sched.c:1622:
+		if (likely((subport->tc_credits[0] + subport->tc_credits_per_period[0]) < subport->tc_rate[0])) {

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#171: FILE: lib/librte_sched/rte_sched.c:1622:
+		if (likely((subport->tc_credits[0] + subport->tc_credits_per_period[0]) < subport->tc_rate[0])) {
[...]
+		else {
[...]

ERROR:CODE_INDENT: code indent should use tabs where possible
#172: FILE: lib/librte_sched/rte_sched.c:1623:
+^I^I        subport->tc_credits[0] += subport->tc_credits_per_period[0];$

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#174: FILE: lib/librte_sched/rte_sched.c:1625:
+		}
+		else {

ERROR:CODE_INDENT: code indent should use tabs where possible
#175: FILE: lib/librte_sched/rte_sched.c:1626:
+^I^I        subport->tc_credits[0] = subport->tc_rate[0];$

WARNING:LONG_LINE: line over 90 characters
#177: FILE: lib/librte_sched/rte_sched.c:1628:
+		if (likely((subport->tc_credits[1] + subport->tc_credits_per_period[1]) < subport->tc_rate[1])) {

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#177: FILE: lib/librte_sched/rte_sched.c:1628:
+		if (likely((subport->tc_credits[1] + subport->tc_credits_per_period[1]) < subport->tc_rate[1])) {
[...]
+		else {
[...]

ERROR:CODE_INDENT: code indent should use tabs where possible
#178: FILE: lib/librte_sched/rte_sched.c:1629:
+^I^I        subport->tc_credits[1] += subport->tc_credits_per_period[1];$

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#180: FILE: lib/librte_sched/rte_sched.c:1631:
+		}
+		else {

ERROR:CODE_INDENT: code indent should use tabs where possible
#181: FILE: lib/librte_sched/rte_sched.c:1632:
+^I^I        subport->tc_credits[1] = subport->tc_rate[1];$

WARNING:LONG_LINE: line over 90 characters
#183: FILE: lib/librte_sched/rte_sched.c:1634:
+		if (likely((subport->tc_credits[2] + subport->tc_credits_per_period[2]) < subport->tc_rate[2])) {

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#183: FILE: lib/librte_sched/rte_sched.c:1634:
+		if (likely((subport->tc_credits[2] + subport->tc_credits_per_period[2]) < subport->tc_rate[2])) {
[...]
+		else {
[...]

ERROR:CODE_INDENT: code indent should use tabs where possible
#184: FILE: lib/librte_sched/rte_sched.c:1635:
+^I^I        subport->tc_credits[2] += subport->tc_credits_per_period[2];$

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#186: FILE: lib/librte_sched/rte_sched.c:1637:
+		}
+		else {

WARNING:LONG_LINE: line over 90 characters
#189: FILE: lib/librte_sched/rte_sched.c:1640:
+		if (likely((subport->tc_credits[3] + subport->tc_credits_per_period[3]) < subport->tc_rate[3])) {

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#189: FILE: lib/librte_sched/rte_sched.c:1640:
+		if (likely((subport->tc_credits[3] + subport->tc_credits_per_period[3]) < subport->tc_rate[3])) {
[...]
+		else {
[...]

ERROR:CODE_INDENT: code indent should use tabs where possible
#190: FILE: lib/librte_sched/rte_sched.c:1641:
+^I^I        subport->tc_credits[3] += subport->tc_credits_per_period[3];$

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#192: FILE: lib/librte_sched/rte_sched.c:1643:
+		}
+		else {

ERROR:CODE_INDENT: code indent should use tabs where possible
#193: FILE: lib/librte_sched/rte_sched.c:1644:
+^I^I        subport->tc_credits[3] = subport->tc_rate[3];$

WARNING:LONG_LINE: line over 90 characters
#205: FILE: lib/librte_sched/rte_sched.c:1652:
+	        if (likely((pipe->tc_credits[0] + params->tc_credits_per_period[0]) < params->tc_rate[0])) {

ERROR:CODE_INDENT: code indent should use tabs where possible
#205: FILE: lib/librte_sched/rte_sched.c:1652:
+^I        if (likely((pipe->tc_credits[0] + params->tc_credits_per_period[0]) < params->tc_rate[0])) {$

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#205: FILE: lib/librte_sched/rte_sched.c:1652:
+	        if (likely((pipe->tc_credits[0] + params->tc_credits_per_period[0]) < params->tc_rate[0])) {
[...]
+		else {
[...]

ERROR:CODE_INDENT: code indent should use tabs where possible
#206: FILE: lib/librte_sched/rte_sched.c:1653:
+^I^I        pipe->tc_credits[0] += params->tc_credits_per_period[0];$

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#208: FILE: lib/librte_sched/rte_sched.c:1655:
+		}
+		else {

ERROR:CODE_INDENT: code indent should use tabs where possible
#209: FILE: lib/librte_sched/rte_sched.c:1656:
+^I^I        pipe->tc_credits[0] = params->tc_rate[0];$

WARNING:LONG_LINE: line over 90 characters
#211: FILE: lib/librte_sched/rte_sched.c:1658:
+	        if (likely((pipe->tc_credits[1] + params->tc_credits_per_period[1]) < params->tc_rate[1])) {

ERROR:CODE_INDENT: code indent should use tabs where possible
#211: FILE: lib/librte_sched/rte_sched.c:1658:
+^I        if (likely((pipe->tc_credits[1] + params->tc_credits_per_period[1]) < params->tc_rate[1])) {$

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#211: FILE: lib/librte_sched/rte_sched.c:1658:
+	        if (likely((pipe->tc_credits[1] + params->tc_credits_per_period[1]) < params->tc_rate[1])) {
[...]
+		else {
[...]

ERROR:CODE_INDENT: code indent should use tabs where possible
#212: FILE: lib/librte_sched/rte_sched.c:1659:
+^I^I        pipe->tc_credits[1] += params->tc_credits_per_period[1];$

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#214: FILE: lib/librte_sched/rte_sched.c:1661:
+		}
+		else {

ERROR:CODE_INDENT: code indent should use tabs where possible
#215: FILE: lib/librte_sched/rte_sched.c:1662:
+^I^I        pipe->tc_credits[1] = params->tc_rate[1];$

WARNING:LONG_LINE: line over 90 characters
#217: FILE: lib/librte_sched/rte_sched.c:1664:
+	        if (likely((pipe->tc_credits[2] + params->tc_credits_per_period[2]) < params->tc_rate[2])) {

ERROR:CODE_INDENT: code indent should use tabs where possible
#217: FILE: lib/librte_sched/rte_sched.c:1664:
+^I        if (likely((pipe->tc_credits[2] + params->tc_credits_per_period[2]) < params->tc_rate[2])) {$

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#217: FILE: lib/librte_sched/rte_sched.c:1664:
+	        if (likely((pipe->tc_credits[2] + params->tc_credits_per_period[2]) < params->tc_rate[2])) {
[...]
+		else {
[...]

ERROR:CODE_INDENT: code indent should use tabs where possible
#218: FILE: lib/librte_sched/rte_sched.c:1665:
+^I^I        pipe->tc_credits[2] += params->tc_credits_per_period[2];$

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#220: FILE: lib/librte_sched/rte_sched.c:1667:
+		}
+		else {

ERROR:CODE_INDENT: code indent should use tabs where possible
#221: FILE: lib/librte_sched/rte_sched.c:1668:
+^I^I        pipe->tc_credits[2] = params->tc_rate[2];$

WARNING:LONG_LINE: line over 90 characters
#223: FILE: lib/librte_sched/rte_sched.c:1670:
+	        if (likely((pipe->tc_credits[3] + params->tc_credits_per_period[3]) < params->tc_rate[3])) {

ERROR:CODE_INDENT: code indent should use tabs where possible
#223: FILE: lib/librte_sched/rte_sched.c:1670:
+^I        if (likely((pipe->tc_credits[3] + params->tc_credits_per_period[3]) < params->tc_rate[3])) {$

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#223: FILE: lib/librte_sched/rte_sched.c:1670:
+	        if (likely((pipe->tc_credits[3] + params->tc_credits_per_period[3]) < params->tc_rate[3])) {
[...]
+		else {
[...]

ERROR:CODE_INDENT: code indent should use tabs where possible
#224: FILE: lib/librte_sched/rte_sched.c:1671:
+^I^I        pipe->tc_credits[3] += params->tc_credits_per_period[3];$

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#226: FILE: lib/librte_sched/rte_sched.c:1673:
+		}
+		else {

ERROR:CODE_INDENT: code indent should use tabs where possible
#227: FILE: lib/librte_sched/rte_sched.c:1674:
+^I^I        pipe->tc_credits[3] = params->tc_rate[3];$

ERROR:CODE_INDENT: code indent should use tabs where possible
#228: FILE: lib/librte_sched/rte_sched.c:1675:
+^I        }$

total: 62 errors, 32 warnings, 196 lines checked


More information about the test-report mailing list