Re: [Xen-devel] [PATCH] drivers: xen-blkfront: only talk_to_blkback() when in XenbusStateInitialising

2015-06-23 Thread Konrad Rzeszutek Wilk
On June 23, 2015 2:23:59 AM EDT, Jan Beulich wrote: On 22.06.15 at 15:56, wrote: >> --- a/drivers/block/xen-blkfront.c >> +++ b/drivers/block/xen-blkfront.c >> @@ -1951,6 +1951,8 @@ static void blkback_changed(struct >xenbus_device *dev, >> >> switch (backend_state) { >> case Xen

Re: [Xen-devel] [PATCH] drivers: xen-blkfront: only talk_to_blkback() when in XenbusStateInitialising

2015-06-22 Thread Jan Beulich
>>> On 22.06.15 at 15:56, wrote: > --- a/drivers/block/xen-blkfront.c > +++ b/drivers/block/xen-blkfront.c > @@ -1951,6 +1951,8 @@ static void blkback_changed(struct xenbus_device *dev, > > switch (backend_state) { > case XenbusStateInitWait: > + if (dev->state != XenbusS

Re: [Xen-devel] [PATCH] drivers: xen-blkfront: only talk_to_blkback() when in XenbusStateInitialising

2015-06-22 Thread Konrad Rzeszutek Wilk
On Mon, Jun 22, 2015 at 09:56:00AM -0400, Konrad Rzeszutek Wilk wrote: > From: Bob Liu > > Patch 69b91ede5cab843dcf345c28bd1f4b5a99dacd9b > "drivers: xen-blkback: delay pending_req allocation to connect_ring" > exposed an problem that Xen blkfront has. There is a race > with XenStored and the dri

[Xen-devel] [PATCH] drivers: xen-blkfront: only talk_to_blkback() when in XenbusStateInitialising

2015-06-22 Thread Konrad Rzeszutek Wilk
From: Bob Liu Patch 69b91ede5cab843dcf345c28bd1f4b5a99dacd9b "drivers: xen-blkback: delay pending_req allocation to connect_ring" exposed an problem that Xen blkfront has. There is a race with XenStored and the drivers such that we can see two: vbd vbd-268440320: blkfront:blkback_changed to stat