netdev - Re: [PATCH 1/2] s2io: add PCI error recovery support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Mon, 21 May 2007 12:07:57 -0500
From:	linas@...tin.ibm.com (Linas Vepstas)
To:	Sivakumar Subramani <Sivakumar.Subramani@...erion.com>
Cc:	Jeff Garzik <jgarzik@...ox.com>,
	linux-pci@...ey.karlin.mff.cuni.cz, netdev@...r.kernel.org,
	James.Bottomley@...elEye.com, linux-kernel@...r.kernel.org,
	Ramkrishna Vepa <Ramkrishna.Vepa@...erion.com>,
	Rastapur Santosh <santosh.rastapur@...erion.com>,
	Leonid Grossman <Leonid.Grossman@...erion.com>,
	Sreenivasa Honnur <Sreenivasa.Honnur@...erion.com>,
	Sriram Rapuru <Sriram.Rapuru@...erion.com>
Subject: Re: [PATCH 1/2] s2io: add PCI error recovery support

On Fri, May 18, 2007 at 03:06:53AM -0400, Sivakumar Subramani wrote:
> Hi,
> 
> Fix looks good. I have couple of comments,
> 
> 1) Return from s2io_updt_stats function if the PCI bus is offline
> (pci_channel_offline). 
>          if (pci_channel_offline(pdev))
>                 return;

OK, missed that one.

> 2) No Need to call netif_wake_queue() in s2io_io_resume as
> netif_device_attach() will take care of calling it.

OK.

> 3) We need to return from s2io_msix_ring_handle(),
> s2io_msix_fifo_handle(), s2io_msi_handle() if the PCI channel is
> offline. Some thing similary to the below condition that is added in the
> s2io_isr()

I was thinking that these wouldn't be needed, as MSI should stop
flowing along with everything else. However, there is a small race
window where the MSI was delivered on the PCI bus, then the error was
detected, and then the kernel delivers the MSI to device driver.
The if statements should close that gap.

I'll send an updated patch shortly.

--linas
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html