* Re: net/tg3: Fix kernel crash
[not found] <20130731204019.059F66609B7@gitolite.kernel.org>
@ 2013-08-06 14:46 ` Dave Jones
0 siblings, 0 replies; only message in thread
From: Dave Jones @ 2013-08-06 14:46 UTC (permalink / raw)
To: netdev; +Cc: Wei Yang, Gavin Shan, Nithin Nayak Sujir, David S. Miller
On Wed, Jul 31, 2013 at 08:40:18PM +0000, Linux Kernel wrote:
> Gitweb: http://git.kernel.org/linus/;a=commit;h=d8af4dfd84329015e18ea3d83a84c46e92f21020
> Commit: d8af4dfd84329015e18ea3d83a84c46e92f21020
> Parent: d738ce8fdc05ebf5b1475f8ae26d908c8c50970b
> Author: Gavin Shan <shangw@linux•vnet.ibm.com>
> AuthorDate: Wed Jul 24 17:25:08 2013 +0800
>
> net/tg3: Fix kernel crash
>
> While EEH error happens, we might not have network device instance
> (struct net_device) yet. So we can't access the instance safely and
> check its link state, which causes kernel crash. The patch fixes it.
> Reported-by: Wei Yang <weiyang@linux•vnet.ibm.com>
> Signed-off-by: Gavin Shan <shangw@linux•vnet.ibm.com>
> Acked-by: Nithin Nayak Sujir <nsujir@broadcom•com>
> Signed-off-by: David S. Miller <davem@davemloft•net>
>
> diff --git a/drivers/net/ethernet/broadcom/tg3.c b/drivers/net/ethernet/broadcom/tg3.c
> index d964f30..aee1b9a 100644
> --- a/drivers/net/ethernet/broadcom/tg3.c
> +++ b/drivers/net/ethernet/broadcom/tg3.c
> @@ -17773,7 +17773,8 @@ static pci_ers_result_t tg3_io_error_detected(struct pci_dev *pdev,
>
> rtnl_lock();
>
> - if (!netif_running(netdev))
> + /* We probably don't have netdev yet */
> + if (!netdev || !netif_running(netdev))
> goto done;
This looks incomplete. 'done' does this..
done:
if (state == pci_channel_io_perm_failure) {
tg3_napi_enable(tp);
dev_close(netdev);
If netdev is null, we'll deref it in dev_close.
Dave
^ permalink raw reply [flat|nested] only message in thread