* Issue with driver i40e stat strings count mismatch
@ 2018-07-10 11:17 ` Jesper Dangaard Brouer
0 siblings, 0 replies; 12+ messages in thread
From: Jesper Dangaard Brouer @ 2018-07-10 11:17 UTC (permalink / raw)
To: Jeff Kirsher
Cc: brouer, Björn Töpel, alexander.h.duyck,
intel-wired-lan, netdev
Hi Intel-fokes,
Your i40e driver have issues with it's ethtool stats. A warning
triggers at drivers/net/ethernet/intel/i40e/i40e_ethtool.c line 1907
(see splash below) in func i40e_get_stat_strings().
[ 5077.779518] ------------[ cut here ]------------
[ 5077.784493] stat strings count mismatch!
[ 5077.784529] WARNING: CPU: 0 PID: 2293 at drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1907 i40e_get_strings+0x477/0x4b0 [i40e]
[ 5077.800941] Modules linked in: act_gact cls_u32 sch_ingress xt_tcpudp iptable_raw ip_tables x_tables tun nfnetlink bridge stp llc bpfilter sunrpc coretemp kvm_intel kvm irqbypass intel_cstate intel_uncore intel_rapl_perf pcspkr i2c_i801 wmi ipmi_si ipmi_devintf ipmi_msghandler acpi_pad sch_fq_codel ixgbe mlx5_core mlxfw i40e devlink hid_generic igb mdio i2c_algo_bit ptp sd_mod i2c_core pps_core [last unloaded: x_tables]
[ 5077.839833] CPU: 0 PID: 2293 Comm: ethtool Not tainted 4.18.0-rc3-net-next-EdwardCree01+ #484
[ 5077.848962] Hardware name: Supermicro Super Server/X10SRi-F, BIOS 2.0a 08/01/2016
[ 5077.857049] RIP: 0010:i40e_get_strings+0x477/0x4b0 [i40e]
[ 5077.862776] Code: 98 49 39 c4 0f 84 2e fc ff ff 80 3d b3 da 03 00 00 0f 85 21 fc ff ff 48 c7 c7 24 42 19 a0 c6 05 9f da 03 00 01 e8 e9 9c ee e0 <0f> 0b e9 07 fc ff ff 48 83 c4 10 48 c7 c1 80 01 19 a0 be 20 00 00
[ 5077.882506] RSP: 0018:ffffc90003af3c18 EFLAGS: 00010296
[ 5077.888063] RAX: 000000000000001c RBX: ffffc90003ce1440 RCX: 0000000000000006
[ 5077.895528] RDX: 0000000000000007 RSI: 0000000000000096 RDI: ffff88087ca15530
[ 5077.902991] RBP: ffffc90003ce1440 R08: 000000000000001c R09: 0000000000000411
[ 5077.910453] R10: 000fffffffe00000 R11: ffffffff82a4e66d R12: ffffffffffffcbc0
[ 5077.917913] R13: ffff88087c50f000 R14: 0000000000000008 R15: ffffffffa0199620
[ 5077.925376] FS: 00007f7a84bcb740(0000) GS:ffff88087ca00000(0000) knlGS:0000000000000000
[ 5077.934061] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[ 5077.940133] CR2: 000055c7d709b000 CR3: 000000081acd0004 CR4: 00000000003606f0
[ 5077.947609] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
[ 5077.955070] DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400
[ 5077.962531] Call Trace:
[ 5077.965309] dev_ethtool+0xf4e/0x2430
[ 5077.969305] ? get_page_from_freelist+0x2bb/0x1240
[ 5077.974428] ? dev_ioctl+0x1e9/0x3c0
[ 5077.978332] dev_ioctl+0x1e9/0x3c0
[ 5077.982062] sock_do_ioctl+0xa8/0x140
[ 5077.986057] ? sock_ioctl+0x1c0/0x300
[ 5077.990051] sock_ioctl+0x1c0/0x300
[ 5077.993864] ? __handle_mm_fault+0xa82/0xfd0
[ 5077.998462] ? do_vfs_ioctl+0x8d/0x5e0
[ 5078.002550] do_vfs_ioctl+0x8d/0x5e0
[ 5078.006456] ? handle_mm_fault+0xd0/0x210
[ 5078.010790] ksys_ioctl+0x70/0x80
[ 5078.014429] __x64_sys_ioctl+0x16/0x20
[ 5078.018505] do_syscall_64+0x42/0xf0
[ 5078.022411] entry_SYSCALL_64_after_hwframe+0x44/0xa9
[ 5078.027789] RIP: 0033:0x7f7a84394dc7
[ 5078.031703] Code: b3 66 90 48 8b 05 d9 00 2d 00 64 c7 00 26 00 00 00 48 c7 c0 ff ff ff ff c3 66 2e 0f 1f 84 00 00 00 00 00 b8 10 00 00 00 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 8b 0d a9 00 2d 00 f7 d8 64 89 01 48
[ 5078.051437] RSP: 002b:00007ffd64d68338 EFLAGS: 00000246 ORIG_RAX: 0000000000000010
[ 5078.059596] RAX: ffffffffffffffda RBX: 000055c7d7099260 RCX: 00007f7a84394dc7
[ 5078.067062] RDX: 00007ffd64d684e0 RSI: 0000000000008946 RDI: 0000000000000003
[ 5078.074524] RBP: 00007ffd64d684d0 R08: ffffffffffffffb0 R09: 000055c7d7099260
[ 5078.081986] R10: 0000000000000000 R11: 0000000000000246 R12: 00000000000001a2
[ 5078.089447] R13: 0000000000000001 R14: 0000000000000000 R15: 00007ffd64d684e0
[ 5078.096911] ---[ end trace faf82a00c0d8b6b1 ]---
$ gdb ./drivers/net/ethernet/intel/i40e/i40e.ko
[...]
Reading symbols from ./drivers/net/ethernet/intel/i40e/i40e.ko...done.
(gdb) list *(i40e_get_strings)+0x477
0x17787 is in i40e_get_strings (drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1906).
1901 snprintf(data, ETH_GSTRING_LEN,
1902 "port.rx_priority_%u_xon_2_xoff", i);
1903 data += ETH_GSTRING_LEN;
1904 }
1905
1906 WARN_ONCE(p - data != i40e_get_stats_count(netdev) * ETH_GSTRING_LEN,
1907 "stat strings count mismatch!");
1908 }
1909
1910 static void i40e_get_priv_flag_strings(struct net_device *netdev, u8 *data)
--
Best regards,
Jesper Dangaard Brouer
MSc.CS, Principal Kernel Engineer at Red Hat
LinkedIn: http://www.linkedin.com/in/brouer
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Intel-wired-lan] Issue with driver i40e stat strings count mismatch
@ 2018-07-10 11:17 ` Jesper Dangaard Brouer
0 siblings, 0 replies; 12+ messages in thread
From: Jesper Dangaard Brouer @ 2018-07-10 11:17 UTC (permalink / raw)
To: intel-wired-lan
Hi Intel-fokes,
Your i40e driver have issues with it's ethtool stats. A warning
triggers at drivers/net/ethernet/intel/i40e/i40e_ethtool.c line 1907
(see splash below) in func i40e_get_stat_strings().
[ 5077.779518] ------------[ cut here ]------------
[ 5077.784493] stat strings count mismatch!
[ 5077.784529] WARNING: CPU: 0 PID: 2293 at drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1907 i40e_get_strings+0x477/0x4b0 [i40e]
[ 5077.800941] Modules linked in: act_gact cls_u32 sch_ingress xt_tcpudp iptable_raw ip_tables x_tables tun nfnetlink bridge stp llc bpfilter sunrpc coretemp kvm_intel kvm irqbypass intel_cstate intel_uncore intel_rapl_perf pcspkr i2c_i801 wmi ipmi_si ipmi_devintf ipmi_msghandler acpi_pad sch_fq_codel ixgbe mlx5_core mlxfw i40e devlink hid_generic igb mdio i2c_algo_bit ptp sd_mod i2c_core pps_core [last unloaded: x_tables]
[ 5077.839833] CPU: 0 PID: 2293 Comm: ethtool Not tainted 4.18.0-rc3-net-next-EdwardCree01+ #484
[ 5077.848962] Hardware name: Supermicro Super Server/X10SRi-F, BIOS 2.0a 08/01/2016
[ 5077.857049] RIP: 0010:i40e_get_strings+0x477/0x4b0 [i40e]
[ 5077.862776] Code: 98 49 39 c4 0f 84 2e fc ff ff 80 3d b3 da 03 00 00 0f 85 21 fc ff ff 48 c7 c7 24 42 19 a0 c6 05 9f da 03 00 01 e8 e9 9c ee e0 <0f> 0b e9 07 fc ff ff 48 83 c4 10 48 c7 c1 80 01 19 a0 be 20 00 00
[ 5077.882506] RSP: 0018:ffffc90003af3c18 EFLAGS: 00010296
[ 5077.888063] RAX: 000000000000001c RBX: ffffc90003ce1440 RCX: 0000000000000006
[ 5077.895528] RDX: 0000000000000007 RSI: 0000000000000096 RDI: ffff88087ca15530
[ 5077.902991] RBP: ffffc90003ce1440 R08: 000000000000001c R09: 0000000000000411
[ 5077.910453] R10: 000fffffffe00000 R11: ffffffff82a4e66d R12: ffffffffffffcbc0
[ 5077.917913] R13: ffff88087c50f000 R14: 0000000000000008 R15: ffffffffa0199620
[ 5077.925376] FS: 00007f7a84bcb740(0000) GS:ffff88087ca00000(0000) knlGS:0000000000000000
[ 5077.934061] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[ 5077.940133] CR2: 000055c7d709b000 CR3: 000000081acd0004 CR4: 00000000003606f0
[ 5077.947609] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
[ 5077.955070] DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400
[ 5077.962531] Call Trace:
[ 5077.965309] dev_ethtool+0xf4e/0x2430
[ 5077.969305] ? get_page_from_freelist+0x2bb/0x1240
[ 5077.974428] ? dev_ioctl+0x1e9/0x3c0
[ 5077.978332] dev_ioctl+0x1e9/0x3c0
[ 5077.982062] sock_do_ioctl+0xa8/0x140
[ 5077.986057] ? sock_ioctl+0x1c0/0x300
[ 5077.990051] sock_ioctl+0x1c0/0x300
[ 5077.993864] ? __handle_mm_fault+0xa82/0xfd0
[ 5077.998462] ? do_vfs_ioctl+0x8d/0x5e0
[ 5078.002550] do_vfs_ioctl+0x8d/0x5e0
[ 5078.006456] ? handle_mm_fault+0xd0/0x210
[ 5078.010790] ksys_ioctl+0x70/0x80
[ 5078.014429] __x64_sys_ioctl+0x16/0x20
[ 5078.018505] do_syscall_64+0x42/0xf0
[ 5078.022411] entry_SYSCALL_64_after_hwframe+0x44/0xa9
[ 5078.027789] RIP: 0033:0x7f7a84394dc7
[ 5078.031703] Code: b3 66 90 48 8b 05 d9 00 2d 00 64 c7 00 26 00 00 00 48 c7 c0 ff ff ff ff c3 66 2e 0f 1f 84 00 00 00 00 00 b8 10 00 00 00 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 8b 0d a9 00 2d 00 f7 d8 64 89 01 48
[ 5078.051437] RSP: 002b:00007ffd64d68338 EFLAGS: 00000246 ORIG_RAX: 0000000000000010
[ 5078.059596] RAX: ffffffffffffffda RBX: 000055c7d7099260 RCX: 00007f7a84394dc7
[ 5078.067062] RDX: 00007ffd64d684e0 RSI: 0000000000008946 RDI: 0000000000000003
[ 5078.074524] RBP: 00007ffd64d684d0 R08: ffffffffffffffb0 R09: 000055c7d7099260
[ 5078.081986] R10: 0000000000000000 R11: 0000000000000246 R12: 00000000000001a2
[ 5078.089447] R13: 0000000000000001 R14: 0000000000000000 R15: 00007ffd64d684e0
[ 5078.096911] ---[ end trace faf82a00c0d8b6b1 ]---
$ gdb ./drivers/net/ethernet/intel/i40e/i40e.ko
[...]
Reading symbols from ./drivers/net/ethernet/intel/i40e/i40e.ko...done.
(gdb) list *(i40e_get_strings)+0x477
0x17787 is in i40e_get_strings (drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1906).
1901 snprintf(data, ETH_GSTRING_LEN,
1902 "port.rx_priority_%u_xon_2_xoff", i);
1903 data += ETH_GSTRING_LEN;
1904 }
1905
1906 WARN_ONCE(p - data != i40e_get_stats_count(netdev) * ETH_GSTRING_LEN,
1907 "stat strings count mismatch!");
1908 }
1909
1910 static void i40e_get_priv_flag_strings(struct net_device *netdev, u8 *data)
--
Best regards,
Jesper Dangaard Brouer
MSc.CS, Principal Kernel Engineer at Red Hat
LinkedIn: http://www.linkedin.com/in/brouer
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: Issue with driver i40e stat strings count mismatch
2018-07-10 11:17 ` [Intel-wired-lan] " Jesper Dangaard Brouer
@ 2018-07-31 7:05 ` Stefan Assmann
-1 siblings, 0 replies; 12+ messages in thread
From: Stefan Assmann @ 2018-07-31 7:05 UTC (permalink / raw)
To: Jesper Dangaard Brouer, Jeff Kirsher
Cc: Björn Töpel, alexander.h.duyck, intel-wired-lan, netdev
On 10.07.2018 13:17, Jesper Dangaard Brouer wrote:
> Hi Intel-fokes,
>
> Your i40e driver have issues with it's ethtool stats. A warning
> triggers at drivers/net/ethernet/intel/i40e/i40e_ethtool.c line 1907
> (see splash below) in func i40e_get_stat_strings().
Hi Jesper,
I ran into the same issue. Here's my proposed fix.
>From 46c74c25496bab06712641c7b2b6b34e365397a2 Mon Sep 17 00:00:00 2001
From: Stefan Assmann <sassmann@kpanic.de>
Date: Mon, 30 Jul 2018 21:38:43 +0200
Subject: [PATCH] i40e: fix i40e_get_stat_strings strings count warning
The current code calculates p - data, which results in a negative value.
Therefore the WARN_ONCE condition will always be true.
Fix this by calculating data - p instead.
Fixes: 9b10df596bd4 ("i40e: use WARN_ONCE to replace the commented BUG_ON size check")
Signed-off-by: Stefan Assmann <sassmann@kpanic.de>
---
drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
index 6947a2a571cb..5d670f4ce5ac 100644
--- a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
+++ b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
@@ -1903,7 +1903,7 @@ static void i40e_get_stat_strings(struct net_device *netdev, u8 *data)
data += ETH_GSTRING_LEN;
}
- WARN_ONCE(p - data != i40e_get_stats_count(netdev) * ETH_GSTRING_LEN,
+ WARN_ONCE(data - p != i40e_get_stats_count(netdev) * ETH_GSTRING_LEN,
"stat strings count mismatch!");
}
--
2.17.1
>
> [ 5077.779518] ------------[ cut here ]------------
> [ 5077.784493] stat strings count mismatch!
> [ 5077.784529] WARNING: CPU: 0 PID: 2293 at drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1907 i40e_get_strings+0x477/0x4b0 [i40e]
> [ 5077.800941] Modules linked in: act_gact cls_u32 sch_ingress xt_tcpudp iptable_raw ip_tables x_tables tun nfnetlink bridge stp llc bpfilter sunrpc coretemp kvm_intel kvm irqbypass intel_cstate intel_uncore intel_rapl_perf pcspkr i2c_i801 wmi ipmi_si ipmi_devintf ipmi_msghandler acpi_pad sch_fq_codel ixgbe mlx5_core mlxfw i40e devlink hid_generic igb mdio i2c_algo_bit ptp sd_mod i2c_core pps_core [last unloaded: x_tables]
> [ 5077.839833] CPU: 0 PID: 2293 Comm: ethtool Not tainted 4.18.0-rc3-net-next-EdwardCree01+ #484
> [ 5077.848962] Hardware name: Supermicro Super Server/X10SRi-F, BIOS 2.0a 08/01/2016
> [ 5077.857049] RIP: 0010:i40e_get_strings+0x477/0x4b0 [i40e]
> [ 5077.862776] Code: 98 49 39 c4 0f 84 2e fc ff ff 80 3d b3 da 03 00 00 0f 85 21 fc ff ff 48 c7 c7 24 42 19 a0 c6 05 9f da 03 00 01 e8 e9 9c ee e0 <0f> 0b e9 07 fc ff ff 48 83 c4 10 48 c7 c1 80 01 19 a0 be 20 00 00
> [ 5077.882506] RSP: 0018:ffffc90003af3c18 EFLAGS: 00010296
> [ 5077.888063] RAX: 000000000000001c RBX: ffffc90003ce1440 RCX: 0000000000000006
> [ 5077.895528] RDX: 0000000000000007 RSI: 0000000000000096 RDI: ffff88087ca15530
> [ 5077.902991] RBP: ffffc90003ce1440 R08: 000000000000001c R09: 0000000000000411
> [ 5077.910453] R10: 000fffffffe00000 R11: ffffffff82a4e66d R12: ffffffffffffcbc0
> [ 5077.917913] R13: ffff88087c50f000 R14: 0000000000000008 R15: ffffffffa0199620
> [ 5077.925376] FS: 00007f7a84bcb740(0000) GS:ffff88087ca00000(0000) knlGS:0000000000000000
> [ 5077.934061] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> [ 5077.940133] CR2: 000055c7d709b000 CR3: 000000081acd0004 CR4: 00000000003606f0
> [ 5077.947609] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
> [ 5077.955070] DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400
> [ 5077.962531] Call Trace:
> [ 5077.965309] dev_ethtool+0xf4e/0x2430
> [ 5077.969305] ? get_page_from_freelist+0x2bb/0x1240
> [ 5077.974428] ? dev_ioctl+0x1e9/0x3c0
> [ 5077.978332] dev_ioctl+0x1e9/0x3c0
> [ 5077.982062] sock_do_ioctl+0xa8/0x140
> [ 5077.986057] ? sock_ioctl+0x1c0/0x300
> [ 5077.990051] sock_ioctl+0x1c0/0x300
> [ 5077.993864] ? __handle_mm_fault+0xa82/0xfd0
> [ 5077.998462] ? do_vfs_ioctl+0x8d/0x5e0
> [ 5078.002550] do_vfs_ioctl+0x8d/0x5e0
> [ 5078.006456] ? handle_mm_fault+0xd0/0x210
> [ 5078.010790] ksys_ioctl+0x70/0x80
> [ 5078.014429] __x64_sys_ioctl+0x16/0x20
> [ 5078.018505] do_syscall_64+0x42/0xf0
> [ 5078.022411] entry_SYSCALL_64_after_hwframe+0x44/0xa9
> [ 5078.027789] RIP: 0033:0x7f7a84394dc7
> [ 5078.031703] Code: b3 66 90 48 8b 05 d9 00 2d 00 64 c7 00 26 00 00 00 48 c7 c0 ff ff ff ff c3 66 2e 0f 1f 84 00 00 00 00 00 b8 10 00 00 00 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 8b 0d a9 00 2d 00 f7 d8 64 89 01 48
> [ 5078.051437] RSP: 002b:00007ffd64d68338 EFLAGS: 00000246 ORIG_RAX: 0000000000000010
> [ 5078.059596] RAX: ffffffffffffffda RBX: 000055c7d7099260 RCX: 00007f7a84394dc7
> [ 5078.067062] RDX: 00007ffd64d684e0 RSI: 0000000000008946 RDI: 0000000000000003
> [ 5078.074524] RBP: 00007ffd64d684d0 R08: ffffffffffffffb0 R09: 000055c7d7099260
> [ 5078.081986] R10: 0000000000000000 R11: 0000000000000246 R12: 00000000000001a2
> [ 5078.089447] R13: 0000000000000001 R14: 0000000000000000 R15: 00007ffd64d684e0
> [ 5078.096911] ---[ end trace faf82a00c0d8b6b1 ]---
>
> $ gdb ./drivers/net/ethernet/intel/i40e/i40e.ko
> [...]
> Reading symbols from ./drivers/net/ethernet/intel/i40e/i40e.ko...done.
> (gdb) list *(i40e_get_strings)+0x477
> 0x17787 is in i40e_get_strings (drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1906).
> 1901 snprintf(data, ETH_GSTRING_LEN,
> 1902 "port.rx_priority_%u_xon_2_xoff", i);
> 1903 data += ETH_GSTRING_LEN;
> 1904 }
> 1905
> 1906 WARN_ONCE(p - data != i40e_get_stats_count(netdev) * ETH_GSTRING_LEN,
> 1907 "stat strings count mismatch!");
> 1908 }
> 1909
> 1910 static void i40e_get_priv_flag_strings(struct net_device *netdev, u8 *data)
>
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [Intel-wired-lan] Issue with driver i40e stat strings count mismatch
@ 2018-07-31 7:05 ` Stefan Assmann
0 siblings, 0 replies; 12+ messages in thread
From: Stefan Assmann @ 2018-07-31 7:05 UTC (permalink / raw)
To: intel-wired-lan
On 10.07.2018 13:17, Jesper Dangaard Brouer wrote:
> Hi Intel-fokes,
>
> Your i40e driver have issues with it's ethtool stats. A warning
> triggers at drivers/net/ethernet/intel/i40e/i40e_ethtool.c line 1907
> (see splash below) in func i40e_get_stat_strings().
Hi Jesper,
I ran into the same issue. Here's my proposed fix.
From 46c74c25496bab06712641c7b2b6b34e365397a2 Mon Sep 17 00:00:00 2001
From: Stefan Assmann <sassmann@kpanic.de>
Date: Mon, 30 Jul 2018 21:38:43 +0200
Subject: [PATCH] i40e: fix i40e_get_stat_strings strings count warning
The current code calculates p - data, which results in a negative value.
Therefore the WARN_ONCE condition will always be true.
Fix this by calculating data - p instead.
Fixes: 9b10df596bd4 ("i40e: use WARN_ONCE to replace the commented BUG_ON size check")
Signed-off-by: Stefan Assmann <sassmann@kpanic.de>
---
drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
index 6947a2a571cb..5d670f4ce5ac 100644
--- a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
+++ b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
@@ -1903,7 +1903,7 @@ static void i40e_get_stat_strings(struct net_device *netdev, u8 *data)
data += ETH_GSTRING_LEN;
}
- WARN_ONCE(p - data != i40e_get_stats_count(netdev) * ETH_GSTRING_LEN,
+ WARN_ONCE(data - p != i40e_get_stats_count(netdev) * ETH_GSTRING_LEN,
"stat strings count mismatch!");
}
--
2.17.1
>
> [ 5077.779518] ------------[ cut here ]------------
> [ 5077.784493] stat strings count mismatch!
> [ 5077.784529] WARNING: CPU: 0 PID: 2293 at drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1907 i40e_get_strings+0x477/0x4b0 [i40e]
> [ 5077.800941] Modules linked in: act_gact cls_u32 sch_ingress xt_tcpudp iptable_raw ip_tables x_tables tun nfnetlink bridge stp llc bpfilter sunrpc coretemp kvm_intel kvm irqbypass intel_cstate intel_uncore intel_rapl_perf pcspkr i2c_i801 wmi ipmi_si ipmi_devintf ipmi_msghandler acpi_pad sch_fq_codel ixgbe mlx5_core mlxfw i40e devlink hid_generic igb mdio i2c_algo_bit ptp sd_mod i2c_core pps_core [last unloaded: x_tables]
> [ 5077.839833] CPU: 0 PID: 2293 Comm: ethtool Not tainted 4.18.0-rc3-net-next-EdwardCree01+ #484
> [ 5077.848962] Hardware name: Supermicro Super Server/X10SRi-F, BIOS 2.0a 08/01/2016
> [ 5077.857049] RIP: 0010:i40e_get_strings+0x477/0x4b0 [i40e]
> [ 5077.862776] Code: 98 49 39 c4 0f 84 2e fc ff ff 80 3d b3 da 03 00 00 0f 85 21 fc ff ff 48 c7 c7 24 42 19 a0 c6 05 9f da 03 00 01 e8 e9 9c ee e0 <0f> 0b e9 07 fc ff ff 48 83 c4 10 48 c7 c1 80 01 19 a0 be 20 00 00
> [ 5077.882506] RSP: 0018:ffffc90003af3c18 EFLAGS: 00010296
> [ 5077.888063] RAX: 000000000000001c RBX: ffffc90003ce1440 RCX: 0000000000000006
> [ 5077.895528] RDX: 0000000000000007 RSI: 0000000000000096 RDI: ffff88087ca15530
> [ 5077.902991] RBP: ffffc90003ce1440 R08: 000000000000001c R09: 0000000000000411
> [ 5077.910453] R10: 000fffffffe00000 R11: ffffffff82a4e66d R12: ffffffffffffcbc0
> [ 5077.917913] R13: ffff88087c50f000 R14: 0000000000000008 R15: ffffffffa0199620
> [ 5077.925376] FS: 00007f7a84bcb740(0000) GS:ffff88087ca00000(0000) knlGS:0000000000000000
> [ 5077.934061] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> [ 5077.940133] CR2: 000055c7d709b000 CR3: 000000081acd0004 CR4: 00000000003606f0
> [ 5077.947609] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
> [ 5077.955070] DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400
> [ 5077.962531] Call Trace:
> [ 5077.965309] dev_ethtool+0xf4e/0x2430
> [ 5077.969305] ? get_page_from_freelist+0x2bb/0x1240
> [ 5077.974428] ? dev_ioctl+0x1e9/0x3c0
> [ 5077.978332] dev_ioctl+0x1e9/0x3c0
> [ 5077.982062] sock_do_ioctl+0xa8/0x140
> [ 5077.986057] ? sock_ioctl+0x1c0/0x300
> [ 5077.990051] sock_ioctl+0x1c0/0x300
> [ 5077.993864] ? __handle_mm_fault+0xa82/0xfd0
> [ 5077.998462] ? do_vfs_ioctl+0x8d/0x5e0
> [ 5078.002550] do_vfs_ioctl+0x8d/0x5e0
> [ 5078.006456] ? handle_mm_fault+0xd0/0x210
> [ 5078.010790] ksys_ioctl+0x70/0x80
> [ 5078.014429] __x64_sys_ioctl+0x16/0x20
> [ 5078.018505] do_syscall_64+0x42/0xf0
> [ 5078.022411] entry_SYSCALL_64_after_hwframe+0x44/0xa9
> [ 5078.027789] RIP: 0033:0x7f7a84394dc7
> [ 5078.031703] Code: b3 66 90 48 8b 05 d9 00 2d 00 64 c7 00 26 00 00 00 48 c7 c0 ff ff ff ff c3 66 2e 0f 1f 84 00 00 00 00 00 b8 10 00 00 00 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 8b 0d a9 00 2d 00 f7 d8 64 89 01 48
> [ 5078.051437] RSP: 002b:00007ffd64d68338 EFLAGS: 00000246 ORIG_RAX: 0000000000000010
> [ 5078.059596] RAX: ffffffffffffffda RBX: 000055c7d7099260 RCX: 00007f7a84394dc7
> [ 5078.067062] RDX: 00007ffd64d684e0 RSI: 0000000000008946 RDI: 0000000000000003
> [ 5078.074524] RBP: 00007ffd64d684d0 R08: ffffffffffffffb0 R09: 000055c7d7099260
> [ 5078.081986] R10: 0000000000000000 R11: 0000000000000246 R12: 00000000000001a2
> [ 5078.089447] R13: 0000000000000001 R14: 0000000000000000 R15: 00007ffd64d684e0
> [ 5078.096911] ---[ end trace faf82a00c0d8b6b1 ]---
>
> $ gdb ./drivers/net/ethernet/intel/i40e/i40e.ko
> [...]
> Reading symbols from ./drivers/net/ethernet/intel/i40e/i40e.ko...done.
> (gdb) list *(i40e_get_strings)+0x477
> 0x17787 is in i40e_get_strings (drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1906).
> 1901 snprintf(data, ETH_GSTRING_LEN,
> 1902 "port.rx_priority_%u_xon_2_xoff", i);
> 1903 data += ETH_GSTRING_LEN;
> 1904 }
> 1905
> 1906 WARN_ONCE(p - data != i40e_get_stats_count(netdev) * ETH_GSTRING_LEN,
> 1907 "stat strings count mismatch!");
> 1908 }
> 1909
> 1910 static void i40e_get_priv_flag_strings(struct net_device *netdev, u8 *data)
>
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: Issue with driver i40e stat strings count mismatch
2018-07-31 7:05 ` [Intel-wired-lan] " Stefan Assmann
@ 2018-07-31 14:28 ` Jesper Dangaard Brouer
-1 siblings, 0 replies; 12+ messages in thread
From: Jesper Dangaard Brouer @ 2018-07-31 14:28 UTC (permalink / raw)
To: Stefan Assmann, Jacob Keller
Cc: Jeff Kirsher, Björn Töpel, alexander.h.duyck,
intel-wired-lan, netdev, brouer
On Tue, 31 Jul 2018 09:05:40 +0200 Stefan Assmann <sassmann@kpanic.de> wrote:
> From: Stefan Assmann <sassmann@kpanic.de>
> To: Jesper Dangaard Brouer <brouer@redhat.com>, Jeff Kirsher <jeffrey.t.kirsher@intel.com>
> Cc: Björn Töpel <bjorn.topel@intel.com>, "alexander.h.duyck@intel.com" <alexander.h.duyck@intel.com>, intel-wired-lan <intel-wired-lan@lists.osuosl.org>, "netdev@vger.kernel.org" <netdev@vger.kernel.org>
> Subject: Re: Issue with driver i40e stat strings count mismatch
> Date: Tue, 31 Jul 2018 09:05:40 +0200
> User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
> Thunderbird/52.9.1
> Message-ID: <64be8e6a-c285-2864-fd91-356eba645acd@kpanic.de>
>
> On 10.07.2018 13:17, Jesper Dangaard Brouer wrote:
> > Hi Intel-fokes,
> >
> > Your i40e driver have issues with it's ethtool stats. A warning
> > triggers at drivers/net/ethernet/intel/i40e/i40e_ethtool.c line 1907
> > (see splash below) in func i40e_get_stat_strings().
>
> Hi Jesper,
>
> I ran into the same issue. Here's my proposed fix.
Thanks for following up Stefan :-)
I'm hoping some Intel people will look at evaluating this fix? ...
> From 46c74c25496bab06712641c7b2b6b34e365397a2 Mon Sep 17 00:00:00 2001
> From: Stefan Assmann <sassmann@kpanic.de>
> Date: Mon, 30 Jul 2018 21:38:43 +0200
> Subject: [PATCH] i40e: fix i40e_get_stat_strings strings count warning
>
> The current code calculates p - data, which results in a negative value.
> Therefore the WARN_ONCE condition will always be true.
> Fix this by calculating data - p instead.
>
> Fixes: 9b10df596bd4 ("i40e: use WARN_ONCE to replace the commented BUG_ON size check")
>
> Signed-off-by: Stefan Assmann <sassmann@kpanic.de>
> ---
> drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
> index 6947a2a571cb..5d670f4ce5ac 100644
> --- a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
> +++ b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
> @@ -1903,7 +1903,7 @@ static void i40e_get_stat_strings(struct net_device *netdev, u8 *data)
> data += ETH_GSTRING_LEN;
> }
>
> - WARN_ONCE(p - data != i40e_get_stats_count(netdev) * ETH_GSTRING_LEN,
> + WARN_ONCE(data - p != i40e_get_stats_count(netdev) * ETH_GSTRING_LEN,
> "stat strings count mismatch!");
> }
>
- -
Best regards,
Jesper Dangaard Brouer
MSc.CS, Principal Kernel Engineer at Red Hat
LinkedIn: http://www.linkedin.com/in/brouer
My error report:
[ 5077.779518] ------------[ cut here ]------------
[ 5077.784493] stat strings count mismatch!
[ 5077.784529] WARNING: CPU: 0 PID: 2293 at drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1907 i40e_get_strings+0x477/0x4b0 [i40e]
[ 5077.800941] Modules linked in: act_gact cls_u32 sch_ingress xt_tcpudp iptable_raw ip_tables x_tables tun nfnetlink bridge stp llc bpfilter sunrpc coretemp kvm_intel kvm irqbypass intel_cstate intel_uncore intel_rapl_perf pcspkr i2c_i801 wmi ipmi_si ipmi_devintf ipmi_msghandler acpi_pad sch_fq_codel ixgbe mlx5_core mlxfw i40e devlink hid_generic igb mdio i2c_algo_bit ptp sd_mod i2c_core pps_core [last unloaded: x_tables]
[ 5077.839833] CPU: 0 PID: 2293 Comm: ethtool Not tainted 4.18.0-rc3-net-next-EdwardCree01+ #484
[ 5077.848962] Hardware name: Supermicro Super Server/X10SRi-F, BIOS 2.0a 08/01/2016
[ 5077.857049] RIP: 0010:i40e_get_strings+0x477/0x4b0 [i40e]
[ 5077.862776] Code: 98 49 39 c4 0f 84 2e fc ff ff 80 3d b3 da 03 00 00 0f 85 21 fc ff ff 48 c7 c7 24 42 19 a0 c6 05 9f da 03 00 01 e8 e9 9c ee e0 <0f> 0b e9 07 fc ff ff 48 83 c4 10 48 c7 c1 80 01 19 a0 be 20 00 00
[ 5077.882506] RSP: 0018:ffffc90003af3c18 EFLAGS: 00010296
[ 5077.888063] RAX: 000000000000001c RBX: ffffc90003ce1440 RCX: 0000000000000006
[ 5077.895528] RDX: 0000000000000007 RSI: 0000000000000096 RDI: ffff88087ca15530
[ 5077.902991] RBP: ffffc90003ce1440 R08: 000000000000001c R09: 0000000000000411
[ 5077.910453] R10: 000fffffffe00000 R11: ffffffff82a4e66d R12: ffffffffffffcbc0
[ 5077.917913] R13: ffff88087c50f000 R14: 0000000000000008 R15: ffffffffa0199620
[ 5077.925376] FS: 00007f7a84bcb740(0000) GS:ffff88087ca00000(0000) knlGS:0000000000000000
[ 5077.934061] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[ 5077.940133] CR2: 000055c7d709b000 CR3: 000000081acd0004 CR4: 00000000003606f0
[ 5077.947609] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
[ 5077.955070] DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400
[ 5077.962531] Call Trace:
[ 5077.965309] dev_ethtool+0xf4e/0x2430
[ 5077.969305] ? get_page_from_freelist+0x2bb/0x1240
[ 5077.974428] ? dev_ioctl+0x1e9/0x3c0
[ 5077.978332] dev_ioctl+0x1e9/0x3c0
[ 5077.982062] sock_do_ioctl+0xa8/0x140
[ 5077.986057] ? sock_ioctl+0x1c0/0x300
[ 5077.990051] sock_ioctl+0x1c0/0x300
[ 5077.993864] ? __handle_mm_fault+0xa82/0xfd0
[ 5077.998462] ? do_vfs_ioctl+0x8d/0x5e0
[ 5078.002550] do_vfs_ioctl+0x8d/0x5e0
[ 5078.006456] ? handle_mm_fault+0xd0/0x210
[ 5078.010790] ksys_ioctl+0x70/0x80
[ 5078.014429] __x64_sys_ioctl+0x16/0x20
[ 5078.018505] do_syscall_64+0x42/0xf0
[ 5078.022411] entry_SYSCALL_64_after_hwframe+0x44/0xa9
[ 5078.027789] RIP: 0033:0x7f7a84394dc7
[ 5078.031703] Code: b3 66 90 48 8b 05 d9 00 2d 00 64 c7 00 26 00 00 00 48 c7 c0 ff ff ff ff c3 66 2e 0f 1f 84 00 00 00 00 00 b8 10 00 00 00 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 8b 0d a9 00 2d 00 f7 d8 64 89 01 48
[ 5078.051437] RSP: 002b:00007ffd64d68338 EFLAGS: 00000246 ORIG_RAX: 0000000000000010
[ 5078.059596] RAX: ffffffffffffffda RBX: 000055c7d7099260 RCX: 00007f7a84394dc7
[ 5078.067062] RDX: 00007ffd64d684e0 RSI: 0000000000008946 RDI: 0000000000000003
[ 5078.074524] RBP: 00007ffd64d684d0 R08: ffffffffffffffb0 R09: 000055c7d7099260
[ 5078.081986] R10: 0000000000000000 R11: 0000000000000246 R12: 00000000000001a2
[ 5078.089447] R13: 0000000000000001 R14: 0000000000000000 R15: 00007ffd64d684e0
[ 5078.096911] ---[ end trace faf82a00c0d8b6b1 ]---
$ gdb ./drivers/net/ethernet/intel/i40e/i40e.ko
[...]
Reading symbols from ./drivers/net/ethernet/intel/i40e/i40e.ko...done.
(gdb) list *(i40e_get_strings)+0x477
0x17787 is in i40e_get_strings (drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1906).
1901 snprintf(data, ETH_GSTRING_LEN,
1902 "port.rx_priority_%u_xon_2_xoff", i);
1903 data += ETH_GSTRING_LEN;
1904 }
1905
1906 WARN_ONCE(p - data != i40e_get_stats_count(netdev) * ETH_GSTRING_LEN,
1907 "stat strings count mismatch!");
1908 }
1909
1910 static void i40e_get_priv_flag_strings(struct net_device *netdev, u8 *data)
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Intel-wired-lan] Issue with driver i40e stat strings count mismatch
@ 2018-07-31 14:28 ` Jesper Dangaard Brouer
0 siblings, 0 replies; 12+ messages in thread
From: Jesper Dangaard Brouer @ 2018-07-31 14:28 UTC (permalink / raw)
To: intel-wired-lan
On Tue, 31 Jul 2018 09:05:40 +0200 Stefan Assmann <sassmann@kpanic.de> wrote:
> From: Stefan Assmann <sassmann@kpanic.de>
> To: Jesper Dangaard Brouer <brouer@redhat.com>, Jeff Kirsher <jeffrey.t.kirsher@intel.com>
> Cc: Bj?rn T?pel <bjorn.topel@intel.com>, "alexander.h.duyck at intel.com" <alexander.h.duyck@intel.com>, intel-wired-lan <intel-wired-lan@lists.osuosl.org>, "netdev at vger.kernel.org" <netdev@vger.kernel.org>
> Subject: Re: Issue with driver i40e stat strings count mismatch
> Date: Tue, 31 Jul 2018 09:05:40 +0200
> User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
> Thunderbird/52.9.1
> Message-ID: <64be8e6a-c285-2864-fd91-356eba645acd@kpanic.de>
>
> On 10.07.2018 13:17, Jesper Dangaard Brouer wrote:
> > Hi Intel-fokes,
> >
> > Your i40e driver have issues with it's ethtool stats. A warning
> > triggers at drivers/net/ethernet/intel/i40e/i40e_ethtool.c line 1907
> > (see splash below) in func i40e_get_stat_strings().
>
> Hi Jesper,
>
> I ran into the same issue. Here's my proposed fix.
Thanks for following up Stefan :-)
I'm hoping some Intel people will look at evaluating this fix? ...
> From 46c74c25496bab06712641c7b2b6b34e365397a2 Mon Sep 17 00:00:00 2001
> From: Stefan Assmann <sassmann@kpanic.de>
> Date: Mon, 30 Jul 2018 21:38:43 +0200
> Subject: [PATCH] i40e: fix i40e_get_stat_strings strings count warning
>
> The current code calculates p - data, which results in a negative value.
> Therefore the WARN_ONCE condition will always be true.
> Fix this by calculating data - p instead.
>
> Fixes: 9b10df596bd4 ("i40e: use WARN_ONCE to replace the commented BUG_ON size check")
>
> Signed-off-by: Stefan Assmann <sassmann@kpanic.de>
> ---
> drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
> index 6947a2a571cb..5d670f4ce5ac 100644
> --- a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
> +++ b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
> @@ -1903,7 +1903,7 @@ static void i40e_get_stat_strings(struct net_device *netdev, u8 *data)
> data += ETH_GSTRING_LEN;
> }
>
> - WARN_ONCE(p - data != i40e_get_stats_count(netdev) * ETH_GSTRING_LEN,
> + WARN_ONCE(data - p != i40e_get_stats_count(netdev) * ETH_GSTRING_LEN,
> "stat strings count mismatch!");
> }
>
- -
Best regards,
Jesper Dangaard Brouer
MSc.CS, Principal Kernel Engineer at Red Hat
LinkedIn: http://www.linkedin.com/in/brouer
My error report:
[ 5077.779518] ------------[ cut here ]------------
[ 5077.784493] stat strings count mismatch!
[ 5077.784529] WARNING: CPU: 0 PID: 2293 at drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1907 i40e_get_strings+0x477/0x4b0 [i40e]
[ 5077.800941] Modules linked in: act_gact cls_u32 sch_ingress xt_tcpudp iptable_raw ip_tables x_tables tun nfnetlink bridge stp llc bpfilter sunrpc coretemp kvm_intel kvm irqbypass intel_cstate intel_uncore intel_rapl_perf pcspkr i2c_i801 wmi ipmi_si ipmi_devintf ipmi_msghandler acpi_pad sch_fq_codel ixgbe mlx5_core mlxfw i40e devlink hid_generic igb mdio i2c_algo_bit ptp sd_mod i2c_core pps_core [last unloaded: x_tables]
[ 5077.839833] CPU: 0 PID: 2293 Comm: ethtool Not tainted 4.18.0-rc3-net-next-EdwardCree01+ #484
[ 5077.848962] Hardware name: Supermicro Super Server/X10SRi-F, BIOS 2.0a 08/01/2016
[ 5077.857049] RIP: 0010:i40e_get_strings+0x477/0x4b0 [i40e]
[ 5077.862776] Code: 98 49 39 c4 0f 84 2e fc ff ff 80 3d b3 da 03 00 00 0f 85 21 fc ff ff 48 c7 c7 24 42 19 a0 c6 05 9f da 03 00 01 e8 e9 9c ee e0 <0f> 0b e9 07 fc ff ff 48 83 c4 10 48 c7 c1 80 01 19 a0 be 20 00 00
[ 5077.882506] RSP: 0018:ffffc90003af3c18 EFLAGS: 00010296
[ 5077.888063] RAX: 000000000000001c RBX: ffffc90003ce1440 RCX: 0000000000000006
[ 5077.895528] RDX: 0000000000000007 RSI: 0000000000000096 RDI: ffff88087ca15530
[ 5077.902991] RBP: ffffc90003ce1440 R08: 000000000000001c R09: 0000000000000411
[ 5077.910453] R10: 000fffffffe00000 R11: ffffffff82a4e66d R12: ffffffffffffcbc0
[ 5077.917913] R13: ffff88087c50f000 R14: 0000000000000008 R15: ffffffffa0199620
[ 5077.925376] FS: 00007f7a84bcb740(0000) GS:ffff88087ca00000(0000) knlGS:0000000000000000
[ 5077.934061] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[ 5077.940133] CR2: 000055c7d709b000 CR3: 000000081acd0004 CR4: 00000000003606f0
[ 5077.947609] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
[ 5077.955070] DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400
[ 5077.962531] Call Trace:
[ 5077.965309] dev_ethtool+0xf4e/0x2430
[ 5077.969305] ? get_page_from_freelist+0x2bb/0x1240
[ 5077.974428] ? dev_ioctl+0x1e9/0x3c0
[ 5077.978332] dev_ioctl+0x1e9/0x3c0
[ 5077.982062] sock_do_ioctl+0xa8/0x140
[ 5077.986057] ? sock_ioctl+0x1c0/0x300
[ 5077.990051] sock_ioctl+0x1c0/0x300
[ 5077.993864] ? __handle_mm_fault+0xa82/0xfd0
[ 5077.998462] ? do_vfs_ioctl+0x8d/0x5e0
[ 5078.002550] do_vfs_ioctl+0x8d/0x5e0
[ 5078.006456] ? handle_mm_fault+0xd0/0x210
[ 5078.010790] ksys_ioctl+0x70/0x80
[ 5078.014429] __x64_sys_ioctl+0x16/0x20
[ 5078.018505] do_syscall_64+0x42/0xf0
[ 5078.022411] entry_SYSCALL_64_after_hwframe+0x44/0xa9
[ 5078.027789] RIP: 0033:0x7f7a84394dc7
[ 5078.031703] Code: b3 66 90 48 8b 05 d9 00 2d 00 64 c7 00 26 00 00 00 48 c7 c0 ff ff ff ff c3 66 2e 0f 1f 84 00 00 00 00 00 b8 10 00 00 00 0f 05 <48> 3d 01 f0 ff ff 73 01 c3 48 8b 0d a9 00 2d 00 f7 d8 64 89 01 48
[ 5078.051437] RSP: 002b:00007ffd64d68338 EFLAGS: 00000246 ORIG_RAX: 0000000000000010
[ 5078.059596] RAX: ffffffffffffffda RBX: 000055c7d7099260 RCX: 00007f7a84394dc7
[ 5078.067062] RDX: 00007ffd64d684e0 RSI: 0000000000008946 RDI: 0000000000000003
[ 5078.074524] RBP: 00007ffd64d684d0 R08: ffffffffffffffb0 R09: 000055c7d7099260
[ 5078.081986] R10: 0000000000000000 R11: 0000000000000246 R12: 00000000000001a2
[ 5078.089447] R13: 0000000000000001 R14: 0000000000000000 R15: 00007ffd64d684e0
[ 5078.096911] ---[ end trace faf82a00c0d8b6b1 ]---
$ gdb ./drivers/net/ethernet/intel/i40e/i40e.ko
[...]
Reading symbols from ./drivers/net/ethernet/intel/i40e/i40e.ko...done.
(gdb) list *(i40e_get_strings)+0x477
0x17787 is in i40e_get_strings (drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1906).
1901 snprintf(data, ETH_GSTRING_LEN,
1902 "port.rx_priority_%u_xon_2_xoff", i);
1903 data += ETH_GSTRING_LEN;
1904 }
1905
1906 WARN_ONCE(p - data != i40e_get_stats_count(netdev) * ETH_GSTRING_LEN,
1907 "stat strings count mismatch!");
1908 }
1909
1910 static void i40e_get_priv_flag_strings(struct net_device *netdev, u8 *data)
^ permalink raw reply [flat|nested] 12+ messages in thread
* RE: [Intel-wired-lan] Issue with driver i40e stat strings count mismatch
2018-07-31 14:28 ` [Intel-wired-lan] " Jesper Dangaard Brouer
@ 2018-07-31 15:53 ` Wyborny, Carolyn
-1 siblings, 0 replies; 12+ messages in thread
From: Wyborny, Carolyn @ 2018-07-31 15:53 UTC (permalink / raw)
To: Jesper Dangaard Brouer, Stefan Assmann, Keller, Jacob E
Cc: netdev, intel-wired-lan, Topel, Bjorn, Keller, Jacob E
> -----Original Message-----
> From: Intel-wired-lan [mailto:intel-wired-lan-bounces@osuosl.org] On
> Behalf Of Jesper Dangaard Brouer
> Sent: Tuesday, July 31, 2018 7:29 AM
> To: Stefan Assmann <sassmann@kpanic.de>; Keller, Jacob E
> <jacob.e.keller@intel.com>
> Cc: netdev@vger.kernel.org; intel-wired-lan <intel-wired-
> lan@lists.osuosl.org>; brouer@redhat.com; Topel, Bjorn
> <bjorn.topel@intel.com>
> Subject: Re: [Intel-wired-lan] Issue with driver i40e stat strings count
> mismatch
>
>
> On Tue, 31 Jul 2018 09:05:40 +0200 Stefan Assmann
> <sassmann@kpanic.de> wrote:
>
> > From: Stefan Assmann <sassmann@kpanic.de>
> > To: Jesper Dangaard Brouer <brouer@redhat.com>, Jeff Kirsher
> <jeffrey.t.kirsher@intel.com>
> > Cc: Björn Töpel <bjorn.topel@intel.com>,
> "alexander.h.duyck@intel.com" <alexander.h.duyck@intel.com>, intel-
> wired-lan <intel-wired-lan@lists.osuosl.org>, "netdev@vger.kernel.org"
> <netdev@vger.kernel.org>
> > Subject: Re: Issue with driver i40e stat strings count mismatch
> > Date: Tue, 31 Jul 2018 09:05:40 +0200
> > User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
> > Thunderbird/52.9.1
> > Message-ID: <64be8e6a-c285-2864-fd91-356eba645acd@kpanic.de>
> >
> > On 10.07.2018 13:17, Jesper Dangaard Brouer wrote:
> > > Hi Intel-fokes,
> > >
> > > Your i40e driver have issues with it's ethtool stats. A warning
> > > triggers at drivers/net/ethernet/intel/i40e/i40e_ethtool.c line 1907
> > > (see splash below) in func i40e_get_stat_strings().
> >
> > Hi Jesper,
> >
> > I ran into the same issue. Here's my proposed fix.
>
> Thanks for following up Stefan :-)
>
> I'm hoping some Intel people will look at evaluating this fix? ...
>
Thanks, we have a patch in process for this fix and other related ones from Jake Keller. We'll expedite it.
Carolyn
Carolyn Wyborny
Linux Development
Networking Division
Intel Corporation
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Intel-wired-lan] Issue with driver i40e stat strings count mismatch
@ 2018-07-31 15:53 ` Wyborny, Carolyn
0 siblings, 0 replies; 12+ messages in thread
From: Wyborny, Carolyn @ 2018-07-31 15:53 UTC (permalink / raw)
To: intel-wired-lan
> -----Original Message-----
> From: Intel-wired-lan [mailto:intel-wired-lan-bounces at osuosl.org] On
> Behalf Of Jesper Dangaard Brouer
> Sent: Tuesday, July 31, 2018 7:29 AM
> To: Stefan Assmann <sassmann@kpanic.de>; Keller, Jacob E
> <jacob.e.keller@intel.com>
> Cc: netdev at vger.kernel.org; intel-wired-lan <intel-wired-
> lan at lists.osuosl.org>; brouer at redhat.com; Topel, Bjorn
> <bjorn.topel@intel.com>
> Subject: Re: [Intel-wired-lan] Issue with driver i40e stat strings count
> mismatch
>
>
> On Tue, 31 Jul 2018 09:05:40 +0200 Stefan Assmann
> <sassmann@kpanic.de> wrote:
>
> > From: Stefan Assmann <sassmann@kpanic.de>
> > To: Jesper Dangaard Brouer <brouer@redhat.com>, Jeff Kirsher
> <jeffrey.t.kirsher@intel.com>
> > Cc: Bj?rn T?pel <bjorn.topel@intel.com>,
> "alexander.h.duyck at intel.com" <alexander.h.duyck@intel.com>, intel-
> wired-lan <intel-wired-lan@lists.osuosl.org>, "netdev at vger.kernel.org"
> <netdev@vger.kernel.org>
> > Subject: Re: Issue with driver i40e stat strings count mismatch
> > Date: Tue, 31 Jul 2018 09:05:40 +0200
> > User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
> > Thunderbird/52.9.1
> > Message-ID: <64be8e6a-c285-2864-fd91-356eba645acd@kpanic.de>
> >
> > On 10.07.2018 13:17, Jesper Dangaard Brouer wrote:
> > > Hi Intel-fokes,
> > >
> > > Your i40e driver have issues with it's ethtool stats. A warning
> > > triggers at drivers/net/ethernet/intel/i40e/i40e_ethtool.c line 1907
> > > (see splash below) in func i40e_get_stat_strings().
> >
> > Hi Jesper,
> >
> > I ran into the same issue. Here's my proposed fix.
>
> Thanks for following up Stefan :-)
>
> I'm hoping some Intel people will look at evaluating this fix? ...
>
Thanks, we have a patch in process for this fix and other related ones from Jake Keller. We'll expedite it.
Carolyn
Carolyn Wyborny
Linux Development
Networking Division
Intel Corporation
^ permalink raw reply [flat|nested] 12+ messages in thread
* RE: [Intel-wired-lan] Issue with driver i40e stat strings count mismatch
2018-07-31 15:53 ` Wyborny, Carolyn
@ 2018-07-31 18:29 ` Keller, Jacob E
-1 siblings, 0 replies; 12+ messages in thread
From: Keller, Jacob E @ 2018-07-31 18:29 UTC (permalink / raw)
To: Wyborny, Carolyn, Jesper Dangaard Brouer, Stefan Assmann
Cc: netdev, intel-wired-lan, Topel, Bjorn
> -----Original Message-----
> From: Wyborny, Carolyn
> Sent: Tuesday, July 31, 2018 8:53 AM
> To: Jesper Dangaard Brouer <brouer@redhat.com>; Stefan Assmann
> <sassmann@kpanic.de>; Keller, Jacob E <jacob.e.keller@intel.com>
> Cc: netdev@vger.kernel.org; intel-wired-lan <intel-wired-lan@lists.osuosl.org>;
> Topel, Bjorn <bjorn.topel@intel.com>; Keller, Jacob E
> <jacob.e.keller@intel.com>
> Subject: RE: [Intel-wired-lan] Issue with driver i40e stat strings count mismatch
>
> > -----Original Message-----
> > From: Intel-wired-lan [mailto:intel-wired-lan-bounces@osuosl.org] On
> > Behalf Of Jesper Dangaard Brouer
> > Sent: Tuesday, July 31, 2018 7:29 AM
> > To: Stefan Assmann <sassmann@kpanic.de>; Keller, Jacob E
> > <jacob.e.keller@intel.com>
> > Cc: netdev@vger.kernel.org; intel-wired-lan <intel-wired-
> > lan@lists.osuosl.org>; brouer@redhat.com; Topel, Bjorn
> > <bjorn.topel@intel.com>
> > Subject: Re: [Intel-wired-lan] Issue with driver i40e stat strings count
> > mismatch
> >
> >
> > On Tue, 31 Jul 2018 09:05:40 +0200 Stefan Assmann
> > <sassmann@kpanic.de> wrote:
> >
> > > From: Stefan Assmann <sassmann@kpanic.de>
> > > To: Jesper Dangaard Brouer <brouer@redhat.com>, Jeff Kirsher
> > <jeffrey.t.kirsher@intel.com>
> > > Cc: Björn Töpel <bjorn.topel@intel.com>,
> > "alexander.h.duyck@intel.com" <alexander.h.duyck@intel.com>, intel-
> > wired-lan <intel-wired-lan@lists.osuosl.org>, "netdev@vger.kernel.org"
> > <netdev@vger.kernel.org>
> > > Subject: Re: Issue with driver i40e stat strings count mismatch
> > > Date: Tue, 31 Jul 2018 09:05:40 +0200
> > > User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
> > > Thunderbird/52.9.1
> > > Message-ID: <64be8e6a-c285-2864-fd91-356eba645acd@kpanic.de>
> > >
> > > On 10.07.2018 13:17, Jesper Dangaard Brouer wrote:
> > > > Hi Intel-fokes,
> > > >
> > > > Your i40e driver have issues with it's ethtool stats. A warning
> > > > triggers at drivers/net/ethernet/intel/i40e/i40e_ethtool.c line 1907
> > > > (see splash below) in func i40e_get_stat_strings().
> > >
> > > Hi Jesper,
> > >
> > > I ran into the same issue. Here's my proposed fix.
> >
> > Thanks for following up Stefan :-)
> >
> > I'm hoping some Intel people will look at evaluating this fix? ...
> >
> Thanks, we have a patch in process for this fix and other related ones from Jake
> Keller. We'll expedite it.
>
> Carolyn
>
> Carolyn Wyborny
> Linux Development
> Networking Division
> Intel Corporation
>
>
Yea I found tihs issue a while ago and we had a patch internally. I suspect it just got lost in a shuffle earlier
I think we have an equivalent patch, but I don't care which gets applied.
Thanks,
Jake
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Intel-wired-lan] Issue with driver i40e stat strings count mismatch
@ 2018-07-31 18:29 ` Keller, Jacob E
0 siblings, 0 replies; 12+ messages in thread
From: Keller, Jacob E @ 2018-07-31 18:29 UTC (permalink / raw)
To: intel-wired-lan
> -----Original Message-----
> From: Wyborny, Carolyn
> Sent: Tuesday, July 31, 2018 8:53 AM
> To: Jesper Dangaard Brouer <brouer@redhat.com>; Stefan Assmann
> <sassmann@kpanic.de>; Keller, Jacob E <jacob.e.keller@intel.com>
> Cc: netdev at vger.kernel.org; intel-wired-lan <intel-wired-lan@lists.osuosl.org>;
> Topel, Bjorn <bjorn.topel@intel.com>; Keller, Jacob E
> <jacob.e.keller@intel.com>
> Subject: RE: [Intel-wired-lan] Issue with driver i40e stat strings count mismatch
>
> > -----Original Message-----
> > From: Intel-wired-lan [mailto:intel-wired-lan-bounces at osuosl.org] On
> > Behalf Of Jesper Dangaard Brouer
> > Sent: Tuesday, July 31, 2018 7:29 AM
> > To: Stefan Assmann <sassmann@kpanic.de>; Keller, Jacob E
> > <jacob.e.keller@intel.com>
> > Cc: netdev at vger.kernel.org; intel-wired-lan <intel-wired-
> > lan at lists.osuosl.org>; brouer at redhat.com; Topel, Bjorn
> > <bjorn.topel@intel.com>
> > Subject: Re: [Intel-wired-lan] Issue with driver i40e stat strings count
> > mismatch
> >
> >
> > On Tue, 31 Jul 2018 09:05:40 +0200 Stefan Assmann
> > <sassmann@kpanic.de> wrote:
> >
> > > From: Stefan Assmann <sassmann@kpanic.de>
> > > To: Jesper Dangaard Brouer <brouer@redhat.com>, Jeff Kirsher
> > <jeffrey.t.kirsher@intel.com>
> > > Cc: Bj?rn T?pel <bjorn.topel@intel.com>,
> > "alexander.h.duyck at intel.com" <alexander.h.duyck@intel.com>, intel-
> > wired-lan <intel-wired-lan@lists.osuosl.org>, "netdev at vger.kernel.org"
> > <netdev@vger.kernel.org>
> > > Subject: Re: Issue with driver i40e stat strings count mismatch
> > > Date: Tue, 31 Jul 2018 09:05:40 +0200
> > > User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
> > > Thunderbird/52.9.1
> > > Message-ID: <64be8e6a-c285-2864-fd91-356eba645acd@kpanic.de>
> > >
> > > On 10.07.2018 13:17, Jesper Dangaard Brouer wrote:
> > > > Hi Intel-fokes,
> > > >
> > > > Your i40e driver have issues with it's ethtool stats. A warning
> > > > triggers at drivers/net/ethernet/intel/i40e/i40e_ethtool.c line 1907
> > > > (see splash below) in func i40e_get_stat_strings().
> > >
> > > Hi Jesper,
> > >
> > > I ran into the same issue. Here's my proposed fix.
> >
> > Thanks for following up Stefan :-)
> >
> > I'm hoping some Intel people will look at evaluating this fix? ...
> >
> Thanks, we have a patch in process for this fix and other related ones from Jake
> Keller. We'll expedite it.
>
> Carolyn
>
> Carolyn Wyborny
> Linux Development
> Networking Division
> Intel Corporation
>
>
Yea I found tihs issue a while ago and we had a patch internally. I suspect it just got lost in a shuffle earlier
I think we have an equivalent patch, but I don't care which gets applied.
Thanks,
Jake
^ permalink raw reply [flat|nested] 12+ messages in thread
* RE: Issue with driver i40e stat strings count mismatch
2018-07-31 7:05 ` [Intel-wired-lan] " Stefan Assmann
@ 2018-07-31 18:35 ` Keller, Jacob E
-1 siblings, 0 replies; 12+ messages in thread
From: Keller, Jacob E @ 2018-07-31 18:35 UTC (permalink / raw)
To: Stefan Assmann, Jesper Dangaard Brouer, Kirsher, Jeffrey T
Cc: Topel, Bjorn, Duyck, Alexander H, intel-wired-lan, netdev
> -----Original Message-----
> From: netdev-owner@vger.kernel.org [mailto:netdev-owner@vger.kernel.org]
> On Behalf Of Stefan Assmann
> Sent: Tuesday, July 31, 2018 12:06 AM
> To: Jesper Dangaard Brouer <brouer@redhat.com>; Kirsher, Jeffrey T
> <jeffrey.t.kirsher@intel.com>
> Cc: Topel, Bjorn <bjorn.topel@intel.com>; Duyck, Alexander H
> <alexander.h.duyck@intel.com>; intel-wired-lan <intel-wired-
> lan@lists.osuosl.org>; netdev@vger.kernel.org
> Subject: Re: Issue with driver i40e stat strings count mismatch
>
> On 10.07.2018 13:17, Jesper Dangaard Brouer wrote:
> > Hi Intel-fokes,
> >
> > Your i40e driver have issues with it's ethtool stats. A warning
> > triggers at drivers/net/ethernet/intel/i40e/i40e_ethtool.c line 1907
> > (see splash below) in func i40e_get_stat_strings().
>
> Hi Jesper,
>
> I ran into the same issue. Here's my proposed fix.
>
> From 46c74c25496bab06712641c7b2b6b34e365397a2 Mon Sep 17 00:00:00 2001
> From: Stefan Assmann <sassmann@kpanic.de>
> Date: Mon, 30 Jul 2018 21:38:43 +0200
> Subject: [PATCH] i40e: fix i40e_get_stat_strings strings count warning
>
> The current code calculates p - data, which results in a negative value.
> Therefore the WARN_ONCE condition will always be true.
> Fix this by calculating data - p instead.
>
> Fixes: 9b10df596bd4 ("i40e: use WARN_ONCE to replace the commented
> BUG_ON size check")
>
> Signed-off-by: Stefan Assmann <sassmann@kpanic.de>
> ---
> drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
> b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
> index 6947a2a571cb..5d670f4ce5ac 100644
> --- a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
> +++ b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
> @@ -1903,7 +1903,7 @@ static void i40e_get_stat_strings(struct net_device
> *netdev, u8 *data)
> data += ETH_GSTRING_LEN;
> }
>
> - WARN_ONCE(p - data != i40e_get_stats_count(netdev) *
> ETH_GSTRING_LEN,
> + WARN_ONCE(data - p != i40e_get_stats_count(netdev) *
> ETH_GSTRING_LEN,
> "stat strings count mismatch!");
> }
>
Thanks Stefan. Sorry about this one. I had a fix for this a while back internally but I think it somehow got lost in the shuffle. It's now in the process of being posted.
It's the same fix though, so I don't feel strongly about which gets applied. Thus...
Acked-by: Jacob Keller <jacob.e.keller@intel.com>
Thanks,
Jake
> --
> 2.17.1
>
> >
> > [ 5077.779518] ------------[ cut here ]------------
> > [ 5077.784493] stat strings count mismatch!
> > [ 5077.784529] WARNING: CPU: 0 PID: 2293 at
> drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1907
> i40e_get_strings+0x477/0x4b0 [i40e]
> > [ 5077.800941] Modules linked in: act_gact cls_u32 sch_ingress xt_tcpudp
> iptable_raw ip_tables x_tables tun nfnetlink bridge stp llc bpfilter sunrpc
> coretemp kvm_intel kvm irqbypass intel_cstate intel_uncore intel_rapl_perf
> pcspkr i2c_i801 wmi ipmi_si ipmi_devintf ipmi_msghandler acpi_pad
> sch_fq_codel ixgbe mlx5_core mlxfw i40e devlink hid_generic igb mdio
> i2c_algo_bit ptp sd_mod i2c_core pps_core [last unloaded: x_tables]
> > [ 5077.839833] CPU: 0 PID: 2293 Comm: ethtool Not tainted 4.18.0-rc3-net-next-
> EdwardCree01+ #484
> > [ 5077.848962] Hardware name: Supermicro Super Server/X10SRi-F, BIOS 2.0a
> 08/01/2016
> > [ 5077.857049] RIP: 0010:i40e_get_strings+0x477/0x4b0 [i40e]
> > [ 5077.862776] Code: 98 49 39 c4 0f 84 2e fc ff ff 80 3d b3 da 03 00 00 0f 85 21 fc ff
> ff 48 c7 c7 24 42 19 a0 c6 05 9f da 03 00 01 e8 e9 9c ee e0 <0f> 0b e9 07 fc ff ff 48 83
> c4 10 48 c7 c1 80 01 19 a0 be 20 00 00
> > [ 5077.882506] RSP: 0018:ffffc90003af3c18 EFLAGS: 00010296
> > [ 5077.888063] RAX: 000000000000001c RBX: ffffc90003ce1440 RCX:
> 0000000000000006
> > [ 5077.895528] RDX: 0000000000000007 RSI: 0000000000000096 RDI:
> ffff88087ca15530
> > [ 5077.902991] RBP: ffffc90003ce1440 R08: 000000000000001c R09:
> 0000000000000411
> > [ 5077.910453] R10: 000fffffffe00000 R11: ffffffff82a4e66d R12: ffffffffffffcbc0
> > [ 5077.917913] R13: ffff88087c50f000 R14: 0000000000000008 R15:
> ffffffffa0199620
> > [ 5077.925376] FS: 00007f7a84bcb740(0000) GS:ffff88087ca00000(0000)
> knlGS:0000000000000000
> > [ 5077.934061] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> > [ 5077.940133] CR2: 000055c7d709b000 CR3: 000000081acd0004 CR4:
> 00000000003606f0
> > [ 5077.947609] DR0: 0000000000000000 DR1: 0000000000000000 DR2:
> 0000000000000000
> > [ 5077.955070] DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7:
> 0000000000000400
> > [ 5077.962531] Call Trace:
> > [ 5077.965309] dev_ethtool+0xf4e/0x2430
> > [ 5077.969305] ? get_page_from_freelist+0x2bb/0x1240
> > [ 5077.974428] ? dev_ioctl+0x1e9/0x3c0
> > [ 5077.978332] dev_ioctl+0x1e9/0x3c0
> > [ 5077.982062] sock_do_ioctl+0xa8/0x140
> > [ 5077.986057] ? sock_ioctl+0x1c0/0x300
> > [ 5077.990051] sock_ioctl+0x1c0/0x300
> > [ 5077.993864] ? __handle_mm_fault+0xa82/0xfd0
> > [ 5077.998462] ? do_vfs_ioctl+0x8d/0x5e0
> > [ 5078.002550] do_vfs_ioctl+0x8d/0x5e0
> > [ 5078.006456] ? handle_mm_fault+0xd0/0x210
> > [ 5078.010790] ksys_ioctl+0x70/0x80
> > [ 5078.014429] __x64_sys_ioctl+0x16/0x20
> > [ 5078.018505] do_syscall_64+0x42/0xf0
> > [ 5078.022411] entry_SYSCALL_64_after_hwframe+0x44/0xa9
> > [ 5078.027789] RIP: 0033:0x7f7a84394dc7
> > [ 5078.031703] Code: b3 66 90 48 8b 05 d9 00 2d 00 64 c7 00 26 00 00 00 48 c7 c0 ff
> ff ff ff c3 66 2e 0f 1f 84 00 00 00 00 00 b8 10 00 00 00 0f 05 <48> 3d 01 f0 ff ff 73 01 c3
> 48 8b 0d a9 00 2d 00 f7 d8 64 89 01 48
> > [ 5078.051437] RSP: 002b:00007ffd64d68338 EFLAGS: 00000246 ORIG_RAX:
> 0000000000000010
> > [ 5078.059596] RAX: ffffffffffffffda RBX: 000055c7d7099260 RCX:
> 00007f7a84394dc7
> > [ 5078.067062] RDX: 00007ffd64d684e0 RSI: 0000000000008946 RDI:
> 0000000000000003
> > [ 5078.074524] RBP: 00007ffd64d684d0 R08: ffffffffffffffb0 R09:
> 000055c7d7099260
> > [ 5078.081986] R10: 0000000000000000 R11: 0000000000000246 R12:
> 00000000000001a2
> > [ 5078.089447] R13: 0000000000000001 R14: 0000000000000000 R15:
> 00007ffd64d684e0
> > [ 5078.096911] ---[ end trace faf82a00c0d8b6b1 ]---
> >
> > $ gdb ./drivers/net/ethernet/intel/i40e/i40e.ko
> > [...]
> > Reading symbols from ./drivers/net/ethernet/intel/i40e/i40e.ko...done.
> > (gdb) list *(i40e_get_strings)+0x477
> > 0x17787 is in i40e_get_strings
> (drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1906).
> > 1901 snprintf(data, ETH_GSTRING_LEN,
> > 1902 "port.rx_priority_%u_xon_2_xoff", i);
> > 1903 data += ETH_GSTRING_LEN;
> > 1904 }
> > 1905
> > 1906 WARN_ONCE(p - data != i40e_get_stats_count(netdev) *
> ETH_GSTRING_LEN,
> > 1907 "stat strings count mismatch!");
> > 1908 }
> > 1909
> > 1910 static void i40e_get_priv_flag_strings(struct net_device *netdev, u8
> *data)
> >
^ permalink raw reply [flat|nested] 12+ messages in thread
* [Intel-wired-lan] Issue with driver i40e stat strings count mismatch
@ 2018-07-31 18:35 ` Keller, Jacob E
0 siblings, 0 replies; 12+ messages in thread
From: Keller, Jacob E @ 2018-07-31 18:35 UTC (permalink / raw)
To: intel-wired-lan
> -----Original Message-----
> From: netdev-owner at vger.kernel.org [mailto:netdev-owner at vger.kernel.org]
> On Behalf Of Stefan Assmann
> Sent: Tuesday, July 31, 2018 12:06 AM
> To: Jesper Dangaard Brouer <brouer@redhat.com>; Kirsher, Jeffrey T
> <jeffrey.t.kirsher@intel.com>
> Cc: Topel, Bjorn <bjorn.topel@intel.com>; Duyck, Alexander H
> <alexander.h.duyck@intel.com>; intel-wired-lan <intel-wired-
> lan at lists.osuosl.org>; netdev at vger.kernel.org
> Subject: Re: Issue with driver i40e stat strings count mismatch
>
> On 10.07.2018 13:17, Jesper Dangaard Brouer wrote:
> > Hi Intel-fokes,
> >
> > Your i40e driver have issues with it's ethtool stats. A warning
> > triggers at drivers/net/ethernet/intel/i40e/i40e_ethtool.c line 1907
> > (see splash below) in func i40e_get_stat_strings().
>
> Hi Jesper,
>
> I ran into the same issue. Here's my proposed fix.
>
> From 46c74c25496bab06712641c7b2b6b34e365397a2 Mon Sep 17 00:00:00 2001
> From: Stefan Assmann <sassmann@kpanic.de>
> Date: Mon, 30 Jul 2018 21:38:43 +0200
> Subject: [PATCH] i40e: fix i40e_get_stat_strings strings count warning
>
> The current code calculates p - data, which results in a negative value.
> Therefore the WARN_ONCE condition will always be true.
> Fix this by calculating data - p instead.
>
> Fixes: 9b10df596bd4 ("i40e: use WARN_ONCE to replace the commented
> BUG_ON size check")
>
> Signed-off-by: Stefan Assmann <sassmann@kpanic.de>
> ---
> drivers/net/ethernet/intel/i40e/i40e_ethtool.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
> b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
> index 6947a2a571cb..5d670f4ce5ac 100644
> --- a/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
> +++ b/drivers/net/ethernet/intel/i40e/i40e_ethtool.c
> @@ -1903,7 +1903,7 @@ static void i40e_get_stat_strings(struct net_device
> *netdev, u8 *data)
> data += ETH_GSTRING_LEN;
> }
>
> - WARN_ONCE(p - data != i40e_get_stats_count(netdev) *
> ETH_GSTRING_LEN,
> + WARN_ONCE(data - p != i40e_get_stats_count(netdev) *
> ETH_GSTRING_LEN,
> "stat strings count mismatch!");
> }
>
Thanks Stefan. Sorry about this one. I had a fix for this a while back internally but I think it somehow got lost in the shuffle. It's now in the process of being posted.
It's the same fix though, so I don't feel strongly about which gets applied. Thus...
Acked-by: Jacob Keller <jacob.e.keller@intel.com>
Thanks,
Jake
> --
> 2.17.1
>
> >
> > [ 5077.779518] ------------[ cut here ]------------
> > [ 5077.784493] stat strings count mismatch!
> > [ 5077.784529] WARNING: CPU: 0 PID: 2293 at
> drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1907
> i40e_get_strings+0x477/0x4b0 [i40e]
> > [ 5077.800941] Modules linked in: act_gact cls_u32 sch_ingress xt_tcpudp
> iptable_raw ip_tables x_tables tun nfnetlink bridge stp llc bpfilter sunrpc
> coretemp kvm_intel kvm irqbypass intel_cstate intel_uncore intel_rapl_perf
> pcspkr i2c_i801 wmi ipmi_si ipmi_devintf ipmi_msghandler acpi_pad
> sch_fq_codel ixgbe mlx5_core mlxfw i40e devlink hid_generic igb mdio
> i2c_algo_bit ptp sd_mod i2c_core pps_core [last unloaded: x_tables]
> > [ 5077.839833] CPU: 0 PID: 2293 Comm: ethtool Not tainted 4.18.0-rc3-net-next-
> EdwardCree01+ #484
> > [ 5077.848962] Hardware name: Supermicro Super Server/X10SRi-F, BIOS 2.0a
> 08/01/2016
> > [ 5077.857049] RIP: 0010:i40e_get_strings+0x477/0x4b0 [i40e]
> > [ 5077.862776] Code: 98 49 39 c4 0f 84 2e fc ff ff 80 3d b3 da 03 00 00 0f 85 21 fc ff
> ff 48 c7 c7 24 42 19 a0 c6 05 9f da 03 00 01 e8 e9 9c ee e0 <0f> 0b e9 07 fc ff ff 48 83
> c4 10 48 c7 c1 80 01 19 a0 be 20 00 00
> > [ 5077.882506] RSP: 0018:ffffc90003af3c18 EFLAGS: 00010296
> > [ 5077.888063] RAX: 000000000000001c RBX: ffffc90003ce1440 RCX:
> 0000000000000006
> > [ 5077.895528] RDX: 0000000000000007 RSI: 0000000000000096 RDI:
> ffff88087ca15530
> > [ 5077.902991] RBP: ffffc90003ce1440 R08: 000000000000001c R09:
> 0000000000000411
> > [ 5077.910453] R10: 000fffffffe00000 R11: ffffffff82a4e66d R12: ffffffffffffcbc0
> > [ 5077.917913] R13: ffff88087c50f000 R14: 0000000000000008 R15:
> ffffffffa0199620
> > [ 5077.925376] FS: 00007f7a84bcb740(0000) GS:ffff88087ca00000(0000)
> knlGS:0000000000000000
> > [ 5077.934061] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> > [ 5077.940133] CR2: 000055c7d709b000 CR3: 000000081acd0004 CR4:
> 00000000003606f0
> > [ 5077.947609] DR0: 0000000000000000 DR1: 0000000000000000 DR2:
> 0000000000000000
> > [ 5077.955070] DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7:
> 0000000000000400
> > [ 5077.962531] Call Trace:
> > [ 5077.965309] dev_ethtool+0xf4e/0x2430
> > [ 5077.969305] ? get_page_from_freelist+0x2bb/0x1240
> > [ 5077.974428] ? dev_ioctl+0x1e9/0x3c0
> > [ 5077.978332] dev_ioctl+0x1e9/0x3c0
> > [ 5077.982062] sock_do_ioctl+0xa8/0x140
> > [ 5077.986057] ? sock_ioctl+0x1c0/0x300
> > [ 5077.990051] sock_ioctl+0x1c0/0x300
> > [ 5077.993864] ? __handle_mm_fault+0xa82/0xfd0
> > [ 5077.998462] ? do_vfs_ioctl+0x8d/0x5e0
> > [ 5078.002550] do_vfs_ioctl+0x8d/0x5e0
> > [ 5078.006456] ? handle_mm_fault+0xd0/0x210
> > [ 5078.010790] ksys_ioctl+0x70/0x80
> > [ 5078.014429] __x64_sys_ioctl+0x16/0x20
> > [ 5078.018505] do_syscall_64+0x42/0xf0
> > [ 5078.022411] entry_SYSCALL_64_after_hwframe+0x44/0xa9
> > [ 5078.027789] RIP: 0033:0x7f7a84394dc7
> > [ 5078.031703] Code: b3 66 90 48 8b 05 d9 00 2d 00 64 c7 00 26 00 00 00 48 c7 c0 ff
> ff ff ff c3 66 2e 0f 1f 84 00 00 00 00 00 b8 10 00 00 00 0f 05 <48> 3d 01 f0 ff ff 73 01 c3
> 48 8b 0d a9 00 2d 00 f7 d8 64 89 01 48
> > [ 5078.051437] RSP: 002b:00007ffd64d68338 EFLAGS: 00000246 ORIG_RAX:
> 0000000000000010
> > [ 5078.059596] RAX: ffffffffffffffda RBX: 000055c7d7099260 RCX:
> 00007f7a84394dc7
> > [ 5078.067062] RDX: 00007ffd64d684e0 RSI: 0000000000008946 RDI:
> 0000000000000003
> > [ 5078.074524] RBP: 00007ffd64d684d0 R08: ffffffffffffffb0 R09:
> 000055c7d7099260
> > [ 5078.081986] R10: 0000000000000000 R11: 0000000000000246 R12:
> 00000000000001a2
> > [ 5078.089447] R13: 0000000000000001 R14: 0000000000000000 R15:
> 00007ffd64d684e0
> > [ 5078.096911] ---[ end trace faf82a00c0d8b6b1 ]---
> >
> > $ gdb ./drivers/net/ethernet/intel/i40e/i40e.ko
> > [...]
> > Reading symbols from ./drivers/net/ethernet/intel/i40e/i40e.ko...done.
> > (gdb) list *(i40e_get_strings)+0x477
> > 0x17787 is in i40e_get_strings
> (drivers/net/ethernet/intel/i40e/i40e_ethtool.c:1906).
> > 1901 snprintf(data, ETH_GSTRING_LEN,
> > 1902 "port.rx_priority_%u_xon_2_xoff", i);
> > 1903 data += ETH_GSTRING_LEN;
> > 1904 }
> > 1905
> > 1906 WARN_ONCE(p - data != i40e_get_stats_count(netdev) *
> ETH_GSTRING_LEN,
> > 1907 "stat strings count mismatch!");
> > 1908 }
> > 1909
> > 1910 static void i40e_get_priv_flag_strings(struct net_device *netdev, u8
> *data)
> >
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2018-07-31 20:17 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-07-10 11:17 Issue with driver i40e stat strings count mismatch Jesper Dangaard Brouer
2018-07-10 11:17 ` [Intel-wired-lan] " Jesper Dangaard Brouer
2018-07-31 7:05 ` Stefan Assmann
2018-07-31 7:05 ` [Intel-wired-lan] " Stefan Assmann
2018-07-31 14:28 ` Jesper Dangaard Brouer
2018-07-31 14:28 ` [Intel-wired-lan] " Jesper Dangaard Brouer
2018-07-31 15:53 ` Wyborny, Carolyn
2018-07-31 15:53 ` Wyborny, Carolyn
2018-07-31 18:29 ` Keller, Jacob E
2018-07-31 18:29 ` Keller, Jacob E
2018-07-31 18:35 ` Keller, Jacob E
2018-07-31 18:35 ` [Intel-wired-lan] " Keller, Jacob E
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.