On 11/09/2016 01:02 PM, Dave Young wrote:
> On 11/09/16 at 11:58am, Wen Congyang wrote:
>> On 11/09/2016 11:17 AM, Dave Young wrote:
>>> Drop qiaonuohan, seems the mail address is wrong..
>>>
>>> On 11/09/16 at 11:01am, Dave Young wrote:
>>>> Hi
this query to qemu devel
>> list. If this is not correct please let me know.
IIRC, 'virsh dump --memory-only' uses dump-guest-memory, and 'virsh dump'
uses migration to dump.
I think I should study kaslr first...
Thanks
Wen Congyang
>>
>> Could you qemu dump people make it work? Or we can not support virt dump
>> as long as KASLR being enabled. Latest Fedora kernel has enabled it in
>> x86_64.
>>
>> Thanks
>> Dave
>
>
>
: Paolo Bonzini
This patch looks fine to me.
Reviewed-by: Wen Congyang
---
block/replication.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/block/replication.c b/block/replication.c
index 3bd1cf1..5231a00 100644
--- a/block/replication.c
+++ b/block/replication.c
@@ -133,6 +133,9
XX.
For example:
Quorum has one child: children.1(children.0 is removed)
We add a new child, its role name is children.2, not children.0.
If we want to attach the child not in the tail, for example:
Quorum has two children: children.0, children.1. And the new child should
be before children.1. In this case, we should rename children.1 to children.2
and the new child role name can be children.1. If we allow such usage, we
should rename the other child role name when add/deleting a child. It means
that we should query the role name again after add/deleting a child.
Thanks
Wen Congyang
>
> Max
>
> probably be addressed before 2.6.
After this commit, iasl's output is changed:
https://github.com/acpica/acpica/commit/1ecbb3d551255dab943f3bbe7e9da0145d154bba
Thanks
Wen Congyang
>
"BXPC"
* Compiler Version 0x0001 (1)
*/
-DefinitionBlock ("t1.aml", "DSDT", 1, "BOCHS ", "BXPCDSDT", 0x0001)
+DefinitionBlock ("", "DSDT", 1, "BOCHS ", "BXPCDSDT", 0x0001)
{
Scope (\)
{
The newest iasl does't output the aml filename: "tl.aml"
In the function normalize_asl(): we hope the output contains the aml filename...
Thanks
Wen Congyang
tdev);
>>> }
>>
>> We'd better handle this at generic layer and don't let a specific net
>> filter need to worry about this.
>>
>> Looks like the issue is we may trigger status_changed() too early (even
>> before the the filter was initialized).
>>
>
> Yes ~
>
>> How about not call status_changed() if the initialization is not done?
>>
>
> But seems that it is difficult to confirm if the filter is initialized
> or not ...
If nfc->setup() is not called, nf->netdev is NULL.
Thanks
Wen Congyang
>
>> .
>>
>
>
>
>
>
hen skipping
> compilation of the nettle-pbkdf.o module
>
> Reported-by: Wen Congyang
> Signed-off-by: Daniel P. Berrange
I build the qemu with this patch. It is OK now.
Thanks
Wen Congyang
> ---
> configure| 16
> crypto/Makefile.objs
iverState *child_bs,
const char *child_name,
const BdrvChildRole *child_role)
{
BdrvChild *child = bdrv_root_attach_child(child_bs, child_name, child_role);
QLIST_INSERT_HEAD(&parent_bs->children, child, next);
return ch
7"}}], "node-name": "#block040"}}
Shoule we hide the node name like "#blockxxx"?
If the bs doesn't have any child, should we output: '"children": [], '?
Can we add a new parameter: depth? For
> 1. Introduce a new driver replication to avoid touch nbd and qcow2.
> V3:
> 1: use error_setg() instead of error_set()
> 2. Add a new block job API
> 3. Active disk, hidden disk and nbd target uses the same AioContext
> 4. Add a testcase to test new hbitmap API
> V2:
> 1. R
The monitor command 'query-block' or 'info block' will output the format
specific
information. So we can get each child's child-name after this patch. This useful
for dynamic reconfiguration.
Signed-off-by: Wen Congyang
---
block/quorum.c | 27 ++
> + * Lesser General Public License for more details.
> + *
> + * You should have received a copy of the GNU Lesser General Public
> + * License along with this library; if not, see
> <http://www.gnu.org/licenses/>.
> + *
> + */
> +
> +#include "qemu/osdep.h"
> +#include "crypto/pbkdf.h"
> +#include "nettle/pbkdf2.h"
I get the following building error:
CCcrypto/pbkdf.o
CCcrypto/pbkdf-nettle.o
CCcrypto/ivgen.o
crypto/pbkdf-nettle.c:23:27: error: nettle/pbkdf2.h: No such file or directory
crypto/pbkdf-nettle.c: In function ‘qcrypto_pbkdf2’:
crypto/pbkdf-nettle.c:46: warning: implicit declaration of function
‘pbkdf2_hmac_sha1’
crypto/pbkdf-nettle.c:46: warning: nested extern declaration of
‘pbkdf2_hmac_sha1’
crypto/pbkdf-nettle.c:53: warning: implicit declaration of function
‘pbkdf2_hmac_sha256’
crypto/pbkdf-nettle.c:53: warning: nested extern declaration of
‘pbkdf2_hmac_sha256’
make: *** [crypto/pbkdf-nettle.o] Error 1
make: *** Waiting for unfinished jobs
rpm -qf /usr/include/nettle/
libnettle-devel-2.4-8.1.2
The nettle version is very old..
The OS is SUSE 11 SP3.
Thanks
Wen Congyang
On 03/23/2016 04:56 PM, Dr. David Alan Gilbert wrote:
> * Changlong Xie (xiecl.f...@cn.fujitsu.com) wrote:
>> On 03/22/2016 08:22 PM, Dr. David Alan Gilbert wrote:
>>> * Changlong Xie (xiecl.f...@cn.fujitsu.com) wrote:
>>>> From: Wen Congyang
>>>>
>
On 03/17/2016 02:10 PM, Jason Wang wrote:
>
>
> On 03/16/2016 05:34 PM, Wen Congyang wrote:
>> On 03/16/2016 04:18 PM, Jason Wang wrote:
>>>>
>>>>
>>>> On 03/15/2016 06:03 PM, Zhang Chen wrote:
>>>>>> Filter-redirector is
On 03/17/2016 05:48 PM, Dr. David Alan Gilbert wrote:
> * Wen Congyang (we...@cn.fujitsu.com) wrote:
>> On 03/17/2016 05:10 PM, Alberto Garcia wrote:
>>> On Thu 17 Mar 2016 02:22:40 AM CET, Wen Congyang
>>> wrote:
>>>>>>>> @@ -81,6 +82,8 @@ t
ot; (hd1.qcow2)
>
> children.0 (hd0.qcow2) is added again:
>
> s->children[0] <--> "children.1" (hd1.qcow2)
> s->children[1] <--> "children.0" (hd0.qcow2)
Yes, it is correct.
>
> Is this correct? Is this the indented behavior? Since you are reading
> in FIFO mode, now hd1.qcow2 will always be read first, so if
> children.1 was the secondary disk, it has just become the primary.
Yes.
Thanks
Wen Congyang
>
> I also think that it would be great to have tests for this
> functionality, but they can be added later.
>
> Thanks,
>
> Berto
>
>
> .
>
On 03/17/2016 05:10 PM, Alberto Garcia wrote:
> On Thu 17 Mar 2016 02:22:40 AM CET, Wen Congyang wrote:
>>>>>> @@ -81,6 +82,8 @@ typedef struct BDRVQuorumState {
>>>>>> bool rewrite_corrupted;/* true if the driver
On 03/17/2016 07:25 PM, Dr. David Alan Gilbert wrote:
> * Wen Congyang (we...@cn.fujitsu.com) wrote:
>> On 03/17/2016 06:07 PM, Alberto Garcia wrote:
>>> On Thu 17 Mar 2016 10:56:09 AM CET, Wen Congyang wrote:
>>>>> We should have the failure modes documented,
On 03/17/2016 06:07 PM, Alberto Garcia wrote:
> On Thu 17 Mar 2016 10:56:09 AM CET, Wen Congyang wrote:
>>> We should have the failure modes documented, and how you'll use it
>>> after failover etc Without that it's really difficult to tell if this
>>> nam
The monitor command 'query-block' or 'info block' will output the filename.
So we can get each children's child-name after this patch. This useful for
dynamic reconfiguration.
Signed-off-by: Wen Congyang
---
block/quorum.c | 10 +++---
1 file changed, 7 insertions
-+
>> |
>> |
>> v
>> filter
>>
>> usage:
>>
>> -netdev user,id=hn0
>> -chardev socket,id=s0,host=ip_primary,port=X,server,nowait
>> -chardev socket,id=s1,
t index)
> [...]
>
> Sorry if I missed a previous discussion, but why is this necessary?
Hi, Alberto Garcia
Do you have any comments about this patch or give a R-B?
Thanks
Wen Congyang
>
> Berto
>
>
> .
>
file included from /home/devel/git/qemu/include/net/filter.h:12:0,
> from net/filter-mirror.c:12:
> /home/devel/git/qemu/include/qom/object.h:300:39: error: unknown type
> name ‘Error’
>Error **errp);
I think he doesn't use t
is, which results in a QEMU crash.
If we use FIFO mode, we don't call quorum_report_bad() in quorum_aio_cb().
But it is OK to iniialize sacb->aiocb for it.
>
> Signed-off-by: Alberto Garcia
> Reviewed-by: Max Reitz
Reviewed-by: Wen Congyang
> ---
> block/quor
,netdev=hn0,queue=tx/rx/all,outdev=mirror0
>
> Signed-off-by: Zhang Chen
> Signed-off-by: Wen Congyang
> Reviewed-by: Yang Hongyang
> Reviewed-by: zhanghailiang
> ---
> net/Makefile.objs | 1 +
> net/filter-mirror.c | 181
> ++
On 03/09/2016 12:26 PM, Li Zhijian wrote:
>
>
> On 03/09/2016 09:36 AM, Wen Congyang wrote:
>> On 03/08/2016 05:54 PM, Peter Maydell wrote:
>>> On 8 March 2016 at 16:06, Zhang Chen wrote:
>>>> I found the reason for this problem is that
>>>> uni
. Otherwise the test is likely to be unreliable
> if run on a heavily loaded or slow machine.
Yes, but there is no way to know when tcp_chr_accept() is called. Add a event
to notify it?
Thanks
Wen Congyang
>
> thanks
> -- PMM
>
>
>
On 03/08/2016 05:06 PM, Zhang Chen wrote:
>
>
> On 03/08/2016 03:56 PM, Jason Wang wrote:
>>
>> On 03/08/2016 03:50 PM, Wen Congyang wrote:
>>> On 03/08/2016 03:33 PM, Jason Wang wrote:
>>>> On 03/08/2016 12:51 PM, Peter Maydell wrote:
>>
tently, every time I run make check, on the same test).
>>
>> thanks
>> -- PMM
>
> Sorry, it manages to pass on my machine before submitting the pull
> request. But when I re-try this several times, it fails.
>
> This probably means we have bug in mirror implementation. Chen and
> Congyang, please try to fix this bug and resubmit a new version of the
> patch.
>
> Will drop mirror from this pull request and submit a V2.
OK. what is the version of the kernel that you use?
Thanks
Wen Congyang
>
> Thanks
>
>
> .
>
On 02/25/2016 12:59 AM, Eric Blake wrote:
> On 02/24/2016 03:11 AM, Changlong Xie wrote:
>> Introduce QuorumOpType, and make QUORUM_REPORT_BAD compatible
>> with it.
>>
>> Cc: Dr. David Alan Gilbert
>> Cc: Wen Congyang
>> Signed-off-by: Wen Co
On 02/19/2016 04:41 PM, Hailiang Zhang wrote:
> Hi,
>
> On 2016/2/15 9:13, Wen Congyang wrote:
>> On 02/15/2016 08:57 AM, Hailiang Zhang wrote:
>>> On 2016/2/5 12:18, Changlong Xie wrote:
>>>> Signed-off-by: Wen Congyang
>>>> Signed-off-by: zhangha
this patch, sometimes quorum returns error, and
sometimes quorum returns 0. In such case, which is better? Always return 0 or
always return error? In my opinion, we can always return 0 if we allow quorum
returns 0 in case 2.
>
> Also, is this only supposed to be used in FIFO mode? Your p
On 02/15/2016 08:57 AM, Hailiang Zhang wrote:
> On 2016/2/5 12:18, Changlong Xie wrote:
>> Signed-off-by: Wen Congyang
>> Signed-off-by: zhanghailiang
>> Signed-off-by: Gonglei
>> Signed-off-by: Changlong Xie
>> ---
>> Makefile.objs
On 02/04/2016 05:07 PM, Dr. David Alan Gilbert wrote:
> * Changlong Xie (xiecl.f...@cn.fujitsu.com) wrote:
>> On 02/01/2016 09:18 AM, Wen Congyang wrote:
>>> On 01/29/2016 06:47 PM, Dr. David Alan Gilbert wrote:
>>>> * Wen Congyang (we...@cn.fujitsu.com) wrote:
&g
On 02/03/2016 05:32 PM, Stefan Hajnoczi wrote:
> On Wed, Feb 03, 2016 at 09:29:15AM +0800, Wen Congyang wrote:
>> On 02/02/2016 10:34 PM, Stefan Hajnoczi wrote:
>>> On Mon, Feb 01, 2016 at 09:13:36AM +0800, Wen Congyang wrote:
>>>> On 01/29/2016 11:46 PM, Stefan Haj
On 02/03/2016 11:35 AM, Eric Blake wrote:
> On 02/02/2016 08:18 PM, Changlong Xie wrote:
>> On 02/02/2016 11:20 PM, Eric Blake wrote:
>>> On 01/13/2016 02:18 AM, Changlong Xie wrote:
>>>> From: Wen Congyang
>>>>
>>>> Signed-off-by: Wen Congya
On 02/02/2016 10:34 PM, Stefan Hajnoczi wrote:
> On Mon, Feb 01, 2016 at 09:13:36AM +0800, Wen Congyang wrote:
>> On 01/29/2016 11:46 PM, Stefan Hajnoczi wrote:
>>> On Fri, Jan 29, 2016 at 11:13:42AM +0800, Changlong Xie wrote:
>>>> On 01/28/2016 11:15 PM, Stefan Haj
On 01/29/2016 06:47 PM, Dr. David Alan Gilbert wrote:
> * Wen Congyang (we...@cn.fujitsu.com) wrote:
>> On 01/29/2016 06:07 PM, Dr. David Alan Gilbert wrote:
>>> * Wen Congyang (we...@cn.fujitsu.com) wrote:
>>>> On 01/27/2016 07:03 PM, Dr. David Alan Gilbert wrote
On 01/29/2016 11:46 PM, Stefan Hajnoczi wrote:
> On Fri, Jan 29, 2016 at 11:13:42AM +0800, Changlong Xie wrote:
>> On 01/28/2016 11:15 PM, Stefan Hajnoczi wrote:
>>> On Thu, Jan 28, 2016 at 09:13:24AM +0800, Wen Congyang wrote:
>>>> On 01/27/2016 10:46 PM, Stefan Haj
On 01/29/2016 06:07 PM, Dr. David Alan Gilbert wrote:
> * Wen Congyang (we...@cn.fujitsu.com) wrote:
>> On 01/27/2016 07:03 PM, Dr. David Alan Gilbert wrote:
>>> Hi,
>>> I've got a block error if I kill the secondary.
>>>
>>> Start b
acb->success_count
> $2 = 0
> (gdb) p acb->is_read
> $5 = false
Sorry for the late reply.
What it the value of acb->count?
If secondary host is down, you should remove quorum's children.1. Otherwise,
you will get
I/O error event.
Thanks
Wen Congyang
>
> (qemu) info bl
On 01/27/2016 10:46 PM, Stefan Hajnoczi wrote:
> On Wed, Jan 13, 2016 at 05:18:31PM +0800, Changlong Xie wrote:
>> From: Wen Congyang
>>
>> Signed-off-by: Wen Congyang
>> Signed-off-by: zhanghailiang
>> Signed-off-by: Gonglei
>> Signed-off-by: Changl
On 01/26/2016 02:59 AM, Dr. David Alan Gilbert wrote:
> * Wen Congyang (we...@cn.fujitsu.com) wrote:
>> On 01/23/2016 03:35 AM, Dr. David Alan Gilbert wrote:
>>> Hi,
>>> I've been looking at what's needed to add a new secondary after
>>> a primary
which number is which; is there a way to give them a name?
No. I think we can improve 'info block' output.
>f) I've not thought about the colo-proxy that much yet - I guess that
> existing connections need to keep their sequence number offset but
> new connections made by what is now the primary dont need to do anything
> special.
Hailiang or Zhijian can answer this question.
Thanks
Wen Congyang
>
> Dave
> --
> Dr. David Alan Gilbert / dgilb...@redhat.com / Manchester, UK
>
>
> .
>
rv_co_complete (acb=0x57815410) at
> /root/colo/jan-2016/qemu/block/io.c:2122
> .
>
> So I guess acb->child_iter is wrong, since we only have one child on that
> quorum?
> and we're trying to do a destroy on the second child.
Can you try the following patch:
&g
e that the
>> destination has the same data before the shallow mirror. How do you do
>> that in this case?
>
> I think in the colo case they're relying on doing a block migrate
> to synchronise the remote disk prior to switching into colo mode.
Yes, we can do a block migration to sync the disk. After the migration finished,
we stop block migration before starting colo.
Thanks
Wen Congyang
>
> Dave
>
>> Berto
> --
> Dr. David Alan Gilbert / dgilb...@redhat.com / Manchester, UK
>
>
> .
>
}
> 173 }
> 174
> 175 g_free(acb->qcrs);
> (gdb) p acb->child_iter
> $1 = 1
> (gdb) p i
> $3 = 1
Thanks for your test. Can you give me the following information:
1. acb->ret's value
2. s->num_children
I think it is quorum's bug, and acb-
Stefan:ping
Do you have time to review this series patchset?
Thanks
Wen Congyang
At 2016/1/13 17:18, Changlong Xie wrote:
Block replication is a very important feature which is used for
continuous checkpoints(for example: COLO).
You can get the detailed information about block replication
lters, next) {
> +/* Don't go through filter if it is off */
> +if (qemu_need_skip_netfilter(nf)) {
> +continue;
> +}
> ret = qemu_netfilter_receive(nf, direction, sender, flags, iov,
> iovcnt, sent_cb);
> if (ret) {
>
qemu_netfilter_pass_to_next() shoule also be updated.
Thanks
Wen Congyang
On 01/22/2016 03:42 PM, Jason Wang wrote:
>
>
> On 01/22/2016 02:47 PM, Wen Congyang wrote:
>> On 01/22/2016 02:21 PM, Jason Wang wrote:
>>>
>>> On 01/22/2016 01:56 PM, Wen Congyang wrote:
>>>> On 01/22/2016 01:41 PM, Jason Wang wrote:
>>>
On 01/22/2016 02:21 PM, Jason Wang wrote:
>
>
> On 01/22/2016 01:56 PM, Wen Congyang wrote:
>> On 01/22/2016 01:41 PM, Jason Wang wrote:
>>>>
>>>>
>>>> On 01/22/2016 11:28 AM, Wen Congyang wrote:
>>>>>> On 01/22/2016 11:1
On 01/22/2016 01:33 PM, Jason Wang wrote:
>
>
> On 01/20/2016 06:34 PM, Wen Congyang wrote:
>> On 01/20/2016 06:03 PM, Jason Wang wrote:
>>>
>>> On 01/20/2016 05:49 PM, Wen Congyang wrote:
>>>> On 01/20/2016 05:20 PM, Jason Wang wrote:
>
On 01/22/2016 01:41 PM, Jason Wang wrote:
>
>
> On 01/22/2016 11:28 AM, Wen Congyang wrote:
>> On 01/22/2016 11:15 AM, Jason Wang wrote:
>>>
>>> On 01/20/2016 06:30 PM, Wen Congyang wrote:
>>>> On 01/20/2016 06:19 PM, Jason Wang wrote:
>>>
On 01/22/2016 11:15 AM, Jason Wang wrote:
>
>
> On 01/20/2016 06:30 PM, Wen Congyang wrote:
>> On 01/20/2016 06:19 PM, Jason Wang wrote:
>>>>
>>>>
>>>> On 01/20/2016 06:01 PM, Wen Congyang wrote:
>>>>>> On 01/20/2016 02:54
to make sure that the data in a new Quorum
> child is consistent with that of the rest of the array?
Quorum can have more than one child when it starts. But we don't do the
similar check. So I don't think we should do such check here.
Thanks
Wen Congyang
>
> Berto
>
>
> .
>
On 01/20/2016 06:03 PM, Jason Wang wrote:
>
>
> On 01/20/2016 05:49 PM, Wen Congyang wrote:
>> On 01/20/2016 05:20 PM, Jason Wang wrote:
>>>
>>> On 01/20/2016 03:44 PM, Wen Congyang wrote:
>>>>>> ...
>>>>>> -chardev socket,id=
On 01/20/2016 06:19 PM, Jason Wang wrote:
>
>
> On 01/20/2016 06:01 PM, Wen Congyang wrote:
>> On 01/20/2016 02:54 PM, Jason Wang wrote:
>>>
>>> On 01/20/2016 11:29 AM, Zhang Chen wrote:
>>>>> Sure.
>>>>>
>>>>>
gt;> In failover scene（primary is down）, the TCP rewriter will keep
>> servicing
>> for the TCP connection which is established after the last checkpoint。
>>
>>
>>
>> How about this plan?
>
> Sounds good.
>
> And there's indeed no
On 01/20/2016 05:20 PM, Jason Wang wrote:
>
>
> On 01/20/2016 03:44 PM, Wen Congyang wrote:
>>>>
>>>> ...
>>>> -chardev socket,id=comparer0,host=ip_primary,port=X,server,nowait
>>>> -chardev socket,id=comparer1,host=ip_primary,po
iffer client/server by reusing the socket
> chardev. E.g:
>
> In primary node:
Thanks for your suggestion.
>
> ...
> -chardev socket,id=comparer0,host=ip_primary,port=X,server,nowait
> -chardev socket,id=comparer1,host=ip_primary,port=Y,server,nowait
> -chardev socket,id=mirrorer0,host=ip_primary,port=Z,server,nowait
> -netdev tap,id=hn0
> -traffic-mirrorer netdev=hn0,id=t0,indev=comparer0,outdev=mirrorer0
> -colo-comparer primary_traffic=comparer0,secondary_traffic=comparer1
> ...
>
> packet comparer compares the packets from two chardev: comparer0 and
> comparer1.
> traffic-mirrorer mirror tx to secondary node through chardev mirrorer0,
> and mirror rx to packet comparer through chardev comparer0.
>
> In secondary node:
>
> ...
> -chardev socket,id=redirector0,host=ip_primary,port=Y
> -chardev socket,id=redirector1,host=ip_primary,port=Z
> -netdev tap,id=hn0
> -traffic-redirector netdev=hn0,id,r0,indev=redirector0,outdev=redirector1
> -colo-rewriter netdev=hn0,id=c0
> ...
>
> traffic-redirector redirect the rx traffic from primary node through
> redirector0 and redirect the tx traffic to promary node through redirector1.
> colo-rewriter rewrite seq number as a normal netfilter.
What are traffic-mirrorer and colo-comparer, traffic-redirector, colo-rewriter?
A netfilter driver?
If not, how to get the packet from the netdev, and send back the packet to
the netdev?
Thanks
Wen Congyang
>
>
>
>>
>>
>>> .
>>>
>>
>
>
>
>
> .
>
ow.
> v2:
> 1. Use bdrv_get_device_or_node_name() instead of new function
>bdrv_get_id_or_node_name()
> 2. Update the error message
> 3. Update the documents in block-core.json
>
> Wen Congyang (3):
> Add new block driver interface to add/delete a BDS's child
&g
On 12/23/2015 05:26 PM, Stefan Hajnoczi wrote:
> On Wed, Dec 02, 2015 at 01:31:46PM +0800, Wen Congyang wrote:
>> +== Failure Handling ==
>> +There are 6 internal errors when block replication is running:
>> +1. I/O error on primary disk
>> +2. Forwarding primary
On 12/23/2015 05:47 PM, Stefan Hajnoczi wrote:
> On Wed, Dec 02, 2015 at 01:37:25PM +0800, Wen Congyang wrote:
>> +/*
>> + * Only write to active disk if the sectors have
>> + * already been allocated in active disk/hidden disk.
>> + */
>> +qem
On 12/23/2015 06:04 PM, Stefan Hajnoczi wrote:
> On Thu, Dec 17, 2015 at 02:22:14PM +0800, Wen Congyang wrote:
>> Stefan:Ping...
>>
>> What about this feature? I have worked for it about 1 year, but it is still
>> in the
>> way...
>
> The code still has
job QMP interface:
>
> * Commands to create a job: block-commit, block-stream, drive-mirror,
> drive-backup.
>
> * Get information on jobs: query-block-jobs
>
> * Pause a job: block-job-pause
>
> * Resume a job: block-job-resume
>
> * Cancel a job: b
Stefan:Ping...
What about this feature? I have worked for it about 1 year, but it is still in
the
way...
On 12/02/2015 01:31 PM, Wen Congyang wrote:
> Block replication is a very important feature which is used for
> continuous checkpoints(for example: COLO).
>
> You can get
at it mark the wakeup as unlikely in rcu_read_unlock.
>
> Signed-off-by: Paolo Bonzini
Reviewed-by: Wen Congyang
> ---
> include/qemu/rcu.h | 6 +-
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/include/qemu/rcu.h b/include/qemu/rcu.h
> index f6
lan Gilbert
> ---
> v11:
> - Turn COLO on in default (Eric's suggestion)
>
> Signed-off-by: zhanghailiang
I think you forgot to remove this line.
Thanks
Wen Congyang
> ---
> configure | 11 +++
> 1 file changed, 11 insertions(+)
>
> diff --git a/co
gt; actually existing child nodes.
I think we should have some way to get the child->name. For example, the
monitor command 'info block' or 'query-block' display it.
Thanks
Wen Congyang
>
> Signed-off-by: Kevin Wolf
> ---
> block.c | 20 +++
On 11/24/2015 05:25 PM, zhanghailiang wrote:
> Rename the 'file' member of MigrationState to 'to_dst_file'.
>
> Signed-off-by: zhanghailiang
> Cc: Dr. David Alan Gilbert
> ---
> v11:
> - Only rename 'file' member of MigrationState
You forg
Kevin: ping
On 11/27/2015 02:06 PM, Wen Congyang wrote:
> If quorum's child is broken, we can use mirror job to replace it.
> But sometimes, the user only need to remove the broken child, and
> add it later when the problem is fixed.
>
> It is based on the Kevin's
On 12/03/2015 11:53 AM, Hailiang Zhang wrote:
> On 2015/12/3 9:17, Wen Congyang wrote:
>> On 11/24/2015 05:25 PM, zhanghailiang wrote:
>>> We add each netdev a default filter-buffer, which will be used for COLO
>>> or Micro-checkpoint to buffer VM's packets. The na
ets in default.
> So it has no side effect for the netdev.
No, filter-buffer doesn't support vhost, so if you add default filter-buffer
for each netdev, you can't use vhost.
Thanks
Wen Congyang
>
> Signed-off-by: zhanghailiang
> Cc: Jason Wang
> Cc: Yang Hongyang
>
On 12/02/2015 09:00 AM, Wen Congyang wrote:
> On 12/01/2015 06:40 PM, Dr. David Alan Gilbert wrote:
>> * Wen Congyang (we...@cn.fujitsu.com) wrote:
>>> Block replication is a very important feature which is used for
>>> continuous checkpoints(for example: COLO).
>&
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
---
block.c | 83 +++
include/block/block.h | 4 +++
2 files changed, 87 insertions(+)
diff --git a/block.c b/block.c
index 213bee8..09ee7f1 100644
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
Reviewed-by: Eric Blake
---
qapi/block-core.json | 21 ++---
1 file changed, 18 insertions(+), 3 deletions(-)
diff --git a/qapi/block-core.json b/qapi/block-core.json
index 2c6bd3f..acc9f8d 100644
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
---
block/Makefile.objs | 1 +
block/replication.c | 549
2 files changed, 550 insertions(+)
create mode 100644 block/replication.c
diff --git a/block
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
Cc: Luiz Capitulino
Cc: Michael Roth
Reviewed-by: Paolo Bonzini
---
block.c | 43 +++
include/block/block.h | 5 +
include/block/block_int.h
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
---
docs/block-replication.txt | 227 +
1 file changed, 227 insertions(+)
create mode 100644 docs/block-replication.txt
diff --git a/docs/block-replication.txt b/docs
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
Reviewed-by: Alberto Garcia
---
block/quorum.c | 78 ++
1 file changed, 78 insertions(+)
diff --git a/block/quorum.c b/block/quorum.c
index b7df14b..6fa54f3
When opening BDS, we need to create backup jobs for
image-fleecing.
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
Reviewed-by: Stefan Hajnoczi
Reviewed-by: Jeff Cody
---
block/Makefile.objs | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a
We need to access the parent BDS to get the root BDS.
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
---
block.c | 1 +
include/block/block_int.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/block.c b/block.c
index eaf479a..0a0468f
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
Reviewed-by: Jeff Cody
---
block/backup.c | 14 ++
blockjob.c | 11 +++
include/block/blockjob.h | 12
3 files changed, 37 insertions(+)
diff --git a/block
testcase to test new hbitmap API
V2:
1. Redesign the secondary qemu(use image-fleecing)
2. Use Error objects to return error message
3. Address the comments from Max Reitz and Eric Blake
Wen Congyang (10):
unblock backup operations in backing file
Store parent BDS in BdrvChild
Backup: clear a
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
---
block.c | 18 ++
1 file changed, 18 insertions(+)
diff --git a/block.c b/block.c
index bfc2be8..eaf479a 100644
--- a/block.c
+++ b/block.c
@@ -1275,6 +1275,24 @@ void bdrv_set_backing_hd
On 12/01/2015 07:58 PM, Hailiang Zhang wrote:
> On 2015/12/1 18:40, Dr. David Alan Gilbert wrote:
>> * Wen Congyang (we...@cn.fujitsu.com) wrote:
>>> Block replication is a very important feature which is used for
>>> continuous checkpoints(for example: COLO).
>&
On 12/01/2015 06:40 PM, Dr. David Alan Gilbert wrote:
> * Wen Congyang (we...@cn.fujitsu.com) wrote:
>> Block replication is a very important feature which is used for
>> continuous checkpoints(for example: COLO).
>>
>> You can get the detailed information about
On 11/27/2015 08:27 PM, Zhang Chen wrote:
> From: zhangchen
>
> Colo-proxy is a plugin of qemu netfilter
> like filter-buffer and dump
>
> Signed-off-by: zhangchen
> ---
> net/Makefile.objs | 1 +
> net/colo-proxy.c | 139
> ++
> net/colo
The option port should be removed
3. The option addr is socket address. The format can be host:port, or fd.
> +
> +colo-proxy
Add more description here.
Thanks
Wen Congyang
> +
> @item -object
> filter-dump,id=@var{id},netdev=@var{dev},file=@var{filename}][,maxlen=@var{len}
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
---
docs/block-replication.txt | 227 +
1 file changed, 227 insertions(+)
create mode 100644 docs/block-replication.txt
diff --git a/docs/block-replication.txt b/docs
When opening BDS, we need to create backup jobs for
image-fleecing.
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
Reviewed-by: Stefan Hajnoczi
Reviewed-by: Jeff Cody
---
block/Makefile.objs | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
Reviewed-by: Alberto Garcia
---
block/quorum.c | 78 ++
1 file changed, 78 insertions(+)
diff --git a/block/quorum.c b/block/quorum.c
index b7df14b..6fa54f3
testcase to test new hbitmap API
V2:
1. Redesign the secondary qemu(use image-fleecing)
2. Use Error objects to return error message
3. Address the comments from Max Reitz and Eric Blake
Wen Congyang (10):
unblock backup operations in backing file
Store parent BDS in BdrvChild
Backup: clear a
We need to access the parent BDS to get the root BDS.
Signed-off-by: Wen Congyang
---
block.c | 1 +
include/block/block_int.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/block.c b/block.c
index eaf479a..0a0468f 100644
--- a/block.c
+++ b/block.c
@@ -1204,6 +1204,7
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
Cc: Luiz Capitulino
Cc: Michael Roth
Reviewed-by: Paolo Bonzini
---
block.c | 43 +++
include/block/block.h | 5 +
include/block/block_int.h
Signed-off-by: Wen Congyang
---
block.c | 18 ++
1 file changed, 18 insertions(+)
diff --git a/block.c b/block.c
index bfc2be8..eaf479a 100644
--- a/block.c
+++ b/block.c
@@ -1275,6 +1275,24 @@ void bdrv_set_backing_hd(BlockDriverState *bs,
BlockDriverState *backing_hd
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
Reviewed-by: Jeff Cody
---
block/backup.c | 14 ++
blockjob.c | 11 +++
include/block/blockjob.h | 12
3 files changed, 37 insertions(+)
diff --git a/block
The new QMP command name is x-blockdev-change. It's just for adding/removing
quorum's child now, and doesn't support all kinds of children, all kinds of
operations, nor all block drivers. So it is experimental now.
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-
by the QMP command blockdev-add.
2. The driver NBD can support filename, path, host:port now.
v2:
1. Use bdrv_get_device_or_node_name() instead of new function
bdrv_get_id_or_node_name()
2. Update the error message
3. Update the documents in block-core.json
Wen Congyang (3):
Add new block dri
1 - 100 of 1192 matches
Mail list logo
