From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934164Ab3ECSBK (ORCPT ); Fri, 3 May 2013 14:01:10 -0400 Received: from mail-wg0-f41.google.com ([74.125.82.41]:53779 "EHLO mail-wg0-f41.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934019Ab3ECSBI (ORCPT ); Fri, 3 May 2013 14:01:08 -0400 MIME-Version: 1.0 Reply-To: linasvepstas@gmail.com In-Reply-To: <1367550808.2093.14.camel@ymzhang.sh.intel.com> References: <1367454625.2093.6.camel@ymzhang.sh.intel.com> <1367541180.2093.10.camel@ymzhang.sh.intel.com> <20130503020039.GA29087@kroah.com> <1367550808.2093.14.camel@ymzhang.sh.intel.com> From: Linas Vepstas Date: Fri, 3 May 2013 13:00:46 -0500 Message-ID: Subject: Re: Subject : [ PATCH ] pci-reset-error_state-to-pci_channel_io_normal-at-report_slot_reset To: yanmin_zhang Cc: Greg Kroah-Hartman , "Zhang, LongX" , "linux-pci@vger.kernel.org" , "bhelgaas@google.com" , "linux-kernel@vger.kernel.org" , "Joseph.Liu@Emulex.Com" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 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 > > > > wrote: > > > > On Fri, 2013-04-26 at 06:28 +0000, Zhang, LongX wrote: > > > > > From: Zhang Long > > > > > > > > > > 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 (supporter:PCI SUBSYSTEM,commit_signer:7/8=88%,commit_signer:10/11=91%) > Linas Vepstas (commit_signer:2/8=25%) > Yijing Wang (commit_signer:2/8=25%,commit_signer:2/11=18%) > Jiang Liu (commit_signer:2/8=25%,commit_signer:2/11=18%) > Stephen Hemminger (commit_signer:1/8=12%) > "Rafael J. Wysocki" (commit_signer:6/11=55%) > Huang Ying (commit_signer:5/11=45%) > linux-pci@vger.kernel.org (open list:PCI SUBSYSTEM) > linux-kernel@vger.kernel.org (open list) > > > I remember Jesse was maintaining PCI subsystem and he responded quickly. > > Yanmin > >