Re: [Qemu-devel] [PATCH v2] block/quorum: add simple read pattern support

2014-07-14 Thread Eric Blake
On 07/14/2014 10:13 PM, Liu Yuan wrote: >>> +'*read-pattern': 'str' } } >> >> Raw strings that encode a finite set of values are bad for type-safety. >> Please add an enum: >> >> { 'enum': 'QuorumReadPattern', 'data': [ 'quorum', 'fifo' ] } >> >> then use '*read-pattern': 'QuorumReadP

Re: [Qemu-devel] [PATCH v2] block/quorum: add simple read pattern support

2014-07-14 Thread Liu Yuan
On Mon, Jul 14, 2014 at 09:33:59PM -0600, Eric Blake wrote: > On 07/14/2014 09:19 PM, Liu Yuan wrote: > > This patch adds single read pattern to quorum driver and quorum vote is > > default > > pattern. > > > > > This patch generalize the above 2 nodes case in the N nodes. That is, > > > > vm -

Re: [Qemu-devel] [PATCH v2] block/quorum: add simple read pattern support

2014-07-14 Thread Eric Blake
On 07/14/2014 09:19 PM, Liu Yuan wrote: > This patch adds single read pattern to quorum driver and quorum vote is > default > pattern. > > This patch generalize the above 2 nodes case in the N nodes. That is, > > vm -> write to all the N nodes, read just one of them. If single read fails, > we

[Qemu-devel] [PATCH v2] block/quorum: add simple read pattern support

2014-07-14 Thread Liu Yuan
This patch adds single read pattern to quorum driver and quorum vote is default pattern. For now we do a quorum vote on all the reads, it is designed for unreliable underlying storage such as non-redundant NFS to make sure data integrity at the cost of the read performance. For some use cases as