automatic DPDK test reports
 help / color / mirror / Atom feed
From: checkpatch@dpdk.org
To: test-report@dpdk.org
Cc: Sunil Pai G <sunil.pai.g@intel.com>
Subject: [dpdk-test-report] |WARNING| pw97222 [PATCH RFC 1/1] vhost: add DMADEV support for async datapath
Date: Mon, 23 Aug 2021 11:55:49 +0200 (CEST)	[thread overview]
Message-ID: <20210823095549.1A3501226B3@dpdk.org> (raw)
In-Reply-To: <20210823095355.2478423-2-sunil.pai.g@intel.com>

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

_coding style issues_


WARNING:SPACING: missing space after struct definition
#336: FILE: lib/vhost/vhost.h:129:
+struct enq_info_t{

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#337: FILE: lib/vhost/vhost.h:130:
+    uint8_t pkt_rcvd; //Make this atomic$

ERROR:C99_COMMENTS: do not use C99 // comments
#337: FILE: lib/vhost/vhost.h:130:
+    uint8_t pkt_rcvd; //Make this atomic

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#342: FILE: lib/vhost/vhost.h:135:
+ * to guarantee correct behaviour. */

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#344: FILE: lib/vhost/vhost.h:137:
+    struct enq_info_t enq_info[DMA_COMPLETION_RING_SIZE];$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#345: FILE: lib/vhost/vhost.h:138:
+    uint16_t count;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#346: FILE: lib/vhost/vhost.h:139:
+    uint16_t read_idx;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#347: FILE: lib/vhost/vhost.h:140:
+    uint16_t write_idx;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#392: FILE: lib/vhost/virtio_net.c:1597:
+    return dma_compl->count == DMA_COMPLETION_RING_SIZE;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#399: FILE: lib/vhost/virtio_net.c:1604:
+    return dma_compl->count == 0;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#407: FILE: lib/vhost/virtio_net.c:1612:
+                    const struct rte_vhost_iov_iter *src_ptr,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#407: FILE: lib/vhost/virtio_net.c:1612:
+                    const struct rte_vhost_iov_iter *src_ptr,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#408: FILE: lib/vhost/virtio_net.c:1613:
+                    const struct rte_vhost_iov_iter *dst_ptr,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#408: FILE: lib/vhost/virtio_net.c:1613:
+                    const struct rte_vhost_iov_iter *dst_ptr,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#409: FILE: lib/vhost/virtio_net.c:1614:
+                    const uint16_t nr_segs,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#409: FILE: lib/vhost/virtio_net.c:1614:
+                    const uint16_t nr_segs,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#410: FILE: lib/vhost/virtio_net.c:1615:
+                    struct enq_info_t *slot_addr)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#410: FILE: lib/vhost/virtio_net.c:1615:
+                    struct enq_info_t *slot_addr)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#412: FILE: lib/vhost/virtio_net.c:1617:
+    uint16_t seg_idx = 0;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#413: FILE: lib/vhost/virtio_net.c:1618:
+    struct enq_info_t *addr = NULL;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#414: FILE: lib/vhost/virtio_net.c:1619:
+    uint64_t dma_flags = RTE_DMA_OP_FLAG_LLC;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#415: FILE: lib/vhost/virtio_net.c:1620:
+    const uint16_t dmadev_ring_mask = VHOST_ASYNC_DMADEV_RING_SIZE-1;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#417: FILE: lib/vhost/virtio_net.c:1622:
+    while (likely(seg_idx < nr_segs)) {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#418: FILE: lib/vhost/virtio_net.c:1623:
+        /* Fetch DMA source start addr. */$

ERROR:CODE_INDENT: code indent should use tabs where possible
#419: FILE: lib/vhost/virtio_net.c:1624:
+        const rte_iova_t s_base = (uintptr_t)(src_ptr->iov[seg_idx].iov_base);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#419: FILE: lib/vhost/virtio_net.c:1624:
+        const rte_iova_t s_base = (uintptr_t)(src_ptr->iov[seg_idx].iov_base);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#420: FILE: lib/vhost/virtio_net.c:1625:
+        const rte_iova_t dma_src_start_addr = src_ptr->offset + s_base;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#420: FILE: lib/vhost/virtio_net.c:1625:
+        const rte_iova_t dma_src_start_addr = src_ptr->offset + s_base;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#421: FILE: lib/vhost/virtio_net.c:1626:
+        /* Fetch DMA destination start addr. */$

ERROR:CODE_INDENT: code indent should use tabs where possible
#422: FILE: lib/vhost/virtio_net.c:1627:
+        const rte_iova_t d_base = (uintptr_t)(dst_ptr->iov[seg_idx].iov_base);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#422: FILE: lib/vhost/virtio_net.c:1627:
+        const rte_iova_t d_base = (uintptr_t)(dst_ptr->iov[seg_idx].iov_base);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#423: FILE: lib/vhost/virtio_net.c:1628:
+        const rte_iova_t dma_dst_start_addr = dst_ptr->offset + d_base;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#423: FILE: lib/vhost/virtio_net.c:1628:
+        const rte_iova_t dma_dst_start_addr = dst_ptr->offset + d_base;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#424: FILE: lib/vhost/virtio_net.c:1629:
+        /* Fetch packet segment length. */$

ERROR:CODE_INDENT: code indent should use tabs where possible
#425: FILE: lib/vhost/virtio_net.c:1630:
+        const uint32_t dma_src_len = src_ptr->iov[seg_idx].iov_len;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#425: FILE: lib/vhost/virtio_net.c:1630:
+        const uint32_t dma_src_len = src_ptr->iov[seg_idx].iov_len;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#426: FILE: lib/vhost/virtio_net.c:1631:
+        /* Check if this segment is the last. */$

ERROR:CODE_INDENT: code indent should use tabs where possible
#427: FILE: lib/vhost/virtio_net.c:1632:
+        if (seg_idx == nr_segs - 1) {$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#427: FILE: lib/vhost/virtio_net.c:1632:
+        if (seg_idx == nr_segs - 1) {$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12)
#427: FILE: lib/vhost/virtio_net.c:1632:
+        if (seg_idx == nr_segs - 1) {
+            addr = slot_addr;

WARNING:BRACES: braces {} are not necessary for single statement blocks
#427: FILE: lib/vhost/virtio_net.c:1632:
+        if (seg_idx == nr_segs - 1) {
+            addr = slot_addr;
+        }

ERROR:CODE_INDENT: code indent should use tabs where possible
#428: FILE: lib/vhost/virtio_net.c:1633:
+            addr = slot_addr;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#428: FILE: lib/vhost/virtio_net.c:1633:
+            addr = slot_addr;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#429: FILE: lib/vhost/virtio_net.c:1634:
+        }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#429: FILE: lib/vhost/virtio_net.c:1634:
+        }$

ERROR:CODE_INDENT: code indent should use tabs where possible
#431: FILE: lib/vhost/virtio_net.c:1636:
+        int enq_index = rte_dmadev_copy(dev_id,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#431: FILE: lib/vhost/virtio_net.c:1636:
+        int enq_index = rte_dmadev_copy(dev_id,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#432: FILE: lib/vhost/virtio_net.c:1637:
+                                        0,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#432: FILE: lib/vhost/virtio_net.c:1637:
+                                        0,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#433: FILE: lib/vhost/virtio_net.c:1638:
+                                        dma_src_start_addr,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#433: FILE: lib/vhost/virtio_net.c:1638:
+                                        dma_src_start_addr,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#434: FILE: lib/vhost/virtio_net.c:1639:
+                                        dma_dst_start_addr,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#434: FILE: lib/vhost/virtio_net.c:1639:
+                                        dma_dst_start_addr,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#435: FILE: lib/vhost/virtio_net.c:1640:
+                                        dma_src_len,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#435: FILE: lib/vhost/virtio_net.c:1640:
+                                        dma_src_len,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#436: FILE: lib/vhost/virtio_net.c:1641:
+                                        dma_flags);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#436: FILE: lib/vhost/virtio_net.c:1641:
+                                        dma_flags);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#437: FILE: lib/vhost/virtio_net.c:1642:
+        if (enq_index < 0)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#437: FILE: lib/vhost/virtio_net.c:1642:
+        if (enq_index < 0)$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12)
#437: FILE: lib/vhost/virtio_net.c:1642:
+        if (enq_index < 0)
+            break;

