[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 0/5] Enable unix socket support on Windows
From: |
Bin Meng |
Subject: |
Re: [PATCH 0/5] Enable unix socket support on Windows |
Date: |
Wed, 27 Jul 2022 19:37:33 +0800 |
On Wed, Jul 27, 2022 at 6:24 PM Daniel P. Berrangé <berrange@redhat.com> wrote:
>
> On Wed, Jul 27, 2022 at 06:15:50PM +0800, Bin Meng wrote:
> > On Wed, Jul 27, 2022 at 5:06 PM Daniel P. Berrangé <berrange@redhat.com>
> > wrote:
> > >
> > > On Wed, Jul 27, 2022 at 03:35:37PM +0800, Bin Meng wrote:
> > > > Support for the unix socket has existed both in BSD and Linux for the
> > > > longest time, but not on Windows. Since Windows 10 build 17063 [1],
> > > > the native support for the unix socket has came to Windows. Starting
> > > > this build, two Win32 processes can use the AF_UNIX address family
> > > > over Winsock API to communicate with each other.
> > > >
> > > > Introduce a new build time config option CONFIG_AF_UNIX when the build
> > > > host has such a capability, and a run-time check afunix_available() for
> > > > Windows host in the QEMU sockets util codes.
> > > >
> > > > [1] https://devblogs.microsoft.com/commandline/af_unix-comes-to-windows/
> > > >
> > > >
> > > > Bin Meng (5):
> > > > util/qemu-sockets: Replace the call to close a socket with
> > > > closesocket()
> > > > util/oslib-win32: Add a helper to get the Windows version
> > > > qga/commands-win32: Use os_get_win_version()
> > > > util/qemu-sockets: Enable unix socket support on Windows
> > > > chardev/char-socket: Update AF_UNIX for Windows
> > > >
> > > > meson.build | 6 +++++
> > > > include/sysemu/os-win32.h | 2 ++
> > > > chardev/char-socket.c | 8 +++++-
> > > > qga/commands-win32.c | 27 +-------------------
> > > > util/oslib-win32.c | 15 +++++++++++
> > > > util/qemu-sockets.c | 52 ++++++++++++++++++++++++++++++++-------
> > > > 6 files changed, 74 insertions(+), 36 deletions(-)
> > >
> > > What about net/socket.c ?
> >
> > It looks net/socket.c does not need to adapt.
> >
> > > Also there are many tests using AF_UNIX and this doesn't appear to
> > > have enablede any of them. I'd at least exepct to see the sockets
> > > tests-io-channel-socket.c test enabled to prove this functionality
> > > is working.
> > >
> >
> > Enabling qtest to run on Windows is underway but that's a separate
> > topic. The qtest itself is using unix socket so as long as we can run
> > qtest on Windows, we should be fine.
> > I feel this series is independent enough of being a standalone one.
>
> That isn't qtest, that is basic unit tests. I would expect those to
> be able to work with this series
>
Ah, I see. Agreed, will do in v2.
Regards,
Bin
- Re: [PATCH 3/5] qga/commands-win32: Use os_get_win_version(), (continued)