Skip to content

L
linux
Commit 876d2d6f authored by Jeff Kirsher's avatar Jeff Kirsher
Browse files
Options

igb: Convert printks to pr_<level> 

Based on original patch from Joe Perches <joe@perches.com>

Use the current logging styles.

pr_<level> conversions are now prefixed with "igb: "

Correct a defect where the trailing NTU may have been printed
on a separate line because of an interleaving hex_dump.

Remove unnecessary uses of KERN_CONT and use single pr_info()s
to avoid any possible output interleaving from other modules.

Coalesce formats as appropriate.

-v2 fix-up to make checkpatch.pl compliant and remove change to
    the copyright line

CC: Joe Perches <joe@perches.com>
Signed-off-by: default avatarJeff Kirsher <jeffrey.t.kirsher@intel.com>
Tested-by: default avatarAaron Brown <aaron.f.brown@intel.com>
parent dbd9636e
Show whitespace changes
Inline Side-by-side
Showing with 80 additions and 85 deletions
drivers/net/ethernet/intel/igb/igb_main.c
View file @ 876d2d6f
...@@ -25,6 +25,8 @@ ...@@ -25,6 +25,8 @@
*******************************************************************************/ *******************************************************************************/
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
#include <linux/module.h> #include <linux/module.h>
#include <linux/types.h> #include <linux/types.h>
#include <linux/init.h> #include <linux/init.h>
...@@ -325,16 +327,13 @@ static void igb_regdump(struct e1000_hw *hw, struct igb_reg_info *reginfo) ...@@ -325,16 +327,13 @@ static void igb_regdump(struct e1000_hw *hw, struct igb_reg_info *reginfo)
regs[n] = rd32(E1000_TXDCTL(n)); regs[n] = rd32(E1000_TXDCTL(n));
break; break;
default: default:
printk(KERN_INFO "%-15s %08x\n", pr_info("%-15s %08x\n", reginfo->name, rd32(reginfo->ofs));
reginfo->name, rd32(reginfo->ofs));
return; return;
} }
snprintf(rname, 16, "%s%s", reginfo->name, "[0-3]"); snprintf(rname, 16, "%s%s", reginfo->name, "[0-3]");
printk(KERN_INFO "%-15s ", rname); pr_info("%-15s %08x %08x %08x %08x\n", rname, regs[0], regs[1],
for (n = 0; n < 4; n++) regs[2], regs[3]);
printk(KERN_CONT "%08x ", regs[n]);
printk(KERN_CONT "\n");
} }
/* /*
...@@ -359,18 +358,15 @@ static void igb_dump(struct igb_adapter *adapter) ...@@ -359,18 +358,15 @@ static void igb_dump(struct igb_adapter *adapter)
/* Print netdevice Info */ /* Print netdevice Info */
if (netdev) { if (netdev) {
dev_info(&adapter->pdev->dev, "Net device Info\n"); dev_info(&adapter->pdev->dev, "Net device Info\n");
printk(KERN_INFO "Device Name state " pr_info("Device Name state trans_start "
"trans_start last_rx\n"); "last_rx\n");
printk(KERN_INFO "%-15s %016lX %016lX %016lX\n", pr_info("%-15s %016lX %016lX %016lX\n", netdev->name,
netdev->name, netdev->state, netdev->trans_start, netdev->last_rx);
netdev->state,
netdev->trans_start,
netdev->last_rx);
} }
/* Print Registers */ /* Print Registers */
dev_info(&adapter->pdev->dev, "Register Dump\n"); dev_info(&adapter->pdev->dev, "Register Dump\n");
printk(KERN_INFO " Register Name Value\n"); pr_info(" Register Name Value\n");
for (reginfo = (struct igb_reg_info *)igb_reg_info_tbl; for (reginfo = (struct igb_reg_info *)igb_reg_info_tbl;
reginfo->name; reginfo++) { reginfo->name; reginfo++) {
igb_regdump(hw, reginfo); igb_regdump(hw, reginfo);
...@@ -381,13 +377,12 @@ static void igb_dump(struct igb_adapter *adapter) ...@@ -381,13 +377,12 @@ static void igb_dump(struct igb_adapter *adapter)
goto exit; goto exit;
dev_info(&adapter->pdev->dev, "TX Rings Summary\n"); dev_info(&adapter->pdev->dev, "TX Rings Summary\n");
printk(KERN_INFO "Queue [NTU] [NTC] [bi(ntc)->dma ]" pr_info("Queue [NTU] [NTC] [bi(ntc)->dma ] leng ntw timestamp\n");
" leng ntw timestamp\n");
for (n = 0; n < adapter->num_tx_queues; n++) { for (n = 0; n < adapter->num_tx_queues; n++) {
struct igb_tx_buffer *buffer_info; struct igb_tx_buffer *buffer_info;
tx_ring = adapter->tx_ring[n]; tx_ring = adapter->tx_ring[n];
buffer_info = &tx_ring->tx_buffer_info[tx_ring->next_to_clean]; buffer_info = &tx_ring->tx_buffer_info[tx_ring->next_to_clean];
printk(KERN_INFO " %5d %5X %5X %016llX %04X %p %016llX\n", pr_info(" %5d %5X %5X %016llX %04X %p %016llX\n",
n, tx_ring->next_to_use, tx_ring->next_to_clean, n, tx_ring->next_to_use, tx_ring->next_to_clean,
(u64)buffer_info->dma, (u64)buffer_info->dma,
buffer_info->length, buffer_info->length,
...@@ -414,36 +409,38 @@ static void igb_dump(struct igb_adapter *adapter) ...@@ -414,36 +409,38 @@ static void igb_dump(struct igb_adapter *adapter)
for (n = 0; n < adapter->num_tx_queues; n++) { for (n = 0; n < adapter->num_tx_queues; n++) {
tx_ring = adapter->tx_ring[n]; tx_ring = adapter->tx_ring[n];
printk(KERN_INFO "------------------------------------\n"); pr_info("------------------------------------\n");
printk(KERN_INFO "TX QUEUE INDEX = %d\n", tx_ring->queue_index); pr_info("TX QUEUE INDEX = %d\n", tx_ring->queue_index);
printk(KERN_INFO "------------------------------------\n"); pr_info("------------------------------------\n");
printk(KERN_INFO "T [desc] [address 63:0 ] " pr_info("T [desc] [address 63:0 ] [PlPOCIStDDM Ln] "
"[PlPOCIStDDM Ln] [bi->dma ] " "[bi->dma ] leng ntw timestamp "
"leng ntw timestamp bi->skb\n"); "bi->skb\n");
for (i = 0; tx_ring->desc && (i < tx_ring->count); i++) { for (i = 0; tx_ring->desc && (i < tx_ring->count); i++) {
const char *next_desc;
struct igb_tx_buffer *buffer_info; struct igb_tx_buffer *buffer_info;
tx_desc = IGB_TX_DESC(tx_ring, i); tx_desc = IGB_TX_DESC(tx_ring, i);
buffer_info = &tx_ring->tx_buffer_info[i]; buffer_info = &tx_ring->tx_buffer_info[i];
u0 = (struct my_u0 *)tx_desc; u0 = (struct my_u0 *)tx_desc;
printk(KERN_INFO "T [0x%03X] %016llX %016llX %016llX" if (i == tx_ring->next_to_use &&
" %04X %p %016llX %p", i, i == tx_ring->next_to_clean)
next_desc = " NTC/U";
else if (i == tx_ring->next_to_use)
next_desc = " NTU";
else if (i == tx_ring->next_to_clean)
next_desc = " NTC";
else
next_desc = "";
pr_info("T [0x%03X] %016llX %016llX %016llX"
" %04X %p %016llX %p%s\n", i,
le64_to_cpu(u0->a), le64_to_cpu(u0->a),
le64_to_cpu(u0->b), le64_to_cpu(u0->b),
(u64)buffer_info->dma, (u64)buffer_info->dma,
buffer_info->length, buffer_info->length,
buffer_info->next_to_watch, buffer_info->next_to_watch,
(u64)buffer_info->time_stamp, (u64)buffer_info->time_stamp,
buffer_info->skb); buffer_info->skb, next_desc);
if (i == tx_ring->next_to_use &&
i == tx_ring->next_to_clean)
printk(KERN_CONT " NTC/U\n");
else if (i == tx_ring->next_to_use)
printk(KERN_CONT " NTU\n");
else if (i == tx_ring->next_to_clean)
printk(KERN_CONT " NTC\n");
else
printk(KERN_CONT "\n");
if (netif_msg_pktdata(adapter) && buffer_info->dma != 0) if (netif_msg_pktdata(adapter) && buffer_info->dma != 0)
print_hex_dump(KERN_INFO, "", print_hex_dump(KERN_INFO, "",
...@@ -456,11 +453,11 @@ static void igb_dump(struct igb_adapter *adapter) ...@@ -456,11 +453,11 @@ static void igb_dump(struct igb_adapter *adapter)
/* Print RX Rings Summary */ /* Print RX Rings Summary */
rx_ring_summary: rx_ring_summary:
dev_info(&adapter->pdev->dev, "RX Rings Summary\n"); dev_info(&adapter->pdev->dev, "RX Rings Summary\n");
printk(KERN_INFO "Queue [NTU] [NTC]\n"); pr_info("Queue [NTU] [NTC]\n");
for (n = 0; n < adapter->num_rx_queues; n++) { for (n = 0; n < adapter->num_rx_queues; n++) {
rx_ring = adapter->rx_ring[n]; rx_ring = adapter->rx_ring[n];
printk(KERN_INFO " %5d %5X %5X\n", n, pr_info(" %5d %5X %5X\n",
rx_ring->next_to_use, rx_ring->next_to_clean); n, rx_ring->next_to_use, rx_ring->next_to_clean);
} }
/* Print RX Rings */ /* Print RX Rings */
...@@ -492,36 +489,43 @@ static void igb_dump(struct igb_adapter *adapter) ...@@ -492,36 +489,43 @@ static void igb_dump(struct igb_adapter *adapter)
for (n = 0; n < adapter->num_rx_queues; n++) { for (n = 0; n < adapter->num_rx_queues; n++) {
rx_ring = adapter->rx_ring[n]; rx_ring = adapter->rx_ring[n];
printk(KERN_INFO "------------------------------------\n"); pr_info("------------------------------------\n");
printk(KERN_INFO "RX QUEUE INDEX = %d\n", rx_ring->queue_index); pr_info("RX QUEUE INDEX = %d\n", rx_ring->queue_index);
printk(KERN_INFO "------------------------------------\n"); pr_info("------------------------------------\n");
printk(KERN_INFO "R [desc] [ PktBuf A0] " pr_info("R [desc] [ PktBuf A0] [ HeadBuf DD] "
"[ HeadBuf DD] [bi->dma ] [bi->skb] " "[bi->dma ] [bi->skb] <-- Adv Rx Read format\n");
"<-- Adv Rx Read format\n"); pr_info("RWB[desc] [PcsmIpSHl PtRs] [vl er S cks ln] -----"
printk(KERN_INFO "RWB[desc] [PcsmIpSHl PtRs] " "----------- [bi->skb] <-- Adv Rx Write-Back format\n");
"[vl er S cks ln] ---------------- [bi->skb] "
"<-- Adv Rx Write-Back format\n");
for (i = 0; i < rx_ring->count; i++) { for (i = 0; i < rx_ring->count; i++) {
const char *next_desc;
struct igb_rx_buffer *buffer_info; struct igb_rx_buffer *buffer_info;
buffer_info = &rx_ring->rx_buffer_info[i]; buffer_info = &rx_ring->rx_buffer_info[i];
rx_desc = IGB_RX_DESC(rx_ring, i); rx_desc = IGB_RX_DESC(rx_ring, i);
u0 = (struct my_u0 *)rx_desc; u0 = (struct my_u0 *)rx_desc;
staterr = le32_to_cpu(rx_desc->wb.upper.status_error); staterr = le32_to_cpu(rx_desc->wb.upper.status_error);
if (i == rx_ring->next_to_use)
next_desc = " NTU";
else if (i == rx_ring->next_to_clean)
next_desc = " NTC";
else
next_desc = "";
if (staterr & E1000_RXD_STAT_DD) { if (staterr & E1000_RXD_STAT_DD) {
/* Descriptor Done */ /* Descriptor Done */
printk(KERN_INFO "RWB[0x%03X] %016llX " pr_info("%s[0x%03X] %016llX %016llX -------"
"%016llX ---------------- %p", i, "--------- %p%s\n", "RWB", i,
le64_to_cpu(u0->a), le64_to_cpu(u0->a),
le64_to_cpu(u0->b), le64_to_cpu(u0->b),
buffer_info->skb); buffer_info->skb, next_desc);
} else { } else {
printk(KERN_INFO "R [0x%03X] %016llX " pr_info("%s[0x%03X] %016llX %016llX %016llX"
"%016llX %016llX %p", i, " %p%s\n", "R ", i,
le64_to_cpu(u0->a), le64_to_cpu(u0->a),
le64_to_cpu(u0->b), le64_to_cpu(u0->b),
(u64)buffer_info->dma, (u64)buffer_info->dma,
buffer_info->skb); buffer_info->skb, next_desc);
if (netif_msg_pktdata(adapter)) { if (netif_msg_pktdata(adapter)) {
print_hex_dump(KERN_INFO, "", print_hex_dump(KERN_INFO, "",
...@@ -538,14 +542,6 @@ static void igb_dump(struct igb_adapter *adapter) ...@@ -538,14 +542,6 @@ static void igb_dump(struct igb_adapter *adapter)
PAGE_SIZE/2, true); PAGE_SIZE/2, true);
} }
} }
if (i == rx_ring->next_to_use)
printk(KERN_CONT " NTU\n");
else if (i == rx_ring->next_to_clean)
printk(KERN_CONT " NTC\n");
else
printk(KERN_CONT "\n");
} }
} }
...@@ -599,10 +595,10 @@ struct net_device *igb_get_hw_dev(struct e1000_hw *hw) ...@@ -599,10 +595,10 @@ struct net_device *igb_get_hw_dev(struct e1000_hw *hw)
static int __init igb_init_module(void) static int __init igb_init_module(void)
{ {
int ret; int ret;
printk(KERN_INFO "%s - version %s\n", pr_info("%s - version %s\n",
igb_driver_string, igb_driver_version); igb_driver_string, igb_driver_version);
printk(KERN_INFO "%s\n", igb_copyright); pr_info("%s\n", igb_copyright);
#ifdef CONFIG_IGB_DCA #ifdef CONFIG_IGB_DCA
dca_register_notify(&dca_notifier); dca_register_notify(&dca_notifier);
...@@ -3640,23 +3636,23 @@ static void igb_watchdog_task(struct work_struct *work) ...@@ -3640,23 +3636,23 @@ static void igb_watchdog_task(struct work_struct *work)
ctrl = rd32(E1000_CTRL); ctrl = rd32(E1000_CTRL);
/* Links status message must follow this format */ /* Links status message must follow this format */
printk(KERN_INFO "igb: %s NIC Link is Up %d Mbps %s, " printk(KERN_INFO "igb: %s NIC Link is Up %d Mbps %s "
"Flow Control: %s\n", "Duplex, Flow Control: %s\n",
netdev->name, netdev->name,
adapter->link_speed, adapter->link_speed,
adapter->link_duplex == FULL_DUPLEX ? adapter->link_duplex == FULL_DUPLEX ?
"Full Duplex" : "Half Duplex", "Full" : "Half",
((ctrl & E1000_CTRL_TFCE) && (ctrl & E1000_CTRL_TFCE) &&
(ctrl & E1000_CTRL_RFCE)) ? "RX/TX" : (ctrl & E1000_CTRL_RFCE) ? "RX/TX" :
((ctrl & E1000_CTRL_RFCE) ? "RX" : (ctrl & E1000_CTRL_RFCE) ? "RX" :
((ctrl & E1000_CTRL_TFCE) ? "TX" : "None"))); (ctrl & E1000_CTRL_TFCE) ? "TX" : "None");
/* check for thermal sensor event */ /* check for thermal sensor event */
if (igb_thermal_sensor_event(hw, E1000_THSTAT_LINK_THROTTLE)) { if (igb_thermal_sensor_event(hw,
printk(KERN_INFO "igb: %s The network adapter " E1000_THSTAT_LINK_THROTTLE)) {
"link speed was downshifted " netdev_info(netdev, "The network adapter link "
"because it overheated.\n", "speed was downshifted because it "
netdev->name); "overheated\n");
} }
/* adjust timeout factor according to speed/duplex */ /* adjust timeout factor according to speed/duplex */
...@@ -3686,11 +3682,10 @@ static void igb_watchdog_task(struct work_struct *work) ...@@ -3686,11 +3682,10 @@ static void igb_watchdog_task(struct work_struct *work)
adapter->link_duplex = 0; adapter->link_duplex = 0;
/* check for thermal sensor event */ /* check for thermal sensor event */
if (igb_thermal_sensor_event(hw, E1000_THSTAT_PWR_DOWN)) { if (igb_thermal_sensor_event(hw,
printk(KERN_ERR "igb: %s The network adapter " E1000_THSTAT_PWR_DOWN)) {
"was stopped because it " netdev_err(netdev, "The network adapter was "
"overheated.\n", "stopped because it overheated\n");
netdev->name);
} }
/* Links status message must follow this format */ /* Links status message must follow this format */
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7