Re: Subject : [ PATCH ] pci-reset-error_state-to-pci_channel_io_normal-at-report_slot_reset

[Linas Vepstas]

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-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/