All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jag Raman <jag.raman@oracle.com>
To: luoyonggang@gmail.com
Cc: "Elena Ufimtseva" <elena.ufimtseva@oracle.com>,
	"Fam Zheng" <fam@euphon.net>,
	"Swapnil Ingle" <swapnil.ingle@nutanix.com>,
	"John G Johnson" <john.g.johnson@oracle.com>,
	qemu-level <qemu-devel@nongnu.org>,
	"Gerd Hoffmann" <kraxel@redhat.com>,
	"Juan Quintela" <quintela@redhat.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	"Markus Armbruster" <armbru@redhat.com>,
	kanth.ghatraju@oracle.com,
	"Felipe Franciosi" <felipe@nutanix.com>,
	"Thomas Huth" <thuth@redhat.com>,
	"Eduardo Habkost" <ehabkost@redhat.com>,
	konrad.wilk@oracle.com,
	"Dr . David Alan Gilbert" <dgilbert@redhat.com>,
	alex.williamson@redhat.com,
	"Stefan Hajnoczi" <stefanha@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Kevin Wolf" <kwolf@redhat.com>,
	"\"Daniel P. Berrangé\"" <berrange@redhat.com>,
	"Max Reitz" <mreitz@redhat.com>,
	ross.lagerwall@citrix.com,
	"Marc-André Lureau" <marcandre.lureau@gmail.com>,
	thanos.makatos@nutanix.com
Subject: Re: [PATCH v16 00/20] Initial support for multi-process Qemu
Date: Wed, 13 Jan 2021 11:45:55 -0500	[thread overview]
Message-ID: <5920F8E4-3B63-49B0-9936-94689AA373B4@oracle.com> (raw)
In-Reply-To: <CAE2XoE_A=mX418VyL_Lxt=0+sZ0YY1RpK6yNWEJQdZ+52tndRQ@mail.gmail.com>



> On Jan 11, 2021, at 3:13 PM, 罗勇刚(Yonggang Luo) <luoyonggang@gmail.com> wrote:
> 
> 
> 
> On Tue, Jan 12, 2021 at 2:02 AM Jag Raman <jag.raman@oracle.com> wrote:
> >
> >
> >
> > > On Jan 11, 2021, at 12:20 AM, 罗勇刚(Yonggang Luo) <luoyonggang@gmail.com> wrote:
> > >
> > > I have a question, does this support/test on Windows?
> >
> > Hi Yonggang, we have not tested this on Windows. The project only builds on Linux for now.
> >
> > Thank you!
> I may interested add Windows support, what's the hard part?

Hi Yonggang,

I think it would be the unix sockets and sharing file descriptors via them.

Thanks!

