[dpdk-test-report] |WARNING| pw50449 [v2] lib/metrics: add unregister api for metrics

automatic DPDK test reports
 help / color / mirror / Atom feed

From: checkpatch@dpdk.org
To: test-report@dpdk.org
Cc: wanjunjie <wan.junjie@foxmail.com>
Subject: [dpdk-test-report] |WARNING| pw50449 [v2] lib/metrics: add unregister api for metrics
Date: Fri, 22 Feb 2019 14:57:48 +0100 (CET)	[thread overview]
Message-ID: <20190222135748.7353B2F42@dpdk.org> (raw)
In-Reply-To: <1550843805-21419-1-git-send-email-wan.junjie@foxmail.com>

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

_coding style issues_


ERROR:EXECUTE_PERMISSIONS: do not set execute permissions for source files
#35: FILE: lib/librte_latencystats/rte_latencystats.c

ERROR:TRAILING_WHITESPACE: trailing whitespace
#53: FILE: lib/librte_latencystats/rte_latencystats.c:294:
+^I$

ERROR:TRAILING_WHITESPACE: trailing whitespace
#56: FILE: lib/librte_latencystats/rte_latencystats.c:297:
+^I$

ERROR:EXECUTE_PERMISSIONS: do not set execute permissions for source files
#62: FILE: lib/librte_metrics/rte_metrics.c

