From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Vrabel Subject: Re: [PATCH 2/6] xen-blkfront: handle backend CLOSED without CLOSING Date: Fri, 5 Oct 2012 12:42:21 +0100 Message-ID: <506EC79D.5080206@citrix.com> References: <1348243464-15903-1-git-send-email-david.vrabel@citrix.com> <1348243464-15903-3-git-send-email-david.vrabel@citrix.com> <5061EFB2.1080407@citrix.com> <5069D097.60606@citrix.com> <20121002200255.GA668@phenom.dumpdata.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20121002200255.GA668@phenom.dumpdata.com> Sender: linux-kernel-owner@vger.kernel.org To: Konrad Rzeszutek Wilk Cc: "xen-devel@lists.xensource.com" , "linux-kernel@vger.kernel.org" , Jan Beulich List-Id: xen-devel@lists.xenproject.org On 02/10/12 21:02, Konrad Rzeszutek Wilk wrote: > On Mon, Oct 01, 2012 at 06:19:19PM +0100, David Vrabel wrote: >> On 25/09/12 18:53, David Vrabel wrote: >>> On 21/09/12 17:04, David Vrabel wrote: >>>> From: David Vrabel >>>> >>>> Backend drivers shouldn't transistion to CLOSED unless the frontend is >>>> CLOSED. If a backend does transition to CLOSED too soon then the >>>> frontend may not see the CLOSING state and will not properly shutdown. >>>> >>>> So, treat an unexpected backend CLOSED state the same as CLOSING. >>> >>> Didn't handle the frontend block device being mounted. Updated patch here. >> >> Konrad, can you ack this updated patch if you're happy with it. > > Acked-by: Konrad Rzeszutek Wilk > > Or should I just carry it in my for-jens-3.7 bug-fixes queue and ask > Jen to pull it once rc0 is out? This seems easiest, if Jan is happy with the patch. Thanks. David