From patchwork Mon Nov 1 17:44:47 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Harman Kalra X-Patchwork-Id: 103357 X-Patchwork-Delegate: thomas@monjalon.net Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 71BDBA0548; Mon, 1 Nov 2021 18:45:06 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 5BBD440E28; Mon, 1 Nov 2021 18:45:06 +0100 (CET) Received: from mx0b-0016f401.pphosted.com (mx0b-0016f401.pphosted.com [67.231.156.173]) by mails.dpdk.org (Postfix) with ESMTP id 0CDF440DF6 for ; Mon, 1 Nov 2021 18:45:04 +0100 (CET) Received: from pps.filterd (m0045851.ppops.net [127.0.0.1]) by mx0b-0016f401.pphosted.com (8.16.1.2/8.16.1.2) with ESMTP id 1A1CDdmx001047 for ; Mon, 1 Nov 2021 10:45:04 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h=from : to : subject : date : message-id : mime-version : content-type; s=pfpt0220; bh=xy9JE7hnh28Ur/LuGc11LLwbG4KzrTEYq+il6G2QzlA=; b=c/s+oL4IgOP5WfrMqUeBWaQxAGRCLusGVWgwMkzNnzkFQrtlFaRa2avbull+cfKPuEPU 8Z+lU/kD2nPgvShySOwKuJJyLrw7d9C8F9jTH3Ac4Fo1aLXqEJgxHyzfcUnVwWOElmxc tQrGBpoa6qkZPHQOSmE4f+0VzQzHCHZtTgsCB4wiqV/Z8m+Nt8qxkg6d2+wJ+/VJxPr4 I3jscrw0aSlmYvQDkwmKCuQ+vW75genD37S91x0+sxuv7AkC9sp/XrqXAjZ7wPQNCVyI edZNUOQv3JoZ5SxrAFqdUq8or8M4kqaDY6GENGrD8ean3Fr5TJK6xn/kd4t/1plfGYAZ fg== Received: from dc5-exch01.marvell.com ([199.233.59.181]) by mx0b-0016f401.pphosted.com (PPS) with ESMTPS id 3c25c4br45-2 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT) for ; Mon, 01 Nov 2021 10:45:04 -0700 Received: from DC5-EXCH02.marvell.com (10.69.176.39) by DC5-EXCH01.marvell.com (10.69.176.38) with Microsoft SMTP Server (TLS) id 15.0.1497.18; Mon, 1 Nov 2021 10:45:02 -0700 Received: from maili.marvell.com (10.69.176.80) by DC5-EXCH02.marvell.com (10.69.176.39) with Microsoft SMTP Server id 15.0.1497.18 via Frontend Transport; Mon, 1 Nov 2021 10:45:02 -0700 Received: from localhost.marvell.com (unknown [10.29.52.211]) by maili.marvell.com (Postfix) with ESMTP id 13A943F70B3; Mon, 1 Nov 2021 10:45:00 -0700 (PDT) From: Harman Kalra To: , Harman Kalra Date: Mon, 1 Nov 2021 23:14:47 +0530 Message-ID: <20211101174447.82017-1-hkalra@marvell.com> X-Mailer: git-send-email 2.18.0 MIME-Version: 1.0 X-Proofpoint-ORIG-GUID: tX9I0K3brmGT8yLvlbtxGJnVu-V1ZvJV X-Proofpoint-GUID: tX9I0K3brmGT8yLvlbtxGJnVu-V1ZvJV X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.182.1,Aquarius:18.0.790,Hydra:6.0.425,FMLib:17.0.607.475 definitions=2021-11-01_06,2021-11-01_01,2020-04-07_01 Subject: [dpdk-dev] [PATCH] interrupts: fix error log level X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Fixing the error logs level, as currently default level is set to debug. Due to this failure is not getting captured. Fixes: b7c984291611 ("interrupts: add allocator and accessors") Signed-off-by: Harman Kalra Acked-by: Stephen Hemminger --- lib/eal/common/eal_common_interrupts.c | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/lib/eal/common/eal_common_interrupts.c b/lib/eal/common/eal_common_interrupts.c index 97b64fed58..96c7e9e40e 100644 --- a/lib/eal/common/eal_common_interrupts.c +++ b/lib/eal/common/eal_common_interrupts.c @@ -15,7 +15,7 @@ /* Macros to check for valid interrupt handle */ #define CHECK_VALID_INTR_HANDLE(intr_handle) do { \ if (intr_handle == NULL) { \ - RTE_LOG(DEBUG, EAL, "Interrupt instance unallocated\n"); \ + RTE_LOG(ERR, EAL, "Interrupt instance unallocated\n"); \ rte_errno = EINVAL; \ goto fail; \ } \ @@ -37,7 +37,7 @@ struct rte_intr_handle *rte_intr_instance_alloc(uint32_t flags) * defined flags. */ if ((flags & ~RTE_INTR_INSTANCE_KNOWN_FLAGS) != 0) { - RTE_LOG(DEBUG, EAL, "Invalid alloc flag passed 0x%x\n", flags); + RTE_LOG(ERR, EAL, "Invalid alloc flag passed 0x%x\n", flags); rte_errno = EINVAL; return NULL; } @@ -100,7 +100,7 @@ struct rte_intr_handle *rte_intr_instance_dup(const struct rte_intr_handle *src) struct rte_intr_handle *intr_handle; if (src == NULL) { - RTE_LOG(DEBUG, EAL, "Source interrupt instance unallocated\n"); + RTE_LOG(ERR, EAL, "Source interrupt instance unallocated\n"); rte_errno = EINVAL; return NULL; } @@ -129,7 +129,7 @@ int rte_intr_event_list_update(struct rte_intr_handle *intr_handle, int size) CHECK_VALID_INTR_HANDLE(intr_handle); if (size == 0) { - RTE_LOG(DEBUG, EAL, "Size can't be zero\n"); + RTE_LOG(ERR, EAL, "Size can't be zero\n"); rte_errno = EINVAL; goto fail; } @@ -253,7 +253,7 @@ int rte_intr_max_intr_set(struct rte_intr_handle *intr_handle, CHECK_VALID_INTR_HANDLE(intr_handle); if (max_intr > intr_handle->nb_intr) { - RTE_LOG(DEBUG, EAL, "Maximum interrupt vector ID (%d) exceeds " + RTE_LOG(ERR, EAL, "Maximum interrupt vector ID (%d) exceeds " "the number of available events (%d)\n", max_intr, intr_handle->nb_intr); rte_errno = ERANGE; @@ -332,7 +332,7 @@ int rte_intr_efds_index_get(const struct rte_intr_handle *intr_handle, CHECK_VALID_INTR_HANDLE(intr_handle); if (index >= intr_handle->nb_intr) { - RTE_LOG(DEBUG, EAL, "Invalid index %d, max limit %d\n", index, + RTE_LOG(ERR, EAL, "Invalid index %d, max limit %d\n", index, intr_handle->nb_intr); rte_errno = EINVAL; goto fail; @@ -349,7 +349,7 @@ int rte_intr_efds_index_set(struct rte_intr_handle *intr_handle, CHECK_VALID_INTR_HANDLE(intr_handle); if (index >= intr_handle->nb_intr) { - RTE_LOG(DEBUG, EAL, "Invalid index %d, max limit %d\n", index, + RTE_LOG(ERR, EAL, "Invalid index %d, max limit %d\n", index, intr_handle->nb_intr); rte_errno = ERANGE; goto fail; @@ -368,7 +368,7 @@ struct rte_epoll_event *rte_intr_elist_index_get( CHECK_VALID_INTR_HANDLE(intr_handle); if (index >= intr_handle->nb_intr) { - RTE_LOG(DEBUG, EAL, "Invalid index %d, max limit %d\n", index, + RTE_LOG(ERR, EAL, "Invalid index %d, max limit %d\n", index, intr_handle->nb_intr); rte_errno = ERANGE; goto fail; @@ -385,7 +385,7 @@ int rte_intr_elist_index_set(struct rte_intr_handle *intr_handle, CHECK_VALID_INTR_HANDLE(intr_handle); if (index >= intr_handle->nb_intr) { - RTE_LOG(DEBUG, EAL, "Invalid index %d, max limit %d\n", index, + RTE_LOG(ERR, EAL, "Invalid index %d, max limit %d\n", index, intr_handle->nb_intr); rte_errno = ERANGE; goto fail; @@ -408,7 +408,7 @@ int rte_intr_vec_list_alloc(struct rte_intr_handle *intr_handle, return 0; if (size > intr_handle->nb_intr) { - RTE_LOG(DEBUG, EAL, "Invalid size %d, max limit %d\n", size, + RTE_LOG(ERR, EAL, "Invalid size %d, max limit %d\n", size, intr_handle->nb_intr); rte_errno = ERANGE; goto fail; @@ -437,7 +437,7 @@ int rte_intr_vec_list_index_get(const struct rte_intr_handle *intr_handle, CHECK_VALID_INTR_HANDLE(intr_handle); if (index >= intr_handle->vec_list_size) { - RTE_LOG(DEBUG, EAL, "Index %d greater than vec list size %d\n", + RTE_LOG(ERR, EAL, "Index %d greater than vec list size %d\n", index, intr_handle->vec_list_size); rte_errno = ERANGE; goto fail; @@ -454,7 +454,7 @@ int rte_intr_vec_list_index_set(struct rte_intr_handle *intr_handle, CHECK_VALID_INTR_HANDLE(intr_handle); if (index >= intr_handle->vec_list_size) { - RTE_LOG(DEBUG, EAL, "Index %d greater than vec list size %d\n", + RTE_LOG(ERR, EAL, "Index %d greater than vec list size %d\n", index, intr_handle->vec_list_size); rte_errno = ERANGE; goto fail;