MAINTAINERS | 5 + hw/net/vmxnet3.c | 4 +- meson.build | 4 + meson_options.txt | 2 + net/clients.h | 11 ++ net/colo-compare.c | 28 ++-- net/filter-mirror.c | 66 +++++++-- net/meson.build | 7 + net/net.c | 10 ++ net/tap-linux.c | 1 + net/tap.c | 1 + net/vmnet-bridged.m | 111 ++++++++++++++ net/vmnet-common.m | 330 ++++++++++++++++++++++++++++++++++++++++++ net/vmnet-host.c | 105 ++++++++++++++ net/vmnet-shared.c | 91 ++++++++++++ net/vmnet_int.h | 48 ++++++ qapi/net.json | 132 ++++++++++++++++- qemu-options.hx | 25 ++++ scripts/meson-buildoptions.sh | 3 + 19 files changed, 954 insertions(+), 30 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
The following changes since commit df722e33d5da26ea8604500ca8f509245a0ea524:
Merge tag 'bsd-user-arm-pull-request' of gitlab.com:bsdimp/qemu into staging (2022-01-08 09:37:59 -0800)
are available in the git repository at:
https://github.com/jasowang/qemu.git tags/net-pull-request
for you to fetch changes up to 5136cc6d3b8b74f4fa572f0874656947a401330e:
net/vmnet: update MAINTAINERS list (2022-01-10 11:30:55 +0800)
----------------------------------------------------------------
----------------------------------------------------------------
Peter Foley (2):
net/tap: Set return code on failure
net: Fix uninitialized data usage
Philippe Mathieu-Daudé (1):
hw/net/vmxnet3: Log guest-triggerable errors using LOG_GUEST_ERROR
Rao Lei (1):
net/filter: Optimize filter_send to coroutine
Vladislav Yaroshchuk (7):
net/vmnet: add vmnet dependency and customizable option
net/vmnet: add vmnet backends to qapi/net
net/vmnet: implement shared mode (vmnet-shared)
net/vmnet: implement host mode (vmnet-host)
net/vmnet: implement bridged mode (vmnet-bridged)
net/vmnet: update qemu-options.hx
net/vmnet: update MAINTAINERS list
Zhang Chen (2):
net/colo-compare.c: Optimize compare order for performance
net/colo-compare.c: Update the default value comments
MAINTAINERS | 5 +
hw/net/vmxnet3.c | 4 +-
meson.build | 4 +
meson_options.txt | 2 +
net/clients.h | 11 ++
net/colo-compare.c | 28 ++--
net/filter-mirror.c | 66 +++++++--
net/meson.build | 7 +
net/net.c | 10 ++
net/tap-linux.c | 1 +
net/tap.c | 1 +
net/vmnet-bridged.m | 111 ++++++++++++++
net/vmnet-common.m | 330 ++++++++++++++++++++++++++++++++++++++++++
net/vmnet-host.c | 105 ++++++++++++++
net/vmnet-shared.c | 91 ++++++++++++
net/vmnet_int.h | 48 ++++++
qapi/net.json | 132 ++++++++++++++++-
qemu-options.hx | 25 ++++
scripts/meson-buildoptions.sh | 3 +
19 files changed, 954 insertions(+), 30 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
On Mon, 10 Jan 2022 at 03:40, Jason Wang <jasowang@redhat.com> wrote:
>
> The following changes since commit df722e33d5da26ea8604500ca8f509245a0ea524:
>
> Merge tag 'bsd-user-arm-pull-request' of gitlab.com:bsdimp/qemu into staging (2022-01-08 09:37:59 -0800)
>
> are available in the git repository at:
>
> https://github.com/jasowang/qemu.git tags/net-pull-request
>
> for you to fetch changes up to 5136cc6d3b8b74f4fa572f0874656947a401330e:
>
> net/vmnet: update MAINTAINERS list (2022-01-10 11:30:55 +0800)
>
> ----------------------------------------------------------------
>
> ----------------------------------------------------------------
Fails to build on OSX Catalina:
../../net/vmnet-common.m:165:10: error: use of undeclared identifier
'VMNET_SHARING_SERVICE_BUSY'
case VMNET_SHARING_SERVICE_BUSY:
^
This constant only got added in macOS 11.0. I guess that technically
our supported-platforms policy only requires us to support 11 (Big Sur)
and 12 (Monterey) at this point, but it would be nice to still be able
to build on Catalina (10.15).
(Personally I would like Catalina still to work at least for a little
while, because my x86 Mac is old enough that it is not supported by
Big Sur. I'll have to dump it once Apple stops doing security support
for Catalina, but they haven't done that quite yet.)
-- PMM
On Tue, Jan 11, 2022 at 12:49 AM Peter Maydell <peter.maydell@linaro.org> wrote: > > On Mon, 10 Jan 2022 at 03:40, Jason Wang <jasowang@redhat.com> wrote: > > > > The following changes since commit df722e33d5da26ea8604500ca8f509245a0ea524: > > > > Merge tag 'bsd-user-arm-pull-request' of gitlab.com:bsdimp/qemu into staging (2022-01-08 09:37:59 -0800) > > > > are available in the git repository at: > > > > https://github.com/jasowang/qemu.git tags/net-pull-request > > > > for you to fetch changes up to 5136cc6d3b8b74f4fa572f0874656947a401330e: > > > > net/vmnet: update MAINTAINERS list (2022-01-10 11:30:55 +0800) > > > > ---------------------------------------------------------------- > > > > ---------------------------------------------------------------- > > Fails to build on OSX Catalina: > > ../../net/vmnet-common.m:165:10: error: use of undeclared identifier > 'VMNET_SHARING_SERVICE_BUSY' > case VMNET_SHARING_SERVICE_BUSY: > ^ > > This constant only got added in macOS 11.0. I guess that technically > our supported-platforms policy only requires us to support 11 (Big Sur) > and 12 (Monterey) at this point, but it would be nice to still be able > to build on Catalina (10.15). Yes, it was only supported by the vmnet framework starting from Catalyst according to https://developer.apple.com/documentation/vmnet?language=objc. > > (Personally I would like Catalina still to work at least for a little > while, because my x86 Mac is old enough that it is not supported by > Big Sur. I'll have to dump it once Apple stops doing security support > for Catalina, but they haven't done that quite yet.) Sure, Vladislav please fix this and send a new version. Thanks > > -- PMM >
вт, 11 янв. 2022 г., 5:10 AM Jason Wang <jasowang@redhat.com>: > On Tue, Jan 11, 2022 at 12:49 AM Peter Maydell <peter.maydell@linaro.org> > wrote: > > > > On Mon, 10 Jan 2022 at 03:40, Jason Wang <jasowang@redhat.com> wrote: > > > > > > The following changes since commit > df722e33d5da26ea8604500ca8f509245a0ea524: > > > > > > Merge tag 'bsd-user-arm-pull-request' of gitlab.com:bsdimp/qemu > into staging (2022-01-08 09:37:59 -0800) > > > > > > are available in the git repository at: > > > > > > https://github.com/jasowang/qemu.git tags/net-pull-request > > > > > > for you to fetch changes up to > 5136cc6d3b8b74f4fa572f0874656947a401330e: > > > > > > net/vmnet: update MAINTAINERS list (2022-01-10 11:30:55 +0800) > > > > > > ---------------------------------------------------------------- > > > > > > ---------------------------------------------------------------- > > > > Fails to build on OSX Catalina: > > > > ../../net/vmnet-common.m:165:10: error: use of undeclared identifier > > 'VMNET_SHARING_SERVICE_BUSY' > > case VMNET_SHARING_SERVICE_BUSY: > > ^ > > > > This constant only got added in macOS 11.0. I guess that technically > > our supported-platforms policy only requires us to support 11 (Big Sur) > > and 12 (Monterey) at this point, but it would be nice to still be able > > to build on Catalina (10.15). > Yes, it was only supported by the vmnet framework starting from > Catalyst according to > https://developer.apple.com/documentation/vmnet?language=objc. > > Yes, there are some symbols from macOS >= 11.0 new backend uses, not only this one, ex. vmnet_enable_isolation_key: https://developer.apple.com/documentation/vmnet/vmnet_enable_isolation_key > > > (Personally I would like Catalina still to work at least for a little > > while, because my x86 Mac is old enough that it is not supported by > > Big Sur. I'll have to dump it once Apple stops doing security support > > for Catalina, but they haven't done that quite yet.) > > Sure, broken builds on old macOSes are bad. For this case I think it's enough to disable vmnet for macOS < 11.0 with a probe while configure build step. Especially given that Apple supports ~three latest macOS versions, support for Catalina is expected to end in 2022, when QEMU releases 7.0. If this workaround is not suitable and it's required to support vmnet in Catalina 10.15 with a subset of available features, it can be done. But I'll be ready to handle this in approximately two-three weeks only. > Sure, Vladislav please fix this and send a new version. > > Quick fix as described above is available in v10: https://patchew.org/QEMU/20220111211422.21789-1-yaroshchuk2000@gmail.com/ > Thanks > > > > > -- PMM > > > > > -- Best Regards, Vladislav Yaroshchuk
在 2022/1/12 上午6:02, Vladislav Yaroshchuk 写道: > > > вт, 11 янв. 2022 г., 5:10 AM Jason Wang <jasowang@redhat.com>: > > On Tue, Jan 11, 2022 at 12:49 AM Peter Maydell > <peter.maydell@linaro.org> wrote: > > > > On Mon, 10 Jan 2022 at 03:40, Jason Wang <jasowang@redhat.com> > wrote: > > > > > > The following changes since commit > df722e33d5da26ea8604500ca8f509245a0ea524: > > > > > > Merge tag 'bsd-user-arm-pull-request' of > gitlab.com:bsdimp/qemu into staging (2022-01-08 09:37:59 -0800) > > > > > > are available in the git repository at: > > > > > > https://github.com/jasowang/qemu.git tags/net-pull-request > > > > > > for you to fetch changes up to > 5136cc6d3b8b74f4fa572f0874656947a401330e: > > > > > > net/vmnet: update MAINTAINERS list (2022-01-10 11:30:55 +0800) > > > > > > ---------------------------------------------------------------- > > > > > > ---------------------------------------------------------------- > > > > Fails to build on OSX Catalina: > > > > ../../net/vmnet-common.m:165:10: error: use of undeclared identifier > > 'VMNET_SHARING_SERVICE_BUSY' > > case VMNET_SHARING_SERVICE_BUSY: > > ^ > > > > This constant only got added in macOS 11.0. I guess that technically > > our supported-platforms policy only requires us to support 11 > (Big Sur) > > and 12 (Monterey) at this point, but it would be nice to still > be able > > to build on Catalina (10.15). > > Yes, it was only supported by the vmnet framework starting from > Catalyst according to > https://developer.apple.com/documentation/vmnet?language=objc. > > > Yes, there are some symbols from macOS >= 11.0 new backend > uses, not only this one, ex. vmnet_enable_isolation_key: > https://developer.apple.com/documentation/vmnet/vmnet_enable_isolation_key > > > > > (Personally I would like Catalina still to work at least for a > little > > while, because my x86 Mac is old enough that it is not supported by > > Big Sur. I'll have to dump it once Apple stops doing security > support > > for Catalina, but they haven't done that quite yet.) > > > Sure, broken builds on old macOSes are bad. For this case I think > it's enough to disable vmnet for macOS < 11.0 with a probe while > configure build step. Especially given that Apple supports ~three > latest macOS versions, support for Catalina is expected to end > in 2022, when QEMU releases 7.0. That should be fine. > > If this workaround is not suitable and it's required to support vmnet > in Catalina 10.15 with a subset of available features, it can be done. > But I'll be ready to handle this in approximately two-three weeks only. > > Sure, Vladislav please fix this and send a new version. > > > Quick fix as described above is available in v10: > https://patchew.org/QEMU/20220111211422.21789-1-yaroshchuk2000@gmail.com/ Have you got chance to test that for macOS < 11.0? Thanks > Thanks > > > > > -- PMM > > > > > > > -- > Best Regards, > > Vladislav Yaroshchuk
ср, 12 янв. 2022 г., 8:39 AM Jason Wang <jasowang@redhat.com>: > > 在 2022/1/12 上午6:02, Vladislav Yaroshchuk 写道: > > > > > > вт, 11 янв. 2022 г., 5:10 AM Jason Wang <jasowang@redhat.com>: > > > > On Tue, Jan 11, 2022 at 12:49 AM Peter Maydell > > <peter.maydell@linaro.org> wrote: > > > > > > On Mon, 10 Jan 2022 at 03:40, Jason Wang <jasowang@redhat.com> > > wrote: > > > > > > > > The following changes since commit > > df722e33d5da26ea8604500ca8f509245a0ea524: > > > > > > > > Merge tag 'bsd-user-arm-pull-request' of > > gitlab.com:bsdimp/qemu into staging (2022-01-08 09:37:59 -0800) > > > > > > > > are available in the git repository at: > > > > > > > > https://github.com/jasowang/qemu.git tags/net-pull-request > > > > > > > > for you to fetch changes up to > > 5136cc6d3b8b74f4fa572f0874656947a401330e: > > > > > > > > net/vmnet: update MAINTAINERS list (2022-01-10 11:30:55 +0800) > > > > > > > > ---------------------------------------------------------------- > > > > > > > > ---------------------------------------------------------------- > > > > > > Fails to build on OSX Catalina: > > > > > > ../../net/vmnet-common.m:165:10: error: use of undeclared > identifier > > > 'VMNET_SHARING_SERVICE_BUSY' > > > case VMNET_SHARING_SERVICE_BUSY: > > > ^ > > > > > > This constant only got added in macOS 11.0. I guess that > technically > > > our supported-platforms policy only requires us to support 11 > > (Big Sur) > > > and 12 (Monterey) at this point, but it would be nice to still > > be able > > > to build on Catalina (10.15). > > > > Yes, it was only supported by the vmnet framework starting from > > Catalyst according to > > https://developer.apple.com/documentation/vmnet?language=objc. > > > > > > Yes, there are some symbols from macOS >= 11.0 new backend > > uses, not only this one, ex. vmnet_enable_isolation_key: > > > https://developer.apple.com/documentation/vmnet/vmnet_enable_isolation_key > > > > > > > > (Personally I would like Catalina still to work at least for a > > little > > > while, because my x86 Mac is old enough that it is not supported by > > > Big Sur. I'll have to dump it once Apple stops doing security > > support > > > for Catalina, but they haven't done that quite yet.) > > > > > > Sure, broken builds on old macOSes are bad. For this case I think > > it's enough to disable vmnet for macOS < 11.0 with a probe while > > configure build step. Especially given that Apple supports ~three > > latest macOS versions, support for Catalina is expected to end > > in 2022, when QEMU releases 7.0. > > > That should be fine. > > > > > > If this workaround is not suitable and it's required to support vmnet > > in Catalina 10.15 with a subset of available features, it can be done. > > But I'll be ready to handle this in approximately two-three weeks only. > > > > Sure, Vladislav please fix this and send a new version. > > > > > > Quick fix as described above is available in v10: > > > https://patchew.org/QEMU/20220111211422.21789-1-yaroshchuk2000@gmail.com/ > > > Have you got chance to test that for macOS < 11.0? > Yes, tested on Catalina 10.15.Works as expected. Thanks > > > > Thanks > > > > > > > > -- PMM > > > > > > > > > > > > > -- > > Best Regards, > > > > Vladislav Yaroshchuk > >
On Wed, Jan 12, 2022 at 2:19 PM Vladislav Yaroshchuk <yaroshchuk2000@gmail.com> wrote: > > > > ср, 12 янв. 2022 г., 8:39 AM Jason Wang <jasowang@redhat.com>: >> >> >> 在 2022/1/12 上午6:02, Vladislav Yaroshchuk 写道: >> > >> > >> > вт, 11 янв. 2022 г., 5:10 AM Jason Wang <jasowang@redhat.com>: >> > >> > On Tue, Jan 11, 2022 at 12:49 AM Peter Maydell >> > <peter.maydell@linaro.org> wrote: >> > > >> > > On Mon, 10 Jan 2022 at 03:40, Jason Wang <jasowang@redhat.com> >> > wrote: >> > > > >> > > > The following changes since commit >> > df722e33d5da26ea8604500ca8f509245a0ea524: >> > > > >> > > > Merge tag 'bsd-user-arm-pull-request' of >> > gitlab.com:bsdimp/qemu into staging (2022-01-08 09:37:59 -0800) >> > > > >> > > > are available in the git repository at: >> > > > >> > > > https://github.com/jasowang/qemu.git tags/net-pull-request >> > > > >> > > > for you to fetch changes up to >> > 5136cc6d3b8b74f4fa572f0874656947a401330e: >> > > > >> > > > net/vmnet: update MAINTAINERS list (2022-01-10 11:30:55 +0800) >> > > > >> > > > ---------------------------------------------------------------- >> > > > >> > > > ---------------------------------------------------------------- >> > > >> > > Fails to build on OSX Catalina: >> > > >> > > ../../net/vmnet-common.m:165:10: error: use of undeclared identifier >> > > 'VMNET_SHARING_SERVICE_BUSY' >> > > case VMNET_SHARING_SERVICE_BUSY: >> > > ^ >> > > >> > > This constant only got added in macOS 11.0. I guess that technically >> > > our supported-platforms policy only requires us to support 11 >> > (Big Sur) >> > > and 12 (Monterey) at this point, but it would be nice to still >> > be able >> > > to build on Catalina (10.15). >> > >> > Yes, it was only supported by the vmnet framework starting from >> > Catalyst according to >> > https://developer.apple.com/documentation/vmnet?language=objc. >> > >> > >> > Yes, there are some symbols from macOS >= 11.0 new backend >> > uses, not only this one, ex. vmnet_enable_isolation_key: >> > https://developer.apple.com/documentation/vmnet/vmnet_enable_isolation_key >> > >> > > >> > > (Personally I would like Catalina still to work at least for a >> > little >> > > while, because my x86 Mac is old enough that it is not supported by >> > > Big Sur. I'll have to dump it once Apple stops doing security >> > support >> > > for Catalina, but they haven't done that quite yet.) >> > >> > >> > Sure, broken builds on old macOSes are bad. For this case I think >> > it's enough to disable vmnet for macOS < 11.0 with a probe while >> > configure build step. Especially given that Apple supports ~three >> > latest macOS versions, support for Catalina is expected to end >> > in 2022, when QEMU releases 7.0. >> >> >> That should be fine. >> >> >> > >> > If this workaround is not suitable and it's required to support vmnet >> > in Catalina 10.15 with a subset of available features, it can be done. >> > But I'll be ready to handle this in approximately two-three weeks only. >> > >> > Sure, Vladislav please fix this and send a new version. >> > >> > >> > Quick fix as described above is available in v10: >> > https://patchew.org/QEMU/20220111211422.21789-1-yaroshchuk2000@gmail.com/ >> >> >> Have you got chance to test that for macOS < 11.0? > > > Yes, tested on Catalina 10.15.Works as expected. Cool. Thanks > >> Thanks >> >> >> > Thanks >> > >> > > >> > > -- PMM >> > > >> > >> > >> > >> > >> > -- >> > Best Regards, >> > >> > Vladislav Yaroshchuk >> >> >>
On Wed, Jan 12, 2022 at 01:39:28PM +0800, Jason Wang wrote: > > 在 2022/1/12 上午6:02, Vladislav Yaroshchuk 写道: > > > > > > вт, 11 янв. 2022 г., 5:10 AM Jason Wang <jasowang@redhat.com>: > > > > On Tue, Jan 11, 2022 at 12:49 AM Peter Maydell > > <peter.maydell@linaro.org> wrote: > > > > > > On Mon, 10 Jan 2022 at 03:40, Jason Wang <jasowang@redhat.com> > > wrote: > > > > > > > > The following changes since commit > > df722e33d5da26ea8604500ca8f509245a0ea524: > > > > > > > > Merge tag 'bsd-user-arm-pull-request' of > > gitlab.com:bsdimp/qemu into staging (2022-01-08 09:37:59 -0800) > > > > > > > > are available in the git repository at: > > > > > > > > https://github.com/jasowang/qemu.git tags/net-pull-request > > > > > > > > for you to fetch changes up to > > 5136cc6d3b8b74f4fa572f0874656947a401330e: > > > > > > > > net/vmnet: update MAINTAINERS list (2022-01-10 11:30:55 +0800) > > > > > > > > ---------------------------------------------------------------- > > > > > > > > ---------------------------------------------------------------- > > > > > > Fails to build on OSX Catalina: > > > > > > ../../net/vmnet-common.m:165:10: error: use of undeclared identifier > > > 'VMNET_SHARING_SERVICE_BUSY' > > > case VMNET_SHARING_SERVICE_BUSY: > > > ^ > > > > > > This constant only got added in macOS 11.0. I guess that technically > > > our supported-platforms policy only requires us to support 11 > > (Big Sur) > > > and 12 (Monterey) at this point, but it would be nice to still > > be able > > > to build on Catalina (10.15). > > > > Yes, it was only supported by the vmnet framework starting from > > Catalyst according to > > https://developer.apple.com/documentation/vmnet?language=objc. > > > > > > Yes, there are some symbols from macOS >= 11.0 new backend > > uses, not only this one, ex. vmnet_enable_isolation_key: > > https://developer.apple.com/documentation/vmnet/vmnet_enable_isolation_key > > > > > > > > (Personally I would like Catalina still to work at least for a > > little > > > while, because my x86 Mac is old enough that it is not supported by > > > Big Sur. I'll have to dump it once Apple stops doing security > > support > > > for Catalina, but they haven't done that quite yet.) > > > > > > Sure, broken builds on old macOSes are bad. For this case I think > > it's enough to disable vmnet for macOS < 11.0 with a probe while > > configure build step. Especially given that Apple supports ~three > > latest macOS versions, support for Catalina is expected to end > > in 2022, when QEMU releases 7.0. > > > That should be fine. > I agree with Peter on this, There's a lot of hardware running with Catalina. I think it's useful to support it a little longer. Regards, Roman > > > > > If this workaround is not suitable and it's required to support vmnet > > in Catalina 10.15 with a subset of available features, it can be done. > > But I'll be ready to handle this in approximately two-three weeks only. > > > > Sure, Vladislav please fix this and send a new version. > > > > > > Quick fix as described above is available in v10: > > https://patchew.org/QEMU/20220111211422.21789-1-yaroshchuk2000@gmail.com/ > > > Have you got chance to test that for macOS < 11.0? > > Thanks > > > > Thanks > > > > > > > > -- PMM > > > > > > > > > > > > > -- > > Best Regards, > > > > Vladislav Yaroshchuk > >
On Wed, Jan 12, 2022 at 3:10 PM Roman Bolshakov <roman@roolebo.dev> wrote: > > On Wed, Jan 12, 2022 at 01:39:28PM +0800, Jason Wang wrote: > > > > 在 2022/1/12 上午6:02, Vladislav Yaroshchuk 写道: > > > > > > > > > вт, 11 янв. 2022 г., 5:10 AM Jason Wang <jasowang@redhat.com>: > > > > > > On Tue, Jan 11, 2022 at 12:49 AM Peter Maydell > > > <peter.maydell@linaro.org> wrote: > > > > > > > > On Mon, 10 Jan 2022 at 03:40, Jason Wang <jasowang@redhat.com> > > > wrote: > > > > > > > > > > The following changes since commit > > > df722e33d5da26ea8604500ca8f509245a0ea524: > > > > > > > > > > Merge tag 'bsd-user-arm-pull-request' of > > > gitlab.com:bsdimp/qemu into staging (2022-01-08 09:37:59 -0800) > > > > > > > > > > are available in the git repository at: > > > > > > > > > > https://github.com/jasowang/qemu.git tags/net-pull-request > > > > > > > > > > for you to fetch changes up to > > > 5136cc6d3b8b74f4fa572f0874656947a401330e: > > > > > > > > > > net/vmnet: update MAINTAINERS list (2022-01-10 11:30:55 +0800) > > > > > > > > > > ---------------------------------------------------------------- > > > > > > > > > > ---------------------------------------------------------------- > > > > > > > > Fails to build on OSX Catalina: > > > > > > > > ../../net/vmnet-common.m:165:10: error: use of undeclared identifier > > > > 'VMNET_SHARING_SERVICE_BUSY' > > > > case VMNET_SHARING_SERVICE_BUSY: > > > > ^ > > > > > > > > This constant only got added in macOS 11.0. I guess that technically > > > > our supported-platforms policy only requires us to support 11 > > > (Big Sur) > > > > and 12 (Monterey) at this point, but it would be nice to still > > > be able > > > > to build on Catalina (10.15). > > > > > > Yes, it was only supported by the vmnet framework starting from > > > Catalyst according to > > > https://developer.apple.com/documentation/vmnet?language=objc. > > > > > > > > > Yes, there are some symbols from macOS >= 11.0 new backend > > > uses, not only this one, ex. vmnet_enable_isolation_key: > > > https://developer.apple.com/documentation/vmnet/vmnet_enable_isolation_key > > > > > > > > > > > (Personally I would like Catalina still to work at least for a > > > little > > > > while, because my x86 Mac is old enough that it is not supported by > > > > Big Sur. I'll have to dump it once Apple stops doing security > > > support > > > > for Catalina, but they haven't done that quite yet.) > > > > > > > > > Sure, broken builds on old macOSes are bad. For this case I think > > > it's enough to disable vmnet for macOS < 11.0 with a probe while > > > configure build step. Especially given that Apple supports ~three > > > latest macOS versions, support for Catalina is expected to end > > > in 2022, when QEMU releases 7.0. > > > > > > That should be fine. > > > > I agree with Peter on this, > > There's a lot of hardware running with Catalina. I think it's useful to > support it a little longer. Right and Vladislav have disabled vmnet on the old versions. Thanks > > Regards, > Roman > > > > > > > > > If this workaround is not suitable and it's required to support vmnet > > > in Catalina 10.15 with a subset of available features, it can be done. > > > But I'll be ready to handle this in approximately two-three weeks only. > > > > > > Sure, Vladislav please fix this and send a new version. > > > > > > > > > Quick fix as described above is available in v10: > > > https://patchew.org/QEMU/20220111211422.21789-1-yaroshchuk2000@gmail.com/ > > > > > > Have you got chance to test that for macOS < 11.0? > > > > Thanks > > > > > > > Thanks > > > > > > > > > > > -- PMM > > > > > > > > > > > > > > > > > > > -- > > > Best Regards, > > > > > > Vladislav Yaroshchuk > > > > >
ср, 12 янв. 2022 г. в 10:51, Jason Wang <jasowang@redhat.com>: > On Wed, Jan 12, 2022 at 3:10 PM Roman Bolshakov <roman@roolebo.dev> wrote: > > > > On Wed, Jan 12, 2022 at 01:39:28PM +0800, Jason Wang wrote: > > > > > > 在 2022/1/12 上午6:02, Vladislav Yaroshchuk 写道: > > > > > > > > > > > > вт, 11 янв. 2022 г., 5:10 AM Jason Wang <jasowang@redhat.com>: > > > > > > > > On Tue, Jan 11, 2022 at 12:49 AM Peter Maydell > > > > <peter.maydell@linaro.org> wrote: > > > > > > > > > > On Mon, 10 Jan 2022 at 03:40, Jason Wang <jasowang@redhat.com> > > > > wrote: > > > > > > > > > > > > The following changes since commit > > > > df722e33d5da26ea8604500ca8f509245a0ea524: > > > > > > > > > > > > Merge tag 'bsd-user-arm-pull-request' of > > > > gitlab.com:bsdimp/qemu into staging (2022-01-08 09:37:59 -0800) > > > > > > > > > > > > are available in the git repository at: > > > > > > > > > > > > https://github.com/jasowang/qemu.git tags/net-pull-request > > > > > > > > > > > > for you to fetch changes up to > > > > 5136cc6d3b8b74f4fa572f0874656947a401330e: > > > > > > > > > > > > net/vmnet: update MAINTAINERS list (2022-01-10 11:30:55 > +0800) > > > > > > > > > > > > > ---------------------------------------------------------------- > > > > > > > > > > > > > ---------------------------------------------------------------- > > > > > > > > > > Fails to build on OSX Catalina: > > > > > > > > > > ../../net/vmnet-common.m:165:10: error: use of undeclared > identifier > > > > > 'VMNET_SHARING_SERVICE_BUSY' > > > > > case VMNET_SHARING_SERVICE_BUSY: > > > > > ^ > > > > > > > > > > This constant only got added in macOS 11.0. I guess that > technically > > > > > our supported-platforms policy only requires us to support 11 > > > > (Big Sur) > > > > > and 12 (Monterey) at this point, but it would be nice to still > > > > be able > > > > > to build on Catalina (10.15). > > > > > > > > Yes, it was only supported by the vmnet framework starting from > > > > Catalyst according to > > > > https://developer.apple.com/documentation/vmnet?language=objc. > > > > > > > > > > > > Yes, there are some symbols from macOS >= 11.0 new backend > > > > uses, not only this one, ex. vmnet_enable_isolation_key: > > > > > https://developer.apple.com/documentation/vmnet/vmnet_enable_isolation_key > > > > > > > > > > > > > > (Personally I would like Catalina still to work at least for a > > > > little > > > > > while, because my x86 Mac is old enough that it is not > supported by > > > > > Big Sur. I'll have to dump it once Apple stops doing security > > > > support > > > > > for Catalina, but they haven't done that quite yet.) > > > > > > > > > > > > Sure, broken builds on old macOSes are bad. For this case I think > > > > it's enough to disable vmnet for macOS < 11.0 with a probe while > > > > configure build step. Especially given that Apple supports ~three > > > > latest macOS versions, support for Catalina is expected to end > > > > in 2022, when QEMU releases 7.0. > > > > > > > > > That should be fine. > > > > > > > I agree with Peter on this, > > > > There's a lot of hardware running with Catalina. I think it's useful to > > support it a little longer. > > Right and Vladislav have disabled vmnet on the old versions. > > Thanks > > Roman requested vmnet support for Catalina (10.15) also: https://patchew.org/QEMU/20220111211422.21789-1-yaroshchuk2000@gmail.com/ After some thought I also found it important to support 10.15. I've found a free hour to update patches and submit as v11 (still not displayed on Patchew for some reason). Tested on Catalina 10.15 and Big Sur 11.5. Built with no errors under Ubuntu 20.04 5.4.0-94-generic. > > > Regards, > > Roman > > > > > > > > > > > > > If this workaround is not suitable and it's required to support vmnet > > > > in Catalina 10.15 with a subset of available features, it can be > done. > > > > But I'll be ready to handle this in approximately two-three weeks > only. > > > > > > > > Sure, Vladislav please fix this and send a new version. > > > > > > > > > > > > Quick fix as described above is available in v10: > > > > > https://patchew.org/QEMU/20220111211422.21789-1-yaroshchuk2000@gmail.com/ > > > > > > > > > Have you got chance to test that for macOS < 11.0? > > > > > > Thanks > > > > > > > > > > Thanks > > > > > > > > > > > > > > -- PMM > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > Best Regards, > > > > > > > > Vladislav Yaroshchuk > > > > > > > > > > -- Best Regards, Vladislav Yaroshchuk
© 2016 - 2024 Red Hat, Inc.