All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rudolf Spring <rudolf.spring@gmail.com>
To: Siva Reddy Kallam <siva.kallam@broadcom.com>
Cc: Michael Chan <michael.chan@broadcom.com>,
	Netdev <netdev@vger.kernel.org>
Subject: Re: Gentoo Linux 5.x - Tigon3
Date: Mon, 11 Nov 2019 08:15:45 +0000	[thread overview]
Message-ID: <3FA0ADFD-6F62-43E4-9119-64F7AC1BCD27@gmail.com> (raw)
In-Reply-To: <7C0AE73E-DB62-45D1-B358-3CC47D0EB6B0@gmail.com>

I found a workaround. Changing the mac of br0 to the mac of the tg3 and so far no more tx mac errors. I’m not sure but its probably a bridge and not a tg3 problem. With kernel < 5.0 this was not necessary.

> On 31 Oct 2019, at 19:42, Rudolf Spring <rudolf.spring@gmail.com> wrote:
> 
> I’m using it in bridged mode with wlan adapters and hostapd. The problem only happens if I access over eth0 not over wlan0 or wlan1. STP is not the problem. Same config with 4.19 has no problems only after 5.0.
> 
> driver: tg3
> version: 3.137
> firmware-version: 57766a-v1.13
> expansion-rom-version: 
> bus-info: 0000:01:00.0
> supports-statistics: yes
> supports-test: yes
> supports-eeprom-access: yes
> supports-register-dump: yes
> supports-priv-flags: no
> 
>> On 31 Oct 2019, at 11:49, Siva Reddy Kallam <siva.kallam@broadcom.com> wrote:
>> 
>> Tried reproducing local with 5.4.0-rc3 kernel and I couldn't observe the issue.
>> I used the internet for almost 15-20 mins and no MAC errors observed.
>> Can you please provide ethtool -i output? I will use the same f/w
>> version and try.
>> 
>> On Fri, Oct 18, 2019 at 12:21 PM Siva Reddy Kallam
>> <siva.kallam@broadcom.com> wrote:
>>> 
>>> On Thu, Oct 17, 2019 at 11:03 PM Rudolf Spring <rudolf.spring@gmail.com> wrote:
>>>> 
>>>> Can you reproduce the error in the lab ? Same behaviour with Kernel 5.3.6.
>>>> 
>>> Yes. We are trying to reproduce in our local lab.  We will update once
>>> we have reproduction.
>>>> 
>>>> eth0: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 9000
>>>>       ether a8:20:66:28:e6:95  txqueuelen 1000  (Ethernet)
>>>>       RX packets 13844854  bytes 2298858755 (2.1 GiB)
>>>>       RX errors 0  dropped 2  overruns 0  frame 0
>>>>       TX packets 14831625  bytes 75684684566 (70.4 GiB)
>>>>       TX errors 158  dropped 0 overruns 0  carrier 0  collisions 0
>>>>       device interrupt 16
>>>> 
>>>>    tx_octets: 75686159197
>>>>    tx_collisions: 0
>>>>    tx_xon_sent: 0
>>>>    tx_xoff_sent: 0
>>>>    tx_flow_control: 0
>>>>    tx_mac_errors: 158
>>>>    tx_single_collisions: 0
>>>>    tx_mult_collisions: 0
>>>>    tx_deferred: 0
>>>>    tx_excessive_collisions: 0
>>>>    tx_late_collisions: 0
>>>>    tx_collide_2times: 0
>>>>    tx_collide_3times: 0
>>>>    tx_collide_4times: 0
>>>>    tx_collide_5times: 0
>>>>    tx_collide_6times: 0
>>>>    tx_collide_7times: 0
>>>>    tx_collide_8times: 0
>>>>    tx_collide_9times: 0
>>>>    tx_collide_10times: 0
>>>>    tx_collide_11times: 0
>>>>    tx_collide_12times: 0
>>>>    tx_collide_13times: 0
>>>>    tx_collide_14times: 0
>>>>    tx_collide_15times: 0
>>>>    tx_ucast_packets: 14400082
>>>>    tx_mcast_packets: 435104
>>>>    tx_bcast_packets: 1542
>>>>    tx_carrier_sense_errors: 0
>>>>    tx_discards: 0
>>>>    tx_errors: 0
>>>> 
>>>> 
>>>>> On 4 Oct 2019, at 14:53, Rudolf Spring <rudolf.spring@gmail.com> wrote:
>>>>> 
>>>>> With Kernel 5.3.2. Interesting all are tx_mac_errors.
>>>>> 
>>>>> ifconfig  eth0
>>>>> eth0: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 9000
>>>>>      ether a8:20:66:28:e6:95  txqueuelen 1000  (Ethernet)
>>>>>      RX packets 1649204  bytes 775261068 (739.3 MiB)
>>>>>      RX errors 0  dropped 0  overruns 0  frame 0
>>>>>      TX packets 1144621  bytes 1241414276 (1.1 GiB)
>>>>>      TX errors 369  dropped 0 overruns 0  carrier 0  collisions 0
>>>>>      device interrupt 16
>>>>> 
>>>>> ethtool -S eth0
>>>>> NIC statistics:
>>>>>   rx_octets: 752756285
>>>>>   rx_fragments: 0
>>>>>   rx_ucast_packets: 1132211
>>>>>   rx_mcast_packets: 128115
>>>>>   rx_bcast_packets: 372162
>>>>>   rx_fcs_errors: 0
>>>>>   rx_align_errors: 0
>>>>>   rx_xon_pause_rcvd: 0
>>>>>   rx_xoff_pause_rcvd: 0
>>>>>   rx_mac_ctrl_rcvd: 0
>>>>>   rx_xoff_entered: 0
>>>>>   rx_frame_too_long_errors: 0
>>>>>   rx_jabbers: 0
>>>>>   rx_undersize_packets: 0
>>>>>   rx_in_length_errors: 0
>>>>>   rx_out_length_errors: 0
>>>>>   rx_64_or_less_octet_packets: 0
>>>>>   rx_65_to_127_octet_packets: 0
>>>>>   rx_128_to_255_octet_packets: 0
>>>>>   rx_256_to_511_octet_packets: 0
>>>>>   rx_512_to_1023_octet_packets: 0
>>>>>   rx_1024_to_1522_octet_packets: 0
>>>>>   rx_1523_to_2047_octet_packets: 0
>>>>>   rx_2048_to_4095_octet_packets: 0
>>>>>   rx_4096_to_8191_octet_packets: 0
>>>>>   rx_8192_to_9022_octet_packets: 0
>>>>>   tx_octets: 1236703101
>>>>>   tx_collisions: 0
>>>>>   tx_xon_sent: 0
>>>>>   tx_xoff_sent: 0
>>>>>   tx_flow_control: 0
>>>>>   tx_mac_errors: 369
>>>>>   tx_single_collisions: 0
>>>>>   tx_mult_collisions: 0
>>>>>   tx_deferred: 0
>>>>>   tx_excessive_collisions: 0
>>>>>   tx_late_collisions: 0
>>>>>   tx_collide_2times: 0
>>>>>   tx_collide_3times: 0
>>>>>   tx_collide_4times: 0
>>>>>   tx_collide_5times: 0
>>>>>   tx_collide_6times: 0
>>>>>   tx_collide_7times: 0
>>>>>   tx_collide_8times: 0
>>>>>   tx_collide_9times: 0
>>>>>   tx_collide_10times: 0
>>>>>   tx_collide_11times: 0
>>>>>   tx_collide_12times: 0
>>>>>   tx_collide_13times: 0
>>>>>   tx_collide_14times: 0
>>>>>   tx_collide_15times: 0
>>>>>   tx_ucast_packets: 986854
>>>>>   tx_mcast_packets: 146951
>>>>>   tx_bcast_packets: 1117
>>>>>   tx_carrier_sense_errors: 0
>>>>>   tx_discards: 0
>>>>>   tx_errors: 0
>>>>>   dma_writeq_full: 0
>>>>>   dma_write_prioq_full: 0
>>>>>   rxbds_empty: 0
>>>>>   rx_discards: 0
>>>>>   rx_errors: 0
>>>>>   rx_threshold_hit: 0
>>>>>   dma_readq_full: 0
>>>>>   dma_read_prioq_full: 0
>>>>>   tx_comp_queue_full: 0
>>>>>   ring_set_send_prod_index: 0
>>>>>   ring_status_update: 0
>>>>>   nic_irqs: 0
>>>>>   nic_avoided_irqs: 0
>>>>>   nic_tx_threshold_hit: 0
>>>>>   mbuf_lwm_thresh_hit: 0
>>>>> 
>>>>>> On 4 Oct 2019, at 12:52, Siva Reddy Kallam <siva.kallam@broadcom.com> wrote:
>>>>>> 
>>>>>> 
>>>>>> 
>>>>>> On Wed, Oct 2, 2019 at 10:05 PM Rudolf Spring <rudolf.spring@gmail.com> wrote:
>>>>>> The output of dmesg and ethtool is identical between 4.19.72 and 5.3.2. Any suggestions ?
>>>>>> Can you please provide the output of "ethtool -S eth0" command?
>>>>>> In the mean time, I will review the register dump and also try to reproduce in our lab.
>>>>>> 0000:01:00.0: enabling device (0000 -> 0002)
>>>>>> [    1.140738] tg3 0000:01:00.0 eth0: Tigon3 [partno(BCM957766a) rev 57766001] (PCI Express) MAC address a8:20:66:28:e6:95
>>>>>> [    1.140741] tg3 0000:01:00.0 eth0: attached PHY is 57765 (10/100/1000Base-T Ethernet) (WireSpeed[1], EEE[1])
>>>>>> [    1.140743] tg3 0000:01:00.0 eth0: RXcsums[1] LinkChgREG[0] MIirq[0] ASF[0] TSOcap[1]
>>>>>> [    1.140744] tg3 0000:01:00.0 eth0: dma_rwctrl[00000001] dma_mask[64-bit]
>>>>>> [   10.290239] tg3 0000:01:00.0 eth0: Link is up at 1000 Mbps, full duplex
>>>>>> [   10.290241] tg3 0000:01:00.0 eth0: Flow control is on for TX and on for RX
>>>>>> [   10.290242] tg3 0000:01:00.0 eth0: EEE is enabled
>>>>>> 
>>>>>> ethtool eth0
>>>>>> Settings for eth0:
>>>>>>      Supported ports: [ TP ]
>>>>>>      Supported link modes:   10baseT/Half 10baseT/Full
>>>>>>                              100baseT/Half 100baseT/Full
>>>>>>                              1000baseT/Half 1000baseT/Full
>>>>>>      Supported pause frame use: No
>>>>>>      Supports auto-negotiation: Yes
>>>>>>      Supported FEC modes: Not reported
>>>>>>      Advertised link modes:  10baseT/Half 10baseT/Full
>>>>>>                              100baseT/Half 100baseT/Full
>>>>>>                              1000baseT/Half 1000baseT/Full
>>>>>>      Advertised pause frame use: Symmetric
>>>>>>      Advertised auto-negotiation: Yes
>>>>>>      Advertised FEC modes: Not reported
>>>>>>      Link partner advertised link modes:  10baseT/Half 10baseT/Full
>>>>>>                                           100baseT/Half 100baseT/Full
>>>>>>                                           1000baseT/Full
>>>>>>      Link partner advertised pause frame use: Symmetric
>>>>>>      Link partner advertised auto-negotiation: Yes
>>>>>>      Link partner advertised FEC modes: Not reported
>>>>>>      Speed: 1000Mb/s
>>>>>>      Duplex: Full
>>>>>>      Port: Twisted Pair
>>>>>>      PHYAD: 1
>>>>>>      Transceiver: internal
>>>>>>      Auto-negotiation: on
>>>>>>      MDI-X: off
>>>>>>      Supports Wake-on: g
>>>>>>      Wake-on: g
>>>>>>      Current message level: 0x000000ff (255)
>>>>>>                             drv probe link timer ifdown ifup rx_err tx_err
>>>>>>      Link detected: yes
>>>>>> 
>>>>>> ethtool -a eth0
>>>>>> Pause parameters for eth0:
>>>>>> Autonegotiate:  on
>>>>>> RX:             on
>>>>>> TX:             on
>>>>>> RX negotiated:  on
>>>>>> TX negotiated:  on
>>>>>> 
>>>>>> ethtool -g eth0
>>>>>> Ring parameters for eth0:
>>>>>> Pre-set maximums:
>>>>>> RX:             511
>>>>>> RX Mini:        0
>>>>>> RX Jumbo:       255
>>>>>> TX:             511
>>>>>> Current hardware settings:
>>>>>> RX:             200
>>>>>> RX Mini:        0
>>>>>> RX Jumbo:       100
>>>>>> TX:             511
>>>>>> 
>>>>>> ethtool -c eth0
>>>>>> Coalesce parameters for eth0:
>>>>>> Adaptive RX: off  TX: off
>>>>>> stats-block-usecs: 0
>>>>>> sample-interval: 0
>>>>>> pkt-rate-low: 0
>>>>>> pkt-rate-high: 0
>>>>>> 
>>>>>> rx-usecs: 20
>>>>>> rx-frames: 5
>>>>>> rx-usecs-irq: 0
>>>>>> rx-frames-irq: 5
>>>>>> 
>>>>>> tx-usecs: 72
>>>>>> tx-frames: 53
>>>>>> tx-usecs-irq: 0
>>>>>> tx-frames-irq: 5
>>>>>> 
>>>>>> rx-usecs-low: 0
>>>>>> rx-frame-low: 0
>>>>>> tx-usecs-low: 0
>>>>>> tx-frame-low: 0
>>>>>> 
>>>>>> rx-usecs-high: 0
>>>>>> rx-frame-high: 0
>>>>>> tx-usecs-high: 0
>>>>>> tx-frame-high: 0
>>>>>> 
>>>>>> ethtool -k eth0
>>>>>> Features for eth0:
>>>>>> rx-checksumming: on
>>>>>> tx-checksumming: on
>>>>>>      tx-checksum-ipv4: on
>>>>>>      tx-checksum-ip-generic: off [fixed]
>>>>>>      tx-checksum-ipv6: on
>>>>>>      tx-checksum-fcoe-crc: off [fixed]
>>>>>>      tx-checksum-sctp: off [fixed]
>>>>>> scatter-gather: on
>>>>>>      tx-scatter-gather: on
>>>>>>      tx-scatter-gather-fraglist: off [fixed]
>>>>>> tcp-segmentation-offload: on
>>>>>>      tx-tcp-segmentation: on
>>>>>>      tx-tcp-ecn-segmentation: on
>>>>>>      tx-tcp-mangleid-segmentation: off
>>>>>>      tx-tcp6-segmentation: on
>>>>>> udp-fragmentation-offload: off
>>>>>> generic-segmentation-offload: on
>>>>>> generic-receive-offload: on
>>>>>> large-receive-offload: off [fixed]
>>>>>> rx-vlan-offload: on [fixed]
>>>>>> tx-vlan-offload: on [fixed]
>>>>>> ntuple-filters: off [fixed]
>>>>>> receive-hashing: off [fixed]
>>>>>> highdma: on
>>>>>> rx-vlan-filter: off [fixed]
>>>>>> vlan-challenged: off [fixed]
>>>>>> tx-lockless: off [fixed]
>>>>>> netns-local: off [fixed]
>>>>>> tx-gso-robust: off [fixed]
>>>>>> tx-fcoe-segmentation: off [fixed]
>>>>>> tx-gre-segmentation: off [fixed]
>>>>>> tx-gre-csum-segmentation: off [fixed]
>>>>>> tx-ipxip4-segmentation: off [fixed]
>>>>>> tx-ipxip6-segmentation: off [fixed]
>>>>>> tx-udp_tnl-segmentation: off [fixed]
>>>>>> tx-udp_tnl-csum-segmentation: off [fixed]
>>>>>> tx-gso-partial: off [fixed]
>>>>>> tx-sctp-segmentation: off [fixed]
>>>>>> tx-esp-segmentation: off [fixed]
>>>>>> tx-udp-segmentation: off [fixed]
>>>>>> fcoe-mtu: off [fixed]
>>>>>> tx-nocache-copy: off
>>>>>> loopback: off [fixed]
>>>>>> rx-fcs: off [fixed]
>>>>>> rx-all: off [fixed]
>>>>>> tx-vlan-stag-hw-insert: off [fixed]
>>>>>> rx-vlan-stag-hw-parse: off [fixed]
>>>>>> rx-vlan-stag-filter: off [fixed]
>>>>>> l2-fwd-offload: off [fixed]
>>>>>> hw-tc-offload: off [fixed]
>>>>>> esp-hw-offload: off [fixed]
>>>>>> esp-tx-csum-hw-offload: off [fixed]
>>>>>> rx-udp_tunnel-port-offload: off [fixed]
>>>>>> tls-hw-tx-offload: off [fixed]
>>>>>> tls-hw-rx-offload: off [fixed]
>>>>>> rx-gro-hw: off [fixed]
>>>>>> tls-hw-record: off [fixed]
>>>>>> 
>>>>>> ethtool -n eth0
>>>>>> 4 RX rings available
>>>>>> rxclass: Cannot get RX class rule count: Operation not supported
>>>>>> RX classification rule retrieval failed
>>>>>> 
>>>>>> ethtool -t eth0
>>>>>> The test result is PASS
>>>>>> The test extra info:
>>>>>> nvram test        (online)       0
>>>>>> link test         (online)       0
>>>>>> register test     (offline)      0
>>>>>> memory test       (offline)      0
>>>>>> mac loopback test (offline)      0
>>>>>> phy loopback test (offline)      0
>>>>>> ext loopback test (offline)      0
>>>>>> interrupt test    (offline)      0
>>>>>> 
>>>>>> ethtool -T eth0
>>>>>> Time stamping parameters for eth0:
>>>>>> Capabilities:
>>>>>>      software-transmit     (SOF_TIMESTAMPING_TX_SOFTWARE)
>>>>>>      software-receive      (SOF_TIMESTAMPING_RX_SOFTWARE)
>>>>>>      software-system-clock (SOF_TIMESTAMPING_SOFTWARE)
>>>>>> PTP Hardware Clock: none
>>>>>> Hardware Transmit Timestamp Modes:
>>>>>>      off                   (HWTSTAMP_TX_OFF)
>>>>>>      on                    (HWTSTAMP_TX_ON)
>>>>>> Hardware Receive Filter Modes:
>>>>>>      none                  (HWTSTAMP_FILTER_NONE)
>>>>>>      ptpv1-l4-event        (HWTSTAMP_FILTER_PTP_V1_L4_EVENT)
>>>>>>      ptpv2-l4-event        (HWTSTAMP_FILTER_PTP_V2_L4_EVENT)
>>>>>>      ptpv2-l2-event        (HWTSTAMP_FILTER_PTP_V2_L2_EVENT)
>>>>>> 
>>>>>> ethtool -l eth0
>>>>>> Channel parameters for eth0:
>>>>>> Pre-set maximums:
>>>>>> RX:             4
>>>>>> TX:             1
>>>>>> Other:          0
>>>>>> Combined:       0
>>>>>> Current hardware settings:
>>>>>> RX:             4
>>>>>> TX:             1
>>>>>> Other:          0
>>>>>> Combined:       0
>>>>>> 
>>>>>> ethtool --show-eee eth0
>>>>>> EEE Settings for eth0:
>>>>>>      EEE status: enabled - active
>>>>>>      Tx LPI: 2047 (us)
>>>>>>      Supported EEE link modes:  100baseT/Full
>>>>>>                                 1000baseT/Full
>>>>>>      Advertised EEE link modes:  100baseT/Full
>>>>>>                                  1000baseT/Full
>>>>>>      Link partner advertised EEE link modes:  100baseT/Full
>>>>>>                                               1000baseT/Full
>>>>>> 
>>>>>>> These are all the tg3 changes between 4.19 and 5.0:
>>>>>>> 
>>>>>>> 750afb08ca71 cross-tree: phase out dma_zalloc_coherent()
>>>>>>> cddaf02bcb73 tg3: optionally use eth_platform_get_mac_address() to get
>>>>>>> mac address
>>>>>>> 3c1bcc8614db net: ethernet: Convert phydev advertize and supported
>>>>>>> from u32 to link mode
>>>>>>> 6fe42e228dc2 tg3: extend PTP gettime function to read system clock
>>>>>>> 310fc0513ea9 tg3: Fix fall-through annotations
>>>>>>> 22b7d29926b5 net: ethernet: Add helper to determine if pause
>>>>>>> configuration is supported
>>>>>>> 70814e819c11 net: ethernet: Add helper for set_pauseparam for Asym Pause
>>>>>>> af8d9bb2f2f4 net: ethernet: Add helper for MACs which support asym pause
>>>>>>> 04b7d41d8046 net: ethernet: Fix up drivers masking pause support
>>>>>>> 58056c1e1b0e net: ethernet: Use phy_set_max_speed() to limit advertised speed
>>>>>>> 
>>>>>>> Most of the changes are related to PHY settings.  I suggest that you
>>>>>>> check the link settings, including speed, pause, asym pause, etc
>>>>>>> between the working kernel and the non-working kernel to see if there
>>>>>>> are differences in the settings.
>>>>>> 
>>>>> 
>>>> 
> 


  reply	other threads:[~2019-11-11  8:15 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-01 14:30 Gentoo Linux 5.x - Tigon3 Rudolf Spring
2019-10-01 18:17 ` Michael Chan
2019-10-02 16:35   ` Rudolf Spring
     [not found]     ` <CAMet4B7vi6yYu2HZd1Pj7rhtxme8FmT4wbXTjQOnQEqJp0Z_3w@mail.gmail.com>
2019-10-04 13:53       ` Rudolf Spring
2019-10-17 17:33         ` Rudolf Spring
2019-10-18  6:51           ` Siva Reddy Kallam
2019-10-31 11:49             ` Siva Reddy Kallam
2019-10-31 19:42               ` Rudolf Spring
2019-11-11  8:15                 ` Rudolf Spring [this message]
2019-11-11 13:44                   ` Rudolf Spring
2019-11-13 12:52                     ` Rudolf Spring
2019-11-13 13:58                       ` Rudolf Spring
2019-12-04 10:03                         ` Siva Reddy Kallam

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3FA0ADFD-6F62-43E4-9119-64F7AC1BCD27@gmail.com \
    --to=rudolf.spring@gmail.com \
    --cc=michael.chan@broadcom.com \
    --cc=netdev@vger.kernel.org \
    --cc=siva.kallam@broadcom.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.