Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1252361 > unrolled thread
| Started by | "Rustad, Mark D" <mark.d.rustad@intel.com> |
|---|---|
| First post | 2015-10-21 01:20 +0200 |
| Last post | 2015-10-21 01:20 +0200 |
| Articles | 1 — 1 participant |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [Intel-wired-lan] [PATCH] fm10k:Fix error handling in the function fm10k_setup_tc "Rustad, Mark D" <mark.d.rustad@intel.com> - 2015-10-21 01:20 +0200
| From | "Rustad, Mark D" <mark.d.rustad@intel.com> |
|---|---|
| Date | 2015-10-21 01:20 +0200 |
| Subject | Re: [Intel-wired-lan] [PATCH] fm10k:Fix error handling in the function fm10k_setup_tc |
| Message-ID | <qlOmd-1Cz-3@gated-at.bofh.it> |
[Multipart message — attachments visible in raw view] — view raw
Nicholas Krause <xerofoify@gmail.com> wrote:
> This fixes error handling in the function fm10k_setup_tc to properly
> check if the call to the function fm10k_open has failed by returning
> a error and if so return immediately to the caller of the function
> fm10k_setup_tc to properly signal this non recoverable failure.
>
> Signed-off-by: Nicholas Krause <xerofoify@gmail.com>
> ---
> drivers/net/ethernet/intel/fm10k/fm10k_netdev.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/intel/fm10k/fm10k_netdev.c b/drivers/net/ethernet/intel/fm10k/fm10k_netdev.c
> index 99228bf..5a4e702 100644
> --- a/drivers/net/ethernet/intel/fm10k/fm10k_netdev.c
> +++ b/drivers/net/ethernet/intel/fm10k/fm10k_netdev.c
> @@ -1146,6 +1146,7 @@ static struct rtnl_link_stats64 *fm10k_get_stats64(struct net_device *netdev,
> int fm10k_setup_tc(struct net_device *dev, u8 tc)
> {
> struct fm10k_intfc *interface = netdev_priv(dev);
> + int err;
>
> /* Currently only the PF supports priority classes */
> if (tc && (interface->hw.mac.type != fm10k_mac_pf))
> @@ -1175,7 +1176,9 @@ int fm10k_setup_tc(struct net_device *dev, u8 tc)
> fm10k_mbx_request_irq(interface);
>
> if (netif_running(dev))
> - fm10k_open(dev);
> + err = fm10k_open(dev);
> + if (err)
> + return err;
>
> /* flag to indicate SWPRI has yet to be updated */
> interface->flags |= FM10K_FLAG_SWPRI_CONFIG;
NAK. This will reference an uninitialized err variable when netif_running returns false. The compiler should complain about this, suggesting that it wasn't compiled.
--
Mark Rustad, Networking Division, Intel Corporation
Back to top | Article view | linux.kernel
csiph-web