ERROR:CODE_INDENT: code indent should use tabs where possible
#438: FILE: lib/vhost/virtio_net.c:1643:
+            break;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#438: FILE: lib/vhost/virtio_net.c:1643:
+            break;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#439: FILE: lib/vhost/virtio_net.c:1644:
+        dmadev_enq_track[dev_id][enq_index & dmadev_ring_mask] = (void *)addr;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#439: FILE: lib/vhost/virtio_net.c:1644:
+        dmadev_enq_track[dev_id][enq_index & dmadev_ring_mask] = (void *)addr;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#440: FILE: lib/vhost/virtio_net.c:1645:
+        seg_idx++;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#440: FILE: lib/vhost/virtio_net.c:1645:
+        seg_idx++;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#441: FILE: lib/vhost/virtio_net.c:1646:
+    }$

ERROR:CODE_INDENT: code indent should use tabs where possible
#447: FILE: lib/vhost/virtio_net.c:1652:
+                  const struct rte_vhost_iov_iter *dst_ptr,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#447: FILE: lib/vhost/virtio_net.c:1652:
+                  const struct rte_vhost_iov_iter *dst_ptr,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#448: FILE: lib/vhost/virtio_net.c:1653:
+                  const uint16_t nr_segs)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#448: FILE: lib/vhost/virtio_net.c:1653:
+                  const uint16_t nr_segs)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#450: FILE: lib/vhost/virtio_net.c:1655:
+    uint16_t seg_idx = 0;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#452: FILE: lib/vhost/virtio_net.c:1657:
+    while (likely(seg_idx < nr_segs)) {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#453: FILE: lib/vhost/virtio_net.c:1658:
+        /* Fetch source start addr. */$

ERROR:CODE_INDENT: code indent should use tabs where possible
#454: FILE: lib/vhost/virtio_net.c:1659:
+        const uintptr_t s_base = (uintptr_t)(src_ptr->iov[seg_idx].iov_base);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#454: FILE: lib/vhost/virtio_net.c:1659:
+        const uintptr_t s_base = (uintptr_t)(src_ptr->iov[seg_idx].iov_base);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#455: FILE: lib/vhost/virtio_net.c:1660:
+        const uintptr_t src_start_addr = src_ptr->offset + s_base;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#455: FILE: lib/vhost/virtio_net.c:1660:
+        const uintptr_t src_start_addr = src_ptr->offset + s_base;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#456: FILE: lib/vhost/virtio_net.c:1661:
+        /* Fetch destination start addr. */$

ERROR:CODE_INDENT: code indent should use tabs where possible
#457: FILE: lib/vhost/virtio_net.c:1662:
+        const uintptr_t d_base = (uintptr_t)(dst_ptr->iov[seg_idx].iov_base);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#457: FILE: lib/vhost/virtio_net.c:1662:
+        const uintptr_t d_base = (uintptr_t)(dst_ptr->iov[seg_idx].iov_base);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#458: FILE: lib/vhost/virtio_net.c:1663:
+        const uintptr_t dst_start_addr = dst_ptr->offset + d_base;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#458: FILE: lib/vhost/virtio_net.c:1663:
+        const uintptr_t dst_start_addr = dst_ptr->offset + d_base;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#459: FILE: lib/vhost/virtio_net.c:1664:
+        /* Fetch segment length. */$

ERROR:CODE_INDENT: code indent should use tabs where possible
#460: FILE: lib/vhost/virtio_net.c:1665:
+        const size_t src_len = src_ptr->iov[seg_idx].iov_len;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#460: FILE: lib/vhost/virtio_net.c:1665:
+        const size_t src_len = src_ptr->iov[seg_idx].iov_len;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#462: FILE: lib/vhost/virtio_net.c:1667:
+        rte_memcpy((void *) dst_start_addr,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#462: FILE: lib/vhost/virtio_net.c:1667:
+        rte_memcpy((void *) dst_start_addr,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#463: FILE: lib/vhost/virtio_net.c:1668:
+                   (void *) src_start_addr,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#463: FILE: lib/vhost/virtio_net.c:1668:
+                   (void *) src_start_addr,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#464: FILE: lib/vhost/virtio_net.c:1669:
+                   src_len);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#464: FILE: lib/vhost/virtio_net.c:1669:
+                   src_len);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#465: FILE: lib/vhost/virtio_net.c:1670:
+        seg_idx++;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#465: FILE: lib/vhost/virtio_net.c:1670:
+        seg_idx++;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#466: FILE: lib/vhost/virtio_net.c:1671:
+    }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#473: FILE: lib/vhost/virtio_net.c:1678:
+    struct enq_info_t *slot_addr$