> If we add Windows/OSX support, we can create same code base for all three platforms, and reduce #ifdef _WIN32 like macros
> 
> 
> > —
> > Jag
> >
> > >
> > > On Mon, Jan 11, 2021 at 1:08 PM Jagannathan Raman <jag.raman@oracle.com> wrote:
> > > >
> > > > Hi
> > > >
> > > > This is the v16 of the patchset. Thank you for your time reviewing v15.
> > > >
> > > > This version has the following changes:
> > > >
> > > > [PATCH v16 04/20] multi-process: Add config option for multi-process QEMU
> > > >   - Using “default_feature” value to enable/disable multiprocess
> > > >
> > > > [PATCH v16 07/20] io: add qio_channel_writev_full_all helper
> > > >   - Removed local variable in qio_channel_writev_full_all(), setting arguments
> > > >     directly
> > > >   - Fixed indentation issues
> > > >   - Updated commit message
> > > >
> > > > [PATCH v16 08/20] io: add qio_channel_readv_full_all_eof &
> > > >                   qio_channel_readv_full_all helpers
> > > >   - Added two variants of readv - _full_all_eof & _full_all based on feedback
> > > >   - Dropped errno return value
> > > >   - Updated commit message
> > > >   - Unable to remove local variables and set arguments directly as the
> > > >     arguments are later needed for cleanup (g_free/close) during failure
> > > >
> > > > Switched to using OBJECT_DECLARE_{SIMPLE_TYPE, TYPE} macros in the
> > > > following patches:
> > > >   - [PATCH v16 05/20] multi-process: setup PCI host bridge for remote device
> > > >   - [PATCH v16 06/20] multi-process: setup a machine object for remote device
> > > >                       process
> > > >   - [PATCH v16 11/20] multi-process: Associate fd of a PCIDevice with its object
> > > >   - [PATCH v16 13/20] multi-process: introduce proxy object
> > > >
> > > > Updated copyright text to use the year 2021 in the files that show them.
> > > >
> > > > To touch upon the history of this project, we posted the Proof Of Concept
> > > > patches before the BoF session in 2018. Subsequently, we have posted 15
> > > > versions on the qemu-devel mailing list. You can find them by following
> > > > the links below ([1] - [15]). Following people contributed to the design and
> > > > implementation of this project:
> > > > Jagannathan Raman <jag.raman@oracle.com>
> > > > Elena Ufimtseva <elena.ufimtseva@oracle.com>
> > > > John G Johnson <john.g.johnson@oracle.com>
> > > > Stefan Hajnoczi <stefanha@redhat.com>
> > > > Konrad Wilk <konrad.wilk@oracle.com>
> > > > Kanth Ghatraju <kanth.ghatraju@oracle.com>
> > > >
> > > > We would like to thank the QEMU community for your feedback in the
> > > > design and implementation of this project. Qemu wiki page:
> > > > https://wiki.qemu.org/Features/MultiProcessQEMU
> > > >
> > > > For the full concept writeup about QEMU multi-process, please
> > > > refer to docs/devel/qemu-multiprocess.rst. Also, see
> > > > docs/qemu-multiprocess.txt for usage information.
> > > >
> > > > Thank you for reviewing this series!
> > > >
> > > > [POC]: https://www.mail-archive.com/qemu-devel@nongnu.org/msg566538.html
> > > > [1]: https://www.mail-archive.com/qemu-devel@nongnu.org/msg602285.html
> > > > [2]: https://www.mail-archive.com/qemu-devel@nongnu.org/msg624877.html
> > > > [3]: https://www.mail-archive.com/qemu-devel@nongnu.org/msg642000.html
> > > > [4]: https://www.mail-archive.com/qemu-devel@nongnu.org/msg655118.html
> > > > [5]: https://www.mail-archive.com/qemu-devel@nongnu.org/msg682429.html
> > > > [6]: https://www.mail-archive.com/qemu-devel@nongnu.org/msg697484.html
> > > > [7]: https://patchew.org/QEMU/cover.1593273671.git.elena.ufimtseva@oracle.com/
> > > > [8]: https://www.mail-archive.com/qemu-devel@nongnu.org/msg727007.html
> > > > [9]: https://www.mail-archive.com/qemu-devel@nongnu.org/msg734275.html
> > > > [10]: https://www.mail-archive.com/qemu-devel@nongnu.org/msg747638.html
> > > > [11]: https://www.mail-archive.com/qemu-devel@nongnu.org/msg750972.html
> > > > [12]: https://patchew.org/QEMU/cover.1606853298.git.jag.raman@oracle.com/
> > > > [13]: https://www.mail-archive.com/qemu-devel@nongnu.org/msg766825.html
> > > > [14]: https://www.mail-archive.com/qemu-devel@nongnu.org/msg768376.html
> > > > [15]: https://www.mail-archive.com/qemu-devel@nongnu.org/msg769178.html
> > > >
> > > > Elena Ufimtseva (8):
> > > >   multi-process: add configure and usage information
> > > >   io: add qio_channel_writev_full_all helper
> > > >   io: add qio_channel_readv_full_all_eof & qio_channel_readv_full_all
> > > >     helpers
> > > >   multi-process: define MPQemuMsg format and transmission functions
> > > >   multi-process: introduce proxy object
> > > >   multi-process: add proxy communication functions
> > > >   multi-process: Forward PCI config space acceses to the remote process
> > > >   multi-process: perform device reset in the remote process
> > > >
> > > > Jagannathan Raman (11):
> > > >   memory: alloc RAM from file at offset
> > > >   multi-process: Add config option for multi-process QEMU
> > > >   multi-process: setup PCI host bridge for remote device
> > > >   multi-process: setup a machine object for remote device process
> > > >   multi-process: Initialize message handler in remote device
> > > >   multi-process: Associate fd of a PCIDevice with its object
> > > >   multi-process: setup memory manager for remote device
> > > >   multi-process: PCI BAR read/write handling for proxy & remote
> > > >     endpoints
> > > >   multi-process: Synchronize remote memory
> > > >   multi-process: create IOHUB object to handle irq
> > > >   multi-process: Retrieve PCI info from remote process
> > > >
> > > > John G Johnson (1):
> > > >   multi-process: add the concept description to
> > > >     docs/devel/qemu-multiprocess
> > > >
> > > >  docs/devel/index.rst                      |   1 +
> > > >  docs/devel/multi-process.rst              | 966 ++++++++++++++++++++++++++++++
> > > >  docs/multi-process.rst                    |  64 ++
> > > >  configure                                 |  10 +
> > > >  meson.build                               |   5 +-
> > > >  hw/remote/trace.h                         |   1 +
> > > >  include/exec/memory.h                     |   2 +
> > > >  include/exec/ram_addr.h                   |   2 +-
> > > >  include/hw/pci-host/remote.h              |  30 +
> > > >  include/hw/pci/pci_ids.h                  |   3 +
> > > >  include/hw/remote/iohub.h                 |  42 ++
> > > >  include/hw/remote/machine.h               |  38 ++
> > > >  include/hw/remote/memory.h                |  19 +
> > > >  include/hw/remote/mpqemu-link.h           |  99 +++
> > > >  include/hw/remote/proxy-memory-listener.h |  28 +
> > > >  include/hw/remote/proxy.h                 |  48 ++
> > > >  include/io/channel.h                      |  76 +++
> > > >  include/qemu/mmap-alloc.h                 |   4 +-
> > > >  include/sysemu/iothread.h                 |   6 +
> > > >  backends/hostmem-memfd.c                  |   2 +-
> > > >  hw/misc/ivshmem.c                         |   3 +-
> > > >  hw/pci-host/remote.c                      |  75 +++
> > > >  hw/remote/iohub.c                         | 119 ++++
> > > >  hw/remote/machine.c                       |  80 +++
> > > >  hw/remote/memory.c                        |  65 ++
> > > >  hw/remote/message.c                       | 230 +++++++
> > > >  hw/remote/mpqemu-link.c                   | 267 +++++++++
> > > >  hw/remote/proxy-memory-listener.c         | 227 +++++++
> > > >  hw/remote/proxy.c                         | 379 ++++++++++++
> > > >  hw/remote/remote-obj.c                    | 203 +++++++
> > > >  io/channel.c                              |  88 ++-
> > > >  iothread.c                                |   6 +
> > > >  softmmu/memory.c                          |   3 +-
> > > >  softmmu/physmem.c                         |  11 +-
> > > >  util/mmap-alloc.c                         |   7 +-
> > > >  util/oslib-posix.c                        |   2 +-
> > > >  Kconfig.host                              |   4 +
> > > >  MAINTAINERS                               |  24 +
> > > >  hw/Kconfig                                |   1 +
> > > >  hw/meson.build                            |   1 +
> > > >  hw/pci-host/Kconfig                       |   3 +
> > > >  hw/pci-host/meson.build                   |   1 +
> > > >  hw/remote/Kconfig                         |   4 +
> > > >  hw/remote/meson.build                     |  13 +
> > > >  hw/remote/trace-events                    |   4 +
> > > >  45 files changed, 3234 insertions(+), 32 deletions(-)
> > > >  create mode 100644 docs/devel/multi-process.rst
> > > >  create mode 100644 docs/multi-process.rst
> > > >  create mode 100644 hw/remote/trace.h
> > > >  create mode 100644 include/hw/pci-host/remote.h
> > > >  create mode 100644 include/hw/remote/iohub.h
> > > >  create mode 100644 include/hw/remote/machine.h
> > > >  create mode 100644 include/hw/remote/memory.h
> > > >  create mode 100644 include/hw/remote/mpqemu-link.h
> > > >  create mode 100644 include/hw/remote/proxy-memory-listener.h
> > > >  create mode 100644 include/hw/remote/proxy.h
> > > >  create mode 100644 hw/pci-host/remote.c
> > > >  create mode 100644 hw/remote/iohub.c
> > > >  create mode 100644 hw/remote/machine.c
> > > >  create mode 100644 hw/remote/memory.c
> > > >  create mode 100644 hw/remote/message.c
> > > >  create mode 100644 hw/remote/mpqemu-link.c
> > > >  create mode 100644 hw/remote/proxy-memory-listener.c
> > > >  create mode 100644 hw/remote/proxy.c
> > > >  create mode 100644 hw/remote/remote-obj.c
> > > >  create mode 100644 hw/remote/Kconfig
> > > >  create mode 100644 hw/remote/meson.build
> > > >  create mode 100644 hw/remote/trace-events
> > > >
> > > > --
> > > > 1.8.3.1
> > > >
> > > >
> > >
> > >
> > > --
> > >          此致
> > > 礼
> > > 罗勇刚
> > > Yours
> > >     sincerely,
> > > Yonggang Luo
> >
> 
> 
> --
>          此致
> 礼
> 罗勇刚
> Yours
>     sincerely,
> Yonggang Luo



  reply	other threads:[~2021-01-13 16:57 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-11  5:05 [PATCH v16 00/20] Initial support for multi-process Qemu Jagannathan Raman
2021-01-11  5:05 ` [PATCH v16 01/20] multi-process: add the concept description to docs/devel/qemu-multiprocess Jagannathan Raman
2021-01-11  5:05 ` [PATCH v16 02/20] multi-process: add configure and usage information Jagannathan Raman
2021-01-11  5:05 ` [PATCH v16 03/20] memory: alloc RAM from file at offset Jagannathan Raman
2021-01-11  5:05 ` [PATCH v16 04/20] multi-process: Add config option for multi-process QEMU Jagannathan Raman
2021-01-11  5:05 ` [PATCH v16 05/20] multi-process: setup PCI host bridge for remote device Jagannathan Raman
2021-01-11  5:05 ` [PATCH v16 06/20] multi-process: setup a machine object for remote device process Jagannathan Raman
2021-01-11  5:05 ` [PATCH v16 07/20] io: add qio_channel_writev_full_all helper Jagannathan Raman
2021-01-11  5:05 ` [PATCH v16 08/20] io: add qio_channel_readv_full_all_eof & qio_channel_readv_full_all helpers Jagannathan Raman
2021-01-12 11:15   ` Stefan Hajnoczi
2021-01-11  5:05 ` [PATCH v16 09/20] multi-process: define MPQemuMsg format and transmission functions Jagannathan Raman
2021-01-11  5:05 ` [PATCH v16 10/20] multi-process: Initialize message handler in remote device Jagannathan Raman
2021-01-11  5:05 ` [PATCH v16 11/20] multi-process: Associate fd of a PCIDevice with its object Jagannathan Raman
2021-01-11  5:05 ` [PATCH v16 12/20] multi-process: setup memory manager for remote device Jagannathan Raman
2021-01-11  5:05 ` [PATCH v16 13/20] multi-process: introduce proxy object Jagannathan Raman
2021-01-11  5:05 ` [PATCH v16 14/20] multi-process: add proxy communication functions Jagannathan Raman
2021-01-11  5:06 ` [PATCH v16 15/20] multi-process: Forward PCI config space acceses to the remote process Jagannathan Raman
2021-01-11  5:06 ` [PATCH v16 16/20] multi-process: PCI BAR read/write handling for proxy & remote endpoints Jagannathan Raman
2021-01-11  5:06 ` [PATCH v16 17/20] multi-process: Synchronize remote memory Jagannathan Raman
2021-01-11  5:06 ` [PATCH v16 18/20] multi-process: create IOHUB object to handle irq Jagannathan Raman
2021-01-11  5:06 ` [PATCH v16 19/20] multi-process: Retrieve PCI info from remote process Jagannathan Raman
2021-01-11  5:06 ` [PATCH v16 20/20] multi-process: perform device reset in the " Jagannathan Raman
2021-01-11  5:20 ` [PATCH v16 00/20] Initial support for multi-process Qemu 罗勇刚(Yonggang Luo)
2021-01-11 18:02   ` Jag Raman
2021-01-11 20:13     ` 罗勇刚(Yonggang Luo)
2021-01-13 16:45       ` Jag Raman [this message]
2021-01-12 11:17 ` Stefan Hajnoczi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5920F8E4-3B63-49B0-9936-94689AA373B4@oracle.com \
    --to=jag.raman@oracle.com \
    --cc=alex.williamson@redhat.com \
    --cc=armbru@redhat.com \
    --cc=berrange@redhat.com \
    --cc=dgilbert@redhat.com \
    --cc=ehabkost@redhat.com \
    --cc=elena.ufimtseva@oracle.com \
    --cc=fam@euphon.net \
    --cc=felipe@nutanix.com \
    --cc=john.g.johnson@oracle.com \
    --cc=kanth.ghatraju@oracle.com \
    --cc=konrad.wilk@oracle.com \
    --cc=kraxel@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=luoyonggang@gmail.com \
    --cc=marcandre.lureau@gmail.com \
    --cc=mreitz@redhat.com \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=quintela@redhat.com \
    --cc=ross.lagerwall@citrix.com \
    --cc=stefanha@redhat.com \
    --cc=swapnil.ingle@nutanix.com \
    --cc=thanos.makatos@nutanix.com \
    --cc=thuth@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.