linux-kernel - Re: [PATCH] pciehp: Checking pci conf reading to new added device instead of sleep 1s

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]

Message-ID: <CAE9FiQW=2toydSnx7H7d6YnQKkZCpMQK8GcarscbyoNWp9VTVg@mail.gmail.com>
Date:	Thu, 15 Dec 2011 11:25:37 -0800
From:	Yinghai Lu <yinghai@...nel.org>
To:	Matthew Wilcox <matthew@....cx>
Cc:	Jesse Barnes <jbarnes@...tuousgeek.org>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	Kenji Kaneshige <kaneshige.kenji@...fujitsu.com>,
	linux-pci@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] pciehp: Checking pci conf reading to new added device
 instead of sleep 1s

On Tue, Dec 13, 2011 at 8:31 PM, Matthew Wilcox <matthew@....cx> wrote:
> This code looks similar to (but not identical to) the code in
> pci_scan_device().  Please make that code a common function that can be
> called from all PCI hotplug drivers, not just this one.

it seems that other hotplug like acpiphp does not check link status.

Please if you are happy with attached version.
it separates some code from pci_scan_device()

Thanks

Yinghai

View attachment "pciehp_debug_x_3_1_access_pci_conf.patch" of type "text/x-patch" (5853 bytes)