mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/chenhuacai/linux-loongson
synced 2025-09-06 13:09:58 +00:00
74ce564a30
The ice_debug_cq() function is called to generate a debug log of control queue messages both sent and received. It currently does this over a potential total of 6 different printk invocations. The main logic prints over 4 calls to ice_debug(): 1. The metadata including opcode, flags, datalength and return value. 2. The cookie in the descriptor. 3. The parameter values. 4. The address for the databuffer. In addition, if the descriptor has a data buffer, it can be logged with two additional prints: 5. A message indicating the start of the data buffer. 6. The actual data buffer, printed using print_hex_dump_debug. This can lead to trouble in the event that two different PFs are logging messages. The messages become intermixed and it may not be possible to determine which part of the output belongs to which control queue message. To fix this, it needs to be possible to unambiguously determine which messages belong together. This is trivial for the messages that comprise the main printing. Combine them together into a single invocation of ice_debug(). The message containing a hex-dump of the data buffer is a bit more complicated. This is printed separately as part of print_hex_dump_debug. This function takes a prefix, which is currently always set to KBUILD_MODNAME. Extend this prefix to include the buffer address for the databuffer, which is printed as part of the main print, and which is guaranteed to be unique for each buffer. Refactor the ice_debug_array(), introducing an ice_debug_array_w_prefix(). Build the prefix by combining KBUILD_MODNAME with the databuffer address using snprintf(). These changes make it possible to unambiguously determine what data belongs to what control queue message. Reported-by: Jacek Wierzbicki <jacek.wierzbicki@intel.com> Signed-off-by: Przemek Kitszel <przemyslaw.kitszel@intel.com> Signed-off-by: Jacob Keller <jacob.e.keller@intel.com> Tested-by: Pucha Himasekhar Reddy <himasekharx.reddy.pucha@intel.com> (A Contingent worker at Intel) Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
90 lines
2.6 KiB
C
90 lines
2.6 KiB
C
/* SPDX-License-Identifier: GPL-2.0 */
|
|
/* Copyright (c) 2018, Intel Corporation. */
|
|
|
|
#ifndef _ICE_OSDEP_H_
|
|
#define _ICE_OSDEP_H_
|
|
|
|
#include <linux/types.h>
|
|
#include <linux/ctype.h>
|
|
#include <linux/delay.h>
|
|
#include <linux/io.h>
|
|
#include <linux/bitops.h>
|
|
#include <linux/ethtool.h>
|
|
#include <linux/etherdevice.h>
|
|
#include <linux/if_ether.h>
|
|
#include <linux/iopoll.h>
|
|
#include <linux/pci_ids.h>
|
|
#ifndef CONFIG_64BIT
|
|
#include <linux/io-64-nonatomic-lo-hi.h>
|
|
#endif
|
|
#include <net/udp_tunnel.h>
|
|
|
|
#define wr32(a, reg, value) writel((value), ((a)->hw_addr + (reg)))
|
|
#define rd32(a, reg) readl((a)->hw_addr + (reg))
|
|
#define wr64(a, reg, value) writeq((value), ((a)->hw_addr + (reg)))
|
|
#define rd64(a, reg) readq((a)->hw_addr + (reg))
|
|
|
|
#define rd32_poll_timeout(a, addr, val, cond, delay_us, timeout_us) \
|
|
read_poll_timeout(rd32, val, cond, delay_us, timeout_us, false, a, addr)
|
|
|
|
#define ice_flush(a) rd32((a), GLGEN_STAT)
|
|
#define ICE_M(m, s) ((m ## U) << (s))
|
|
|
|
struct ice_dma_mem {
|
|
void *va;
|
|
dma_addr_t pa;
|
|
size_t size;
|
|
};
|
|
|
|
struct ice_hw;
|
|
struct device *ice_hw_to_dev(struct ice_hw *hw);
|
|
|
|
#ifdef CONFIG_DYNAMIC_DEBUG
|
|
#define ice_debug(hw, type, fmt, args...) \
|
|
dev_dbg(ice_hw_to_dev(hw), fmt, ##args)
|
|
|
|
#define _ice_debug_array(hw, type, prefix, rowsize, groupsize, buf, len) \
|
|
print_hex_dump_debug(prefix, DUMP_PREFIX_OFFSET, \
|
|
rowsize, groupsize, buf, len, false)
|
|
#else /* CONFIG_DYNAMIC_DEBUG */
|
|
#define ice_debug(hw, type, fmt, args...) \
|
|
do { \
|
|
if ((type) & (hw)->debug_mask) \
|
|
dev_info(ice_hw_to_dev(hw), fmt, ##args); \
|
|
} while (0)
|
|
|
|
#ifdef DEBUG
|
|
#define _ice_debug_array(hw, type, prefix, rowsize, groupsize, buf, len) \
|
|
do { \
|
|
if ((type) & (hw)->debug_mask) \
|
|
print_hex_dump_debug(prefix, DUMP_PREFIX_OFFSET,\
|
|
rowsize, groupsize, buf, \
|
|
len, false); \
|
|
} while (0)
|
|
#else /* DEBUG */
|
|
#define _ice_debug_array(hw, type, prefix, rowsize, groupsize, buf, len) \
|
|
do { \
|
|
struct ice_hw *hw_l = hw; \
|
|
if ((type) & (hw_l)->debug_mask) { \
|
|
u16 len_l = len; \
|
|
u8 *buf_l = buf; \
|
|
int i; \
|
|
for (i = 0; i < (len_l - 16); i += 16) \
|
|
ice_debug(hw_l, type, "0x%04X %16ph\n",\
|
|
i, ((buf_l) + i)); \
|
|
if (i < len_l) \
|
|
ice_debug(hw_l, type, "0x%04X %*ph\n", \
|
|
i, ((len_l) - i), ((buf_l) + i));\
|
|
} \
|
|
} while (0)
|
|
#endif /* DEBUG */
|
|
#endif /* CONFIG_DYNAMIC_DEBUG */
|
|
|
|
#define ice_debug_array(hw, type, rowsize, groupsize, buf, len) \
|
|
_ice_debug_array(hw, type, KBUILD_MODNAME, rowsize, groupsize, buf, len)
|
|
|
|
#define ice_debug_array_w_prefix(hw, type, prefix, buf, len) \
|
|
_ice_debug_array(hw, type, prefix, 16, 1, buf, len)
|
|
|
|
#endif /* _ICE_OSDEP_H_ */
|