On Fri, 2013-05-03 at 13:00 -0500, Linas Vepstas wrote: > 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 powerpc uses the similar method. See function eeh_report_reset. We worked out the patch after checking powerpc codes. Joseph Liu helped test the patch and the patch does work well. > . > > -- 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
