• Vladimir Oltean's avatar
    net: enetc: initialize RFS/RSS memories for unused ports too · 3222b5b6
    Vladimir Oltean authored
    Michael reports that since linux-next-20210211, the AER messages for ECC
    errors have started reappearing, and this time they can be reliably
    reproduced with the first ping on one of his LS1028A boards.
    
    $ ping 1[   33.258069] pcieport 0000:00:1f.0: AER: Multiple Corrected error received: 0000:00:00.0
    72.16.0.1
    PING [   33.267050] pcieport 0000:00:1f.0: AER: can't find device of ID0000
    172.16.0.1 (172.16.0.1): 56 data bytes
    64 bytes from 172.16.0.1: seq=0 ttl=64 time=17.124 ms
    64 bytes from 172.16.0.1: seq=1 ttl=64 time=0.273 ms
    
    $ devmem 0x1f8010e10 32
    0xC0000006
    
    It isn't clear why this is necessary, but it seems that for the errors
    to go away, we must clear the entire RFS and RSS memory, not just for
    the ports in use.
    
    Sadly the code is structured in such a way that we can't have unified
    logic for the used and unused ports. For the minimal initialization of
    an unused port, we need just to enable and ioremap the PF memory space,
    and a control buffer descriptor ring. Unused ports must then free the
    CBDR because the driver will exit, but used ports can not pick up from
    where that code path left, since the CBDR API does not reinitialize a
    ring when setting it up, so its producer and consumer indices are out of
    sync between the software and hardware state. So a separate
    enetc_init_unused_port function was created, and it gets called right
    after the PF memory space is enabled.
    
    Fixes: 07bf34a5 ("net: enetc: initialize the RFS and RSS memories")
    Reported-by: default avatarMichael Walle <michael@walle.cc>
    Cc: Jesse Brandeburg <jesse.brandeburg@intel.com>
    Signed-off-by: default avatarVladimir Oltean <vladimir.oltean@nxp.com>
    Tested-by: default avatarMichael Walle <michael@walle.cc>
    Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
    3222b5b6
enetc.c 44.7 KB