Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965209AbWIVWGS (ORCPT ); Fri, 22 Sep 2006 18:06:18 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S965213AbWIVWGS (ORCPT ); Fri, 22 Sep 2006 18:06:18 -0400 Received: from nf-out-0910.google.com ([64.233.182.185]:20140 "EHLO nf-out-0910.google.com") by vger.kernel.org with ESMTP id S965209AbWIVWGR (ORCPT ); Fri, 22 Sep 2006 18:06:17 -0400 DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:date:from:to:cc:subject:message-id:mime-version:content-type:content-disposition:in-reply-to:user-agent; b=RuaKko6iBaNQoeUncxCWCzqw8warmxYtnqpgwPYCZSUuFJIfXXxh80ae8WV08Fscuf5zJG0pJDDx/g61GnRS6W1yOgq9D82Qws9cE3asA7WXB7wXJFqrKWk1YhsSyyybT0+VWXe3xBnpk2JufLcOy9J/TQ9yFNKQJKDxtMdWOeg= Date: Sat, 23 Sep 2006 00:06:29 +0200 From: Luca To: Linas Vepstas Cc: linux-scsi@vger.kernel.org, linux-pci@atrey.karlin.mff.cuni.cz, linuxppc-dev@ozlabs.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH]: PCI Error Recovery: Symbios SCSI device driver Message-ID: <20060922220629.GA4600@dreamland.darkstar.lan> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20060921231314.GW29167@austin.ibm.com> User-Agent: Mutt/1.5.13 (2006-08-11) Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2232 Lines: 70 Hi Linas, Linas Vepstas ha scritto: > Index: linux-2.6.18-rc7-git1/drivers/scsi/sym53c8xx_2/sym_glue.c > =================================================================== > --- linux-2.6.18-rc7-git1.orig/drivers/scsi/sym53c8xx_2/sym_glue.c 2006-09-21 17:32:54.000000000 -0500 > +++ linux-2.6.18-rc7-git1/drivers/scsi/sym53c8xx_2/sym_glue.c 2006-09-21 17:35:37.000000000 -0500 > +/** > + * sym2_io_slot_reset() -- called when the pci bus has been reset. > + * @pdev: pointer to PCI device > + * > + * Restart the card from scratch. > + */ > +static pci_ers_result_t sym2_io_slot_reset (struct pci_dev *pdev) > +{ > + struct sym_hcb *np = pci_get_drvdata(pdev); > + > + printk (KERN_INFO "%s: recovering from a PCI slot reset\n", Space after function name? You put in other places too, it's not consistent with the rest of the patch. > + sym_name(np)); > + > + if (pci_enable_device(pdev)) > + printk (KERN_ERR "%s: device setup failed most egregiously\n", > + sym_name(np)); Is the failure of pci_enable_device ignored on purpose? (plus extra space) > + > + pci_set_master(pdev); > + enable_irq (pdev->irq); Spurious space. > + > + /* Perform host reset only on one instance of the card */ > + if (0 == PCI_FUNC (pdev->devfn)) { Ditto. > + if (sym_reset_scsi_bus(np, 0)) { > + printk(KERN_ERR "%s: Unable to reset scsi host controller\n", > + sym_name(np)); > + return PCI_ERS_RESULT_DISCONNECT; > + } > + sym_start_up (np, 1); Ditto. > + } > + > + return PCI_ERS_RESULT_RECOVERED; > +} Luca -- "L'abilita` politica e` l'abilita` di prevedere quello che accadra` domani, la prossima settimana, il prossimo mese e l'anno prossimo. E di essere cosi` abili, piu` tardi, da spiegare perche' non e` accaduto." - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/