ERROR:TRAILING_WHITESPACE: trailing whitespace
#121: FILE: lib/librte_metrics/rte_metrics.c:79:
+^I^Irte_exit(EXIT_FAILURE, $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#128: FILE: lib/librte_metrics/rte_metrics.c:86:
+^I^Irte_exit(EXIT_FAILURE, $

WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline
#129: FILE: lib/librte_metrics/rte_metrics.c:87:
+				"Failed to init vlan bitmap for bonded device 
");

ERROR:SPACING: space required after that ',' (ctx:VxV)
#141: FILE: lib/librte_metrics/rte_metrics.c:108:
+	uint16_t idx_name,idx;
 	                 ^

ERROR:OPEN_BRACE: that open brace { should be on the previous line
#160: FILE: lib/librte_metrics/rte_metrics.c:129:
+	for (idx_name = 0; idx_name < RTE_METRICS_MAX_METRICS; idx_name++)
+	{

ERROR:OPEN_BRACE: that open brace { should be on the previous line
#162: FILE: lib/librte_metrics/rte_metrics.c:131:
+		if(!rte_bitmap_get(stats->bits,idx_name))
+		{

ERROR:SPACING: space required after that ',' (ctx:VxV)
#162: FILE: lib/librte_metrics/rte_metrics.c:131:
+		if(!rte_bitmap_get(stats->bits,idx_name))
 		                              ^

ERROR:SPACING: space required before the open parenthesis '('
#162: FILE: lib/librte_metrics/rte_metrics.c:131:
+		if(!rte_bitmap_get(stats->bits,idx_name))

ERROR:SPACING: space required before the open parenthesis '('
#165: FILE: lib/librte_metrics/rte_metrics.c:134:
+			if(idx_base + cnt_names > RTE_METRICS_MAX_METRICS)

ERROR:OPEN_BRACE: that open brace { should be on the previous line
#167: FILE: lib/librte_metrics/rte_metrics.c:136:
+			for(idx = idx_base; idx < idx_base+cnt_names; idx++)
+			{

ERROR:SPACING: space required before the open parenthesis '('
#167: FILE: lib/librte_metrics/rte_metrics.c:136:
+			for(idx = idx_base; idx < idx_base+cnt_names; idx++)

ERROR:OPEN_BRACE: that open brace { should be on the previous line
#169: FILE: lib/librte_metrics/rte_metrics.c:138:
+				if(rte_bitmap_get(stats->bits, idx))
+				{

ERROR:SPACING: space required before the open parenthesis '('
#169: FILE: lib/librte_metrics/rte_metrics.c:138:
+				if(rte_bitmap_get(stats->bits, idx))

ERROR:OPEN_BRACE: that open brace { should be on the previous line
#174: FILE: lib/librte_metrics/rte_metrics.c:143:
+			if(idx == idx_base+cnt_names)
+			{

ERROR:SPACING: space required before the open parenthesis '('
#174: FILE: lib/librte_metrics/rte_metrics.c:143:
+			if(idx == idx_base+cnt_names)

ERROR:OPEN_BRACE: that open brace { should be on the previous line
#183: FILE: lib/librte_metrics/rte_metrics.c:150:
+	for(idx = idx_base; idx < idx_base+cnt_names; idx++)
+	{

ERROR:SPACING: space required before the open parenthesis '('
#183: FILE: lib/librte_metrics/rte_metrics.c:150:
+	for(idx = idx_base; idx < idx_base+cnt_names; idx++)

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#190: FILE: lib/librte_metrics/rte_metrics.c:157:
+    }$

ERROR:SPACING: space required before the open brace '{'
#216: FILE: lib/librte_metrics/rte_metrics.c:200:
+		if(rte_bitmap_get(stats->bits, idx_metric)){

ERROR:SPACING: space required before the open parenthesis '('
#216: FILE: lib/librte_metrics/rte_metrics.c:200:
+		if(rte_bitmap_get(stats->bits, idx_metric)){

ERROR:SPACING: space required after that close brace '}'
#219: FILE: lib/librte_metrics/rte_metrics.c:203:
+		}else

ERROR:SPACING: space prohibited before that close parenthesis ')'
#239: FILE: lib/librte_metrics/rte_metrics.c:238:
+	if (count < 1 )

ERROR:TRAILING_WHITESPACE: trailing whitespace
#245: FILE: lib/librte_metrics/rte_metrics.c:244:
+^I$

WARNING:BRACES: braces {} are not necessary for single statement blocks
#250: FILE: lib/librte_metrics/rte_metrics.c:249:
+	if (key >= RTE_METRICS_MAX_METRICS) {
+		return -EINVAL;
+	}

ERROR:SPACING: space required after that ',' (ctx:VxV)
#259: FILE: lib/librte_metrics/rte_metrics.c:258:
+		if(rte_bitmap_get(stats->bits,idx_metric)){
 		                             ^

ERROR:SPACING: space required before the open brace '{'
#259: FILE: lib/librte_metrics/rte_metrics.c:258:
+		if(rte_bitmap_get(stats->bits,idx_metric)){

ERROR:SPACING: space required before the open parenthesis '('
#259: FILE: lib/librte_metrics/rte_metrics.c:258:
+		if(rte_bitmap_get(stats->bits,idx_metric)){

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (16, 16)
#262: FILE: lib/librte_metrics/rte_metrics.c:261:
+		}else
+		break;

ERROR:SPACING: space required after that close brace '}'
#262: FILE: lib/librte_metrics/rte_metrics.c:261:
+		}else

ERROR:TRAILING_WHITESPACE: trailing whitespace
#271: FILE: lib/librte_metrics/rte_metrics.c:270:
+^I$

WARNING:SPACING: space prohibited before semicolon
#274: FILE: lib/librte_metrics/rte_metrics.c:273:
+		memset(stats->metadata[idx_metric].name, 0, RTE_METRICS_MAX_NAME_LEN) ;

ERROR:SPACING: spaces required around that '=' (ctx:VxV)
#291: FILE: lib/librte_metrics/rte_metrics.c:289:
+	uint16_t idx_name, idx=0;
 	                      ^

WARNING:LONG_LINE: line over 90 characters
#303: FILE: lib/librte_metrics/rte_metrics.c:304:
+		for (idx_name = 0;idx< stats->cnt_stats && idx_name < RTE_METRICS_MAX_METRICS; idx_name++)

ERROR:SPACING: space required after that ';' (ctx:VxV)
#303: FILE: lib/librte_metrics/rte_metrics.c:304:
+		for (idx_name = 0;idx< stats->cnt_stats && idx_name < RTE_METRICS_MAX_METRICS; idx_name++)
 		                 ^

ERROR:SPACING: spaces required around that '<' (ctx:VxW)
#303: FILE: lib/librte_metrics/rte_metrics.c:304:
+		for (idx_name = 0;idx< stats->cnt_stats && idx_name < RTE_METRICS_MAX_METRICS; idx_name++)
 		                     ^

ERROR:OPEN_BRACE: that open brace { should be on the previous line
#304: FILE: lib/librte_metrics/rte_metrics.c:305:
+			if(rte_bitmap_get(stats->bits,idx_name))
+			{

ERROR:SPACING: space required after that ',' (ctx:VxV)
#304: FILE: lib/librte_metrics/rte_metrics.c:305:
+			if(rte_bitmap_get(stats->bits,idx_name))
 			                             ^

ERROR:SPACING: space required before the open parenthesis '('
#304: FILE: lib/librte_metrics/rte_metrics.c:305:
+			if(rte_bitmap_get(stats->bits,idx_name))

ERROR:SPACING: space prohibited before that '++' (ctx:WxO)
#309: FILE: lib/librte_metrics/rte_metrics.c:310:
+				idx ++;
 				    ^

ERROR:SPACING: spaces required around that '=' (ctx:VxV)
#319: FILE: lib/librte_metrics/rte_metrics.c:326:
+	uint16_t idx_name, idx=0;
 	                      ^

ERROR:SPACING: space required after that ';' (ctx:VxV)
#340: FILE: lib/librte_metrics/rte_metrics.c:347:
+		for (idx_name = 0;idx< stats->cnt_stats &&
 		                 ^

ERROR:SPACING: spaces required around that '<' (ctx:VxW)
#340: FILE: lib/librte_metrics/rte_metrics.c:347:
+		for (idx_name = 0;idx< stats->cnt_stats &&
 		                     ^

ERROR:OPEN_BRACE: that open brace { should be on the previous line
#343: FILE: lib/librte_metrics/rte_metrics.c:350:
+			if(rte_bitmap_get(stats->bits,idx_name))
+			{

ERROR:SPACING: space required after that ',' (ctx:VxV)
#343: FILE: lib/librte_metrics/rte_metrics.c:350:
+			if(rte_bitmap_get(stats->bits,idx_name))
 			                             ^

ERROR:SPACING: space required before the open parenthesis '('
#343: FILE: lib/librte_metrics/rte_metrics.c:350:
+			if(rte_bitmap_get(stats->bits,idx_name))

ERROR:TRAILING_WHITESPACE: trailing whitespace
#355: FILE: lib/librte_metrics/rte_metrics.c:360:
+^I^I}^I^I$

WARNING:TYPO_SPELLING: 'registerd' may be misspelled - perhaps 'registered'?
#372: FILE: lib/librte_metrics/rte_metrics.h:128:
+ * Remove the metrics previously registerd

total: 44 errors, 7 warnings, 329 lines checked

          parent reply	other threads:[~2019-02-22 13:57 UTC|newest]

Thread overview: expand[flat|nested]  mbox.gz  Atom feed
 [parent not found: <1550843805-21419-1-git-send-email-wan.junjie@foxmail.com>]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190222135748.7353B2F42@dpdk.org \
    --to=checkpatch@dpdk.org \
    --cc=test-report@dpdk.org \
    --cc=wan.junjie@foxmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Be sure your reply has a Subject: header at the top and a blank line before the message body.

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).