[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 5/8] quorum: fix quorum_aio_cancel()
From: |
Liu Yuan |
Subject: |
Re: [Qemu-devel] [PATCH 5/8] quorum: fix quorum_aio_cancel() |
Date: |
Mon, 1 Sep 2014 17:46:56 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, Sep 01, 2014 at 11:32:04AM +0200, Benoît Canet wrote:
> The Monday 01 Sep 2014 à 17:26:09 (+0800), Liu Yuan wrote :
> > On Mon, Sep 01, 2014 at 10:35:27AM +0200, Benoît Canet wrote:
> > > The Monday 01 Sep 2014 à 15:43:11 (+0800), Liu Yuan wrote :
> > > > For a fifo read pattern, we only have one running aio
> > >
> > > >(possible other cases that has less number than num_children in the
> > > >future)
> > > I have trouble understanding this part of the commit message could you try
> > > to clarify it ?
> >
> > Until this patch, we have two cases, read single or read all. But later
> > patch
> > allow VMs to continue if some devices are down. So the discrete number 1
> > and N
> > becomes a range [1, N], that is possible running requests are from 1 to N.
>
> Why not
>
> (In some other cases some children of the num_children BDS could be disabled
> reducing the number of requests needed) ?
>
Sounds better, I'll take yours, thanks!
Yuan
- Re: [Qemu-devel] [PATCH 2/8] block: add driver operation callbacks, (continued)
[Qemu-devel] [PATCH 6/8] block/quorum: add broken state to BlockDriverState, Liu Yuan, 2014/09/01
[Qemu-devel] [PATCH 7/8] block: add two helpers, Liu Yuan, 2014/09/01
[Qemu-devel] [PATCH 8/8] quorum: add basic device recovery logic, Liu Yuan, 2014/09/01
Re: [Qemu-devel] [PATCH 0/8] add basic recovery logic to quorum driver, Benoît Canet, 2014/09/01
Re: [Qemu-devel] [PATCH 0/8] add basic recovery logic to quorum driver, Benoît Canet, 2014/09/02