From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jianfeng Tan Subject: [PATCH v3 5/5] bus/vdev: normalize log type Date: Mon, 9 Oct 2017 03:20:29 +0000 Message-ID: <1507519229-80692-6-git-send-email-jianfeng.tan@intel.com> References: <1503654052-84730-1-git-send-email-jianfeng.tan@intel.com> <1507519229-80692-1-git-send-email-jianfeng.tan@intel.com> Cc: jblunck@infradead.org, bruce.richardson@intel.com, konstantin.ananyev@intel.com, pablo.de.lara.guarch@intel.com, thomas@monjalon.net, yliu@fridaylinux.org, maxime.coquelin@redhat.com, mtetsuyah@gmail.com, ferruh.yigit@intel.com, Jianfeng Tan To: dev@dpdk.org Return-path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id 85FC51B1B3 for ; Mon, 9 Oct 2017 05:19:15 +0200 (CEST) In-Reply-To: <1507519229-80692-1-git-send-email-jianfeng.tan@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Use RTE_LOGTYPE_BUS for logging in vdev bus. Signed-off-by: Jianfeng Tan --- drivers/bus/vdev/vdev.c | 9 +++++---- drivers/bus/vdev/vdev_logs.h | 40 ++++++++++++++++++++++++++++++++++++++++ 2 files changed, 45 insertions(+), 4 deletions(-) create mode 100644 drivers/bus/vdev/vdev_logs.h diff --git a/drivers/bus/vdev/vdev.c b/drivers/bus/vdev/vdev.c index 8a1b00a..e0ba0da 100644 --- a/drivers/bus/vdev/vdev.c +++ b/drivers/bus/vdev/vdev.c @@ -47,6 +47,7 @@ #include #include "rte_vdev.h" +#include "vdev_logs.h" /* Forward declare to access virtual bus name */ static struct rte_bus rte_vdev_bus; @@ -103,7 +104,7 @@ vdev_probe_all_drivers(struct rte_vdev_device *dev) name = rte_vdev_device_name(dev); - RTE_LOG(DEBUG, EAL, "Search driver %s to probe device %s\n", name, + VDEV_LOG(DEBUG, "Search driver %s to probe device %s\n", name, rte_vdev_device_name(dev)); if (vdev_parse(name, &driver)) @@ -189,7 +190,7 @@ rte_vdev_init(const char *name, const char *args) ret = vdev_probe_all_drivers(dev); if (ret) { if (ret > 0) - RTE_LOG(ERR, EAL, "no driver found for %s\n", name); + VDEV_LOG(ERR, "no driver found for %s\n", name); goto fail; } @@ -212,7 +213,7 @@ vdev_remove_driver(struct rte_vdev_device *dev) const struct rte_vdev_driver *driver; if (!dev->device.driver) { - RTE_LOG(DEBUG, EAL, "no driver attach to device %s\n", name); + VDEV_LOG(DEBUG, "no driver attach to device %s\n", name); return 1; } @@ -293,7 +294,7 @@ vdev_probe(void) continue; if (vdev_probe_all_drivers(dev)) { - RTE_LOG(ERR, EAL, "failed to initialize %s device\n", + VDEV_LOG(ERR, "failed to initialize %s device\n", rte_vdev_device_name(dev)); return -1; } diff --git a/drivers/bus/vdev/vdev_logs.h b/drivers/bus/vdev/vdev_logs.h new file mode 100644 index 0000000..09ff973 --- /dev/null +++ b/drivers/bus/vdev/vdev_logs.h @@ -0,0 +1,40 @@ +/*- + * BSD LICENSE + * + * Copyright(c) 2017 Intel Corporation. All rights reserved. + * All rights reserved. + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted provided that the following conditions + * are met: + * + * * Redistributions of source code must retain the above copyright + * notice, this list of conditions and the following disclaimer. + * * Redistributions in binary form must reproduce the above copyright + * notice, this list of conditions and the following disclaimer in + * the documentation and/or other materials provided with the + * distribution. + * * Neither the name of Intel Corporation nor the names of its + * contributors may be used to endorse or promote products derived + * from this software without specific prior written permission. + * + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR + * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT + * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, + * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT + * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, + * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY + * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT + * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE + * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. + */ + +#ifndef _VDEV_LOGS_H_ +#define _VDEV_LOGS_H_ + +#define VDEV_LOG(level, fmt, args...) \ + RTE_LOG(level, BUS, "%s(): " fmt "\n", __func__, ##args) + +#endif /* _VDEV_LOGS_H_ */ -- 2.7.4