ERROR:CODE_INDENT: code indent should use tabs where possible
#474: FILE: lib/vhost/virtio_net.c:1679:
+                        = &(dma_compl->enq_info[dma_compl->write_idx]);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#474: FILE: lib/vhost/virtio_net.c:1679:
+                        = &(dma_compl->enq_info[dma_compl->write_idx]);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#475: FILE: lib/vhost/virtio_net.c:1680:
+    const uint16_t ring_mask = DMA_COMPLETION_RING_SIZE - 1;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#477: FILE: lib/vhost/virtio_net.c:1682:
+    dma_compl->write_idx++;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#478: FILE: lib/vhost/virtio_net.c:1683:
+    dma_compl->write_idx &= ring_mask;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#479: FILE: lib/vhost/virtio_net.c:1684:
+    dma_compl->count++;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#480: FILE: lib/vhost/virtio_net.c:1685:
+    return slot_addr;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#486: FILE: lib/vhost/virtio_net.c:1691:
+                        const int max_pkts)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#486: FILE: lib/vhost/virtio_net.c:1691:
+                        const int max_pkts)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#488: FILE: lib/vhost/virtio_net.c:1693:
+    uint32_t pkts;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#489: FILE: lib/vhost/virtio_net.c:1694:
+    int count = dma_compl->count;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#490: FILE: lib/vhost/virtio_net.c:1695:
+    int read_idx = dma_compl->read_idx;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#491: FILE: lib/vhost/virtio_net.c:1696:
+    uint8_t pkt_rcvd = 0;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#492: FILE: lib/vhost/virtio_net.c:1697:
+    const uint16_t ring_mask = DMA_COMPLETION_RING_SIZE - 1;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#494: FILE: lib/vhost/virtio_net.c:1699:
+    for (pkts = 0; (pkts < (uint32_t)max_pkts) && (count > 0); pkts++) {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#495: FILE: lib/vhost/virtio_net.c:1700:
+        read_idx &= ring_mask;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#495: FILE: lib/vhost/virtio_net.c:1700:
+        read_idx &= ring_mask;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#496: FILE: lib/vhost/virtio_net.c:1701:
+        pkt_rcvd = dma_compl->enq_info[read_idx].pkt_rcvd;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#496: FILE: lib/vhost/virtio_net.c:1701:
+        pkt_rcvd = dma_compl->enq_info[read_idx].pkt_rcvd;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#497: FILE: lib/vhost/virtio_net.c:1702:
+        if (!pkt_rcvd) {$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#497: FILE: lib/vhost/virtio_net.c:1702:
+        if (!pkt_rcvd) {$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12)
#497: FILE: lib/vhost/virtio_net.c:1702:
+        if (!pkt_rcvd) {
+            break;

WARNING:BRACES: braces {} are not necessary for single statement blocks
#497: FILE: lib/vhost/virtio_net.c:1702:
+        if (!pkt_rcvd) {
+            break;
+        }

ERROR:CODE_INDENT: code indent should use tabs where possible
#498: FILE: lib/vhost/virtio_net.c:1703:
+            break;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#498: FILE: lib/vhost/virtio_net.c:1703:
+            break;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#499: FILE: lib/vhost/virtio_net.c:1704:
+        }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#499: FILE: lib/vhost/virtio_net.c:1704:
+        }$

ERROR:CODE_INDENT: code indent should use tabs where possible
#501: FILE: lib/vhost/virtio_net.c:1706:
+        dma_compl->enq_info[read_idx].pkt_rcvd = 0;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#501: FILE: lib/vhost/virtio_net.c:1706:
+        dma_compl->enq_info[read_idx].pkt_rcvd = 0;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#502: FILE: lib/vhost/virtio_net.c:1707:
+        count--;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#502: FILE: lib/vhost/virtio_net.c:1707:
+        count--;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#503: FILE: lib/vhost/virtio_net.c:1708:
+        read_idx++;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#503: FILE: lib/vhost/virtio_net.c:1708:
+        read_idx++;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#504: FILE: lib/vhost/virtio_net.c:1709:
+    }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#505: FILE: lib/vhost/virtio_net.c:1710:
+    dma_compl->count = count;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#506: FILE: lib/vhost/virtio_net.c:1711:
+    dma_compl->read_idx = read_idx;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#507: FILE: lib/vhost/virtio_net.c:1712:
+    return pkts;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#513: FILE: lib/vhost/virtio_net.c:1718:
+                     struct dma_completions_t *compl,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#513: FILE: lib/vhost/virtio_net.c:1718:
+                     struct dma_completions_t *compl,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#514: FILE: lib/vhost/virtio_net.c:1719:
+                     struct rte_vhost_async_desc *descs,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#514: FILE: lib/vhost/virtio_net.c:1719:
+                     struct rte_vhost_async_desc *descs,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#515: FILE: lib/vhost/virtio_net.c:1720:
+                     uint16_t count)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#515: FILE: lib/vhost/virtio_net.c:1720:
+                     uint16_t count)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#517: FILE: lib/vhost/virtio_net.c:1722:
+    uint16_t desc_idx = 0;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#518: FILE: lib/vhost/virtio_net.c:1723:
+    struct enq_info_t *slot_addr = NULL;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#520: FILE: lib/vhost/virtio_net.c:1725:
+    if (is_compl_ring_full(compl)) {$

WARNING:BRACES: braces {} are not necessary for single statement blocks
#520: FILE: lib/vhost/virtio_net.c:1725:
+    if (is_compl_ring_full(compl)) {
+        goto out;
+    }

ERROR:CODE_INDENT: code indent should use tabs where possible
#521: FILE: lib/vhost/virtio_net.c:1726:
+        goto out;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#521: FILE: lib/vhost/virtio_net.c:1726:
+        goto out;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#522: FILE: lib/vhost/virtio_net.c:1727:
+    }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#525: FILE: lib/vhost/virtio_net.c:1730:
+    uint16_t dmadev_space_left = rte_dmadev_burst_capacity(dev_id);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#526: FILE: lib/vhost/virtio_net.c:1731:
+    const int compl_space_left = DMA_COMPLETION_RING_SIZE - compl->count;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#527: FILE: lib/vhost/virtio_net.c:1732:
+    if (count > compl_space_left) {$

WARNING:BRACES: braces {} are not necessary for single statement blocks
#527: FILE: lib/vhost/virtio_net.c:1732:
+    if (count > compl_space_left) {
+        count = compl_space_left;
+    }

ERROR:CODE_INDENT: code indent should use tabs where possible
#528: FILE: lib/vhost/virtio_net.c:1733:
+        count = compl_space_left;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#528: FILE: lib/vhost/virtio_net.c:1733:
+        count = compl_space_left;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#529: FILE: lib/vhost/virtio_net.c:1734:
+    }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#531: FILE: lib/vhost/virtio_net.c:1736:
+    while (desc_idx < count) {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#532: FILE: lib/vhost/virtio_net.c:1737:
+        const struct rte_vhost_iov_iter *src_ptr = descs[desc_idx].src;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#532: FILE: lib/vhost/virtio_net.c:1737:
+        const struct rte_vhost_iov_iter *src_ptr = descs[desc_idx].src;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#533: FILE: lib/vhost/virtio_net.c:1738:
+        const struct rte_vhost_iov_iter *dst_ptr = descs[desc_idx].dst;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#533: FILE: lib/vhost/virtio_net.c:1738:
+        const struct rte_vhost_iov_iter *dst_ptr = descs[desc_idx].dst;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#534: FILE: lib/vhost/virtio_net.c:1739:
+        const uint16_t nr_segs = src_ptr->nr_segs;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#534: FILE: lib/vhost/virtio_net.c:1739:
+        const uint16_t nr_segs = src_ptr->nr_segs;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#535: FILE: lib/vhost/virtio_net.c:1740:
+        if (dmadev_space_left < nr_segs) {$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#535: FILE: lib/vhost/virtio_net.c:1740:
+        if (dmadev_space_left < nr_segs) {$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12)
#535: FILE: lib/vhost/virtio_net.c:1740:
+        if (dmadev_space_left < nr_segs) {
+            goto ring_doorbell;

WARNING:BRACES: braces {} are not necessary for single statement blocks
#535: FILE: lib/vhost/virtio_net.c:1740:
+        if (dmadev_space_left < nr_segs) {
+            goto ring_doorbell;
+        }

ERROR:CODE_INDENT: code indent should use tabs where possible
#536: FILE: lib/vhost/virtio_net.c:1741:
+            goto ring_doorbell;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#536: FILE: lib/vhost/virtio_net.c:1741:
+            goto ring_doorbell;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#537: FILE: lib/vhost/virtio_net.c:1742:
+        }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#537: FILE: lib/vhost/virtio_net.c:1742:
+        }$

ERROR:CODE_INDENT: code indent should use tabs where possible
#538: FILE: lib/vhost/virtio_net.c:1743:
+        slot_addr = compl_slot_get_and_inc(compl);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#538: FILE: lib/vhost/virtio_net.c:1743:
+        slot_addr = compl_slot_get_and_inc(compl);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#539: FILE: lib/vhost/virtio_net.c:1744:
+        dmadev_enqueue_packet(dev_id, src_ptr, dst_ptr, nr_segs, slot_addr);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#539: FILE: lib/vhost/virtio_net.c:1744:
+        dmadev_enqueue_packet(dev_id, src_ptr, dst_ptr, nr_segs, slot_addr);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#540: FILE: lib/vhost/virtio_net.c:1745:
+        dmadev_space_left -= nr_segs;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#540: FILE: lib/vhost/virtio_net.c:1745:
+        dmadev_space_left -= nr_segs;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#541: FILE: lib/vhost/virtio_net.c:1746:
+        desc_idx++;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#541: FILE: lib/vhost/virtio_net.c:1746:
+        desc_idx++;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#542: FILE: lib/vhost/virtio_net.c:1747:
+    }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#545: FILE: lib/vhost/virtio_net.c:1750:
+    if (desc_idx != 0) {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#546: FILE: lib/vhost/virtio_net.c:1751:
+        /* Ring the doorbell. */$

ERROR:CODE_INDENT: code indent should use tabs where possible
#547: FILE: lib/vhost/virtio_net.c:1752:
+        rte_dmadev_submit(dev_id, 0);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#547: FILE: lib/vhost/virtio_net.c:1752:
+        rte_dmadev_submit(dev_id, 0);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#548: FILE: lib/vhost/virtio_net.c:1753:
+    }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#551: FILE: lib/vhost/virtio_net.c:1756:
+    while (desc_idx < count) {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#552: FILE: lib/vhost/virtio_net.c:1757:
+        const struct rte_vhost_iov_iter *src_ptr = descs[desc_idx].src;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#552: FILE: lib/vhost/virtio_net.c:1757:
+        const struct rte_vhost_iov_iter *src_ptr = descs[desc_idx].src;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#553: FILE: lib/vhost/virtio_net.c:1758:
+        const struct rte_vhost_iov_iter *dst_ptr = descs[desc_idx].dst;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#553: FILE: lib/vhost/virtio_net.c:1758:
+        const struct rte_vhost_iov_iter *dst_ptr = descs[desc_idx].dst;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#554: FILE: lib/vhost/virtio_net.c:1759:
+        slot_addr = compl_slot_get_and_inc(compl);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#554: FILE: lib/vhost/virtio_net.c:1759:
+        slot_addr = compl_slot_get_and_inc(compl);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#555: FILE: lib/vhost/virtio_net.c:1760:
+        sw_enqueue_packet(src_ptr, dst_ptr, src_ptr->nr_segs);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#555: FILE: lib/vhost/virtio_net.c:1760:
+        sw_enqueue_packet(src_ptr, dst_ptr, src_ptr->nr_segs);$

ERROR:CODE_INDENT: code indent should use tabs where possible
#556: FILE: lib/vhost/virtio_net.c:1761:
+        slot_addr->pkt_rcvd = 1;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#556: FILE: lib/vhost/virtio_net.c:1761:
+        slot_addr->pkt_rcvd = 1;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#557: FILE: lib/vhost/virtio_net.c:1762:
+        desc_idx++;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#557: FILE: lib/vhost/virtio_net.c:1762:
+        desc_idx++;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#558: FILE: lib/vhost/virtio_net.c:1763:
+    }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#561: FILE: lib/vhost/virtio_net.c:1766:
+    return desc_idx;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#567: FILE: lib/vhost/virtio_net.c:1772:
+                              struct dma_completions_t *compl,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#567: FILE: lib/vhost/virtio_net.c:1772:
+                              struct dma_completions_t *compl,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#568: FILE: lib/vhost/virtio_net.c:1773:
+                              uint16_t max_pkts)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#568: FILE: lib/vhost/virtio_net.c:1773:
+                              uint16_t max_pkts)$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#570: FILE: lib/vhost/virtio_net.c:1775:
+    bool error;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#571: FILE: lib/vhost/virtio_net.c:1776:
+    uint16_t last_idx;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#572: FILE: lib/vhost/virtio_net.c:1777:
+    uint32_t nr_pkts = 0;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#573: FILE: lib/vhost/virtio_net.c:1778:
+    struct enq_info_t *slots;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#574: FILE: lib/vhost/virtio_net.c:1779:
+    const uint16_t mask = VHOST_ASYNC_DMADEV_RING_SIZE-1;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#576: FILE: lib/vhost/virtio_net.c:1781:
+    if (unlikely(is_compl_ring_empty(compl))) {$

WARNING:BRACES: braces {} are not necessary for single statement blocks
#576: FILE: lib/vhost/virtio_net.c:1781:
+    if (unlikely(is_compl_ring_empty(compl))) {
+        goto out;
+    }

ERROR:CODE_INDENT: code indent should use tabs where possible
#577: FILE: lib/vhost/virtio_net.c:1782:
+        goto out;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#577: FILE: lib/vhost/virtio_net.c:1782:
+        goto out;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#578: FILE: lib/vhost/virtio_net.c:1783:
+    }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#581: FILE: lib/vhost/virtio_net.c:1786:
+    const int ret_segs = rte_dmadev_completed(dev_id,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#582: FILE: lib/vhost/virtio_net.c:1787:
+                                              0,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#582: FILE: lib/vhost/virtio_net.c:1787:
+                                              0,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#583: FILE: lib/vhost/virtio_net.c:1788:
+                                              MAX_PKT_BURST,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#583: FILE: lib/vhost/virtio_net.c:1788:
+                                              MAX_PKT_BURST,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#584: FILE: lib/vhost/virtio_net.c:1789:
+                                              &last_idx,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#584: FILE: lib/vhost/virtio_net.c:1789:
+                                              &last_idx,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#585: FILE: lib/vhost/virtio_net.c:1790:
+                                              &error);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#585: FILE: lib/vhost/virtio_net.c:1790:
+                                              &error);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#586: FILE: lib/vhost/virtio_net.c:1791:
+    if (unlikely(error)) {$

WARNING:BRACES: braces {} are not necessary for single statement blocks
#586: FILE: lib/vhost/virtio_net.c:1791:
+    if (unlikely(error)) {
+        return -1;
+    }

ERROR:CODE_INDENT: code indent should use tabs where possible
#587: FILE: lib/vhost/virtio_net.c:1792:
+        return -1;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#587: FILE: lib/vhost/virtio_net.c:1792:
+        return -1;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#588: FILE: lib/vhost/virtio_net.c:1793:
+    }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#590: FILE: lib/vhost/virtio_net.c:1795:
+    uint16_t idx = (last_idx - ret_segs + 1);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#591: FILE: lib/vhost/virtio_net.c:1796:
+    for (int i = 0; i < ret_segs; i++) {$

ERROR:CODE_INDENT: code indent should use tabs where possible
#592: FILE: lib/vhost/virtio_net.c:1797:
+        slots = (struct enq_info_t* )dmadev_enq_track[dev_id][idx & mask];$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#592: FILE: lib/vhost/virtio_net.c:1797:
+        slots = (struct enq_info_t* )dmadev_enq_track[dev_id][idx & mask];$

ERROR:POINTER_LOCATION: "(foo* )" should be "(foo *)"
#592: FILE: lib/vhost/virtio_net.c:1797:
+        slots = (struct enq_info_t* )dmadev_enq_track[dev_id][idx & mask];

ERROR:SPACING: space prohibited before that close parenthesis ')'
#592: FILE: lib/vhost/virtio_net.c:1797:
+        slots = (struct enq_info_t* )dmadev_enq_track[dev_id][idx & mask];

ERROR:CODE_INDENT: code indent should use tabs where possible
#593: FILE: lib/vhost/virtio_net.c:1798:
+        if (slots) {$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#593: FILE: lib/vhost/virtio_net.c:1798:
+        if (slots) {$

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12)
#593: FILE: lib/vhost/virtio_net.c:1798:
+        if (slots) {
[...]
+            slots->pkt_rcvd = 1;

WARNING:TYPO_SPELLING: 'recieved' may be misspelled - perhaps 'received'?
#594: FILE: lib/vhost/virtio_net.c:1799:
+            /* Mark the packet slot as recieved.

ERROR:CODE_INDENT: code indent should use tabs where possible
#594: FILE: lib/vhost/virtio_net.c:1799:
+            /* Mark the packet slot as recieved.$

ERROR:CODE_INDENT: code indent should use tabs where possible
#595: FILE: lib/vhost/virtio_net.c:1800:
+             * The slot could belong to another queue but writes are atomic. */$

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#595: FILE: lib/vhost/virtio_net.c:1800:
+             * The slot could belong to another queue but writes are atomic. */

ERROR:CODE_INDENT: code indent should use tabs where possible
#596: FILE: lib/vhost/virtio_net.c:1801:
+            slots->pkt_rcvd = 1;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#596: FILE: lib/vhost/virtio_net.c:1801:
+            slots->pkt_rcvd = 1;$

ERROR:CODE_INDENT: code indent should use tabs where possible
#597: FILE: lib/vhost/virtio_net.c:1802:
+        }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#597: FILE: lib/vhost/virtio_net.c:1802:
+        }$

ERROR:CODE_INDENT: code indent should use tabs where possible
#598: FILE: lib/vhost/virtio_net.c:1803:
+        idx++;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#598: FILE: lib/vhost/virtio_net.c:1803:
+        idx++;$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#599: FILE: lib/vhost/virtio_net.c:1804:
+    }$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#601: FILE: lib/vhost/virtio_net.c:1806:
+    nr_pkts = count_completed_packets(compl, max_pkts);$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#604: FILE: lib/vhost/virtio_net.c:1809:
+    return nr_pkts;$

total: 91 errors, 156 warnings, 762 lines checked
Warning in lib/vhost/virtio_net.c:
Declaring a variable inside for()

           reply	other threads:[~2021-08-23  9:55 UTC|newest]

Thread overview: expand[flat|nested]  mbox.gz  Atom feed
 [parent not found: <20210823095355.2478423-2-sunil.pai.g@intel.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=20210823095549.1A3501226B3@dpdk.org \
    --to=checkpatch@dpdk.org \
    --cc=sunil.pai.g@intel.com \
    --cc=test-report@dpdk.org \
    /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).