From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from e23smtp07.au.ibm.com ([202.81.31.140]:54993 "EHLO e23smtp07.au.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751904AbbFCDl0 (ORCPT ); Tue, 2 Jun 2015 23:41:26 -0400 Received: from /spool/local by e23smtp07.au.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Wed, 3 Jun 2015 13:41:24 +1000 Received: from d23relay07.au.ibm.com (d23relay07.au.ibm.com [9.190.26.37]) by d23dlp03.au.ibm.com (Postfix) with ESMTP id 7E94F3578052 for ; Wed, 3 Jun 2015 13:41:21 +1000 (EST) Received: from d23av01.au.ibm.com (d23av01.au.ibm.com [9.190.234.96]) by d23relay07.au.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id t533dwYo20250814 for ; Wed, 3 Jun 2015 13:40:06 +1000 Received: from d23av01.au.ibm.com (localhost [127.0.0.1]) by d23av01.au.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id t533dXZA000638 for ; Wed, 3 Jun 2015 13:39:33 +1000 Date: Wed, 3 Jun 2015 11:39:15 +0800 From: Wei Yang To: Bjorn Helgaas Cc: Wei Yang , linux-pci@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, gwshan@linux.vnet.ibm.com Subject: Re: [PATCH V7 06/10] powerpc/eeh: Create PE for VFs Message-ID: <20150603033915.GB22584@richard> Reply-To: Wei Yang References: <1431999312-10517-1-git-send-email-weiyang@linux.vnet.ibm.com> <1432032612-21701-1-git-send-email-weiyang@linux.vnet.ibm.com> <1432032612-21701-7-git-send-email-weiyang@linux.vnet.ibm.com> <20150601234958.GG3631@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20150601234958.GG3631@google.com> Sender: linux-pci-owner@vger.kernel.org List-ID: On Mon, Jun 01, 2015 at 06:49:58PM -0500, Bjorn Helgaas wrote: >On Tue, May 19, 2015 at 06:50:08PM +0800, Wei Yang wrote: >> Current EEH recovery code works with the assumption: the PE has primary >> bus. Unfortunately, that's not true to VF PEs, which generally contains > >"Primary bus" normally means the bus on the upstream side of a PCI bridge. >But a PE is not a bridge, so I don't know what it means here. > Before VF PE introduced, a PE is a "Bus PE" which contains a pci bus. Yes, the "primary bus" may be a little confusing, since this may refer to the upstream side of a PCI bridge. I think the log here tries to emphasize the Bus PE contains a whole pci bus. >s/not true to VF PEs/not true for VF PEs/ Thanks, changed > >> one or multiple VFs (for VF group case). The patch creates PEs for VFs >> at PCI final fixup time. Those PEs for VFs are indentified with newly > >s/indentified/identified/ Thanks, changed > >> introduced flag EEH_PE_VF so that we handle them differently during >> EEH recovery. >_______________________________________________ >Linuxppc-dev mailing list >Linuxppc-dev@lists.ozlabs.org >https://lists.ozlabs.org/listinfo/linuxppc-dev -- Richard Yang Help you, Help me