From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from e23smtp07.au.ibm.com (e23smtp07.au.ibm.com [202.81.31.140]) (using TLSv1 with cipher CAMELLIA256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 485F11A01ED for ; Fri, 28 Aug 2015 11:10:08 +1000 (AEST) Received: from /spool/local by e23smtp07.au.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Fri, 28 Aug 2015 11:10:06 +1000 Received: from d23relay08.au.ibm.com (d23relay08.au.ibm.com [9.185.71.33]) by d23dlp01.au.ibm.com (Postfix) with ESMTP id 965482CE8052 for ; Fri, 28 Aug 2015 11:10:03 +1000 (EST) Received: from d23av04.au.ibm.com (d23av04.au.ibm.com [9.190.235.139]) by d23relay08.au.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id t7S19mGW52428892 for ; Fri, 28 Aug 2015 11:09:56 +1000 Received: from d23av04.au.ibm.com (localhost [127.0.0.1]) by d23av04.au.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id t7S19USj005179 for ; Fri, 28 Aug 2015 11:09:31 +1000 Date: Fri, 28 Aug 2015 11:09:05 +1000 From: Gavin Shan To: Michael Ellerman Cc: Gavin Shan , linuxppc-dev@lists.ozlabs.org Subject: Re: [1/2] powerpc/pseries: Fix corrupted pdn list Message-ID: <20150828010905.GA21981@gwshan> Reply-To: Gavin Shan References: <1440648757-23359-1-git-send-email-gwshan@linux.vnet.ibm.com> <20150828005535.5B29814018C@ozlabs.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20150828005535.5B29814018C@ozlabs.org> List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Fri, Aug 28, 2015 at 10:55:35AM +1000, Michael Ellerman wrote: >On Thu, 2015-27-08 at 04:12:36 UTC, Gavin Shan wrote: >> Commit cca87d30 ("powerpc/pci: Refactor pci_dn") introduced pdn >> list for SRIOV VFs. It means the pdn is be put into the child list >> of its parent pdn when the pdn is created. When doing PCI hot >> unplugging on pSeries, the PCI device node as well as its pdn are >> released through procfs entry "powerpc/ofdt". Some one else grabs >> the memory chunk of the pdn and update it accordingly. At the same >> time, the pdn is still tracked in the child list of parent pdn. It >> leads to corrupted child list in the parent pdn. >> >> This fixes above issue by removing the pdn from the child list of >> its parent pdn when the device node is detached from the system. >> Note the pdn is free'd when the device node is released if the >> device node is dynamic one. Otherwise, the device node as well >> as the pdn won't be released. >> >> Fixes: cca87d30 ("powerpc/pci: Refactor pci_dn") > >That went into 4.1, so this should presumably go to stable? > Yeah, This should be put into 4.1 stable. Thanks, Gavin >cheers >