Hi Michael,
On Mon, Dec 09, 2019 at 11:29:03AM -0500, Michael S. Tsirkin wrote:
> This allows incrementing the correct timeout statistic without any mess.
> Down the road, devices can learn to reset just the specific queue.
>
> The patch was generated with the following script:
>
<snip>
>
> where the list of files and functions is simply from:
>
> git grep ndo_tx_timeout, with manual addition of headers
> in the rare cases where the function is from a header,
> then manually changing the few places which actually
> call ndo_tx_timeout.
>
> Signed-off-by: Michael S. Tsirkin <mst@xxxxxxxxxx>
> Acked-by: Heiner Kallweit <hkallweit1@xxxxxxxxx>
> Acked-by: Jakub Kicinski <jakub.kicinski@xxxxxxxxxxxxx>
> Acked-by: Shannon Nelson <snelson@xxxxxxxxxxx>
> Reviewed-by: Martin Habets <mhabets@xxxxxxxxxxxxxx>
>
> changes from v8:
> fix up a missing direct call to timeout
> rebased on net-next
> changes from v7:
> fixup leftovers from v3 change
> changes from v6:
> fix typo in rtl driver
> changes from v5:
> add missing files (allow any net device argument name)
> changes from v4:
> add a missing driver header
> changes from v3:
> change queue # to unsigned
> Changes from v2:
> added headers
> Changes from v1:
> Fix errors found by kbuild:
> generalize the pattern a bit, to pick up
> a couple of instances missed by the previous
> version.
> ---
<snip>
> diff --git a/drivers/net/ethernet/freescale/dpaa/dpaa_eth.c b/drivers/net/ethernet/freescale/dpaa/dpaa_eth.c
> index 6a9d12dad5d9..ad0ecebb1b34 100644
> --- a/drivers/net/ethernet/freescale/dpaa/dpaa_eth.c
> +++ b/drivers/net/ethernet/freescale/dpaa/dpaa_eth.c
> @@ -288,7 +288,7 @@ static int dpaa_stop(struct net_device *net_dev)
> return err;
> }
>
> -static void dpaa_tx_timeout(struct net_device *net_dev)
> +static void dpaa_tx_timeout(struct net_device *net_dev, int txqueue)
This needs to be unsigned int, otherwise there is a build error:
../drivers/net/ethernet/freescale/dpaa/dpaa_eth.c:2622:20: error: incompatible pointer types initializing 'void (*)(struct net_device *, unsigned int)' with an expression of type 'void (struct net_device *, int)' [-Werror,-Wincompatible-pointer-types]
.ndo_tx_timeout = dpaa_tx_timeout,
^~~~~~~~~~~~~~~
1 error generated.
Cheers,
Nathan
_______________________________________________
Virtualization mailing list
Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linuxfoundation.org/mailman/listinfo/virtualization
