On Wed, Aug 07, 2013 at 10:42:26AM +0800, Wenchao Xia wrote: > >On Fri, Jul 26, 2013 at 02:43:44PM +0800, Wenchao Xia wrote: > >>Reviewed-by: Wenchao Xia <xiaw...@linux.vnet.ibm.com> > >> > >>One question: old code missed itself in bdrv_drain_all(), is that a bug? > > > >Sorry, I don't understand the question. Can you rephrase it? > > > Before this patch, in the code path: bdrv_close()->bdrv_drain_all(), > the *bs does not exist in bdrv_states, so the code missed the chance to > drain the request on *bs. That is a bug, and this patch is actually a > bugfix?
Yes, exactly. It's a bug fix and therefore I CCed qemu-sta...@nongnu.org. Stefan
