* Re: [Patch net] ipv4: restore rt->fi for reference counting
From: Julian Anastasov @ 2017-05-12 6:39 UTC (permalink / raw)
To: Cong Wang
Cc: Eric Dumazet, David Miller, Linux Kernel Network Developers,
Andrey Konovalov, Eric Dumazet
In-Reply-To: <CAM_iQpWxDKHF4N1xor7qj+5s91GrctgcGnzN0buw18qs7__-gQ@mail.gmail.com>
Hello,
On Thu, 11 May 2017, Cong Wang wrote:
> On Thu, May 11, 2017 at 5:07 PM, Cong Wang <xiyou.wangcong@gmail.com> wrote:
> > So, if I understand you correctly it is safe to NULL'ing
> > nh_dev in NETDEV_UNREGISTER_FINAL, right?
> >
> > If still not, how about transfer nh_dev's to loopback_dev
> > too in NETDEV_UNREGISTER? Like we transfer dst->dev.
> >
> > I don't want to touch the fast path to check for NULL, as
> > it will change more code and slow down performance.
>
> Finally I come up with the attached patch. Please let me know if
> I still miss anything.
fib_flush will unlink the FIB infos at NETDEV_UNREGISTER
time, so we can not see them in any hash tables later on
NETDEV_UNREGISTER_FINAL. fib_put_nh_devs() can not work
except if moving NHs in another hash table but that should
not be needed.
I'm thinking for the following solution which
is a bit hackish:
- on NETDEV_UNREGISTER we want to put the nh_dev references,
so fib_release_info is a good place. But as fib_release_info
is not always called, we will have two places that put
references. We can use such hack:
- for example, use nh_oif to know if dev_put is
already called
- fib_release_info() should set nh_oif to 0 because
it will now call dev_put without clearing nh_dev
- free_fib_info_rcu() will not call dev_put if nh_oif is 0:
if (nexthop_nh->nh_dev && nexthop_nh->nh_oif)
dev_put(nexthop_nh->nh_dev);
- as fi is unlinked, there is no chance fib_info_hashfn()
to use the cleared nh_oif for hashing
- we expect noone to touch nh_dev fields after fi is
unlinked, this includes free_fib_info and free_fib_info_rcu
What we achieve:
- between NETDEV_UNREGISTER and NETDEV_UNREGISTER_FINAL nh_dev
still points to our dev (and not to loopback, I think, this
answers your question in previous email), so route lookups
will not find loopback in the FIB tree. We do not want
some packets to be wrongly rerouted. Even if we don't
hold dev reference, the RCU grace period helps here.
- fib_dev/nh_dev != NULL checks are not needed, so this addresses
Eric's concerns. BTW, fib_route_seq_show actually checks
for fi->fib_dev, may be not in a safe way (lockless_dereference
needed?) but as we don't set nh_dev to NULL this is not
needed.
What more? What about nh_pcpu_rth_output and
nh_rth_input holding routes? We should think about
them too. I should think more if nh_oif trick can work
for them, may be not because nh_oif is optional...
May be we should purge them somehow?
Regards
^ permalink raw reply
* Re: [PATCH v4 1/4] can: m_can: move Message RAM initialization to function
From: Quentin Schulz @ 2017-05-12 6:37 UTC (permalink / raw)
To: wg, mkl, mario.huettel, socketcan
Cc: linux-can, netdev, linux-kernel, alexandre.belloni,
thomas.petazzoni
In-Reply-To: <20170505135033.8349-1-quentin.schulz@free-electrons.com>
Hi all,
On 05/05/2017 15:50, Quentin Schulz wrote:
> To avoid possible ECC/parity checksum errors when reading an
> uninitialized buffer, the entire Message RAM is initialized when probing
> the driver. This initialization is done in the same function reading the
> Device Tree properties.
>
> This patch moves the RAM initialization to a separate function so it can
> be called separately from device initialization from Device Tree.
>
> Signed-off-by: Quentin Schulz <quentin.schulz@free-electrons.com>
It's been a week since I sent this patch series. Any comments?
Thanks,
Quentin
> ---
>
> v4:
> - remove unused variables from m_can_of_parse_mram,
>
> drivers/net/can/m_can/m_can.c | 26 +++++++++++++++-----------
> 1 file changed, 15 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/net/can/m_can/m_can.c b/drivers/net/can/m_can/m_can.c
> index bf8fdaeb955e..5da1bdb202a3 100644
> --- a/drivers/net/can/m_can/m_can.c
> +++ b/drivers/net/can/m_can/m_can.c
> @@ -1489,11 +1489,23 @@ static int register_m_can_dev(struct net_device *dev)
> return register_candev(dev);
> }
>
> +static void m_can_init_ram(struct m_can_priv *priv)
> +{
> + int end, i, start;
> +
> + /* initialize the entire Message RAM in use to avoid possible
> + * ECC/parity checksum errors when reading an uninitialized buffer
> + */
> + start = priv->mcfg[MRAM_SIDF].off;
> + end = priv->mcfg[MRAM_TXB].off +
> + priv->mcfg[MRAM_TXB].num * TXB_ELEMENT_SIZE;
> + for (i = start; i < end; i += 4)
> + writel(0x0, priv->mram_base + i);
> +}
> +
> static void m_can_of_parse_mram(struct m_can_priv *priv,
> const u32 *mram_config_vals)
> {
> - int i, start, end;
> -
> priv->mcfg[MRAM_SIDF].off = mram_config_vals[0];
> priv->mcfg[MRAM_SIDF].num = mram_config_vals[1];
> priv->mcfg[MRAM_XIDF].off = priv->mcfg[MRAM_SIDF].off +
> @@ -1529,15 +1541,7 @@ static void m_can_of_parse_mram(struct m_can_priv *priv,
> priv->mcfg[MRAM_TXE].off, priv->mcfg[MRAM_TXE].num,
> priv->mcfg[MRAM_TXB].off, priv->mcfg[MRAM_TXB].num);
>
> - /* initialize the entire Message RAM in use to avoid possible
> - * ECC/parity checksum errors when reading an uninitialized buffer
> - */
> - start = priv->mcfg[MRAM_SIDF].off;
> - end = priv->mcfg[MRAM_TXB].off +
> - priv->mcfg[MRAM_TXB].num * TXB_ELEMENT_SIZE;
> - for (i = start; i < end; i += 4)
> - writel(0x0, priv->mram_base + i);
> -
> + m_can_init_ram(priv);
> }
>
> static int m_can_plat_probe(struct platform_device *pdev)
>
--
Quentin Schulz, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
^ permalink raw reply
* Re: [PATCH v5 15/17] dt-bindings: qca7000: append UART interface to binding
From: Michael Heimpold @ 2017-05-12 6:15 UTC (permalink / raw)
To: Jakub Kicinski
Cc: Rob Herring, Stefan Wahren, Mark Rutland, Greg Kroah-Hartman,
Jiri Slaby, linux-serial, linux-kernel, netdev, devicetree
In-Reply-To: <20170511194541.4e58cb7d@cakuba.netronome.com>
Hi,
Zitat von Jakub Kicinski <kubakici@wp.pl>:
> On Thu, 11 May 2017 21:12:22 +0200, Michael Heimpold wrote:
>> Am Mittwoch, 10. Mai 2017, 10:53:26 CEST schrieb Stefan Wahren:
>> > This merges the serdev binding for the QCA7000 UART driver (Ethernet over
>> > UART) into the existing document.
>> >
>> > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
>> > ---
>> > .../devicetree/bindings/net/qca-qca7000.txt | 32
>> > ++++++++++++++++++++++ 1 file changed, 32 insertions(+)
>> >
>> > diff --git a/Documentation/devicetree/bindings/net/qca-qca7000.txt
>> > b/Documentation/devicetree/bindings/net/qca-qca7000.txt index
>> > a37f656..08364c3 100644
>> > --- a/Documentation/devicetree/bindings/net/qca-qca7000.txt
>> > +++ b/Documentation/devicetree/bindings/net/qca-qca7000.txt
>> > @@ -54,3 +54,35 @@ ssp2: spi@80014000 {
>> > local-mac-address = [ A0 B0 C0 D0 E0 F0 ];
>> > };
>> > };
>> > +
>> > +(b) Ethernet over UART
>> > +
>> > +In order to use the QCA7000 as UART slave it must be defined as
>> a child of
>> > a +UART master in the device tree. It is possible to preconfigure the UART
>> > +settings of the QCA7000 firmware, but it's not possible to change them
>> > during +runtime.
>> > +
>> > +Required properties:
>> > +- compatible : Should be "qca,qca7000-uart"
>>
>> I already discussed this with Stefan off-list a little bit, but I would like
>> to bring this to a broader audience: I'm not sure whether the compatible
>> should contain the "-uart" suffix, because the hardware chip is the
>> very same
>> QCA7000 chip which can also be used with SPI protocol.
>> The only difference is the loaded firmware within the chip which can either
>> speak SPI or UART protocol (but not both at the same time - due to shared
>> pins). So the hardware design decides which interface type is used.
>>
>> At the moment, this patch series adds a dedicated driver for the UART
>> protocol, in parallel to the existing SPI driver. So a different compatible
>> string is needed here to match against the new driver.
>>
>> An alternative approach would be to re-use the existing compatible string
>> "qca,qca7000" for both, the SPI and UART protocol, because a "smarter"
>> (combined) driver would detect which protocol to use. For example the driver
>> could check for spi-cpha and/or spi-cpol which are required for SPI
>> protocol:
>> if these exists the driver could assume that SPI must be used, if both are
>> missing then UART protocol should be used.
>> (This way it would not be necessary to check whether the node is a child of
>> a SPI or UART master node - but maybe this is even easier - I don't know)
>>
>> Or in shorter words: my concern is that while "qca7000-uart" describes the
>> hardware, it's too closely coupled to the driver implementation. Having
>> some feedback of the experts would be nice :-)
>
> I'm no expert, but devices which can do both I2C and SPI are quite
> common, and they usually have the same compatible string for both
> buses.
do you have an example driver at hand? I only found GPIO mcp23s08 driver,
which can handle both I2C and SPI chips, but there are different compatible
strings used to distinguish several chip models.
Regards,
Michael
^ permalink raw reply
* [PATCH v4] net/mlx4_core: Use min3 to select number of MSI-X vectors
From: Yuval Shaia @ 2017-05-12 6:10 UTC (permalink / raw)
To: yishaih-VPRAkNaXOzVWk0Htik3J/w, netdev-u79uwXL29TY76Z2rM5mHXA,
linux-rdma-u79uwXL29TY76Z2rM5mHXA
Signed-off-by: Yuval Shaia <yuval.shaia-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>
---
v0 -> v1:
* s/"min_t("/"min_t(int"
v1 -> v2:
* Use min3 instead of min_t twice
v2 -> v3:
* Change commit log header message to reflect the changes made in
v2
v3 -> v4:
* Cast return value from num_online_cpus to int to avoid
compilation errors from "sparse"
---
drivers/net/ethernet/mellanox/mlx4/main.c | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/drivers/net/ethernet/mellanox/mlx4/main.c b/drivers/net/ethernet/mellanox/mlx4/main.c
index 7032054..83aab1e 100644
--- a/drivers/net/ethernet/mellanox/mlx4/main.c
+++ b/drivers/net/ethernet/mellanox/mlx4/main.c
@@ -2862,12 +2862,10 @@ static void mlx4_enable_msi_x(struct mlx4_dev *dev)
int port = 0;
if (msi_x) {
- int nreq = dev->caps.num_ports * num_online_cpus() + 1;
-
- nreq = min_t(int, dev->caps.num_eqs - dev->caps.reserved_eqs,
- nreq);
- if (nreq > MAX_MSIX)
- nreq = MAX_MSIX;
+ int nreq = min3(dev->caps.num_ports *
+ (int)num_online_cpus() + 1,
+ dev->caps.num_eqs - dev->caps.reserved_eqs,
+ MAX_MSIX);
entries = kcalloc(nreq, sizeof *entries, GFP_KERNEL);
if (!entries)
--
2.7.4
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related
* [PATCH] net: netfilter: netlink: delete extra spaces
From: linzhang @ 2017-05-12 5:11 UTC (permalink / raw)
To: pablo
Cc: kadlec, fw, davem, netfilter-devel, coreteam, netdev,
linux-kernel, linzhang
This patch cleans up extra spaces.
Signed-off-by: linzhang <xiaolou4617@gmail.com>
---
net/netfilter/nf_conntrack_netlink.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/net/netfilter/nf_conntrack_netlink.c b/net/netfilter/nf_conntrack_netlink.c
index dcf561b..356e6f0 100644
--- a/net/netfilter/nf_conntrack_netlink.c
+++ b/net/netfilter/nf_conntrack_netlink.c
@@ -634,11 +634,11 @@ static size_t ctnetlink_nlmsg_size(const struct nf_conn *ct)
if (events & (1 << IPCT_DESTROY)) {
type = IPCTNL_MSG_CT_DELETE;
group = NFNLGRP_CONNTRACK_DESTROY;
- } else if (events & ((1 << IPCT_NEW) | (1 << IPCT_RELATED))) {
+ } else if (events & ((1 << IPCT_NEW) | (1 << IPCT_RELATED))) {
type = IPCTNL_MSG_CT_NEW;
flags = NLM_F_CREATE|NLM_F_EXCL;
group = NFNLGRP_CONNTRACK_NEW;
- } else if (events) {
+ } else if (events) {
type = IPCTNL_MSG_CT_NEW;
group = NFNLGRP_CONNTRACK_UPDATE;
} else
--
1.8.3.1
^ permalink raw reply related
* Re: [PATCH net-next] selftests/bpf: get rid of -D__x86_64__
From: Alexei Starovoitov @ 2017-05-12 5:07 UTC (permalink / raw)
To: David Miller; +Cc: daniel, netdev
In-Reply-To: <20170511.212940.1950696136811799858.davem@davemloft.net>
On 5/11/17 6:29 PM, David Miller wrote:
> This whole thing go me thinking however. What do you expect to happen
> on 32-bit architectures implementing an eBPF JIT?
I doubt any 32-bit cpu architectures will do JIT in the near future.
Doing 64-bit operations everywhere is too painful in assembler.
HW offloading is a different story. Like nfp has 32-bit registers
and everything 64-bit in bpf is not the ideal for them.
For such cases the idea is to teach verifier to recognize that
registers don't use upper 32-bits and mark them, so JITs can be
more efficient.
> That's going to
> create some serious conflicts and consternation wrt. tracing which is
> going to want to use headers which are for sizeof(void *)==4 whereas
> for eBPF natively it's 8.
that is indeed a problem. The bpf tracing approach that includes
kernel headers mostly doesn't work on 32-bit archs, though
few iovisor/bcc scripts should be functional. Those that don't
walk pointers and relying on tracepoints instead.
^ permalink raw reply
* Queries regarding TAP interface
From: Kashyap, Saurav @ 2017-05-12 5:02 UTC (permalink / raw)
To: netdev@vger.kernel.org
Hi,
I am using upstream kernel 4.11.0 and base is RHEL 7.2. I am playing around with TAP devices and created it using following commands
=======
ip tuntap add tap10 mode tap
ip addr add 172.28.12.1/24 dev tap10
ip link set tap10 up
ethtool -s tap10 msglvl 1
=======
I enabled debug in driver and also added some prints in tun_net_xmit function. I tried to ping using tap10 but I don’t see request reaching driver. I also tried to create a bridge using brctl and link with physical ethernet interface but it didn’t work.
Second experiment I did was to create two tap device on same system and try pinging one from another, that also didn’t worked.
I have following queries
1) Is I am missing something?
2) Is is possible to hook my own tx/rx to that tap device, so that data packets actually goes on wire?
3) Does DHCP works with tap devices?
Thanks,
~Saurav
^ permalink raw reply
* Re: [Patch net] ipv4: restore rt->fi for reference counting
From: Eric Dumazet @ 2017-05-12 4:55 UTC (permalink / raw)
To: Cong Wang
Cc: Julian Anastasov, David Miller, Linux Kernel Network Developers,
Andrey Konovalov, Eric Dumazet
In-Reply-To: <CAM_iQpWxDKHF4N1xor7qj+5s91GrctgcGnzN0buw18qs7__-gQ@mail.gmail.com>
On Thu, 2017-05-11 at 18:22 -0700, Cong Wang wrote:
> On Thu, May 11, 2017 at 5:07 PM, Cong Wang <xiyou.wangcong@gmail.com> wrote:
> > So, if I understand you correctly it is safe to NULL'ing
> > nh_dev in NETDEV_UNREGISTER_FINAL, right?
> >
> > If still not, how about transfer nh_dev's to loopback_dev
> > too in NETDEV_UNREGISTER? Like we transfer dst->dev.
> >
> > I don't want to touch the fast path to check for NULL, as
> > it will change more code and slow down performance.
>
> Finally I come up with the attached patch. Please let me know if
> I still miss anything.
You have not addressed my prior feedback, unless I am mistaken ?
fib_route_seq_show() and others do not expect fi->fib_dev suddenly
becoming NULL.
RCU contract is more complicated than simply adding rcu grace period
before delete.
Thanks.
^ permalink raw reply
* Re: [Problem] Broadcom BCM5762 Ethernet tg3 times out with stack trace
From: Siva Reddy Kallam @ 2017-05-12 4:09 UTC (permalink / raw)
To: Daniel Kim; +Cc: Prashant Sreedharan, Michael Chan, Linux Netdev List
On Thu, May 11, 2017 at 8:55 PM, Daniel Kim <dkim@playmechanix.com> wrote:
> Summary:
> Broadcom BCM5762 Ethernet tg3 times out with stack trace
>
> Description:
> The ethernet device will disconnect with dmesg reporting a stack trace
> and a time out (visible in DMesg Output below). The ethernet device
> will periodically reconnect and disconnect and the driver will output
> a lot of hex values into logs.
>
> The computer doesn't have to have network activity in order to trigger
> this to happen, but I believe it can trigger faster if there was. It
> usually occurs as soon as a few minutes after bootup to upwards of a
> few hours in my test cases.
>
> Kernel version:
> Linux version 4.11.0-041100-generic (kernel@tangerine) (gcc version
> 6.3.0 20170415 (Ubuntu 6.3.0-14ubuntu3) ) #201705041534 SMP Thu May 4
> 19:36:05 UTC 2017
>
> Full DMesg Output:
> https://launchpadlibrarian.net/319135862/error_dmesg.txt
> Bug has been reported with additional information at:
> https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1689383
>
> Please let me know if there is other data or logs that you may need.
>
Thank you for reporting this. We will look into the logs and contact
you directly if we need any other data.
>
> Cut DMesg Output:
> [ 6244.283996] ------------[ cut here ]------------
> [ 6244.284051] WARNING: CPU: 0 PID: 0 at
> /home/kernel/COD/linux/net/sched/sch_generic.c:316
> dev_watchdog+0x22c/0x230
> [ 6244.284058] NETDEV WATCHDOG: enp4s0 (tg3): transmit queue 0 timed out
> [ 6244.284064] Modules linked in: snd_hda_codec_hdmi hp_wmi
> snd_hda_codec_realtek snd_hda_codec_generic sparse_keymap
> snd_hda_intel snd_hda_codec edac_mce_amd edac_core crct10dif_pclmul
> crc32_pclmul snd_hda_core snd_hwdep ghash_clmulni_intel pcbc joydev
> input_leds snd_pcm aesni_intel aes_x86_64 snd_seq_midi
> snd_seq_midi_event snd_rawmidi crypto_simd snd_seq snd_seq_device
> snd_timer glue_helper snd cryptd soundcore serio_raw k10temp shpchp
> i2c_piix4 mac_hid tpm_infineon wmi parport_pc ppdev lp parport autofs4
> hid_generic usbhid hid psmouse tg3 ptp pps_core ahci libahci video
> [ 6244.284129] CPU: 0 PID: 0 Comm: swapper/0 Not tainted
> 4.11.0-041100-generic #201705041534
> [ 6244.284131] Hardware name: Hewlett-Packard HP EliteDesk 705 G1
> MT/2215, BIOS L06 v02.17 12/11/2014
> [ 6244.284132] Call Trace:
> [ 6244.284135] <IRQ>
> [ 6244.284141] dump_stack+0x63/0x81
> [ 6244.284143] __warn+0xcb/0xf0
> [ 6244.284146] warn_slowpath_fmt+0x5a/0x80
> [ 6244.284149] ? cpu_load_update+0xdd/0x150
> [ 6244.284155] dev_watchdog+0x22c/0x230
> [ 6244.284159] ? qdisc_rcu_free+0x50/0x50
> [ 6244.284162] call_timer_fn+0x35/0x140
> [ 6244.284165] run_timer_softirq+0x1db/0x440
> [ 6244.284168] ? ktime_get+0x41/0xb0
> [ 6244.284172] ? lapic_next_event+0x1d/0x30
> [ 6244.284175] ? clockevents_program_event+0x7f/0x120
> [ 6244.284179] __do_softirq+0x104/0x2af
> [ 6244.284183] irq_exit+0xb6/0xc0
> [ 6244.284187] smp_apic_timer_interrupt+0x3d/0x50
> [ 6244.284190] apic_timer_interrupt+0x89/0x90
> [ 6244.284193] RIP: 0010:cpuidle_enter_state+0x122/0x2c0
> [ 6244.284195] RSP: 0018:ffffffff86203dc8 EFLAGS: 00000246 ORIG_RAX:
> ffffffffffffff10
> [ 6244.284198] RAX: 0000000000000000 RBX: 0000000000000002 RCX: 000000000000001f
> [ 6244.284200] RDX: 000005addc53de33 RSI: ffffa01a2ec189d8 RDI: 0000000000000000
> [ 6244.284202] RBP: ffffffff86203e08 R08: 000000000000cd01 R09: 0000000000000018
> [ 6244.284203] R10: ffffffff86203d98 R11: 0000000000000ef9 R12: ffffa01a1d6ab200
> [ 6244.284205] R13: ffffffff862f92d8 R14: 0000000000000002 R15: ffffffff862f92c0
> [ 6244.284207] </IRQ>
> [ 6244.284211] ? cpuidle_enter_state+0x110/0x2c0
> [ 6244.284213] cpuidle_enter+0x17/0x20
> [ 6244.284218] call_cpuidle+0x23/0x40
> [ 6244.284221] do_idle+0x189/0x200
> [ 6244.284224] cpu_startup_entry+0x71/0x80
> [ 6244.284227] rest_init+0x77/0x80
> [ 6244.284230] start_kernel+0x455/0x476
> [ 6244.284235] ? early_idt_handler_array+0x120/0x120
> [ 6244.284239] x86_64_start_reservations+0x24/0x26
> [ 6244.284243] x86_64_start_kernel+0x14d/0x170
> [ 6244.284246] start_cpu+0x14/0x14
> [ 6244.284251] ---[ end trace b14ec0a8b1e2a4e8 ]---
> [ 6244.284259] tg3 0000:04:00.0 enp4s0: transmit timed out, resetting
> [ 6244.398892] hrtimer: interrupt took 65509334 ns
> [ 6247.090588] tg3 0000:04:00.0 enp4s0: 0x00000000: 0x168714e4,
> 0x00100506, 0x02000010, 0x00000010
> [ 6247.090601] tg3 0000:04:00.0 enp4s0: 0x00000010: 0xe212000c,
> 0x00000000, 0xe211000c, 0x00000000
> [ 6247.090607] tg3 0000:04:00.0 enp4s0: 0x00000020: 0xe210000c,
> 0x00000000, 0x00000000, 0x2215103c
> [ 6247.090610] tg3 0000:04:00.0 enp4s0: 0x00000030: 0x00000000,
> 0x00000048, 0x00000000, 0x0000010a
> [ 6247.090613] tg3 0000:04:00.0 enp4s0: 0x00000040: 0x00000000,
> 0x39000000, 0xc8035001, 0x16002008
> [ 6247.090617] tg3 0000:04:00.0 enp4s0: 0x00000050: 0x00005803,
> 0x00000000, 0x0086a005, 0x00000000
> [ 6247.090622] tg3 0000:04:00.0 enp4s0: 0x00000060: 0x00000000,
> 0x00000000, 0xf1000298, 0x01f802d1
> [ 6247.090625] tg3 0000:04:00.0 enp4s0: 0x00000070: 0x00071010,
> 0xf6001500, 0x00000000, 0x00000000
> [ 6247.090628] tg3 0000:04:00.0 enp4s0: 0x00000080: 0x168714e4,
> 0x00000018, 0x00000000, 0x00000800
> [ 6247.090632] tg3 0000:04:00.0 enp4s0: 0x00000090: 0x00000000,
> 0x00000171, 0x00000000, 0x0000021d
> [ 6247.090636] tg3 0000:04:00.0 enp4s0: 0x000000a0: 0x8005ac11,
> 0x00000004, 0x00000122, 0x00020010
> [ 6247.090639] tg3 0000:04:00.0 enp4s0: 0x000000b0: 0x10008d82,
> 0x00115400, 0x00475c12, 0x10120043
> [ 6247.090643] tg3 0000:04:00.0 enp4s0: 0x000000d0: 0x0008081f,
> 0x00000000, 0x00000000, 0x00010001
> [ 6247.090649] tg3 0000:04:00.0 enp4s0: 0x000000f0: 0x00000000,
> 0x05762100, 0x00000000, 0xffffffff
> [ 6247.090652] tg3 0000:04:00.0 enp4s0: 0x00000100: 0x13c10001,
> 0x00000000, 0x00000000, 0x00062030
> [ 6247.090654] tg3 0000:04:00.0 enp4s0: 0x00000110: 0x00001000,
> 0x00002000, 0x000000a0, 0x00000000
> [ 6247.090658] tg3 0000:04:00.0 enp4s0: 0x00000130: 0x00000000,
> 0x00000000, 0x00000000, 0x15010003
> [ 6247.090660] tg3 0000:04:00.0 enp4s0: 0x00000140: 0xd45792bf,
> 0x00008cdc, 0x00000000, 0x00000000
> [ 6247.090663] tg3 0000:04:00.0 enp4s0: 0x00000150: 0x16010004,
> 0x00000000, 0x00078116, 0x00000001
> [ 6247.090666] tg3 0000:04:00.0 enp4s0: 0x00000160: 0x1b010002,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.090669] tg3 0000:04:00.0 enp4s0: 0x00000170: 0x00000000,
> 0x80000001, 0x00000000, 0x00000000
> [ 6247.090672] tg3 0000:04:00.0 enp4s0: 0x000001b0: 0x23010018,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.090676] tg3 0000:04:00.0 enp4s0: 0x00000200: 0x00000000,
> 0x39000000, 0x00000000, 0xc3000000
> [ 6247.090679] tg3 0000:04:00.0 enp4s0: 0x00000210: 0x00000000,
> 0x51000000, 0x00000000, 0x00000001
> [ 6247.090682] tg3 0000:04:00.0 enp4s0: 0x00000220: 0x00000000,
> 0x00000001, 0x00000000, 0x00000000
> [ 6247.090685] tg3 0000:04:00.0 enp4s0: 0x00000260: 0x00000000,
> 0x00000000, 0x00000000, 0x0000021d
> [ 6247.090689] tg3 0000:04:00.0 enp4s0: 0x00000280: 0x00000000,
> 0x00000800, 0x00000000, 0x00000955
> [ 6247.090692] tg3 0000:04:00.0 enp4s0: 0x00000300: 0x00000000,
> 0x00000171, 0x00000000, 0x00000171
> [ 6247.090695] tg3 0000:04:00.0 enp4s0: 0x00000310: 0x00000000,
> 0x00000171, 0x00000000, 0x00000171
> [ 6247.090698] tg3 0000:04:00.0 enp4s0: 0x00000320: 0x00000000,
> 0x00000171, 0x00000000, 0x00000171
> [ 6247.090701] tg3 0000:04:00.0 enp4s0: 0x00000330: 0x00000000,
> 0x00000171, 0x00000000, 0x00000171
> [ 6247.090704] tg3 0000:04:00.0 enp4s0: 0x00000340: 0x00000000,
> 0x00000171, 0x00000000, 0x00000171
> [ 6247.090707] tg3 0000:04:00.0 enp4s0: 0x00000350: 0x00000000,
> 0x00000171, 0x00000000, 0x00000171
> [ 6247.090710] tg3 0000:04:00.0 enp4s0: 0x00000360: 0x00000000,
> 0x00000171, 0x00000000, 0x00000171
> [ 6247.090713] tg3 0000:04:00.0 enp4s0: 0x00000370: 0x00000000,
> 0x00000171, 0x00000000, 0x00000171
> [ 6247.090716] tg3 0000:04:00.0 enp4s0: 0x00000380: 0x00000000,
> 0x00000171, 0x00000000, 0x00000171
> [ 6247.090718] tg3 0000:04:00.0 enp4s0: 0x00000390: 0x00000000,
> 0x00000171, 0x00000000, 0x00000171
> [ 6247.090721] tg3 0000:04:00.0 enp4s0: 0x000003a0: 0x00000000,
> 0x00000171, 0x00000000, 0x00000171
> [ 6247.090724] tg3 0000:04:00.0 enp4s0: 0x000003b0: 0x00000000,
> 0x00000171, 0x00000000, 0x00000171
> [ 6247.090727] tg3 0000:04:00.0 enp4s0: 0x000003c0: 0x00000000,
> 0x00000171, 0x00000000, 0x00000171
> [ 6247.090730] tg3 0000:04:00.0 enp4s0: 0x000003d0: 0x00000000,
> 0x00000171, 0x00000000, 0x00000171
> [ 6247.090733] tg3 0000:04:00.0 enp4s0: 0x000003e0: 0x00000000,
> 0x00000171, 0x00000000, 0x00000171
> [ 6247.090736] tg3 0000:04:00.0 enp4s0: 0x000003f0: 0x00000000,
> 0x00000171, 0x00000000, 0x00000171
> [ 6247.090738] tg3 0000:04:00.0 enp4s0: 0x00000400: 0x18e04808,
> 0x00400000, 0x00001000, 0x00009b00
> [ 6247.090741] tg3 0000:04:00.0 enp4s0: 0x00000410: 0x00008cdc,
> 0xd45792bf, 0x00008cdc, 0xd45792bf
> [ 6247.090744] tg3 0000:04:00.0 enp4s0: 0x00000420: 0x00008cdc,
> 0xd45792bf, 0x00008cdc, 0xd45792bf
> [ 6247.090747] tg3 0000:04:00.0 enp4s0: 0x00000430: 0x00000000,
> 0x00000000, 0x00000008, 0x000005f2
> [ 6247.090750] tg3 0000:04:00.0 enp4s0: 0x00000440: 0x00000000,
> 0x00000000, 0x00000000, 0x082e0006
> [ 6247.090753] tg3 0000:04:00.0 enp4s0: 0x00000450: 0x00000001,
> 0x00008000, 0x00000000, 0x00000112
> [ 6247.090756] tg3 0000:04:00.0 enp4s0: 0x00000460: 0x0000000b,
> 0x00002620, 0x03ff0006, 0x00000000
> [ 6247.090759] tg3 0000:04:00.0 enp4s0: 0x00000470: 0xa0000000,
> 0x00000000, 0x00000000, 0x50000001
> [ 6247.090762] tg3 0000:04:00.0 enp4s0: 0x00000480: 0x42000000,
> 0x7fffffff, 0x06000004, 0x7fffffff
> [ 6247.090765] tg3 0000:04:00.0 enp4s0: 0x00000500: 0x00000008,
> 0x00000002, 0x00000000, 0x00000000
> [ 6247.090768] tg3 0000:04:00.0 enp4s0: 0x00000590: 0x00901001,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.090771] tg3 0000:04:00.0 enp4s0: 0x00000600: 0xffffffff,
> 0x00f80011, 0x00000000, 0x40001f0c
> [ 6247.090774] tg3 0000:04:00.0 enp4s0: 0x00000610: 0xffffffff,
> 0x00000000, 0x00000044, 0x00000000
> [ 6247.090777] tg3 0000:04:00.0 enp4s0: 0x00000620: 0x00000040,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.090780] tg3 0000:04:00.0 enp4s0: 0x00000630: 0x01010101,
> 0x01010101, 0x01010101, 0x01010101
> [ 6247.090782] tg3 0000:04:00.0 enp4s0: 0x00000640: 0x01010101,
> 0x01010101, 0x01010101, 0x01010101
> [ 6247.090785] tg3 0000:04:00.0 enp4s0: 0x00000650: 0x01010101,
> 0x01010101, 0x01010101, 0x01010101
> [ 6247.090788] tg3 0000:04:00.0 enp4s0: 0x00000660: 0x01010101,
> 0x01010101, 0x01010101, 0x01010101
> [ 6247.090790] tg3 0000:04:00.0 enp4s0: 0x00000670: 0x43cb3b20,
> 0x0defd9a3, 0x767fe840, 0x1442a189
> [ 6247.090793] tg3 0000:04:00.0 enp4s0: 0x00000680: 0xf210c361,
> 0xdb0f8709, 0xee76c0a3, 0x3753b863
> [ 6247.090796] tg3 0000:04:00.0 enp4s0: 0x00000690: 0x8c5ffafc,
> 0xba4254bd, 0x00000000, 0x00000000
> [ 6247.090798] tg3 0000:04:00.0 enp4s0: 0x000006c0: 0x00000000,
> 0x00000000, 0x04000000, 0x00000000
> [ 6247.090801] tg3 0000:04:00.0 enp4s0: 0x00000800: 0x00000000,
> 0xffffffff, 0x00000000, 0x00000000
> [ 6247.090804] tg3 0000:04:00.0 enp4s0: 0x00000810: 0x00000000,
> 0xffffffff, 0x00000000, 0x00000000
> [ 6247.090806] tg3 0000:04:00.0 enp4s0: 0x00000820: 0x00000000,
> 0x00000000, 0xffffffff, 0x00000000
> [ 6247.090809] tg3 0000:04:00.0 enp4s0: 0x00000830: 0x00000000,
> 0xffffffff, 0xffffffff, 0xffffffff
> [ 6247.090812] tg3 0000:04:00.0 enp4s0: 0x00000840: 0xffffffff,
> 0xffffffff, 0xffffffff, 0xffffffff
> [ 6247.090815] tg3 0000:04:00.0 enp4s0: 0x00000850: 0xffffffff,
> 0xffffffff, 0xffffffff, 0xffffffff
> [ 6247.090817] tg3 0000:04:00.0 enp4s0: 0x00000860: 0xffffffff,
> 0xffffffff, 0xffffffff, 0x00000000
> [ 6247.090820] tg3 0000:04:00.0 enp4s0: 0x00000880: 0x00000754,
> 0x00001633, 0x00000000, 0x00000000
> [ 6247.090822] tg3 0000:04:00.0 enp4s0: 0x00000890: 0x00000005,
> 0x0000000c, 0x00000000, 0x00000000
> [ 6247.090825] tg3 0000:04:00.0 enp4s0: 0x000008f0: 0x007c0001,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.090828] tg3 0000:04:00.0 enp4s0: 0x00000900: 0x00095b83,
> 0xffffffff, 0x00000000, 0x00000000
> [ 6247.090831] tg3 0000:04:00.0 enp4s0: 0x00000910: 0x00000002,
> 0xffffffff, 0x00000000, 0x00000000
> [ 6247.090834] tg3 0000:04:00.0 enp4s0: 0x00000920: 0x00000000,
> 0x00000000, 0xffffffff, 0x00000000
> [ 6247.090837] tg3 0000:04:00.0 enp4s0: 0x00000930: 0x00000000,
> 0xffffffff, 0xffffffff, 0xffffffff
> [ 6247.090840] tg3 0000:04:00.0 enp4s0: 0x00000940: 0xffffffff,
> 0xffffffff, 0xffffffff, 0xffffffff
> [ 6247.090843] tg3 0000:04:00.0 enp4s0: 0x00000950: 0xffffffff,
> 0xffffffff, 0xffffffff, 0xffffffff
> [ 6247.090846] tg3 0000:04:00.0 enp4s0: 0x00000960: 0xffffffff,
> 0xffffffff, 0xffffffff, 0x000015d7
> [ 6247.090849] tg3 0000:04:00.0 enp4s0: 0x00000970: 0x00000036,
> 0x00000164, 0x00000000, 0x00000000
> [ 6247.090852] tg3 0000:04:00.0 enp4s0: 0x00000980: 0x010a92c7,
> 0x00001633, 0x00000000, 0x000015c8
> [ 6247.090855] tg3 0000:04:00.0 enp4s0: 0x00000990: 0x0000a81a,
> 0x00019e10, 0x00000000, 0x00000000
> [ 6247.090858] tg3 0000:04:00.0 enp4s0: 0x00000c00: 0x0000000a,
> 0x00000000, 0x00000003, 0x00000001
> [ 6247.090861] tg3 0000:04:00.0 enp4s0: 0x00000c10: 0x00000000,
> 0x00000000, 0x00000000, 0x00620000
> [ 6247.090864] tg3 0000:04:00.0 enp4s0: 0x00000c80: 0x00001771,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.090867] tg3 0000:04:00.0 enp4s0: 0x00000ce0: 0xffd07a02,
> 0x00000000, 0x00000062, 0x00040028
> [ 6247.090870] tg3 0000:04:00.0 enp4s0: 0x00000cf0: 0x00000000,
> 0x50000071, 0x00000000, 0x00000000
> [ 6247.090873] tg3 0000:04:00.0 enp4s0: 0x00001000: 0x00000002,
> 0x00000000, 0xa000e5e6, 0x00000000
> [ 6247.090876] tg3 0000:04:00.0 enp4s0: 0x00001010: 0x01711711,
> 0x0000e5e6, 0x00000000, 0x00000000
> [ 6247.090879] tg3 0000:04:00.0 enp4s0: 0x00001400: 0x00000006,
> 0x00000000, 0x00000000, 0x00000001
> [ 6247.090881] tg3 0000:04:00.0 enp4s0: 0x00001440: 0x00000171,
> 0x00000171, 0x00000171, 0x00000171
> [ 6247.090884] tg3 0000:04:00.0 enp4s0: 0x00001450: 0x00000171,
> 0x00000171, 0x00000171, 0x00000171
> [ 6247.090887] tg3 0000:04:00.0 enp4s0: 0x00001460: 0x00000171,
> 0x00000171, 0x00000171, 0x00000171
> [ 6247.090890] tg3 0000:04:00.0 enp4s0: 0x00001470: 0x00000171,
> 0x00000171, 0x00000171, 0x00000171
> [ 6247.090893] tg3 0000:04:00.0 enp4s0: 0x00001800: 0x00000016,
> 0x00000000, 0x00000171, 0x00000000
> [ 6247.090896] tg3 0000:04:00.0 enp4s0: 0x00001840: 0x00000000,
> 0x00000000, 0x00000200, 0x00000010
> [ 6247.090899] tg3 0000:04:00.0 enp4s0: 0x00001850: 0x0000001f,
> 0x00000000, 0x00004300, 0x01710171
> [ 6247.090902] tg3 0000:04:00.0 enp4s0: 0x00001860: 0x01000000,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.090904] tg3 0000:04:00.0 enp4s0: 0x00001c00: 0x00000002,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.090907] tg3 0000:04:00.0 enp4s0: 0x00002000: 0x00000002,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.090910] tg3 0000:04:00.0 enp4s0: 0x00002010: 0x00000181,
> 0x00000001, 0x007bfffd, 0x00000000
> [ 6247.090913] tg3 0000:04:00.0 enp4s0: 0x00002100: 0x000d62b3,
> 0x0008dc6f, 0x0000009e, 0x00000000
> [ 6247.090916] tg3 0000:04:00.0 enp4s0: 0x00002110: 0x000d62b3,
> 0x0008dc6f, 0x0000009e, 0x00000000
> [ 6247.090918] tg3 0000:04:00.0 enp4s0: 0x00002120: 0x000d62b3,
> 0x0008dc6f, 0x0000009e, 0x00000000
> [ 6247.090921] tg3 0000:04:00.0 enp4s0: 0x00002130: 0x000d62b3,
> 0x0008dc6f, 0x0000009e, 0x00000000
> [ 6247.090924] tg3 0000:04:00.0 enp4s0: 0x00002140: 0x000d62b3,
> 0x0008dc6f, 0x0000009e, 0x00000000
> [ 6247.090928] tg3 0000:04:00.0 enp4s0: 0x00002150: 0x000d62b3,
> 0x0008dc6f, 0x0000009e, 0x00000000
> [ 6247.090931] tg3 0000:04:00.0 enp4s0: 0x00002160: 0x000d62b3,
> 0x0008dc6f, 0x0000009e, 0x00000000
> [ 6247.090934] tg3 0000:04:00.0 enp4s0: 0x00002170: 0x000d62b3,
> 0x0008dc6f, 0x0000009e, 0x00000000
> [ 6247.090937] tg3 0000:04:00.0 enp4s0: 0x00002180: 0x000d62b3,
> 0x0008dc6f, 0x0000009e, 0x00000000
> [ 6247.090940] tg3 0000:04:00.0 enp4s0: 0x00002190: 0x000d62b3,
> 0x0008dc6f, 0x0000009e, 0x00000000
> [ 6247.090942] tg3 0000:04:00.0 enp4s0: 0x000021a0: 0x000d62b3,
> 0x0008dc6f, 0x0000009e, 0x00000000
> [ 6247.090945] tg3 0000:04:00.0 enp4s0: 0x000021b0: 0x000d62b3,
> 0x0008dc6f, 0x0000009e, 0x00000000
> [ 6247.090948] tg3 0000:04:00.0 enp4s0: 0x000021c0: 0x000d62b3,
> 0x0008dc6f, 0x0000009e, 0x00000000
> [ 6247.090951] tg3 0000:04:00.0 enp4s0: 0x000021d0: 0x000d62b3,
> 0x0008dc6f, 0x0000009e, 0x00000000
> [ 6247.090953] tg3 0000:04:00.0 enp4s0: 0x000021e0: 0x000d62b3,
> 0x0008dc6f, 0x0000009e, 0x00000000
> [ 6247.090956] tg3 0000:04:00.0 enp4s0: 0x000021f0: 0x000d62b3,
> 0x0008dc6f, 0x0000009e, 0x00000000
> [ 6247.090959] tg3 0000:04:00.0 enp4s0: 0x00002200: 0x00025b63,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.090962] tg3 0000:04:00.0 enp4s0: 0x00002250: 0x00009a2d,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.090965] tg3 0000:04:00.0 enp4s0: 0x00002400: 0x00010012,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.090968] tg3 0000:04:00.0 enp4s0: 0x00002440: 0x00000000,
> 0x00000000, 0x00000002, 0x00000000
> [ 6247.090970] tg3 0000:04:00.0 enp4s0: 0x00002450: 0x00000000,
> 0xfffe0000, 0x08001800, 0x00006000
> [ 6247.090973] tg3 0000:04:00.0 enp4s0: 0x00002470: 0x00000000,
> 0x0000015f, 0x00000000, 0x00000000
> [ 6247.090976] tg3 0000:04:00.0 enp4s0: 0x000024c0: 0x02005101,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.090979] tg3 0000:04:00.0 enp4s0: 0x00002800: 0x00000006,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.090982] tg3 0000:04:00.0 enp4s0: 0x00002c00: 0x00000006,
> 0x00000000, 0x00000000, 0x000001d5
> [ 6247.090985] tg3 0000:04:00.0 enp4s0: 0x00002c10: 0x00000000,
> 0x00000000, 0x00000019, 0x0000000c
> [ 6247.090988] tg3 0000:04:00.0 enp4s0: 0x00002c20: 0x00020002,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.090990] tg3 0000:04:00.0 enp4s0: 0x00002d00: 0x00000080,
> 0x00000040, 0x00000000, 0x00000000
> [ 6247.090993] tg3 0000:04:00.0 enp4s0: 0x00003000: 0x00000006,
> 0x00000000, 0x00000000, 0x000001d5
> [ 6247.090997] tg3 0000:04:00.0 enp4s0: 0x00003600: 0x00034400,
> 0x00130000, 0x00110000, 0x00000000
> [ 6247.090999] tg3 0000:04:00.0 enp4s0: 0x00003610: 0x00131100,
> 0x00130000, 0x00130000, 0x00000000
> [ 6247.091002] tg3 0000:04:00.0 enp4s0: 0x00003620: 0x80110011,
> 0x00000000, 0x00000004, 0x3c822080
> [ 6247.091005] tg3 0000:04:00.0 enp4s0: 0x00003630: 0x00000000,
> 0x80008000, 0x00001080, 0x00000000
> [ 6247.091008] tg3 0000:04:00.0 enp4s0: 0x00003640: 0x0000000a,
> 0x33f00003, 0x00000020, 0x00000019
> [ 6247.091010] tg3 0000:04:00.0 enp4s0: 0x00003650: 0x00000171,
> 0x00000bff, 0x05762100, 0x00000000
> [ 6247.091014] tg3 0000:04:00.0 enp4s0: 0x00003660: 0x00000000,
> 0x00000000, 0x000400a3, 0x00000202
> [ 6247.091017] tg3 0000:04:00.0 enp4s0: 0x00003670: 0x0000002a,
> 0xfeffff63, 0x0000000a, 0x00000000
> [ 6247.091020] tg3 0000:04:00.0 enp4s0: 0x000036b0: 0x001003cc,
> 0x07ff07ff, 0x07ff07ff, 0x01000004
> [ 6247.091022] tg3 0000:04:00.0 enp4s0: 0x000036c0: 0xffffffff,
> 0x0000a441, 0x00c84caa, 0x0000a558
> [ 6247.091025] tg3 0000:04:00.0 enp4s0: 0x000036d0: 0x0000019d,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.091028] tg3 0000:04:00.0 enp4s0: 0x000036e0: 0x80000b19,
> 0xa0800799, 0x00500799, 0x00000000
> [ 6247.091031] tg3 0000:04:00.0 enp4s0: 0x000036f0: 0x19090900,
> 0x13180600, 0x00000301, 0x00000001
> [ 6247.091034] tg3 0000:04:00.0 enp4s0: 0x00003700: 0x00000000,
> 0x00000000, 0x00000000, 0x00001388
> [ 6247.091037] tg3 0000:04:00.0 enp4s0: 0x00003710: 0x87fffffd,
> 0x00000000, 0x00000000, 0x000010dc
> [ 6247.091039] tg3 0000:04:00.0 enp4s0: 0x00003720: 0x00000000,
> 0x00002040, 0x88006000, 0xa0002000
> [ 6247.091042] tg3 0000:04:00.0 enp4s0: 0x00003750: 0x00000000,
> 0x00000000, 0x0000080b, 0x00000000
> [ 6247.091045] tg3 0000:04:00.0 enp4s0: 0x00003780: 0x0000bff9,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.091049] tg3 0000:04:00.0 enp4s0: 0x00003c00: 0x00000306,
> 0x00000000, 0x00000000, 0x00000048
> [ 6247.091052] tg3 0000:04:00.0 enp4s0: 0x00003c10: 0x00000000,
> 0x00000035, 0x00000000, 0x00000000
> [ 6247.091055] tg3 0000:04:00.0 enp4s0: 0x00003c20: 0x00000000,
> 0x00000005, 0x00000000, 0x00000000
> [ 6247.091058] tg3 0000:04:00.0 enp4s0: 0x00003c30: 0x00000000,
> 0x00000000, 0x00000000, 0xffffc000
> [ 6247.091061] tg3 0000:04:00.0 enp4s0: 0x00003c40: 0x00000000,
> 0x00000b00, 0x00000000, 0x00000000
> [ 6247.091064] tg3 0000:04:00.0 enp4s0: 0x00003c50: 0x00000000,
> 0x00000155, 0x00000000, 0x00000000
> [ 6247.091066] tg3 0000:04:00.0 enp4s0: 0x00003c80: 0x00000800,
> 0x00000955, 0x00000000, 0x00000000
> [ 6247.091069] tg3 0000:04:00.0 enp4s0: 0x00003cc0: 0x00000171,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.091072] tg3 0000:04:00.0 enp4s0: 0x00003cd0: 0x00000000,
> 0x0000000f, 0x00000000, 0x00000000
> [ 6247.091075] tg3 0000:04:00.0 enp4s0: 0x00003d00: 0x00000000,
> 0xffffb000, 0x00000000, 0xffffa000
> [ 6247.091078] tg3 0000:04:00.0 enp4s0: 0x00003d80: 0x00000014,
> 0x00000000, 0x00000005, 0x00000000
> [ 6247.091080] tg3 0000:04:00.0 enp4s0: 0x00003d90: 0x00000005,
> 0x00000000, 0x00000014, 0x00000000
> [ 6247.091083] tg3 0000:04:00.0 enp4s0: 0x00003da0: 0x00000005,
> 0x00000000, 0x00000005, 0x00000000
> [ 6247.091086] tg3 0000:04:00.0 enp4s0: 0x00004000: 0x00000002,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.091088] tg3 0000:04:00.0 enp4s0: 0x00004010: 0x00000000,
> 0x0026f012, 0x00800420, 0x01006222
> [ 6247.091091] tg3 0000:04:00.0 enp4s0: 0x00004020: 0x00000000,
> 0x00000010, 0x00000010, 0x00000050
> [ 6247.091094] tg3 0000:04:00.0 enp4s0: 0x00004030: 0x00000000,
> 0x01086020, 0x00296010, 0x00000002
> [ 6247.091096] tg3 0000:04:00.0 enp4s0: 0x00004040: 0x00400000,
> 0x00000000, 0x00000010, 0x004e5062
> [ 6247.091100] tg3 0000:04:00.0 enp4s0: 0x00004400: 0x00000016,
> 0x00000000, 0x00010000, 0x0000a000
> [ 6247.091103] tg3 0000:04:00.0 enp4s0: 0x00004410: 0x00000000,
> 0x0000002a, 0x000000a0, 0x00000000
> [ 6247.091106] tg3 0000:04:00.0 enp4s0: 0x00004420: 0x000000fd,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.091109] tg3 0000:04:00.0 enp4s0: 0x00004440: 0x00000000,
> 0x00000000, 0x00000000, 0x0e5398e8
> [ 6247.091112] tg3 0000:04:00.0 enp4s0: 0x00004450: 0x02340226,
> 0x00970071, 0x00000000, 0x00000000
> [ 6247.091116] tg3 0000:04:00.0 enp4s0: 0x00004700: 0x00030002,
> 0x00100000, 0x00100010, 0xc1250000
> [ 6247.091119] tg3 0000:04:00.0 enp4s0: 0x00004710: 0xfffff6e0,
> 0x00060484, 0x00100010, 0x00000000
> [ 6247.091121] tg3 0000:04:00.0 enp4s0: 0x00004720: 0x00000000,
> 0x00000000, 0xf02c0000, 0xfffff710
> [ 6247.091124] tg3 0000:04:00.0 enp4s0: 0x00004770: 0x000c0404,
> 0x00000002, 0x00000010, 0x00000000
> [ 6247.091127] tg3 0000:04:00.0 enp4s0: 0x00004800: 0x380303fe,
> 0x00000000, 0x00000000, 0x00000100
> [ 6247.091130] tg3 0000:04:00.0 enp4s0: 0x00004810: 0x00000000,
> 0x00000008, 0x00429f80, 0x00008000
> [ 6247.091133] tg3 0000:04:00.0 enp4s0: 0x00004820: 0x000000dd,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.091136] tg3 0000:04:00.0 enp4s0: 0x00004860: 0x000000dd,
> 0x11400062, 0x001ff800, 0x62000008
> [ 6247.091139] tg3 0000:04:00.0 enp4s0: 0x00004870: 0x05ea0080,
> 0x003e1820, 0x003e1820, 0x00000000
> [ 6247.091141] tg3 0000:04:00.0 enp4s0: 0x00004890: 0x280c0c04,
> 0x00305400, 0x00000000, 0x00000000
> [ 6247.091144] tg3 0000:04:00.0 enp4s0: 0x000048a0: 0x000f0010,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.091147] tg3 0000:04:00.0 enp4s0: 0x00004900: 0x18030002,
> 0x00000003, 0x30000000, 0x00000120
> [ 6247.091149] tg3 0000:04:00.0 enp4s0: 0x00004910: 0x00000040,
> 0x00000003, 0x0000df94, 0x00000008
> [ 6247.091152] tg3 0000:04:00.0 enp4s0: 0x00004920: 0x000000e7,
> 0x02728000, 0xffd07a01, 0x0f000068
> [ 6247.091155] tg3 0000:04:00.0 enp4s0: 0x00004930: 0xffd31201,
> 0x0f000068, 0xffcdf401, 0x0f000068
> [ 6247.091158] tg3 0000:04:00.0 enp4s0: 0x00004940: 0xffebcc01,
> 0x0f000068, 0x4f4f1212, 0x3e3ea3a3
> [ 6247.091161] tg3 0000:04:00.0 enp4s0: 0x00004950: 0xf0330000,
> 0xffd07a68, 0xaf000000, 0xe7e500e6
> [ 6247.091164] tg3 0000:04:00.0 enp4s0: 0x00004960: 0x00000000,
> 0xffd31268, 0xaf000000, 0x40000171
> [ 6247.091167] tg3 0000:04:00.0 enp4s0: 0x00004970: 0x00028202,
> 0x00205400, 0x0000001c, 0x000000ff
> [ 6247.091170] tg3 0000:04:00.0 enp4s0: 0x00004980: 0x000000e7,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.091172] tg3 0000:04:00.0 enp4s0: 0x00004990: 0x00000000,
> 0x00000000, 0xffcdf468, 0xaf000000
> [ 6247.091175] tg3 0000:04:00.0 enp4s0: 0x000049a0: 0x00f00049,
> 0x000000e7, 0x00000000, 0x00000000
> [ 6247.091178] tg3 0000:04:00.0 enp4s0: 0x000049b0: 0xffd09601,
> 0xffd2c401, 0xffce7e01, 0xffec7e01
> [ 6247.091181] tg3 0000:04:00.0 enp4s0: 0x000049c0: 0xffd07a02,
> 0xffd31202, 0xffcdf402, 0xffebcc02
> [ 6247.091183] tg3 0000:04:00.0 enp4s0: 0x000049d0: 0xffd09602,
> 0xffd2c402, 0xffce7e02, 0xffec7e02
> [ 6247.091186] tg3 0000:04:00.0 enp4s0: 0x000049f0: 0xf0330000,
> 0xffebcc68, 0xaf000000, 0x0000ffff
> [ 6247.091190] tg3 0000:04:00.0 enp4s0: 0x00004c00: 0x200003fe,
> 0x00000020, 0x00000000, 0x00000000
> [ 6247.091193] tg3 0000:04:00.0 enp4s0: 0x00004c10: 0x0000003f,
> 0x00000000, 0x00000006, 0x00000000
> [ 6247.091196] tg3 0000:04:00.0 enp4s0: 0x00004c20: 0x00000000,
> 0x00000000, 0x00000000, 0x00000006
> [ 6247.091199] tg3 0000:04:00.0 enp4s0: 0x00004c30: 0x00000000,
> 0x00200000, 0x0000007c, 0x0000007c
> [ 6247.091201] tg3 0000:04:00.0 enp4s0: 0x00004c40: 0x00000000,
> 0x09550000, 0x00000000, 0x095407ff
> [ 6247.091204] tg3 0000:04:00.0 enp4s0: 0x00004c50: 0x0fff0fff,
> 0x01550fff, 0x00000000, 0x00553939
> [ 6247.091207] tg3 0000:04:00.0 enp4s0: 0x00004c60: 0x00000020,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.091210] tg3 0000:04:00.0 enp4s0: 0x00005000: 0x00009800,
> 0x80000000, 0x00000000, 0x00000000
> [ 6247.091213] tg3 0000:04:00.0 enp4s0: 0x00005010: 0x00000000,
> 0x00000000, 0x00000000, 0x08001028
> [ 6247.091216] tg3 0000:04:00.0 enp4s0: 0x00005020: 0x30840fff,
> 0x00000000, 0x00000000, 0x40000020
> [ 6247.091218] tg3 0000:04:00.0 enp4s0: 0x00005030: 0x00000000,
> 0x00000025, 0x00000000, 0x00000000
> [ 6247.091222] tg3 0000:04:00.0 enp4s0: 0x00005040: 0x00000000,
> 0x00000000, 0x080011f4, 0x00000000
> [ 6247.091225] tg3 0000:04:00.0 enp4s0: 0x00005080: 0x00009800,
> 0x80004000, 0x00000000, 0x00000000
> [ 6247.091227] tg3 0000:04:00.0 enp4s0: 0x00005090: 0x00000000,
> 0x00000000, 0x00000000, 0x08001028
> [ 6247.091230] tg3 0000:04:00.0 enp4s0: 0x000050a0: 0x30840fff,
> 0x00000000, 0x00000000, 0x40000020
> [ 6247.091233] tg3 0000:04:00.0 enp4s0: 0x000050b0: 0x00000000,
> 0x00000025, 0x00000000, 0x00000000
> [ 6247.091236] tg3 0000:04:00.0 enp4s0: 0x000050c0: 0x00000000,
> 0x00000000, 0x080011f4, 0x00000000
> [ 6247.091238] tg3 0000:04:00.0 enp4s0: 0x00005100: 0x00009800,
> 0x80000000, 0x00000000, 0x00000000
> [ 6247.091241] tg3 0000:04:00.0 enp4s0: 0x00005110: 0x00000000,
> 0x00000000, 0x00000000, 0x08001028
> [ 6247.091244] tg3 0000:04:00.0 enp4s0: 0x00005120: 0x30840fff,
> 0x00000000, 0x00000000, 0x40000020
> [ 6247.091247] tg3 0000:04:00.0 enp4s0: 0x00005130: 0x00000000,
> 0x00000025, 0x00000000, 0x00000000
> [ 6247.091250] tg3 0000:04:00.0 enp4s0: 0x00005140: 0x00000000,
> 0x00000000, 0x080011f4, 0x00000000
> [ 6247.091253] tg3 0000:04:00.0 enp4s0: 0x00005180: 0x00009800,
> 0x80000000, 0x00000000, 0x00000000
> [ 6247.091256] tg3 0000:04:00.0 enp4s0: 0x00005190: 0x00000000,
> 0x00000000, 0x00000000, 0x08001028
> [ 6247.091258] tg3 0000:04:00.0 enp4s0: 0x000051a0: 0x30840fff,
> 0x00000000, 0x00000000, 0x40000020
> [ 6247.091261] tg3 0000:04:00.0 enp4s0: 0x000051b0: 0x00000000,
> 0x00000025, 0x00000000, 0x00000000
> [ 6247.091264] tg3 0000:04:00.0 enp4s0: 0x000051c0: 0x00000000,
> 0x00000000, 0x080011f4, 0x00000000
> [ 6247.091266] tg3 0000:04:00.0 enp4s0: 0x00005200: 0x05762100,
> 0x00000000, 0x21100010, 0x00000000
> [ 6247.091270] tg3 0000:04:00.0 enp4s0: 0x00005210: 0x00010000,
> 0x00831824, 0xc0000000, 0x14400016
> [ 6247.091273] tg3 0000:04:00.0 enp4s0: 0x00005220: 0xc0000000,
> 0x21100010, 0x00000000, 0x00000000
> [ 6247.091276] tg3 0000:04:00.0 enp4s0: 0x00005230: 0x080012bc,
> 0x03608821, 0x14400016, 0x14830018
> [ 6247.091279] tg3 0000:04:00.0 enp4s0: 0x00005240: 0x0800150c,
> 0x00000000, 0x3c038000, 0x54000003
> [ 6247.091282] tg3 0000:04:00.0 enp4s0: 0x00005250: 0x03608821,
> 0x00010000, 0x14830018, 0x21100010
> [ 6247.091285] tg3 0000:04:00.0 enp4s0: 0x00005260: 0x00000000,
> 0x3c038000, 0x54000003, 0x03608821
> [ 6247.091288] tg3 0000:04:00.0 enp4s0: 0x00005270: 0xc0010000,
> 0x14830018, 0x21100010, 0x00000000
> [ 6247.091291] tg3 0000:04:00.0 enp4s0: 0x00005280: 0x00009800,
> 0x80004000, 0x00000000, 0x00000000
> [ 6247.091294] tg3 0000:04:00.0 enp4s0: 0x00005290: 0x00000000,
> 0x00000000, 0x00000000, 0x0800102c
> [ 6247.091297] tg3 0000:04:00.0 enp4s0: 0x000052a0: 0x3c04ff00,
> 0x00000000, 0x00000000, 0x40000020
> [ 6247.091300] tg3 0000:04:00.0 enp4s0: 0x000052b0: 0x00000000,
> 0x00000025, 0x00000000, 0x00000000
> [ 6247.091302] tg3 0000:04:00.0 enp4s0: 0x000052c0: 0x00000000,
> 0x00000000, 0x08000110, 0x00000000
> [ 6247.091305] tg3 0000:04:00.0 enp4s0: 0x00005300: 0x00009800,
> 0x80004000, 0x00000000, 0x00000000
> [ 6247.091308] tg3 0000:04:00.0 enp4s0: 0x00005310: 0x00000000,
> 0x00000000, 0x00000000, 0x08000100
> [ 6247.091311] tg3 0000:04:00.0 enp4s0: 0x00005320: 0x3c04ff00,
> 0x00000000, 0x00000000, 0x40000020
> [ 6247.091314] tg3 0000:04:00.0 enp4s0: 0x00005330: 0x00000000,
> 0x00000025, 0x00000000, 0x00000000
> [ 6247.091317] tg3 0000:04:00.0 enp4s0: 0x00005340: 0x00000000,
> 0x00000000, 0x08000110, 0x00000000
> [ 6247.091320] tg3 0000:04:00.0 enp4s0: 0x00005380: 0x00009800,
> 0x80004000, 0x00000000, 0x00000000
> [ 6247.091323] tg3 0000:04:00.0 enp4s0: 0x00005390: 0x00000000,
> 0x00000000, 0x00000000, 0x080000f8
> [ 6247.091325] tg3 0000:04:00.0 enp4s0: 0x000053a0: 0x3c04ff00,
> 0x00000000, 0x00000000, 0x40000020
> [ 6247.091328] tg3 0000:04:00.0 enp4s0: 0x000053b0: 0x00000000,
> 0x00000025, 0x00000000, 0x00000000
> [ 6247.091331] tg3 0000:04:00.0 enp4s0: 0x000053c0: 0x00000000,
> 0x00000000, 0x080011f4, 0x00000000
> [ 6247.091334] tg3 0000:04:00.0 enp4s0: 0x00005800: 0x00000000,
> 0x39000000, 0x00000000, 0xc3000000
> [ 6247.091337] tg3 0000:04:00.0 enp4s0: 0x00005810: 0x00000000,
> 0x51000000, 0x00000000, 0x00000001
> [ 6247.091340] tg3 0000:04:00.0 enp4s0: 0x00005820: 0x00000000,
> 0x00000001, 0x00000000, 0x00000000
> [ 6247.091342] tg3 0000:04:00.0 enp4s0: 0x00005860: 0x00000000,
> 0x00000000, 0x00000000, 0x0000021d
> [ 6247.091345] tg3 0000:04:00.0 enp4s0: 0x00005880: 0x00000000,
> 0x00000800, 0x00000000, 0x00000955
> [ 6247.091348] tg3 0000:04:00.0 enp4s0: 0x00005900: 0x00000000,
> 0x00000171, 0x00000000, 0x00000000
> [ 6247.091350] tg3 0000:04:00.0 enp4s0: 0x00005980: 0x00000171,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.091353] tg3 0000:04:00.0 enp4s0: 0x00005a00: 0x000f601f,
> 0x00000000, 0x00010000, 0x00000000
> [ 6247.091356] tg3 0000:04:00.0 enp4s0: 0x00006000: 0x20010082,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.091359] tg3 0000:04:00.0 enp4s0: 0x00006400: 0x00000000,
> 0x00000000, 0x00010091, 0xc0000000
> [ 6247.091362] tg3 0000:04:00.0 enp4s0: 0x00006410: 0x05000016,
> 0x05000016, 0x00000000, 0x00000000
> [ 6247.091364] tg3 0000:04:00.0 enp4s0: 0x00006430: 0x00000000,
> 0x14e41687, 0x2215103c, 0x10020000
> [ 6247.091367] tg3 0000:04:00.0 enp4s0: 0x00006440: 0x0000304f,
> 0x000002e4, 0x00000000, 0x00000080
> [ 6247.091371] tg3 0000:04:00.0 enp4s0: 0x000064c0: 0x00000005,
> 0x00000004, 0x00000122, 0x00000000
> [ 6247.091374] tg3 0000:04:00.0 enp4s0: 0x000064d0: 0x00000040,
> 0x10008d82, 0x00000000, 0x00d75e12
> [ 6247.091377] tg3 0000:04:00.0 enp4s0: 0x000064e0: 0x00000031,
> 0x0008001f, 0x00000000, 0x00000000
> [ 6247.091379] tg3 0000:04:00.0 enp4s0: 0x000064f0: 0x00000002,
> 0x00000031, 0x00000000, 0x00000000
> [ 6247.091382] tg3 0000:04:00.0 enp4s0: 0x00006500: 0x03e10003,
> 0xd45792bf, 0x00008cdc, 0x00000003
> [ 6247.091385] tg3 0000:04:00.0 enp4s0: 0x00006510: 0x00078116,
> 0x0005810b, 0x00046105, 0x00000000
> [ 6247.091388] tg3 0000:04:00.0 enp4s0: 0x00006530: 0x00000001,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.091390] tg3 0000:04:00.0 enp4s0: 0x00006540: 0x0028081f,
> 0x0001001e, 0x00000000, 0x00000000
> [ 6247.091393] tg3 0000:04:00.0 enp4s0: 0x00006550: 0x00000001,
> 0x02800000, 0x0000000f, 0x00000000
> [ 6247.091396] tg3 0000:04:00.0 enp4s0: 0x00006560: 0x0000000f,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.091399] tg3 0000:04:00.0 enp4s0: 0x000065f0: 0x00000000,
> 0x00000059, 0x00000000, 0x00000000
> [ 6247.091402] tg3 0000:04:00.0 enp4s0: 0x00006800: 0x141b0034,
> 0x20081082, 0x00029118, 0x730cfbb9
> [ 6247.091405] tg3 0000:04:00.0 enp4s0: 0x00006810: 0x21100010,
> 0xffffffff, 0x00000000, 0x000000f0
> [ 6247.091408] tg3 0000:04:00.0 enp4s0: 0x00006880: 0x77fff020,
> 0x00000040, 0x00801687, 0x00000000
> [ 6247.091410] tg3 0000:04:00.0 enp4s0: 0x00006890: 0x00800000,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.091413] tg3 0000:04:00.0 enp4s0: 0x000068a0: 0x00000000,
> 0x00010001, 0x00000000, 0x00000000
> [ 6247.091416] tg3 0000:04:00.0 enp4s0: 0x000068b0: 0x00040000,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.091419] tg3 0000:04:00.0 enp4s0: 0x000068c0: 0x00000044,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.091422] tg3 0000:04:00.0 enp4s0: 0x000068f0: 0x00000000,
> 0x00000000, 0x00000000, 0x04444444
> [ 6247.091424] tg3 0000:04:00.0 enp4s0: 0x00006900: 0x6f95a788,
> 0x14bd0ec2, 0x00000000, 0x00000000
> [ 6247.091427] tg3 0000:04:00.0 enp4s0: 0x00006920: 0x00000000,
> 0x00000000, 0x00000001, 0x00000000
> [ 6247.091430] tg3 0000:04:00.0 enp4s0: 0x00006c00: 0x168714e4,
> 0x00100506, 0x02000010, 0x00000010
> [ 6247.091433] tg3 0000:04:00.0 enp4s0: 0x00006c10: 0xe212000c,
> 0x00000000, 0xe211000c, 0x00000000
> [ 6247.091436] tg3 0000:04:00.0 enp4s0: 0x00006c20: 0xe210000c,
> 0x00000000, 0x00000000, 0x2215103c
> [ 6247.091438] tg3 0000:04:00.0 enp4s0: 0x00006c30: 0x00000000,
> 0x00000048, 0x00000000, 0x0000010a
> [ 6247.091441] tg3 0000:04:00.0 enp4s0: 0x00006c40: 0x00000000,
> 0x00000000, 0xc8035001, 0x16002008
> [ 6247.091444] tg3 0000:04:00.0 enp4s0: 0x00006c50: 0x00005803,
> 0x00000000, 0x0086a005, 0x00000000
> [ 6247.091447] tg3 0000:04:00.0 enp4s0: 0x00006c60: 0x00000000,
> 0x00000000, 0xf1000298, 0x01f802d1
> [ 6247.091450] tg3 0000:04:00.0 enp4s0: 0x00006c70: 0x00071010,
> 0xf6001500, 0x00000000, 0x00000000
> [ 6247.091453] tg3 0000:04:00.0 enp4s0: 0x00006ca0: 0x8005ac11,
> 0x00000004, 0x00000122, 0x00020010
> [ 6247.091456] tg3 0000:04:00.0 enp4s0: 0x00006cb0: 0x10008d82,
> 0x00115400, 0x00475c12, 0x10120043
> [ 6247.091459] tg3 0000:04:00.0 enp4s0: 0x00006cd0: 0x0008081f,
> 0x00000000, 0x00000000, 0x00010001
> [ 6247.091462] tg3 0000:04:00.0 enp4s0: 0x00006cf0: 0x00000000,
> 0x05762100, 0x00000000, 0x00000000
> [ 6247.091464] tg3 0000:04:00.0 enp4s0: 0x00006d00: 0x13c10001,
> 0x00000000, 0x00000000, 0x00062030
> [ 6247.091467] tg3 0000:04:00.0 enp4s0: 0x00006d10: 0x00001000,
> 0x00002000, 0x000000a0, 0x00000000
> [ 6247.091470] tg3 0000:04:00.0 enp4s0: 0x00006d30: 0x00000000,
> 0x00000000, 0x00000000, 0x15010003
> [ 6247.091473] tg3 0000:04:00.0 enp4s0: 0x00006d40: 0xd45792bf,
> 0x00008cdc, 0x00000000, 0x00000000
> [ 6247.091476] tg3 0000:04:00.0 enp4s0: 0x00006d50: 0x16010004,
> 0x00000000, 0x00078116, 0x00000001
> [ 6247.091479] tg3 0000:04:00.0 enp4s0: 0x00006d60: 0x1b010002,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.091482] tg3 0000:04:00.0 enp4s0: 0x00006d70: 0x00000000,
> 0x80000001, 0x00000000, 0x00000000
> [ 6247.091484] tg3 0000:04:00.0 enp4s0: 0x00006db0: 0x23010018,
> 0x00000000, 0x00000000, 0x00000000
> [ 6247.091487] tg3 0000:04:00.0 enp4s0: 0x00006e30: 0x00010017,
> 0x00050403, 0x00000000, 0x00000000
> [ 6247.091490] tg3 0000:04:00.0 enp4s0: 0x00007000: 0x08000008,
> 0x00000000, 0x00000000, 0x000038d4
> [ 6247.091493] tg3 0000:04:00.0 enp4s0: 0x00007010: 0x9928504d,
> 0x02808083, 0x800500db, 0x03000a00
> [ 6247.091496] tg3 0000:04:00.0 enp4s0: 0x00007020: 0x00000000,
> 0x00000008, 0x00000406, 0x10004000
> [ 6247.091499] tg3 0000:04:00.0 enp4s0: 0x00007030: 0x000e0000,
> 0x000038d8, 0x00230030, 0x80000000
> [ 6247.091502] tg3 0000:04:00.0 enp4s0: 0x00007500: 0x00000000,
> 0x00000000, 0x00000081, 0x00000000
> [ 6247.091505] tg3 0000:04:00.0 enp4s0: 0x00007510: 0x00000000,
> 0xffffdfff, 0x00000000, 0x00000000
> [ 6247.091510] tg3 0000:04:00.0 enp4s0: 0: Host status block
> [00000001:00000039:(0000:0145:0000):(0000:0167)]
> [ 6247.091514] tg3 0000:04:00.0 enp4s0: 0: NAPI info
> [00000039:00000039:(0171:0167:01ff):0000:(021d:0000:0000:0000)]
> [ 6247.091518] tg3 0000:04:00.0 enp4s0: 1: Host status block
> [00000001:000000c3:(0000:0000:0000):(0800:0000)]
> [ 6247.091521] tg3 0000:04:00.0 enp4s0: 1: NAPI info
> [000000c3:000000c3:(0000:0000:01ff):0800:(0000:0000:0000:0000)]
> [ 6247.091524] tg3 0000:04:00.0 enp4s0: 2: Host status block
> [00000001:00000051:(0955:0000:0000):(0000:0000)]
> [ 6247.091528] tg3 0000:04:00.0 enp4s0: 2: NAPI info
> [00000051:00000051:(0000:0000:01ff):0955:(0155:0155:0000:0000)]
> [ 6247.135111] tg3 0000:04:00.0 enp4s0: Link is down
> [ 6250.868825] tg3 0000:04:00.0 enp4s0: Link is up at 1000 Mbps, full duplex
> [ 6250.868845] tg3 0000:04:00.0 enp4s0: Flow control is on for TX and on for RX
> [ 6250.868849] tg3 0000:04:00.0 enp4s0: EEE is enabled
--
Thanks,
Siva
^ permalink raw reply
* Re: [PATCH 0/2] net: Set maximum receive packet size on veth interfaces
From: Cong Wang @ 2017-05-12 3:27 UTC (permalink / raw)
To: Fredrik Markström
Cc: Eric Dumazet, Daniel Borkmann, Linux Kernel Network Developers,
bridge, Alexei Starovoitov, LKML, David Miller
In-Reply-To: <CAKdL+dQh9ZEWZKHam0Z3iaVpxgZY0mziT3ZWN1OcuS55zUjEXg@mail.gmail.com>
On Tue, May 9, 2017 at 5:42 PM, Fredrik Markström
<fredrik.markstrom@gmail.com> wrote:
>
> Maybe I was unclear, the veth implementation drops all packers larger then the
> configured MTU (on the receiving interface).
> Most ethernet drivers accepts packets up to the ethernet MTU no matter the
> configured MTU. As far as I can tell from the RFC:s that is ok.
This is because IP layer does the fragmentation for you. But some drivers,
for example tg3, drop packet larger than its dev->mtu very early too.
>
> A simpler solution would be to only drop packets larger then ethernet MTU (like
> most network drivers), but I guess that will break existing stuff
> already out there.
I wonder why did we introduce that mtu check for veth when IP layer
could either fragment or reject with ICMP?
^ permalink raw reply
* Re: Implementing Dynamic Rerouting in Kernel
From: Ravish Kumar @ 2017-05-12 3:11 UTC (permalink / raw)
To: Florian Fainelli; +Cc: Networking, linux-kernel, mst, Jason
In-Reply-To: <3b6e4d7b-0e35-5944-1200-6b0721afca82@gmail.com>
Hi,
We do not want to add routes at run time rather i would prefer to have
a NetFilter driver which can intercept the packet and forward it to
desired interface.
On Thu, May 11, 2017 at 9:52 PM, Florian Fainelli <f.fainelli@gmail.com> wrote:
> On 05/11/2017 02:59 AM, Ravish Kumar wrote:
>> Hi Experts,
>>
>> Need expert advice for the one of the requirement Where in VPN
>> solution we want to dynaically route the packets to different adapter.
>> We will manage our own DNS cache and , based on DNS to IP lookup, we
>> can redirect the packet either to Tun device or to a physical adapter.
>>
>> Please suggest some design what i need to do.
>
> What you are looking for can be done using ipset-dns from Jason:
>
> https://git.zx2c4.com/ipset-dns/about/
> --
> Florian
^ permalink raw reply
* [PATCH] net: dsa: mv88e6xxx: add default case to switch
From: Gustavo A. R. Silva @ 2017-05-12 3:11 UTC (permalink / raw)
To: Andrew Lunn, Vivien Didelot, Florian Fainelli
Cc: netdev, linux-kernel, Gustavo A. R. Silva
In-Reply-To: <20170511214814.Horde.otwLVFIaWFNJZ23PSdgsq_Z@gator4166.hostgator.com>
Add default case to switch in order to avoid any chance of using an
uninitialized variable _low_, in case s->type does not match any of
the listed case values.
Addresses-Coverity-ID: 1398130
Suggested-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com>
---
drivers/net/dsa/mv88e6xxx/chip.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/net/dsa/mv88e6xxx/chip.c b/drivers/net/dsa/mv88e6xxx/chip.c
index 03dc886..d39e210 100644
--- a/drivers/net/dsa/mv88e6xxx/chip.c
+++ b/drivers/net/dsa/mv88e6xxx/chip.c
@@ -879,6 +879,9 @@ static uint64_t _mv88e6xxx_get_ethtool_stat(struct mv88e6xxx_chip *chip,
mv88e6xxx_g1_stats_read(chip, reg, &low);
if (s->sizeof_stat == 8)
mv88e6xxx_g1_stats_read(chip, reg + 1, &high);
+ break;
+ default:
+ return UINT64_MAX;
}
value = (((u64)high) << 16) | low;
return value;
--
2.5.0
^ permalink raw reply related
* Re: [PATCH net] net: phy: Call bus->reset() after releasing PHYs from reset
From: David Miller @ 2017-05-12 3:07 UTC (permalink / raw)
To: f.fainelli; +Cc: netdev, rogerq, andrew, linux-kernel
In-Reply-To: <20170511182417.17708-1-f.fainelli@gmail.com>
From: Florian Fainelli <f.fainelli@gmail.com>
Date: Thu, 11 May 2017 11:24:16 -0700
> The API convention makes it that a given MDIO bus reset should be able
> to access PHY devices in its reset() callback and perform additional
> MDIO accesses in order to bring the bus and PHYs in a working state.
>
> Commit 69226896ad63 ("mdio_bus: Issue GPIO RESET to PHYs.") broke that
> contract by first calling bus->reset() and then release all PHYs from
> reset using their shared GPIO line, so restore the expected
> functionality here.
>
> Fixes: 69226896ad63 ("mdio_bus: Issue GPIO RESET to PHYs.")
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Applied, thanks Florian.
^ permalink raw reply
* Re: [net-dsa-mv88e6xxx] question about potential use of uninitialized variable
From: Gustavo A. R. Silva @ 2017-05-12 2:48 UTC (permalink / raw)
To: Andrew Lunn; +Cc: Vivien Didelot, Florian Fainelli, netdev, linux-kernel
In-Reply-To: <20170512023333.GA18767@lunn.ch>
Hi Andrew,
Quoting Andrew Lunn <andrew@lunn.ch>:
> On Thu, May 11, 2017 at 04:35:37PM -0500, Gustavo A. R. Silva wrote:
>>
>> Hello everybody,
>>
>> While looking into Coverity ID 1398130 I ran into the following
>> piece of code at drivers/net/dsa/mv88e6xxx/chip.c:849:
>>
>> 849static uint64_t _mv88e6xxx_get_ethtool_stat(struct mv88e6xxx_chip *chip,
>> 850 struct mv88e6xxx_hw_stat *s,
>> 851 int port, u16 bank1_select,
>> 852 u16 histogram)
>> 853{
>> 854 u32 low;
>> 855 u32 high = 0;
>> 856 u16 reg = 0;
>> 857 int err;
>> 858 u64 value;
>> 859
>> 860 switch (s->type) {
>> 861 case STATS_TYPE_PORT:
>> 862 err = mv88e6xxx_port_read(chip, port, s->reg, ®);
>> 863 if (err)
>> 864 return UINT64_MAX;
>> 865
>> 866 low = reg;
>> 867 if (s->sizeof_stat == 4) {
>> 868 err = mv88e6xxx_port_read(chip, port,
>> s->reg + 1, ®);
>> 869 if (err)
>> 870 return UINT64_MAX;
>> 871 high = reg;
>> 872 }
>> 873 break;
>> 874 case STATS_TYPE_BANK1:
>> 875 reg = bank1_select;
>> 876 /* fall through */
>> 877 case STATS_TYPE_BANK0:
>> 878 reg |= s->reg | histogram;
>> 879 mv88e6xxx_g1_stats_read(chip, reg, &low);
>> 880 if (s->sizeof_stat == 8)
>> 881 mv88e6xxx_g1_stats_read(chip, reg + 1, &high);
>> 882 }
>> 883 value = (((u64)high) << 16) | low;
>> 884 return value;
>> 885}
>>
>> My question here is if there is any chance for the execution path to
>> directly jump from line 860 to line 883, hence ending up using the
>> uninitialized variable _low_?
>
> Hi Gustavo
>
> It would require that s->type not have one of the listed case values.
> Currently all members of mv88e6xxx_hw_stats due use expected values.
> However, it would not hurt to add a
>
> default:
> return UINT64_MAX;
>
> Do you want to submit a patch?
>
Sure, I'll send it shortly.
Thanks for clarifying!
--
Gustavo A. R. Silva
^ permalink raw reply
* Re: [PATCH v5 15/17] dt-bindings: qca7000: append UART interface to binding
From: Jakub Kicinski @ 2017-05-12 2:45 UTC (permalink / raw)
To: Michael Heimpold
Cc: Rob Herring, Stefan Wahren, Mark Rutland, Greg Kroah-Hartman,
Jiri Slaby, linux-serial, linux-kernel, netdev, devicetree
In-Reply-To: <1702237.66ccflAQAJ@kerker>
On Thu, 11 May 2017 21:12:22 +0200, Michael Heimpold wrote:
> Am Mittwoch, 10. Mai 2017, 10:53:26 CEST schrieb Stefan Wahren:
> > This merges the serdev binding for the QCA7000 UART driver (Ethernet over
> > UART) into the existing document.
> >
> > Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com>
> > ---
> > .../devicetree/bindings/net/qca-qca7000.txt | 32
> > ++++++++++++++++++++++ 1 file changed, 32 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/net/qca-qca7000.txt
> > b/Documentation/devicetree/bindings/net/qca-qca7000.txt index
> > a37f656..08364c3 100644
> > --- a/Documentation/devicetree/bindings/net/qca-qca7000.txt
> > +++ b/Documentation/devicetree/bindings/net/qca-qca7000.txt
> > @@ -54,3 +54,35 @@ ssp2: spi@80014000 {
> > local-mac-address = [ A0 B0 C0 D0 E0 F0 ];
> > };
> > };
> > +
> > +(b) Ethernet over UART
> > +
> > +In order to use the QCA7000 as UART slave it must be defined as a child of
> > a +UART master in the device tree. It is possible to preconfigure the UART
> > +settings of the QCA7000 firmware, but it's not possible to change them
> > during +runtime.
> > +
> > +Required properties:
> > +- compatible : Should be "qca,qca7000-uart"
>
> I already discussed this with Stefan off-list a little bit, but I would like
> to bring this to a broader audience: I'm not sure whether the compatible
> should contain the "-uart" suffix, because the hardware chip is the very same
> QCA7000 chip which can also be used with SPI protocol.
> The only difference is the loaded firmware within the chip which can either
> speak SPI or UART protocol (but not both at the same time - due to shared
> pins). So the hardware design decides which interface type is used.
>
> At the moment, this patch series adds a dedicated driver for the UART
> protocol, in parallel to the existing SPI driver. So a different compatible
> string is needed here to match against the new driver.
>
> An alternative approach would be to re-use the existing compatible string
> "qca,qca7000" for both, the SPI and UART protocol, because a "smarter"
> (combined) driver would detect which protocol to use. For example the driver
> could check for spi-cpha and/or spi-cpol which are required for SPI protocol:
> if these exists the driver could assume that SPI must be used, if both are
> missing then UART protocol should be used.
> (This way it would not be necessary to check whether the node is a child of
> a SPI or UART master node - but maybe this is even easier - I don't know)
>
> Or in shorter words: my concern is that while "qca7000-uart" describes the
> hardware, it's too closely coupled to the driver implementation. Having
> some feedback of the experts would be nice :-)
I'm no expert, but devices which can do both I2C and SPI are quite
common, and they usually have the same compatible string for both
buses.
^ permalink raw reply
* Re: [net-dsa-mv88e6xxx] question about potential use of uninitialized variable
From: Andrew Lunn @ 2017-05-12 2:33 UTC (permalink / raw)
To: Gustavo A. R. Silva
Cc: Vivien Didelot, Florian Fainelli, netdev, linux-kernel
In-Reply-To: <20170511163537.Horde.CH1Q_6Lo2SOIbQB6MQJFV3g@gator4166.hostgator.com>
On Thu, May 11, 2017 at 04:35:37PM -0500, Gustavo A. R. Silva wrote:
>
> Hello everybody,
>
> While looking into Coverity ID 1398130 I ran into the following
> piece of code at drivers/net/dsa/mv88e6xxx/chip.c:849:
>
> 849static uint64_t _mv88e6xxx_get_ethtool_stat(struct mv88e6xxx_chip *chip,
> 850 struct mv88e6xxx_hw_stat *s,
> 851 int port, u16 bank1_select,
> 852 u16 histogram)
> 853{
> 854 u32 low;
> 855 u32 high = 0;
> 856 u16 reg = 0;
> 857 int err;
> 858 u64 value;
> 859
> 860 switch (s->type) {
> 861 case STATS_TYPE_PORT:
> 862 err = mv88e6xxx_port_read(chip, port, s->reg, ®);
> 863 if (err)
> 864 return UINT64_MAX;
> 865
> 866 low = reg;
> 867 if (s->sizeof_stat == 4) {
> 868 err = mv88e6xxx_port_read(chip, port,
> s->reg + 1, ®);
> 869 if (err)
> 870 return UINT64_MAX;
> 871 high = reg;
> 872 }
> 873 break;
> 874 case STATS_TYPE_BANK1:
> 875 reg = bank1_select;
> 876 /* fall through */
> 877 case STATS_TYPE_BANK0:
> 878 reg |= s->reg | histogram;
> 879 mv88e6xxx_g1_stats_read(chip, reg, &low);
> 880 if (s->sizeof_stat == 8)
> 881 mv88e6xxx_g1_stats_read(chip, reg + 1, &high);
> 882 }
> 883 value = (((u64)high) << 16) | low;
> 884 return value;
> 885}
>
> My question here is if there is any chance for the execution path to
> directly jump from line 860 to line 883, hence ending up using the
> uninitialized variable _low_?
Hi Gustavo
It would require that s->type not have one of the listed case values.
Currently all members of mv88e6xxx_hw_stats due use expected values.
However, it would not hurt to add a
default:
return UINT64_MAX;
Do you want to submit a patch?
Thanks
Andrew
^ permalink raw reply
* Re: [PATCH v2 1/7] bpf: Track alignment of register values in the verifier.
From: David Miller @ 2017-05-12 2:31 UTC (permalink / raw)
To: ast; +Cc: daniel, alexei.starovoitov, netdev
In-Reply-To: <20170511.211406.1201174799693258613.davem@davemloft.net>
From: David Miller <davem@davemloft.net>
Date: Thu, 11 May 2017 21:14:06 -0400 (EDT)
> Indeed, we have to accumulate. I was just thinking about this earlier
> today.
I've just pushed the following fix, thanks a lot.
====================
[PATCH] bpf: Handle multiple variable additions into packet pointers in verifier.
We must accumulate into reg->aux_off rather than use a plain assignment.
Add a test for this situation to test_align.
Reported-by: Alexei Starovoitov <ast@fb.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
---
kernel/bpf/verifier.c | 2 +-
tools/testing/selftests/bpf/test_align.c | 37 ++++++++++++++++++++++++++++++++
2 files changed, 38 insertions(+), 1 deletion(-)
diff --git a/kernel/bpf/verifier.c b/kernel/bpf/verifier.c
index e74fb1b..39f2dcb 100644
--- a/kernel/bpf/verifier.c
+++ b/kernel/bpf/verifier.c
@@ -1531,7 +1531,7 @@ static int check_packet_ptr_add(struct bpf_verifier_env *env,
dst_reg->id = ++env->id_gen;
/* something was added to pkt_ptr, set range to zero */
- dst_reg->aux_off = dst_reg->off;
+ dst_reg->aux_off += dst_reg->off;
dst_reg->off = 0;
dst_reg->range = 0;
if (had_id)
diff --git a/tools/testing/selftests/bpf/test_align.c b/tools/testing/selftests/bpf/test_align.c
index ed24255..9644d4e 100644
--- a/tools/testing/selftests/bpf/test_align.c
+++ b/tools/testing/selftests/bpf/test_align.c
@@ -273,6 +273,20 @@ static struct bpf_align_test tests[] = {
BPF_EXIT_INSN(),
BPF_LDX_MEM(BPF_W, BPF_REG_4, BPF_REG_5, 0),
+ /* Test multiple accumulations of unknown values
+ * into a packet pointer.
+ */
+ BPF_MOV64_REG(BPF_REG_5, BPF_REG_2),
+ BPF_ALU64_IMM(BPF_ADD, BPF_REG_5, 14),
+ BPF_ALU64_REG(BPF_ADD, BPF_REG_5, BPF_REG_6),
+ BPF_ALU64_IMM(BPF_ADD, BPF_REG_5, 4),
+ BPF_ALU64_REG(BPF_ADD, BPF_REG_5, BPF_REG_6),
+ BPF_MOV64_REG(BPF_REG_4, BPF_REG_5),
+ BPF_ALU64_IMM(BPF_ADD, BPF_REG_4, 4),
+ BPF_JMP_REG(BPF_JGE, BPF_REG_3, BPF_REG_4, 1),
+ BPF_EXIT_INSN(),
+ BPF_LDX_MEM(BPF_W, BPF_REG_4, BPF_REG_5, 0),
+
BPF_MOV64_IMM(BPF_REG_0, 0),
BPF_EXIT_INSN(),
},
@@ -314,6 +328,29 @@ static struct bpf_align_test tests[] = {
* requirements.
*/
"23: R0=pkt(id=0,off=8,r=8) R1=ctx R2=pkt(id=0,off=0,r=8) R3=pkt_end R4=pkt(id=2,off=18,r=18),aux_off_align=4 R5=pkt(id=2,off=14,r=18),aux_off_align=4 R6=inv54,min_align=4 R10=fp",
+
+ /* Constant offset is added to R5 packet pointer,
+ * resulting in reg->off value of 14.
+ */
+ "26: R0=pkt(id=0,off=8,r=8) R1=ctx R2=pkt(id=0,off=0,r=8) R3=pkt_end R4=inv,aux_off_align=4 R5=pkt(id=0,off=14,r=8) R6=inv54,min_align=4 R10=fp",
+ /* Variable offset is added to R5, resulting in an
+ * auxiliary offset of 14, and an auxiliary alignment of 4.
+ */
+ "27: R0=pkt(id=0,off=8,r=8) R1=ctx R2=pkt(id=0,off=0,r=8) R3=pkt_end R4=inv,aux_off_align=4 R5=pkt(id=3,off=0,r=0),aux_off=14,aux_off_align=4 R6=inv54,min_align=4 R10=fp",
+ /* Constant is added to R5 again, setting reg->off to 4. */
+ "28: R0=pkt(id=0,off=8,r=8) R1=ctx R2=pkt(id=0,off=0,r=8) R3=pkt_end R4=inv,aux_off_align=4 R5=pkt(id=3,off=4,r=0),aux_off=14,aux_off_align=4 R6=inv54,min_align=4 R10=fp",
+ /* And once more we add a variable, which causes an accumulation
+ * of reg->off into reg->aux_off_align, with resulting value of
+ * 18. The auxiliary alignment stays at 4.
+ */
+ "29: R0=pkt(id=0,off=8,r=8) R1=ctx R2=pkt(id=0,off=0,r=8) R3=pkt_end R4=inv,aux_off_align=4 R5=pkt(id=4,off=0,r=0),aux_off=18,aux_off_align=4 R6=inv54,min_align=4 R10=fp",
+ /* At the time the word size load is performed from R5,
+ * it's total offset is NET_IP_ALIGN + reg->off (0) +
+ * reg->aux_off (18) which is 20. Then the variable offset
+ * is considered using reg->aux_off_align which is 4 and meets
+ * the load's requirements.
+ */
+ "33: R0=pkt(id=0,off=8,r=8) R1=ctx R2=pkt(id=0,off=0,r=8) R3=pkt_end R4=pkt(id=4,off=4,r=4),aux_off=18,aux_off_align=4 R5=pkt(id=4,off=0,r=4),aux_off=18,aux_off_align=4 R6=inv54,min_align=4 R10=fp",
},
},
};
--
2.1.2.532.g19b5d50
^ permalink raw reply related
* Re: [PATCH net 1/1] tipc: make macro tipc_wait_for_cond() smp safe
From: David Miller @ 2017-05-12 2:20 UTC (permalink / raw)
To: jon.maloy; +Cc: netdev, parthasarathy.bhuvaragan, ying.xue, tipc-discussion
In-Reply-To: <1494527295-20646-1-git-send-email-jon.maloy@ericsson.com>
From: Jon Maloy <jon.maloy@ericsson.com>
Date: Thu, 11 May 2017 20:28:15 +0200
> The macro tipc_wait_for_cond() is embedding the macro sk_wait_event()
> to fulfil its task. The latter, in turn, is evaluating the stated
> condition outside the socket lock context. This is problematic if
> the condition is accessing non-trivial data structures which may be
> altered by incoming interrupts, as is the case with the cong_links()
> linked list, used by socket to keep track of the current set of
> congested links. We sometimes see crashes when this list is accessed
> by a condition function at the same time as a SOCK_WAKEUP interrupt
> is removing an element from the list.
>
> We fix this by expanding selected parts of sk_wait_event() into the
> outer macro, while ensuring that all evaluations of a given condition
> are performed under socket lock protection.
>
> Fixes: commit 365ad353c256 ("tipc: reduce risk of user starvation
> during link congestion")
>
> Reviewed-by: Parthasarathy Bhuvaragan <parthasarathy.bhuvaragan@ericsson.com>
> Signed-off-by: Jon Maloy <jon.maloy@ericsson.com>
Applied and queued up for -stable, thanks Jon.
^ permalink raw reply
* Re: BPF relocations
From: Alexei Starovoitov @ 2017-05-12 2:17 UTC (permalink / raw)
To: David Miller; +Cc: daniel, netdev
In-Reply-To: <20170511.211920.2220259143750653022.davem@davemloft.net>
On 5/11/17 6:19 PM, David Miller wrote:
> From: Alexei Starovoitov <ast@fb.com>
> Date: Thu, 11 May 2017 16:10:35 -0700
>
>> I don't see what we would use pc-relative relo for.
>
> We must have them at least for banches.
>
> Internally, we have to emit some kind of relocation as GAS makes it's
> first pass over the instructions.
>
> Afterwards, it walks the relocations and resolves all that can be done
> at assembly time, and preserves as relocations in the object file for
> those which it cannot.
got it. yes.
llvm has this thing as well. It calls it FK_PCRel_2
(FK stands for Fixup Kind) which is 16-bit pc relative relo
into 'off' bits of insn.
Fortunately for LLVM this fixup is never converted to actual relo
and it is applied before final .o is emitted.
So, indeed, defining 16-bit pc-relative relo for branches is
definitely useful.
I can imagine someone combining two .o files with 'goto's
crossing two functions or something.
Let's pick a code for it.
> Thinking further down the line many other kinds of PC relative
> relocations are possible, even if you don't allow calls. For example:
>
> ldimm64 R1, 24 + (. - external_label)
>
> This would be a 64-bit PC relative reloc, with the value 24 in the
> addend.
yes. that would be useful as well.
I don't mind defining it for our semi-official bpf relo spec :)
> And eventually we want full linking.
>
> The example above may seem silly, but every other full CPU ELF
> specification handles these things completely and we should seek to be
> complete as well.
I agree that the further we go the more complete it will become.
My concern that if we define too many things because other archs
have them we may end up not using them when time comes, since we
didn't think of some minor detail.
So today I would only add 16-bit pc-relative and 64-bit pc-relative.
The former is clearly useful for branches and the latter for calls
and jmp tables.
Right now we don't have simple 'switch()'. llvm has to convert it
into a sequence of branches. It's ok-ish. Ideally we need jmp-table
like normal cpus do and ldimm64 + relo should work for such purpose.
^ permalink raw reply
* Re: [PATCH net] samples/bpf: run cleanup routines when receiving SIGTERM
From: David Miller @ 2017-05-12 1:43 UTC (permalink / raw)
To: andy; +Cc: netdev, shahid.habib
In-Reply-To: <1494532350-23945-1-git-send-email-andy@greyhouse.net>
From: Andy Gospodarek <andy@greyhouse.net>
Date: Thu, 11 May 2017 15:52:30 -0400
> Shahid Habib noticed that when xdp1 was killed from a different console the xdp
> program was not cleaned-up properly in the kernel and it continued to forward
> traffic.
>
> Most of the applications in samples/bpf cleanup properly, but only when getting
> SIGINT. Since kill defaults to using SIGTERM, add support to cleanup when the
> application receives either SIGINT or SIGTERM.
>
> Signed-off-by: Andy Gospodarek <andy@greyhouse.net>
> Reported-by: Shahid Habib <shahid.habib@broadcom.com>
Applied, thanks Andy.
^ permalink raw reply
* Re: [PATCH][V2] ethernet: aquantia: remove redundant checks on error status
From: David Miller @ 2017-05-12 1:42 UTC (permalink / raw)
To: colin.king
Cc: pavel.belous, vomlehn, Alexander.Loktionov, Dmitry.Bezrukov,
Dmitrii.Tarakanov, netdev, kernel-janitors, linux-kernel
In-Reply-To: <20170511182940.18774-1-colin.king@canonical.com>
From: Colin King <colin.king@canonical.com>
Date: Thu, 11 May 2017 19:29:40 +0100
> From: Colin Ian King <colin.king@canonical.com>
>
> The error status err is initialized as zero and then being checked
> several times to see if it is less than zero even when it has not
> been updated. It may seem that the err should be assigned to the
> return code of the call to the various *offload_en_set calls and
> then we check for failure, however, these functions are void and
> never actually return any status.
>
> Since these error checks are redundant we can remove these
> as well as err and the error exit label err_exit.
>
> Detected by CoverityScan, CID#1398313 and CID#1398306 ("Logically
> dead code")
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
Applied, thank you.
^ permalink raw reply
* Re: [PATCH net 0/2] qlcnic: Bug fix and update version
From: David Miller @ 2017-05-12 1:40 UTC (permalink / raw)
To: manish.chopra; +Cc: netdev, Dept-GELinuxNICDev
In-Reply-To: <20170511141248.996-1-manish.chopra@cavium.com>
From: Manish Chopra <manish.chopra@cavium.com>
Date: Thu, 11 May 2017 07:12:46 -0700
> This series has one fix and bumps up driver version.
> Please consider applying to "net"
Series applied, thanks.
^ permalink raw reply
* Re: [PATCH v2] xen-netfront: avoid crashing on resume after a failure in talk_to_netback()
From: David Miller @ 2017-05-12 1:39 UTC (permalink / raw)
To: vkuznets; +Cc: xen-devel, netdev, linux-kernel, boris.ostrovsky, jgross
In-Reply-To: <20170511115806.25322-1-vkuznets@redhat.com>
From: Vitaly Kuznetsov <vkuznets@redhat.com>
Date: Thu, 11 May 2017 13:58:06 +0200
> Unavoidable crashes in netfront_resume() and netback_changed() after a
> previous fail in talk_to_netback() (e.g. when we fail to read MAC from
> xenstore) were discovered. The failure path in talk_to_netback() does
> unregister/free for netdev but we don't reset drvdata and we try accessing
> it after resume.
>
> Fix the bug by removing the whole xen device completely with
> device_unregister(), this guarantees we won't have any calls into netfront
> after a failure.
>
> Signed-off-by: Vitaly Kuznetsov <vkuznets@redhat.com>
> ---
> Changes since v1: instead of cleaning drvdata and checking for it in
> netfront_resume() and netback_changed() remove the device completely with
> device_unregister() [David Miller]
This looks a lot better, applied, thanks!
^ permalink raw reply
* Re: [PATCH net] net: sched: optimize class dumps
From: David Miller @ 2017-05-12 1:38 UTC (permalink / raw)
To: eric.dumazet; +Cc: netdev, jkosina, jhs, xiyou.wangcong, jiri
In-Reply-To: <1494478768.7796.108.camel@edumazet-glaptop3.roam.corp.google.com>
From: Eric Dumazet <eric.dumazet@gmail.com>
Date: Wed, 10 May 2017 21:59:28 -0700
> From: Eric Dumazet <edumazet@google.com>
>
> In commit 59cc1f61f09c ("net: sched: convert qdisc linked list to
> hashtable") we missed the opportunity to considerably speed up
> tc_dump_tclass_root() if a qdisc handle is provided by user.
>
> Instead of iterating all the qdiscs, use qdisc_match_from_root()
> to directly get the one we look for.
>
> Signed-off-by: Eric Dumazet <edumazet@google.com>
Applied, thanks Eric.
^ permalink raw reply
* Re: [PATCH net] tcp: avoid fragmenting peculiar skbs in SACK
From: David Miller @ 2017-05-12 1:36 UTC (permalink / raw)
To: ycheng; +Cc: netdev, ncardwell, edumazet, soheil
In-Reply-To: <20170511000127.4249-1-ycheng@google.com>
From: Yuchung Cheng <ycheng@google.com>
Date: Wed, 10 May 2017 17:01:27 -0700
> This patch fixes a bug in splitting an SKB during SACK
> processing. Specifically if an skb contains multiple
> packets and is only partially sacked in the higher sequences,
> tcp_match_sack_to_skb() splits the skb and marks the second fragment
> as SACKed.
>
> The current code further attempts rounding up the first fragment
> to MSS boundaries. But it misses a boundary condition when the
> rounded-up fragment size (pkt_len) is exactly skb size. Spliting
> such an skb is pointless and causses a kernel warning and aborts
> the SACK processing. This patch universally checks such over-split
> before calling tcp_fragment to prevent these unnecessary warnings.
>
> Fixes: adb92db857ee ("tcp: Make SACK code to split only at mss boundaries")
> Signed-off-by: Yuchung Cheng <ycheng@google.com>
> Signed-off-by: Eric Dumazet <edumazet@google.com>
> Signed-off-by: Soheil Hassas Yeganeh <soheil@google.com>
Hehe, a 2.6.29 bug, funny that it lived for so long.
Applied and queued up for -stable, thanks!
^ permalink raw reply
page: next (older) | prev (newer) | latest
- recent:[subjects (threaded)|topics (new)|topics (active)]
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox