qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v13 2/7] net/vmnet: add vmnet backends to qapi/net


From: Vladislav Yaroshchuk
Subject: Re: [PATCH v13 2/7] net/vmnet: add vmnet backends to qapi/net
Date: Fri, 28 Jan 2022 17:29:56 +0300



пт, 21 янв. 2022 г. в 16:03, Akihiko Odaki <akihiko.odaki@gmail.com>:
On Fri, Jan 21, 2022 at 9:19 PM Vladislav Yaroshchuk
<yaroshchuk2000@gmail.com> wrote:
>
>
> чт, 20 янв. 2022 г. в 11:32, Roman Bolshakov <roman@roolebo.dev>:
>>
>> On Thu, Jan 13, 2022 at 08:22:14PM +0300, Vladislav Yaroshchuk wrote:
>> > Create separate netdevs for each vmnet operating mode:
>> > - vmnet-host
>> > - vmnet-shared
>> > - vmnet-bridged
>> >
>>
>> Sure I'm late to the party but what if we add only one backend - vmnet
>> with default mode set to shared and all parameters are added there?
>>
>> The CLI would look more reasonable for the most typical use case:
>>  -netdev vmnet,id=if1 -device virtio-net,netdev=if1
>>
>> That would remove duplication of options in QAPI schema (e.g. isolated
>> is available in all backends now, altough I'm not sure if it makes sense
>> for bridged mode):
>>
>>  -netdev vmnet,id=if1,isolated=yes
>>
>> start-address, end-address and subnet-mask are also used by both shared
>> and host modes.
>>
>> Bridged netdev would lool like:
>>
>>  -netdev vmnet,id=if1,mode=bridged,ifname=en1
>>
>> Checksum offloading also seems to be available for all backends from
>> Monterey.
>>
>> The approach might simplify integration of the changes to libvirt and
>> discovery of upcoming vmnet features via qapi.
>>
>
> I can rewrite this if it sounds more suitable to use
> single `vmnet` netdev instead of three different ones.
> We can discuss this with Markus as a QAPI reviewer.
> Markus, what is your opinion about single netdev?
>
> P.S. Seems we have enough time for discussion:
> I started fixing memory leaks found by Akihiko and
> met a strange deadlock on QEMU shutdown on
> `qemu_mutex_lock_iothread()` during careful
> interface destruction with added semaphore.
> Need to go deeper to understand what's the
> problem, it will take some time.
>
> Mentioned part of Akihiko's review:
> 20220113172219.66372-1-yaroshchuk2000@gmail.com/20220113172219.66372-4-yaroshchuk2000@gmail.com/" rel="noreferrer" target="_blank">https://patchew.org/QEMU/20220113172219.66372-1-yaroshchuk2000@gmail.com/20220113172219.66372-4-yaroshchuk2000@gmail.com/


> Actually I thought it would be tricky to implement.
 
You mean single netdev type?

Actually I thought it would be tricky to implement. A deadlock will
occur in a simple implementation if vmnet_send is already queued but
not executed yet when destructing:
- vmnet_send tries to lock the iothread and waits for the destructor to unlock.
- vmnet_stop_interface waits for vmnet_send finishing.


Sounds like that's what happens actually.
With added semaphore:
--- a/net/vmnet-common.m
+++ b/net/vmnet-common.m
@@ -320,13 +320,14 @@ void vmnet_cleanup_common(NetClientState *nc)
         "org.qemu.vmnet.destroy",
         DISPATCH_QUEUE_SERIAL
     );
-
+    dispatch_semaphore_t if_destroy_sem = dispatch_semaphore_create(0);
     vmnet_stop_interface(
         s->vmnet_if,
         if_destroy_q,
         ^(vmnet_return_t status) {
+            dispatch_semaphore_signal(if_destroy_sem);
         });
-
+    dispatch_semaphore_wait(if_destroy_sem, DISPATCH_TIME_FOREVER);
     for (int i = 0; i < VMNET_PACKETS_LIMIT; ++i) {
         g_free(s->iov_buf[i].iov_base);
     }
-- 

I see this thread dump on deadlock:

(lldb) bt all
* thread #1, queue = 'com.apple.main-thread', stop reason = signal SIGSTOP
  * frame #0: 0x00007fff2037ebb2 libsystem_kernel.dylib`__semwait_signal + 10
    frame #1: 0x00007fff202fec1a libsystem_c.dylib`nanosleep + 196
    frame #2: 0x00007fff212c4bb8 Foundation`+[NSThread sleepForTimeInterval:] + 170
    frame #3: 0x0000000101ebce3a qemu-system-x86_64`-[QemuCocoaAppController applicationWillTerminate:](self=0x00007fa9f91a1de0, _cmd="applicationWillTerminate:", aNotification=@"NSApplicationWillTerminateNotification") at cocoa.m:1202:5
    frame #4: 0x00007fff204a00cd CoreFoundation`__CFNOTIFICATIONCENTER_IS_CALLING_OUT_TO_AN_OBSERVER__ + 12
    frame #5: 0x00007fff2053bb1c CoreFoundation`___CFXRegistrationPost_block_invoke + 49
    frame #6: 0x00007fff2053ba9a CoreFoundation`_CFXRegistrationPost + 454
    frame #7: 0x00007fff2047134e CoreFoundation`_CFXNotificationPost + 736
    frame #8: 0x00007fff211e1bb8 Foundation`-[NSNotificationCenter postNotificationName:object:userInfo:] + 59
    frame #9: 0x00007fff22f585b3 AppKit`-[NSApplication terminate:] + 1377
    frame #10: 0x0000000101ebcf1f qemu-system-x86_64`-[QemuCocoaAppController windowShouldClose:](self=0x00007fa9f91a1de0, _cmd="windowShouldClose:", sender=0x00007fa9f91a7810) at cocoa.m:1231:5
    frame #11: 0x00007fff230417c9 AppKit`__19-[NSWindow __close]_block_invoke + 153
    frame #12: 0x00007fff23041723 AppKit`-[NSWindow __close] + 284
    frame #13: 0x00007fff22ed12bb AppKit`-[NSApplication(NSResponder) sendAction:to:from:] + 288
    frame #14: 0x00007fff22ed115f AppKit`-[NSControl sendAction:to:] + 86
    frame #15: 0x00007fff22ed1091 AppKit`__26-[NSCell _sendActionFrom:]_block_invoke + 131
    frame #16: 0x00007fff22ed0f98 AppKit`-[NSCell _sendActionFrom:] + 171
    frame #17: 0x00007fff22ed0ede AppKit`-[NSButtonCell _sendActionFrom:] + 96
    frame #18: 0x00007fff22ecdfc7 AppKit`NSControlTrackMouse + 1820
    frame #19: 0x00007fff22ecd883 AppKit`-[NSCell trackMouse:inRect:ofView:untilMouseUp:] + 130
    frame #20: 0x00007fff22ecd74a AppKit`-[NSButtonCell trackMouse:inRect:ofView:untilMouseUp:] + 697
    frame #21: 0x00007fff22ecca72 AppKit`-[NSControl mouseDown:] + 722
    frame #22: 0x00007fff22ecae5e AppKit`-[NSWindow(NSEventRouting) _handleMouseDownEvent:isDelayedEvent:] + 4961
    frame #23: 0x00007fff22e3a648 AppKit`-[NSWindow(NSEventRouting) _reallySendEvent:isDelayedEvent:] + 2594
    frame #24: 0x00007fff22e39a06 AppKit`-[NSWindow(NSEventRouting) sendEvent:] + 347
    frame #25: 0x00007fff22e37e14 AppKit`-[NSApplication(NSEvent) sendEvent:] + 352
    frame #26: 0x0000000101ebec29 qemu-system-x86_64`-[QemuApplication sendEvent:](self=0x00007fa9f90374e0, _cmd="sendEvent:", event=0x00007fa9f6ef1920) at cocoa.m:1582:9
    frame #27: 0x00007fff23110be1 AppKit`-[NSApplication _handleEvent:] + 65
    frame #28: 0x00007fff22ca0c8e AppKit`-[NSApplication run] + 623
    frame #29: 0x0000000101ebef45 qemu-system-x86_64`main(argc=3, argv=0x00007ffeedd71a20) at cocoa.m:1947:5
    frame #30: 0x00007fff203ccf3d libdyld.dylib`start + 1
  thread #2
    frame #0: 0x00007fff2037ecde libsystem_kernel.dylib`__psynch_cvwait + 10
    frame #1: 0x00007fff203b1e49 libsystem_pthread.dylib`_pthread_cond_wait + 1298
    frame #2: 0x00000001026b7837 qemu-system-x86_64`qemu_futex_wait(ev=0x0000000102e88eb8, val=4294967295) at qemu-thread-posix.c:386:9
    frame #3: 0x00000001026b77b0 qemu-system-x86_64`qemu_event_wait(ev=0x0000000102e88eb8) at qemu-thread-posix.c:481:9
    frame #4: 0x00000001026c56be qemu-system-x86_64`call_rcu_thread(opaque=0x0000000000000000) at rcu.c:261:21
    frame #5: 0x00000001026b7b76 qemu-system-x86_64`qemu_thread_start(args=0x00007fa9f9105320) at qemu-thread-posix.c:556:9
    frame #6: 0x00007fff203b18fc libsystem_pthread.dylib`_pthread_start + 224
    frame #7: 0x00007fff203ad443 libsystem_pthread.dylib`thread_start + 15
  thread #3
    frame #0: 0x00007fff2037c2f6 libsystem_kernel.dylib`semaphore_wait_trap + 10
    frame #1: 0x00007fff20207c9b libdispatch.dylib`_dispatch_sema4_wait + 16
    frame #2: 0x00007fff2020816d libdispatch.dylib`_dispatch_semaphore_wait_slow + 98
    frame #3: 0x0000000101f6ac1b qemu-system-x86_64`vmnet_cleanup_common(nc=0x00007fa9f7019400) at vmnet-common.m:330:5
    frame #4: 0x0000000101f5c6cd qemu-system-x86_64`qemu_cleanup_net_client(nc=0x00007fa9f7019400) at net.c:361:9
    frame #5: 0x0000000101f5c4cf qemu-system-x86_64`qemu_del_net_client(nc=0x00007fa9f7019400) at net.c:417:13
    frame #6: 0x0000000101f5e751 qemu-system-x86_64`net_cleanup at net.c:1447:13
    frame #7: 0x000000010228f548 qemu-system-x86_64`qemu_cleanup at runstate.c:820:5
    frame #8: 0x0000000101e9950d qemu-system-x86_64`qemu_main(argc=3, argv=0x00007ffeedd71a20, envp=0x00007ffeedd71a40) at main.c:51:5
    frame #9: 0x0000000101ebefb4 qemu-system-x86_64`call_qemu_main(opaque=0x0000000000000000) at cocoa.m:1897:14
    frame #10: 0x00000001026b7b76 qemu-system-x86_64`qemu_thread_start(args=0x00007fa9f9105320) at qemu-thread-posix.c:556:9
    frame #11: 0x00007fff203b18fc libsystem_pthread.dylib`_pthread_start + 224
    frame #12: 0x00007fff203ad443 libsystem_pthread.dylib`thread_start + 15
  thread #4
    frame #0: 0x00007fff20384e22 libsystem_kernel.dylib`__sigwait + 10
    frame #1: 0x00007fff203b1fff libsystem_pthread.dylib`sigwait + 29
    frame #2: 0x00000001026b3541 qemu-system-x86_64`sigwait_compat(opaque=0x00007fa9f6d0a3f0) at compatfd.c:36:15
    frame #3: 0x00000001026b7b76 qemu-system-x86_64`qemu_thread_start(args=0x00007fa9f6d09ec0) at qemu-thread-posix.c:556:9
    frame #4: 0x00007fff203b18fc libsystem_pthread.dylib`_pthread_start + 224
    frame #5: 0x00007fff203ad443 libsystem_pthread.dylib`thread_start + 15
  thread #7
    frame #0: 0x00007fff2037e4ca libsystem_kernel.dylib`__psynch_mutexwait + 10
    frame #1: 0x00007fff203af2ab libsystem_pthread.dylib`_pthread_mutex_firstfit_lock_wait + 76
    frame #2: 0x00007fff203ad192 libsystem_pthread.dylib`_pthread_mutex_firstfit_lock_slow + 204
    frame #3: 0x00007fff203b1ee1 libsystem_pthread.dylib`_pthread_cond_wait + 1450
    frame #4: 0x00000001026b6d92 qemu-system-x86_64`qemu_cond_wait_impl(cond=0x00007fa9f6c2edf0, mutex=0x0000000102e758f0, file="../softmmu/cpus.c", line=423) at qemu-thread-posix.c:195:11
    frame #5: 0x000000010227c716 qemu-system-x86_64`qemu_wait_io_event(cpu=0x00007fa9f949a000) at cpus.c:423:9
    frame #6: 0x000000010239c94e qemu-system-x86_64`mttcg_cpu_thread_fn(arg=0x00007fa9f949a000) at tcg-accel-ops-mttcg.c:123:9
    frame #7: 0x00000001026b7b76 qemu-system-x86_64`qemu_thread_start(args=0x00007fa9f6c2f5e0) at qemu-thread-posix.c:556:9
    frame #8: 0x00007fff203b18fc libsystem_pthread.dylib`_pthread_start + 224
    frame #9: 0x00007fff203ad443 libsystem_pthread.dylib`thread_start + 15
  thread #8
    frame #0: 0x00007fff2037d95e libsystem_kernel.dylib`__workq_kernreturn + 10
    frame #1: 0x00007fff203ae4c1 libsystem_pthread.dylib`_pthread_wqthread + 414
    frame #2: 0x00007fff203ad42f libsystem_pthread.dylib`start_wqthread + 15
  thread #9, queue = 'org.qemu.vmnet.if_queue'
    frame #0: 0x00007fff2037e4ca libsystem_kernel.dylib`__psynch_mutexwait + 10
    frame #1: 0x00007fff203af2ab libsystem_pthread.dylib`_pthread_mutex_firstfit_lock_wait + 76
    frame #2: 0x00007fff203ad192 libsystem_pthread.dylib`_pthread_mutex_firstfit_lock_slow + 204
    frame #3: 0x00000001026b67ea qemu-system-x86_64`qemu_mutex_lock_impl(mutex=0x0000000102e758f0, file="../net/vmnet-common.m", line=91) at qemu-thread-posix.c:80:11
    frame #4: 0x000000010227c9d2 qemu-system-x86_64`qemu_mutex_lock_iothread_impl(file="../net/vmnet-common.m", line=91) at cpus.c:497:5
    frame #5: 0x0000000101f6ae9e qemu-system-x86_64`vmnet_send(nc=0x00007fa9f7019400, event_id=VMNET_INTERFACE_PACKETS_AVAILABLE, event=0x00007fa9f6d3b860) at vmnet-common.m:91:5
    frame #6: 0x0000000101f6acf3 qemu-system-x86_64`__vmnet_register_event_callback_block_invoke(.block_descriptor=0x00007fa9f9034240, event_id=VMNET_INTERFACE_PACKETS_AVAILABLE, event=0x00007fa9f6d3b860) at vmnet-common.m:121:15
    frame #7: 0x00007fff6f385a3d vmnet`__vmnet_interface_set_event_callback_block_invoke_3 + 195
    frame #8: 0x00007fff20207806 libdispatch.dylib`_dispatch_client_callout + 8
    frame #9: 0x00007fff2020a1b0 libdispatch.dylib`_dispatch_continuation_pop + 423
    frame #10: 0x00007fff2021a564 libdispatch.dylib`_dispatch_source_invoke + 2061
    frame #11: 0x00007fff2020d493 libdispatch.dylib`_dispatch_lane_serial_drain + 263
    frame #12: 0x00007fff2020e0ad libdispatch.dylib`_dispatch_lane_invoke + 366
    frame #13: 0x00007fff20217c0d libdispatch.dylib`_dispatch_workloop_worker_thread + 811
    frame #14: 0x00007fff203ae45d libsystem_pthread.dylib`_pthread_wqthread + 314
    frame #15: 0x00007fff203ad42f libsystem_pthread.dylib`start_wqthread + 15
  thread #11, name = 'com.apple.NSEventThread'
    frame #0: 0x00007fff2037c2ba libsystem_kernel.dylib`mach_msg_trap + 10
    frame #1: 0x00007fff2037c62c libsystem_kernel.dylib`mach_msg + 60
    frame #2: 0x00007fff204aab2f CoreFoundation`__CFRunLoopServiceMachPort + 316
    frame #3: 0x00007fff204a920f CoreFoundation`__CFRunLoopRun + 1328
    frame #4: 0x00007fff204a861c CoreFoundation`CFRunLoopRunSpecific + 563
    frame #5: 0x00007fff22e3623a AppKit`_NSEventThread + 124
    frame #6: 0x00007fff203b18fc libsystem_pthread.dylib`_pthread_start + 224
    frame #7: 0x00007fff203ad443 libsystem_pthread.dylib`thread_start + 15

Though I doubt it is the cause of your deadlock. This deadlock would
not happen frequently since vmnet_send will not be queued if the
device is not used.


Having two-three connected nics causes deadlock almost every
time. With only one nic it happens one time out of two or three.

An interesting thing is that using qemu_send_packet() instead
of qemu_send_packet_async() fixes the problem. Hope to
start investigating this next week.

Regards,
Akihiko Odaki

>
>
>> Thanks,
>> Roman
>>
>> > Signed-off-by: Vladislav Yaroshchuk <yaroshchuk2000@gmail.com>
>> > ---
>> >  net/clients.h       |  11 ++++
>> >  net/meson.build     |   7 +++
>> >  net/net.c           |  10 ++++
>> >  net/vmnet-bridged.m |  25 +++++++++
>> >  net/vmnet-common.m  |  20 +++++++
>> >  net/vmnet-host.c    |  24 ++++++++
>> >  net/vmnet-shared.c  |  25 +++++++++
>> >  net/vmnet_int.h     |  25 +++++++++
>> >  qapi/net.json       | 133 +++++++++++++++++++++++++++++++++++++++++++-
>> >  9 files changed, 278 insertions(+), 2 deletions(-)
>> >  create mode 100644 net/vmnet-bridged.m
>> >  create mode 100644 net/vmnet-common.m
>> >  create mode 100644 net/vmnet-host.c
>> >  create mode 100644 net/vmnet-shared.c
>> >  create mode 100644 net/vmnet_int.h
>> >
>> > diff --git a/net/net.c b/net/net.c
>> > index f0d14dbfc1..1dbb64b935 100644
>> > --- a/net/net.c
>> > +++ b/net/net.c
>> > @@ -1021,6 +1021,11 @@ static int (* const net_client_init_fun[NET_CLIENT_DRIVER__MAX])(
>> >  #ifdef CONFIG_L2TPV3
>> >          [NET_CLIENT_DRIVER_L2TPV3]    = net_init_l2tpv3,
>> >  #endif
>> > +#ifdef CONFIG_VMNET
>> > +        [NET_CLIENT_DRIVER_VMNET_HOST] = net_init_vmnet_host,
>> > +        [NET_CLIENT_DRIVER_VMNET_SHARED] = net_init_vmnet_shared,
>> > +        [NET_CLIENT_DRIVER_VMNET_BRIDGED] = net_init_vmnet_bridged,
>> > +#endif /* CONFIG_VMNET */
>> >  };
>> >
>> >
>> > @@ -1106,6 +1111,11 @@ void show_netdevs(void)
>> >  #endif
>> >  #ifdef CONFIG_VHOST_VDPA
>> >          "vhost-vdpa",
>> > +#endif
>> > +#ifdef CONFIG_VMNET
>> > +        "vmnet-host",
>> > +        "vmnet-shared",
>> > +        "vmnet-bridged",
>> >  #endif
>> >      };
>> >
>> > diff --git a/net/vmnet-bridged.m b/net/vmnet-bridged.m
>> > new file mode 100644
>> > index 0000000000..4e42a90391
>> > --- /dev/null
>> > +++ b/net/vmnet-bridged.m
>> > @@ -0,0 +1,25 @@
>> > +/*
>> > + * vmnet-bridged.m
>> > + *
>> > + * Copyright(c) 2021 Vladislav Yaroshchuk <yaroshchuk2000@gmail.com>
>> > + *
>> > + * This work is licensed under the terms of the GNU GPL, version 2 or later.
>> > + * See the COPYING file in the top-level directory.
>> > + *
>> > + */
>> > +
>> > +#include "qemu/osdep.h"
>> > +#include "qapi/qapi-types-net.h"
>> > +#include "vmnet_int.h"
>> > +#include "clients.h"
>> > +#include "qemu/error-report.h"
>> > +#include "qapi/error.h"
>> > +
>> > +#include <vmnet/vmnet.h>
>> > +
>> > +int net_init_vmnet_bridged(const Netdev *netdev, const char *name,
>> > +                           NetClientState *peer, Error **errp)
>> > +{
>> > +  error_setg(errp, "vmnet-bridged is not implemented yet");
>> > +  return -1;
>> > +}
>> > diff --git a/net/vmnet-common.m b/net/vmnet-common.m
>> > new file mode 100644
>> > index 0000000000..532d152840
>> > --- /dev/null
>> > +++ b/net/vmnet-common.m
>> > @@ -0,0 +1,20 @@
>> > +/*
>> > + * vmnet-common.m - network client wrapper for Apple vmnet.framework
>> > + *
>> > + * Copyright(c) 2021 Vladislav Yaroshchuk <yaroshchuk2000@gmail.com>
>> > + * Copyright(c) 2021 Phillip Tennen <phillip@axleos.com>
>> > + *
>> > + * This work is licensed under the terms of the GNU GPL, version 2 or later.
>> > + * See the COPYING file in the top-level directory.
>> > + *
>> > + */
>> > +
>> > +#include "qemu/osdep.h"
>> > +#include "qapi/qapi-types-net.h"
>> > +#include "vmnet_int.h"
>> > +#include "clients.h"
>> > +#include "qemu/error-report.h"
>> > +#include "qapi/error.h"
>> > +
>> > +#include <vmnet/vmnet.h>
>> > +
>> > diff --git a/net/vmnet-host.c b/net/vmnet-host.c
>> > new file mode 100644
>> > index 0000000000..4a5ef99dc7
>> > --- /dev/null
>> > +++ b/net/vmnet-host.c
>> > @@ -0,0 +1,24 @@
>> > +/*
>> > + * vmnet-host.c
>> > + *
>> > + * Copyright(c) 2021 Vladislav Yaroshchuk <yaroshchuk2000@gmail.com>
>> > + *
>> > + * This work is licensed under the terms of the GNU GPL, version 2 or later.
>> > + * See the COPYING file in the top-level directory.
>> > + *
>> > + */
>> > +
>> > +#include "qemu/osdep.h"
>> > +#include "qapi/qapi-types-net.h"
>> > +#include "vmnet_int.h"
>> > +#include "clients.h"
>> > +#include "qemu/error-report.h"
>> > +#include "qapi/error.h"
>> > +
>> > +#include <vmnet/vmnet.h>
>> > +
>> > +int net_init_vmnet_host(const Netdev *netdev, const char *name,
>> > +                        NetClientState *peer, Error **errp) {
>> > +  error_setg(errp, "vmnet-host is not implemented yet");
>> > +  return -1;
>> > +}
>> > diff --git a/net/vmnet-shared.c b/net/vmnet-shared.c
>> > new file mode 100644
>> > index 0000000000..f8c4a4f3b8
>> > --- /dev/null
>> > +++ b/net/vmnet-shared.c
>> > @@ -0,0 +1,25 @@
>> > +/*
>> > + * vmnet-shared.c
>> > + *
>> > + * Copyright(c) 2021 Vladislav Yaroshchuk <yaroshchuk2000@gmail.com>
>> > + *
>> > + * This work is licensed under the terms of the GNU GPL, version 2 or later.
>> > + * See the COPYING file in the top-level directory.
>> > + *
>> > + */
>> > +
>> > +#include "qemu/osdep.h"
>> > +#include "qapi/qapi-types-net.h"
>> > +#include "vmnet_int.h"
>> > +#include "clients.h"
>> > +#include "qemu/error-report.h"
>> > +#include "qapi/error.h"
>> > +
>> > +#include <vmnet/vmnet.h>
>> > +
>> > +int net_init_vmnet_shared(const Netdev *netdev, const char *name,
>> > +                          NetClientState *peer, Error **errp)
>> > +{
>> > +  error_setg(errp, "vmnet-shared is not implemented yet");
>> > +  return -1;
>> > +}
>> > diff --git a/net/vmnet_int.h b/net/vmnet_int.h
>> > new file mode 100644
>> > index 0000000000..c5982259a4
>> > --- /dev/null
>> > +++ b/net/vmnet_int.h
>> > @@ -0,0 +1,25 @@
>> > +/*
>> > + * vmnet_int.h
>> > + *
>> > + * Copyright(c) 2021 Vladislav Yaroshchuk <yaroshchuk2000@gmail.com>
>> > + *
>> > + * This work is licensed under the terms of the GNU GPL, version 2 or later.
>> > + * See the COPYING file in the top-level directory.
>> > + *
>> > + */
>> > +#ifndef VMNET_INT_H
>> > +#define VMNET_INT_H
>> > +
>> > +#include "qemu/osdep.h"
>> > +#include "vmnet_int.h"
>> > +#include "clients.h"
>> > +
>> > +#include <vmnet/vmnet.h>
>> > +
>> > +typedef struct VmnetCommonState {
>> > +  NetClientState nc;
>> > +
>> > +} VmnetCommonState;
>> > +
>> > +
>> > +#endif /* VMNET_INT_H */
>> > diff --git a/qapi/net.json b/qapi/net.json
>> > index 7fab2e7cd8..b922e2e34f 100644
>> > --- a/qapi/net.json
>> > +++ b/qapi/net.json
>> > @@ -452,6 +452,120 @@
>> >      '*vhostdev':     'str',
>> >      '*queues':       'int' } }
>> >
>> > +##
>> > +# @NetdevVmnetHostOptions:
>> > +#
>> > +# vmnet (host mode) network backend.
>> > +#
>> > +# Allows the vmnet interface to communicate with other vmnet
>> > +# interfaces that are in host mode and also with the host.
>> > +#
>> > +# @start-address: The starting IPv4 address to use for the interface.
>> > +#                 Must be in the private IP range (RFC 1918). Must be
>> > +#                 specified along with @end-address and @subnet-mask.
>> > +#                 This address is used as the gateway address. The
>> > +#                 subsequent address up to and including end-address are
>> > +#                 placed in the DHCP pool.
>> > +#
>> > +# @end-address: The DHCP IPv4 range end address to use for the
>> > +#               interface. Must be in the private IP range (RFC 1918).
>> > +#               Must be specified along with @start-address and
>> > +#               @subnet-mask.
>> > +#
>> > +# @subnet-mask: The IPv4 subnet mask to use on the interface. Must
>> > +#               be specified along with @start-address and @subnet-mask.
>> > +#
>> > +# @isolated: Enable isolation for this interface. Interface isolation
>> > +#            ensures that vmnet interface is not able to communicate
>> > +#            with any other vmnet interfaces. Only communication with
>> > +#            host is allowed. Available since macOS Big Sur 11.0.
>> > +#
>> > +# @net-uuid: The identifier (UUID) to uniquely identify the isolated
>> > +#            network vmnet interface should be added to. If
>> > +#            set, no DHCP service is provided for this interface and
>> > +#            network communication is allowed only with other interfaces
>> > +#            added to this network identified by the UUID. Available
>> > +#            since macOS Big Sur 11.0.
>> > +#
>> > +# Since: 7.0
>> > +##
>> > +{ 'struct': 'NetdevVmnetHostOptions',
>> > +  'data': {
>> > +    '*start-address': 'str',
>> > +    '*end-address':   'str',
>> > +    '*subnet-mask':   'str',
>> > +    '*isolated':      'bool',
>> > +    '*net-uuid':      'str' },
>> > +  'if': 'CONFIG_VMNET' }
>> > +
>> > +##
>> > +# @NetdevVmnetSharedOptions:
>> > +#
>> > +# vmnet (shared mode) network backend.
>> > +#
>> > +# Allows traffic originating from the vmnet interface to reach the
>> > +# Internet through a network address translator (NAT).
>> > +# The vmnet interface can communicate with the host and with
>> > +# other shared mode interfaces on the same subnet. If no DHCP
>> > +# settings, subnet mask and IPv6 prefix specified, the interface can
>> > +# communicate with any of other interfaces in shared mode.
>> > +#
>> > +# @start-address: The starting IPv4 address to use for the interface.
>> > +#                 Must be in the private IP range (RFC 1918). Must be
>> > +#                 specified along with @end-address and @subnet-mask.
>> > +#                 This address is used as the gateway address. The
>> > +#                 subsequent address up to and including end-address are
>> > +#                 placed in the DHCP pool.
>> > +#
>> > +# @end-address: The DHCP IPv4 range end address to use for the
>> > +#               interface. Must be in the private IP range (RFC 1918).
>> > +#               Must be specified along with @start-address and @subnet-mask.
>> > +#
>> > +# @subnet-mask: The IPv4 subnet mask to use on the interface. Must
>> > +#                be specified along with @start-address and @subnet-mask.
>> > +#
>> > +# @isolated: Enable isolation for this interface. Interface isolation
>> > +#            ensures that vmnet interface is not able to communicate
>> > +#            with any other vmnet interfaces. Only communication with
>> > +#            host is allowed. Available since macOS Big Sur 11.0.
>> > +#
>> > +# @nat66-prefix: The IPv6 prefix to use into guest network. Must be a
>> > +#                unique local address i.e. start with fd00::/8 and have
>> > +#                length of 64.
>> > +#
>> > +# Since: 7.0
>> > +##
>> > +{ 'struct': 'NetdevVmnetSharedOptions',
>> > +  'data': {
>> > +    '*start-address': 'str',
>> > +    '*end-address':   'str',
>> > +    '*subnet-mask':   'str',
>> > +    '*isolated':      'bool',
>> > +    '*nat66-prefix':  'str' },
>> > +  'if': 'CONFIG_VMNET' }
>> > +
>> > +##
>> > +# @NetdevVmnetBridgedOptions:
>> > +#
>> > +# vmnet (bridged mode) network backend.
>> > +#
>> > +# Bridges the vmnet interface with a physical network interface.
>> > +#
>> > +# @ifname: The name of the physical interface to be bridged.
>> > +#
>> > +# @isolated: Enable isolation for this interface. Interface isolation
>> > +#            ensures that vmnet interface is not able to communicate
>> > +#            with any other vmnet interfaces. Only communication with
>> > +#            host is allowed. Available since macOS Big Sur 11.0.
>> > +#
>> > +# Since: 7.0
>> > +##
>> > +{ 'struct': 'NetdevVmnetBridgedOptions',
>> > +  'data': {
>> > +    'ifname':     'str',
>> > +    '*isolated':  'bool' },
>> > +  'if': 'CONFIG_VMNET' }
>> > +
>> >  ##
>> >  # @NetClientDriver:
>> >  #
>> > @@ -460,10 +574,16 @@
>> >  # Since: 2.7
>> >  #
>> >  #        @vhost-vdpa since 5.1
>> > +#        @vmnet-host since 7.0
>> > +#        @vmnet-shared since 7.0
>> > +#        @vmnet-bridged since 7.0
>> >  ##
>> >  { 'enum': 'NetClientDriver',
>> >    'data': [ 'none', 'nic', 'user', 'tap', 'l2tpv3', 'socket', 'vde',
>> > -            'bridge', 'hubport', 'netmap', 'vhost-user', 'vhost-vdpa' ] }
>> > +            'bridge', 'hubport', 'netmap', 'vhost-user', 'vhost-vdpa',
>> > +            { 'name': 'vmnet-host', 'if': 'CONFIG_VMNET' },
>> > +            { 'name': 'vmnet-shared', 'if': 'CONFIG_VMNET' },
>> > +            { 'name': 'vmnet-bridged', 'if': 'CONFIG_VMNET' }] }
>> >
>> >  ##
>> >  # @Netdev:
>> > @@ -477,6 +597,9 @@
>> >  # Since: 1.2
>> >  #
>> >  #        'l2tpv3' - since 2.1
>> > +#        'vmnet-host' - since 7.0
>> > +#        'vmnet-shared' - since 7.0
>> > +#        'vmnet-bridged' - since 7.0
>> >  ##
>> >  { 'union': 'Netdev',
>> >    'base': { 'id': 'str', 'type': 'NetClientDriver' },
>> > @@ -492,7 +615,13 @@
>> >      'hubport':  'NetdevHubPortOptions',
>> >      'netmap':   'NetdevNetmapOptions',
>> >      'vhost-user': 'NetdevVhostUserOptions',
>> > -    'vhost-vdpa': 'NetdevVhostVDPAOptions' } }
>> > +    'vhost-vdpa': 'NetdevVhostVDPAOptions',
>> > +    'vmnet-host': { 'type': 'NetdevVmnetHostOptions',
>> > +                    'if': 'CONFIG_VMNET' },
>> > +    'vmnet-shared': { 'type': 'NetdevVmnetSharedOptions',
>> > +                      'if': 'CONFIG_VMNET' },
>> > +    'vmnet-bridged': { 'type': 'NetdevVmnetBridgedOptions',
>> > +                       'if': 'CONFIG_VMNET' } } }
>> >
>> >  ##
>> >  # @RxState:
>> > --
>> > 2.23.0
>> >
>
>
>
> --
> Best Regards,
>
> Vladislav Yaroshchuk


--
Best Regards,

Vladislav Yaroshchuk

reply via email to

[Prev in Thread] Current Thread [Next in Thread]