automatic DPDK test reports
 help / color / mirror / Atom feed
* [dpdk-test-report] |WARNING| pw31093 [PATCH] eal: force IOVA PA mode if KNI module inserted
       [not found] <3a029ac2-2b6b-b0c2-a577-967b6feba0c9@caviumnetworks.com>
@ 2017-11-02  4:00 ` checkpatch
  0 siblings, 0 replies; only message in thread
From: checkpatch @ 2017-11-02  4:00 UTC (permalink / raw)
  To: test-report; +Cc: Santosh Shukla

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

_coding style issues_


WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#46: 
> diff --git a/lib/librte_eal/linuxapp/eal/eal.c b/lib/librte_eal/linuxapp/eal/eal.c

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#82: FILE: drivers/bus/pci/linux/pci.c:589:
+       bool has_kni;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#88: FILE: drivers/bus/pci/linux/pci.c:595:
+       has_kni = rte_eal_check_module("rte_kni");$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#97: FILE: drivers/bus/pci/linux/pci.c:603:
+       if (has_iova_va && !is_bound_uio && !has_kni && !is_vfio_noiommu_enabled)$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (7, 15)
#97: FILE: drivers/bus/pci/linux/pci.c:603:
+       if (has_iova_va && !is_bound_uio && !has_kni && !is_vfio_noiommu_enabled)
                return RTE_IOVA_VA;

ERROR:CODE_INDENT: code indent should use tabs where possible
#105: FILE: drivers/bus/pci/linux/pci.c:612:
+               if (has_kni)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#105: FILE: drivers/bus/pci/linux/pci.c:612:
+               if (has_kni)$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (15, 23)
#105: FILE: drivers/bus/pci/linux/pci.c:612:
+               if (has_kni)
+                       RTE_LOG(WARNING, EAL, "Some devices want iova as va but pa will be used because.."

ERROR:CODE_INDENT: code indent should use tabs where possible
#106: FILE: drivers/bus/pci/linux/pci.c:613:
+                       RTE_LOG(WARNING, EAL, "Some devices want iova as va but pa will be used because.."$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#106: FILE: drivers/bus/pci/linux/pci.c:613:
+                       RTE_LOG(WARNING, EAL, "Some devices want iova as va but pa will be used because.."$

ERROR:CODE_INDENT: code indent should use tabs where possible
#107: FILE: drivers/bus/pci/linux/pci.c:614:
+                                       "KNI module inserted
");$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#107: FILE: drivers/bus/pci/linux/pci.c:614:
+                                       "KNI module inserted
");$

ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s)

total: 4 errors, 9 warnings, 30 lines checked

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2017-11-02  4:00 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <3a029ac2-2b6b-b0c2-a577-967b6feba0c9@caviumnetworks.com>
2017-11-02  4:00 ` [dpdk-test-report] |WARNING| pw31093 [PATCH] eal: force IOVA PA mode if KNI module inserted checkpatch

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).