[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 16/36] multi-process: remote process initialization
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [PATCH v6 16/36] multi-process: remote process initialization |
Date: |
Thu, 9 Apr 2020 19:00:02 +0100 |
User-agent: |
Mutt/1.13.4 (2020-02-15) |
* address@hidden (address@hidden) wrote:
> From: Jagannathan Raman <address@hidden>
>
> Adds the handler to process message from QEMU,
> Initialize remote process main loop, handles SYNC_SYSMEM
> message by updating its "system_memory" container using
> shared file descriptors received from QEMU.
>
> Signed-off-by: John G Johnson <address@hidden>
> Signed-off-by: Elena Ufimtseva <address@hidden>
> Signed-off-by: Jagannathan Raman <address@hidden>
> ---
> remote/remote-main.c | 87 ++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 87 insertions(+)
>
> diff --git a/remote/remote-main.c b/remote/remote-main.c
> index ecf30e0cba..51595f3141 100644
> --- a/remote/remote-main.c
> +++ b/remote/remote-main.c
> @@ -12,6 +12,7 @@
> #include "qemu-common.h"
>
> #include <stdio.h>
> +#include <unistd.h>
>
> #include "qemu/module.h"
> #include "remote/pcihost.h"
> @@ -19,12 +20,98 @@
> #include "hw/boards.h"
> #include "hw/qdev-core.h"
> #include "qemu/main-loop.h"
> +#include "remote/memory.h"
> +#include "io/mpqemu-link.h"
> +#include "qapi/error.h"
> +#include "qemu/main-loop.h"
> +#include "sysemu/cpus.h"
> +#include "qemu-common.h"
> +#include "hw/pci/pci.h"
> +#include "qemu/thread.h"
> +#include "qemu/main-loop.h"
> +#include "qemu/config-file.h"
> +#include "sysemu/sysemu.h"
> +#include "block/block.h"
> +#include "exec/ramlist.h"
> +
> +static MPQemuLinkState *mpqemu_link;
> +
> +static void process_msg(GIOCondition cond, MPQemuLinkState *link,
> + MPQemuChannel *chan)
> +{
> + MPQemuMsg *msg = NULL;
> + Error *err = NULL;
> +
> + if ((cond & G_IO_HUP) || (cond & G_IO_ERR)) {
> + goto finalize_loop;
> + }
> +
> + msg = g_malloc0(sizeof(MPQemuMsg));
> +
> + if (mpqemu_msg_recv(msg, chan) < 0) {
> + error_setg(&err, "Failed to receive message");
> + goto finalize_loop;
> + }
> +
> + switch (msg->cmd) {
> + case INIT:
> + break;
> + default:
> + error_setg(&err, "Unknown command");
Again this doesn't seem to have changed since my 4th March review where
I asked for better error messages.
Dave
> + goto finalize_loop;
> + }
> +
> + g_free(msg->data2);
> + g_free(msg);
> +
> + return;
> +
> +finalize_loop:
> + if (err) {
> + error_report_err(err);
> + }
> + g_free(msg);
> + mpqemu_link_finalize(mpqemu_link);
> + mpqemu_link = NULL;
> +}
>
> int main(int argc, char *argv[])
> {
> + Error *err = NULL;
> +
> module_call_init(MODULE_INIT_QOM);
>
> + bdrv_init_with_whitelist();
> +
> + if (qemu_init_main_loop(&err)) {
> + error_report_err(err);
> + return -EBUSY;
> + }
> +
> + qemu_init_cpu_loop();
> +
> + page_size_init();
> +
> + qemu_mutex_init(&ram_list.mutex);
> +
> current_machine =
> MACHINE(REMOTE_MACHINE(object_new(TYPE_REMOTE_MACHINE)));
>
> + mpqemu_link = mpqemu_link_create();
> + if (!mpqemu_link) {
> + printf("Could not create MPQemu link\n");
> + return -1;
> + }
> +
> + mpqemu_init_channel(mpqemu_link, &mpqemu_link->com, STDIN_FILENO);
> +
> + mpqemu_link_set_callback(mpqemu_link, process_msg);
> +
> + qdev_machine_creation_done();
> + qemu_mutex_lock_iothread();
> + qemu_run_machine_init_done_notifiers();
> + qemu_mutex_unlock_iothread();
> +
> + mpqemu_start_coms(mpqemu_link, mpqemu_link->com);
> +
> return 0;
> }
> --
> 2.25.GIT
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
- [PATCH v6 01/36] memory: alloc RAM from file at offset, (continued)
- [PATCH v6 01/36] memory: alloc RAM from file at offset, elena . ufimtseva, 2020/04/06
- [PATCH v6 08/36] multi-process: Add stub functions to facilate build of multi-process, elena . ufimtseva, 2020/04/06
- [PATCH v6 07/36] multi-process: add a command line option for debug file, elena . ufimtseva, 2020/04/06
- [PATCH v6 10/36] multi-process: build system for remote device process, elena . ufimtseva, 2020/04/06
- [PATCH v6 12/36] multi-process: add functions to synchronize proxy and remote endpoints, elena . ufimtseva, 2020/04/06
- [PATCH v6 15/36] multi-process: setup memory manager for remote device, elena . ufimtseva, 2020/04/06
- [PATCH v6 17/36] multi-process: introduce proxy object, elena . ufimtseva, 2020/04/06
- [PATCH v6 16/36] multi-process: remote process initialization, elena . ufimtseva, 2020/04/06
- Re: [PATCH v6 16/36] multi-process: remote process initialization,
Dr. David Alan Gilbert <=
- [PATCH v6 14/36] multi-process: setup a machine object for remote device process, elena . ufimtseva, 2020/04/06
- [PATCH v6 21/36] multi-process: PCI BAR read/write handling for proxy & remote endpoints, elena . ufimtseva, 2020/04/06
- [PATCH v6 18/36] multi-process: Initialize Proxy Object's communication channel, elena . ufimtseva, 2020/04/06
- [PATCH v6 13/36] multi-process: setup PCI host bridge for remote device, elena . ufimtseva, 2020/04/06
- [PATCH v6 19/36] multi-process: Connect Proxy Object with device in the remote process, elena . ufimtseva, 2020/04/06
- [PATCH v6 22/36] multi-process: Synchronize remote memory, elena . ufimtseva, 2020/04/06
- [PATCH v6 26/36] multi-process: add parse_cmdline in remote process, elena . ufimtseva, 2020/04/06
- [PATCH v6 20/36] multi-process: Forward PCI config space acceses to the remote process, elena . ufimtseva, 2020/04/06