[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V11 15/20] net/net.c: Add net client type check
From: |
Zhang Chen |
Subject: |
Re: [Qemu-devel] [PATCH V11 15/20] net/net.c: Add net client type check function for COLO |
Date: |
Tue, 21 Aug 2018 17:26:35 +0800 |
On Tue, Aug 21, 2018 at 11:27 AM Jason Wang <address@hidden> wrote:
>
>
> On 2018年08月12日 04:59, Zhang Chen wrote:
> > From: Zhang Chen <address@hidden>
> >
> > We add is_colo_support_client_type() to check the net client type for
> > COLO-compare. Currently we just support TAP.
> > Suggested by Jason.
> >
> > Signed-off-by: Zhang Chen <address@hidden>
> > Signed-off-by: Zhang Chen <address@hidden>
> > ---
>
> This needs more works:
>
> - Forbid vhost
> - Deal with hotplug
>
> Looks not a must for this series, so to speed up the process, you may do
> it on top.
>
>
Do you mean this patch we can stay this status currently?
Thanks
Zhang Chen
> Thanks
>
> > include/net/net.h | 1 +
> > net/colo-compare.c | 5 +++++
> > net/net.c | 14 ++++++++++++++
> > 3 files changed, 20 insertions(+)
> >
> > diff --git a/include/net/net.h b/include/net/net.h
> > index 1425960f76..dcbc7ba9c0 100644
> > --- a/include/net/net.h
> > +++ b/include/net/net.h
> > @@ -212,6 +212,7 @@ void hmp_host_net_add(Monitor *mon, const QDict
> *qdict);
> > void hmp_host_net_remove(Monitor *mon, const QDict *qdict);
> > void netdev_add(QemuOpts *opts, Error **errp);
> > void qmp_netdev_add(QDict *qdict, QObject **ret, Error **errp);
> > +bool is_colo_support_client_type(void);
> >
> > int net_hub_id_for_client(NetClientState *nc, int *id);
> > NetClientState *net_hub_port_find(int hub_id);
> > diff --git a/net/colo-compare.c b/net/colo-compare.c
> > index 426eab5973..b8c0240725 100644
> > --- a/net/colo-compare.c
> > +++ b/net/colo-compare.c
> > @@ -996,6 +996,11 @@ static void colo_compare_complete(UserCreatable
> *uc, Error **errp)
> > return;
> > }
> >
> > + if (!is_colo_support_client_type()) {
> > + error_setg(errp, "COLO-compare: Net client type is not
> supported");
> > + return;
> > + }
> > +
> > net_socket_rs_init(&s->pri_rs, compare_pri_rs_finalize,
> s->vnet_hdr);
> > net_socket_rs_init(&s->sec_rs, compare_sec_rs_finalize,
> s->vnet_hdr);
> >
> > diff --git a/net/net.c b/net/net.c
> > index 2a3133990c..a77ea88fff 100644
> > --- a/net/net.c
> > +++ b/net/net.c
> > @@ -1733,3 +1733,17 @@ int net_fill_rstate(SocketReadState *rs, const
> uint8_t *buf, int size)
> > assert(size == 0);
> > return 0;
> > }
> > +
> > +/* Currently, COLO just support TAP */
> > +bool is_colo_support_client_type(void)
> > +{
> > + NetClientState *nc;
> > +
> > + QTAILQ_FOREACH(nc, &net_clients, next) {
> > + if (nc->info->type != NET_CLIENT_DRIVER_TAP) {
> > + return false;
> > + }
> > + }
> > +
> > + return true;
> > +}
>
>
- [Qemu-devel] [PATCH V11 07/20] COLO: Load dirty pages into SVM's RAM cache firstly, (continued)
- [Qemu-devel] [PATCH V11 07/20] COLO: Load dirty pages into SVM's RAM cache firstly, Zhang Chen, 2018/08/11
- [Qemu-devel] [PATCH V11 08/20] ram/COLO: Record the dirty pages that SVM received, Zhang Chen, 2018/08/11
- [Qemu-devel] [PATCH V11 09/20] COLO: Flush memory data from ram cache, Zhang Chen, 2018/08/11
- [Qemu-devel] [PATCH V11 10/20] qmp event: Add COLO_EXIT event to notify users while exited COLO, Zhang Chen, 2018/08/11
- [Qemu-devel] [PATCH V11 11/20] qapi/migration.json: Rename COLO unknown mode to none mode., Zhang Chen, 2018/08/11
- [Qemu-devel] [PATCH V11 12/20] qapi: Add new command to query colo status, Zhang Chen, 2018/08/11
- [Qemu-devel] [PATCH V11 13/20] savevm: split the process of different stages for loadvm/savevm, Zhang Chen, 2018/08/11
- [Qemu-devel] [PATCH V11 14/20] COLO: flush host dirty ram from cache, Zhang Chen, 2018/08/11
- [Qemu-devel] [PATCH V11 15/20] net/net.c: Add net client type check function for COLO, Zhang Chen, 2018/08/11
[Qemu-devel] [PATCH V11 16/20] filter: Add handle_event method for NetFilterClass, Zhang Chen, 2018/08/11
- Re: [Qemu-devel] [PATCH V11 16/20] filter: Add handle_event method for NetFilterClass, Jason Wang, 2018/08/20
- Re: [Qemu-devel] [PATCH V11 16/20] filter: Add handle_event method for NetFilterClass, Zhang Chen, 2018/08/21
- Re: [Qemu-devel] [PATCH V11 16/20] filter: Add handle_event method for NetFilterClass, Jason Wang, 2018/08/22
- Re: [Qemu-devel] [PATCH V11 16/20] filter: Add handle_event method for NetFilterClass, Zhang Chen, 2018/08/24
- Re: [Qemu-devel] [PATCH V11 16/20] filter: Add handle_event method for NetFilterClass, Jason Wang, 2018/08/28
- Re: [Qemu-devel] [PATCH V11 16/20] filter: Add handle_event method for NetFilterClass, Zhang Chen, 2018/08/30
[Qemu-devel] [PATCH V11 17/20] filter-rewriter: handle checkpoint and failover event, Zhang Chen, 2018/08/11