From: checkpatch@dpdk.org
To: test-report@dpdk.org
Cc: Ziyang Xuan <xuanziyang2@huawei.com>
Subject: [dpdk-test-report] |WARNING| pw55447 [PATCH v6 09/15] net/hinic/base: add context and work queue support
Date: Thu, 27 Jun 2019 10:05:47 +0200 (CEST) [thread overview]
Message-ID: <20190627080547.7079337B0@dpdk.org> (raw)
In-Reply-To: <2f43f8cfa40cb2d673648543424c1fd3644ea2f1.1561620219.git.xuanziyang2@huawei.com>
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/55447
_coding style issues_
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'max_sqs' - possible side-effects?
#973: FILE: drivers/net/hinic/base/hinic_pmd_nicio.h:32:
+#define RQ_CTXT_OFFSET(max_sqs, max_rqs, q_id) \
+ (((max_rqs) + (max_sqs)) * TSO_LRO_CTXT_SIZE + \
+ (max_sqs) * Q_CTXT_SIZE + (q_id) * Q_CTXT_SIZE)
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects?
#1407: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:10:
+#define WQ_SIZE(wq) (u32)((u64)(wq)->q_depth * (wq)->wqebb_size)
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects?
#1409: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:12:
+#define WQE_PAGE_NUM(wq, idx) (((idx) >> ((wq)->wqebbs_per_page_shift)) & \
+ ((wq)->num_q_pages - 1))
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects?
#1412: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:15:
+#define WQE_PAGE_OFF(wq, idx) ((u64)((wq)->wqebb_size) * \
+ ((idx) & ((wq)->num_wqebbs_per_page - 1)))
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects?
#1417: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:20:
+#define WQ_PAGE_ADDR(wq, idx) \
+ (u8 *)(*(u64 *)((u64)((wq)->shadow_block_vaddr) + \
+ (WQE_PAGE_NUM(wq, idx) << WQ_PAGE_ADDR_SIZE_SHIFT)))
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects?
#1433: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:36:
+#define WQ_BASE_VADDR(wqs, wq) \
+ (u64 *)(((u64)((wqs)->page_vaddr[(wq)->page_idx])) \
+ + (wq)->block_idx * WQ_BLOCK_SIZE)
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects?
#1437: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:40:
+#define WQ_BASE_PADDR(wqs, wq) (((wqs)->page_paddr[(wq)->page_idx]) \
+ + (u64)(wq)->block_idx * WQ_BLOCK_SIZE)
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects?
#1440: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:43:
+#define WQ_BASE_ADDR(wqs, wq) \
+ (u64 *)(((u64)((wqs)->shadow_page_vaddr[(wq)->page_idx])) \
+ + (wq)->block_idx * WQ_BLOCK_SIZE)
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects?
#1458: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:61:
+#define WQE_SHADOW_PAGE(wq, wqe) \
+ (u16)(((unsigned long)(wqe) - (unsigned long)(wq)->shadow_wqe) \
+ / (wq)->max_wqe_size)
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wqe' - possible side-effects?
#1462: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:65:
+#define WQE_IN_RANGE(wqe, start, end) \
+ (((unsigned long)(wqe) >= (unsigned long)(start)) && \
+ ((unsigned long)(wqe) < (unsigned long)(end)))
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'wq' - possible side-effects?
#1469: FILE: drivers/net/hinic/base/hinic_pmd_wq.h:72:
+#define WQ_WQE_ADDR(wq, idx) ((void *)((u64)((wq)->queue_buf_vaddr) + \
+ ((idx) << (wq)->wqebb_shift)))
total: 0 errors, 0 warnings, 11 checks, 1475 lines checked
parent reply other threads:[~2019-06-27 8:05 UTC|newest]
Thread overview: expand[flat|nested] mbox.gz Atom feed
[parent not found: <2f43f8cfa40cb2d673648543424c1fd3644ea2f1.1561620219.git.xuanziyang2@huawei.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=20190627080547.7079337B0@dpdk.org \
--to=checkpatch@dpdk.org \
--cc=test-report@dpdk.org \
--cc=xuanziyang2@huawei.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).