From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754112AbcLNBBg (ORCPT ); Tue, 13 Dec 2016 20:01:36 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:32794 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754280AbcLNBBb (ORCPT ); Tue, 13 Dec 2016 20:01:31 -0500 Date: Tue, 13 Dec 2016 17:01:40 -0800 From: Greg Kroah-Hartman To: Andrew Donnellan Cc: linux-kernel@vger.kernel.org, stable@vger.kernel.org, Pradipta Ghosh , Russell Currey , Michael Ellerman Subject: Re: [PATCH 4.4 01/16] powerpc/eeh: Fix deadlock when PE frozen state cant be cleared Message-ID: <20161214010140.GC1141@kroah.com> References: <20161213171518.862135257@linuxfoundation.org> <20161213171519.044222027@linuxfoundation.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.7.2 (2016-11-26) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Dec 14, 2016 at 10:59:32AM +1100, Andrew Donnellan wrote: > On 14/12/16 04:15, Greg Kroah-Hartman wrote: > > 4.4-stable review patch. If anyone has any objections, please let me know. > > > > ------------------ > > > > From: Andrew Donnellan > > > > commit 409bf7f8a02ef88db5a0f2cdcf9489914f4b8508 upstream. > > > > In eeh_reset_device(), we take the pci_rescan_remove_lock immediately after > > after we call eeh_reset_pe() to reset the PCI controller. We then call > > eeh_clear_pe_frozen_state(), which can return an error. In this case, we > > bail out of eeh_reset_device() without calling pci_unlock_rescan_remove(). > > > > Add a call to pci_unlock_rescan_remove() in the eeh_clear_pe_frozen_state() > > error path so that we don't cause a deadlock later on. > > > > Reported-by: Pradipta Ghosh > > Fixes: 78954700631f ("powerpc/eeh: Avoid I/O access during PE reset") > > Signed-off-by: Andrew Donnellan > > Acked-by: Russell Currey > > Signed-off-by: Michael Ellerman > > Signed-off-by: Greg Kroah-Hartman > > It seems that something in your scripts munged the apostrophe in the summary > line when generating these emails. Though it looks fine in the actual tree. Odd, the original patch has the subject right, so git will do it correctly when it commits the patch. But, something dropped the ' character when the email was created. Given that it goes through quilt a few different ways to get here to the mailing list, I'll blame it :) thanks, greg k-h