Greg, I've been receiving (and reading!) these messages; I replied that I am not maintaining a tree, and have no way of testing these patches (no access to hardware.) I believe it unlikely that my situation will change, and so I should probably be removed from the maintainers file. No acked-by or anything; the patches are not "obviously correct" by visual inspection; they may be right, but would require deeper thinking about what is actually happening than I'm capable of at this time; I'm a bit rusty with this code base, and might miss something important. -- Linas p.s. you didn't see my earlier reply because I forgot to hit 'plain text reply' On 2 May 2013 22:13, Yanmin Zhang <yanmin_zhang@xxxxxxxxxxxxxxx> wrote: > > On Thu, 2013-05-02 at 19:00 -0700, Greg Kroah-Hartman wrote: > > On Fri, May 03, 2013 at 08:33:00AM +0800, Yanmin Zhang wrote: > > > On Wed, 2013-05-01 at 20:20 -0500, Linas Vepstas wrote: > > > > Hi, > > > > > > > > On 1 May 2013 19:30, Yanmin Zhang <yanmin_zhang@xxxxxxxxxxxxxxx> > > > > wrote: > > > > On Fri, 2013-04-26 at 06:28 +0000, Zhang, LongX wrote: > > > > > From: Zhang Long <longx.zhang@xxxxxxxxx> > > > > > > > > > > Specific pci device drivers might have many functions to > > > > call > > > > > pci_channel_offline to check device states. When slot_reset > > > > happens, > > > > > drivers' slot_reset callback might call such functions and > > > > eventually > > > > > abort the reset. > > > > > > > > > > The patch resets pdev->error_state to pci_channel_io_normal > > > > at > > > > > the begining of report_slot_reset. > > > > > > > > > > Thank Liu Joseph for pointing it out. > > > > > > > > Linas, Bjorn, > > > > > > > > Would you like to merge the patch to your testing tree? > > > > The patch resolves one issue pointed out by Joseph. > > > > > > > > > > > > I'm not maintaining a tree at this time, and am not able to test. > > > > Sorry. > > > Thanks Linas. > > > > > > Greg, > > > > > > Would you like to merge it into your testing tree? Joseph Liu tested > > > the patch and it does work. > > > > You do know about the scripts/get_maintainer.pl script, right? > > > > Hint, try it out :) > Greg, > > Thanks. We did send to the right people, Linas and Bjorn. > > scripts/get_maintainer.pl 0001-pci-reset-error_state-to-pci_channel_io_normal-at-re.patch > Bjorn Helgaas <bhelgaas@xxxxxxxxxx> (supporter:PCI SUBSYSTEM,commit_signer:7/8=88%,commit_signer:10/11=91%) > Linas Vepstas <linasvepstas@xxxxxxxxx> (commit_signer:2/8=25%) > Yijing Wang <wangyijing@xxxxxxxxxx> (commit_signer:2/8=25%,commit_signer:2/11=18%) > Jiang Liu <jiang.liu@xxxxxxxxxx> (commit_signer:2/8=25%,commit_signer:2/11=18%) > Stephen Hemminger <shemminger@xxxxxxxxxx> (commit_signer:1/8=12%) > "Rafael J. Wysocki" <rafael.j.wysocki@xxxxxxxxx> (commit_signer:6/11=55%) > Huang Ying <ying.huang@xxxxxxxxx> (commit_signer:5/11=45%) > linux-pci@xxxxxxxxxxxxxxx (open list:PCI SUBSYSTEM) > linux-kernel@xxxxxxxxxxxxxxx (open list) > > > I remember Jesse was maintaining PCI subsystem and he responded quickly. > > Yanmin > > -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
