From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id CECECA0A0C for ; Thu, 15 Apr 2021 06:57:24 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id CAD27161F67; Thu, 15 Apr 2021 06:57:24 +0200 (CEST) Received: from dpdk.org (dpdk.org [92.243.24.197]) by mails.dpdk.org (Postfix) with ESMTP id 10E0E161F53 for ; Thu, 15 Apr 2021 06:57:24 +0200 (CEST) Received: by dpdk.org (Postfix, from userid 65534) id 05CA71209EC; Thu, 15 Apr 2021 06:57:24 +0200 (CEST) In-Reply-To: <20210415045431.2082256-2-lizh@nvidia.com> References: <20210415045431.2082256-2-lizh@nvidia.com> To: test-report@dpdk.org From: checkpatch@dpdk.org Cc: Li Zhang Message-Id: <20210415045724.05CA71209EC@dpdk.org> Date: Thu, 15 Apr 2021 06:57:24 +0200 (CEST) Subject: [dpdk-test-report] |WARNING| pw91522 [PATCH v7 1/2] ethdev: add pre-defined meter policy API X-BeenThere: test-report@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: automatic DPDK test reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: test-report-bounces@dpdk.org Sender: "test-report" Test-Label: checkpatch Test-Status: WARNING http://dpdk.org/patch/91522 _coding style issues_ WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon #2543: FILE: lib/librte_ethdev/rte_mtr.h:520: +#define rte_mtr_policy_pass_color(policy) \ +struct rte_mtr_meter_policy_params policy = \ +{ \ + .actions[RTE_COLOR_GREEN] = (struct rte_flow_action[]) { \ + { \ + .type = RTE_FLOW_ACTION_TYPE_METER_COLOR, \ + .conf = &(struct rte_flow_action_meter_color) { \ + .color = RTE_COLOR_GREEN, \ + }, \ + }, \ + { \ + .type = RTE_FLOW_ACTION_TYPE_END, \ + }, \ + }, \ + .actions[RTE_COLOR_YELLOW] = (struct rte_flow_action[]) { \ + { \ + .type = RTE_FLOW_ACTION_TYPE_METER_COLOR, \ + .conf = &(struct rte_flow_action_meter_color) { \ + .color = RTE_COLOR_YELLOW, \ + }, \ + }, \ + { \ + .type = RTE_FLOW_ACTION_TYPE_END, \ + }, \ + }, \ + .actions[RTE_COLOR_RED] = (struct rte_flow_action[]) { \ + { \ + .type = RTE_FLOW_ACTION_TYPE_METER_COLOR, \ + .conf = &(struct rte_flow_action_meter_color) { \ + .color = RTE_COLOR_RED, \ + }, \ + }, \ + { \ + .type = RTE_FLOW_ACTION_TYPE_END, \ + }, \ + }, \ +}; WARNING:TRAILING_SEMICOLON: macros should not use a trailing semicolon #2585: FILE: lib/librte_ethdev/rte_mtr.h:562: +#define rte_mtr_policy_drop_red(policy) \ +struct rte_mtr_meter_policy_params policy = \ +{ \ + .actions[RTE_COLOR_GREEN] = NULL, \ + .actions[RTE_COLOR_YELLOW] = NULL, \ + .actions[RTE_COLOR_RED] = (struct rte_flow_action[]) { \ + { \ + .type = RTE_FLOW_ACTION_TYPE_DROP, \ + }, \ + { \ + .type = RTE_FLOW_ACTION_TYPE_END, \ + }, \ + }, \ +}; total: 0 errors, 2 warnings, 2489 lines checked