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  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

Powered by blists - more mailing lists