RE: [PATCH] s2io: add PCI error recovery support

[Ramkrishna Vepa]

Linas,

We reviewed this patch and it looks good.

Thanks,
Ram
> > -----Original Message-----
> > From: netdev-owner@xxxxxxxxxxxxxxx
[mailto:netdev-owner@xxxxxxxxxxxxxxx]
> > On Behalf Of Linas Vepstas
> > Sent: Tuesday, October 24, 2006 2:55 PM
> > To: Raghavendra Koushik; Ananda Raju; Wen Xiong
> > Cc: linux-kernel@xxxxxxxxxxxxxxx;
linux-pci@xxxxxxxxxxxxxxxxxxxxxxxx;
> > netdev@xxxxxxxxxxxxxxx; Jeff Garzik; Andrew Morton
> > Subject: [PATCH] s2io: add PCI error recovery support
> >
> >
> > Koushik, Raju,
> > Please review, comment, and if you find this acceptable,
> > please forward upstream.
> >
> > --linas
> >
> > This patch adds PCI error recovery support to the
> > s2io 10-Gigabit ethernet device driver.
> >
> > Tested, seems to work well.
> >
> > Signed-off-by: Linas Vepstas <linas@xxxxxxxxxxxxxx>
> > Cc: Raghavendra Koushik <raghavendra.koushik@xxxxxxxxxxxx>
> > Cc: Ananda Raju <Ananda.Raju@xxxxxxxxxxxx>
> > Cc: Wen Xiong <wenxiong@xxxxxxxxxx>
> >
> > ----
> >  drivers/net/s2io.c |   77
> > +++++++++++++++++++++++++++++++++++++++++++++++++++++
> >  drivers/net/s2io.h |    5 +++
> >  2 files changed, 82 insertions(+)
> >
> > Index: linux-2.6.19-rc1-git11/drivers/net/s2io.c
> > ===================================================================
> > --- linux-2.6.19-rc1-git11.orig/drivers/net/s2io.c	2006-10-20
> > 12:24:17.000000000 -0500
> > +++ linux-2.6.19-rc1-git11/drivers/net/s2io.c	2006-10-24
> > 16:19:49.000000000 -0500
> > @@ -434,11 +434,18 @@ static struct pci_device_id s2io_tbl[] _
> >
> >  MODULE_DEVICE_TABLE(pci, s2io_tbl);
> >
> > +static struct pci_error_handlers s2io_err_handler = {
> > +	.error_detected = s2io_io_error_detected,
> > +	.slot_reset = s2io_io_slot_reset,
> > +	.resume = s2io_io_resume,
> > +};
> > +
> >  static struct pci_driver s2io_driver = {
> >        .name = "S2IO",
> >        .id_table = s2io_tbl,
> >        .probe = s2io_init_nic,
> >        .remove = __devexit_p(s2io_rem_nic),
> > +      .err_handler = &s2io_err_handler,
> >  };
> >
> >  /* A simplifier macro used both by init and free shared_mem Fns().
*/
> > @@ -7564,3 +7571,73 @@ static void lro_append_pkt(nic_t *sp, lr
> >  	sp->mac_control.stats_info->sw_stat.clubbed_frms_cnt++;
> >  	return;
> >  }
> > +
> > +/**
> > + * s2io_io_error_detected - called when PCI error is detected
> > + * @pdev: Pointer to PCI device
> > + * @state: The current pci conneection state
> > + *
> > + * This function is called after a PCI bus error affecting
> > + * this device has been detected.
> > + */
> > +static pci_ers_result_t s2io_io_error_detected(struct pci_dev
*pdev,
> > pci_channel_state_t state)
> > +{
> > +	struct net_device *netdev = pci_get_drvdata(pdev);
> > +	nic_t *sp = netdev->priv;
> > +
> > +	netif_device_detach(netdev);
> > +
> > +	if (netif_running(netdev)) {
> > +		/* Reset card */
> > +		s2io_card_down(sp);
> > +		sp->device_close_flag = TRUE;	/* Device is shut down.
*/
> > +	}
> > +	pci_disable_device(pdev);
> > +
> > +	return PCI_ERS_RESULT_NEED_RESET;
> > +}
> > +
> > +/**
> > + * s2io_io_slot_reset - called after the pci bus has been reset.
> > + * @pdev: Pointer to PCI device
> > + *
> > + * Restart the card from scratch, as if from a cold-boot.
> > + */
> > +static pci_ers_result_t s2io_io_slot_reset(struct pci_dev *pdev)
> > +{
> > +	struct net_device *netdev = pci_get_drvdata(pdev);
> > +	nic_t *sp = netdev->priv;
> > +
> > +	if (pci_enable_device(pdev)) {
> > +		printk(KERN_ERR "s2io: Cannot re-enable PCI device after
> > reset.\n");
> > +		return PCI_ERS_RESULT_DISCONNECT;
> > +	}
> > +
> > +	pci_set_master(pdev);
> > +	s2io_reset(sp);
> > +
> > +	return PCI_ERS_RESULT_RECOVERED;
> > +}
> > +
> > +/**
> > + * s2io_io_resume - called when traffic can start flowing again.
> > + * @pdev: Pointer to PCI device
> > + *
> > + * This callback is called when the error recovery driver tells us
that
> > + * its OK to resume normal operation.
> > + */
> > +static void s2io_io_resume(struct pci_dev *pdev)
> > +{
> > +	struct net_device *netdev = pci_get_drvdata(pdev);
> > +	nic_t *sp = netdev->priv;
> > +
> > +	if (netif_running(netdev)) {
> > +		if (s2io_card_up(sp)) {
> > +			printk(KERN_ERR "s2io: can't bring device back
up after
> > reset\n");
> > +			return;
> > +		}
> > +	}
> > +
> > +	netif_device_attach(netdev);
> > +	netif_wake_queue(netdev);
> > +}
> > Index: linux-2.6.19-rc1-git11/drivers/net/s2io.h
> > ===================================================================
> > --- linux-2.6.19-rc1-git11.orig/drivers/net/s2io.h	2006-10-20
> > 12:24:17.000000000 -0500
> > +++ linux-2.6.19-rc1-git11/drivers/net/s2io.h	2006-10-20
> > 12:41:39.000000000 -0500
> > @@ -1013,6 +1013,11 @@ static void queue_rx_frame(struct sk_buf
> >  static void update_L3L4_header(nic_t *sp, lro_t *lro);
> >  static void lro_append_pkt(nic_t *sp, lro_t *lro, struct sk_buff
*skb,
> > u32 tcp_len);
> >
> > +static pci_ers_result_t s2io_io_error_detected(struct pci_dev
*pdev,
> > +
pci_channel_state_t
> > state);
> > +static pci_ers_result_t s2io_io_slot_reset(struct pci_dev *pdev);
> > +static void s2io_io_resume(struct pci_dev *pdev);
> > +
> >  #define s2io_tcp_mss(skb) skb_shinfo(skb)->gso_size
> >  #define s2io_udp_mss(skb) skb_shinfo(skb)->gso_size
> >  #define s2io_offload_type(skb) skb_shinfo(skb)->gso_type
> > -
> > To unsubscribe from this list: send the line "unsubscribe netdev" in
> > the body of a message to majordomo@xxxxxxxxxxxxxxx
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/