Cover Letter Detail
Show a cover letter.
GET /api/covers/813233/?format=api
{ "id": 813233, "url": "http://patchwork.ozlabs.org/api/covers/813233/?format=api", "web_url": "http://patchwork.ozlabs.org/project/qemu-devel/cover/20170913090522.4022-1-Sergio.G.DelReal@gmail.com/", "project": { "id": 14, "url": "http://patchwork.ozlabs.org/api/projects/14/?format=api", "name": "QEMU Development", "link_name": "qemu-devel", "list_id": "qemu-devel.nongnu.org", "list_email": "qemu-devel@nongnu.org", "web_url": "", "scm_url": "", "webscm_url": "", "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20170913090522.4022-1-Sergio.G.DelReal@gmail.com>", "list_archive_url": null, "date": "2017-09-13T09:05:08", "name": "[v4,00/14] add support for Hypervisor.framework in QEMU", "submitter": { "id": 70675, "url": "http://patchwork.ozlabs.org/api/people/70675/?format=api", "name": "Sergio Andres Gomez Del Real", "email": "sergio.g.delreal@gmail.com" }, "mbox": "http://patchwork.ozlabs.org/project/qemu-devel/cover/20170913090522.4022-1-Sergio.G.DelReal@gmail.com/mbox/", "series": [ { "id": 2843, "url": "http://patchwork.ozlabs.org/api/series/2843/?format=api", "web_url": "http://patchwork.ozlabs.org/project/qemu-devel/list/?series=2843", "date": "2017-09-13T09:05:08", "name": "add support for Hypervisor.framework in QEMU", "version": 4, "mbox": "http://patchwork.ozlabs.org/series/2843/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/covers/813233/comments/", "headers": { "Return-Path": "<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>", "X-Original-To": "incoming@patchwork.ozlabs.org", "Delivered-To": "patchwork-incoming@bilbo.ozlabs.org", "Authentication-Results": [ "ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=nongnu.org\n\t(client-ip=2001:4830:134:3::11; helo=lists.gnu.org;\n\tenvelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n\treceiver=<UNKNOWN>)", "ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"U44//yFG\"; dkim-atps=neutral" ], "Received": [ "from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11])\n\t(using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xsbPB5kqWz9sMN\n\tfor <incoming@patchwork.ozlabs.org>;\n\tWed, 13 Sep 2017 19:06:05 +1000 (AEST)", "from localhost ([::1]:41053 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>)\n\tid 1ds3cP-0001Kx-FK\n\tfor incoming@patchwork.ozlabs.org; Wed, 13 Sep 2017 05:06:02 -0400", "from eggs.gnu.org ([2001:4830:134:3::10]:33811)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <sergio.g.delreal@gmail.com>) id 1ds3c5-0001Kl-D3\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 05:05:42 -0400", "from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from <sergio.g.delreal@gmail.com>) id 1ds3c1-0007EJ-W1\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 05:05:41 -0400", "from mail-ua0-x241.google.com ([2607:f8b0:400c:c08::241]:34363)\n\tby eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16)\n\t(Exim 4.71) (envelope-from <sergio.g.delreal@gmail.com>)\n\tid 1ds3c1-0007DK-Oy\n\tfor qemu-devel@nongnu.org; Wed, 13 Sep 2017 05:05:37 -0400", "by mail-ua0-x241.google.com with SMTP id c27so3900199uah.1\n\tfor <qemu-devel@nongnu.org>; Wed, 13 Sep 2017 02:05:36 -0700 (PDT)", "from localhost.localdomain ([190.66.154.128])\n\tby smtp.gmail.com with ESMTPSA id\n\tr74sm2677601vkf.19.2017.09.13.02.05.34\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tWed, 13 Sep 2017 02:05:35 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=from:to:cc:subject:date:message-id;\n\tbh=sfN/1hMM1pSwfn8ePYMHm9qSsqBRglrk1dOFFTWrQIk=;\n\tb=U44//yFGqlKztlDtd1FDZOG7c1DJjUAGV4+smI+G8WkRVnfwS8N3u9Ezw3iQkll3Pm\n\tgUucpntbB7fyaSJ7HSsl+DiX1VqzXDOxELQ98y+RGbG1Ct6nJ8JtKdG8L2EZg+j5gFOe\n\tCpd/4diH97jmzILIunGEfxrdBoPmXryyZN1JAWQnK5nZ4W5JsLREgZBVt5XO1tQN/cD9\n\tPQx1l63kmuijV57XjN7zKcrdV/BYAsGuQVF8OlQdF0e1iXXqUOy9pGIMkuUEG8EbU1Yy\n\t11MaiPiRWmV1XOPBC+3VyrkoYgUqnbkUNYlC4rDs9i4PSbUW9WvkXrCkTpvxVN//uL6b\n\tFjpg==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id;\n\tbh=sfN/1hMM1pSwfn8ePYMHm9qSsqBRglrk1dOFFTWrQIk=;\n\tb=bqLecCiH3ZjzhTrNw6hc+SZKb41owD81Cy0L7uIc3ZgYy8BtaFNqwxWcUL9IBIV1Ok\n\tfRvvOWkLurHWfMGh7O+8ycE5ZinmmbwEDzGs7s0lfMGRFka+gDhRWKfLIQ4xCaJtfGCo\n\t1JGOd6a9rjmfJgbYt2e9tk97fvRtW9segvJKG6E69517B1Cmc1c8Hpz26Zti3Dxcc/ar\n\tA1i0HUsQpLMUvn5AXdsw+7G3fG69T7oyV3uUh1ckCk+qewIf2iHjpdhpzrjaqVOf5KY6\n\tcBpVinSbtU7o8XHzbpnGQuQqI9xubVW148afTcZMgM0eL3fh0p7nf1QT/w7jbiovtgxX\n\teFnA==", "X-Gm-Message-State": "AHPjjUjpL2Zf1Ojwui90ajNa1PdcoJk3Arekag19tmcCKInI0V/x3XtU\n\tVDH92ecchrxSCxnT", "X-Google-Smtp-Source": "AOwi7QBogzW1DwI35vLEmScSALXs7mGUB/bh0XQsjUUncaQw4Chge2h4bL7zD73JyvWObBU0ya0APw==", "X-Received": "by 10.176.72.129 with SMTP id x1mr12973125uac.163.1505293535624; \n\tWed, 13 Sep 2017 02:05:35 -0700 (PDT)", "From": "Sergio Andres Gomez Del Real <sergio.g.delreal@gmail.com>", "X-Google-Original-From": "Sergio Andres Gomez Del Real\n\t<Sergio.G.DelReal@gmail.com>", "To": "qemu-devel@nongnu.org", "Date": "Wed, 13 Sep 2017 04:05:08 -0500", "Message-Id": "<20170913090522.4022-1-Sergio.G.DelReal@gmail.com>", "X-Mailer": "git-send-email 2.11.0", "X-detected-operating-system": "by eggs.gnu.org: Genre and OS details not\n\trecognized.", "X-Received-From": "2607:f8b0:400c:c08::241", "Subject": "[Qemu-devel] [PATCH v4 00/14] add support for Hypervisor.framework\n\tin QEMU", "X-BeenThere": "qemu-devel@nongnu.org", "X-Mailman-Version": "2.1.21", "Precedence": "list", "List-Id": "<qemu-devel.nongnu.org>", "List-Unsubscribe": "<https://lists.nongnu.org/mailman/options/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>", "List-Archive": "<http://lists.nongnu.org/archive/html/qemu-devel/>", "List-Post": "<mailto:qemu-devel@nongnu.org>", "List-Help": "<mailto:qemu-devel-request@nongnu.org?subject=help>", "List-Subscribe": "<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=subscribe>", "Cc": "Sergio Andres Gomez Del Real <Sergio.G.DelReal@gmail.com>,\n\tpbonzini@redhat.com, stefanha@gmail.com", "Errors-To": "qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org", "Sender": "\"Qemu-devel\"\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>" }, "content": "================\nChanges in v4:\n (1) Use g_new0 instead of g_malloc.\n (2) hvf TODO's in cpu.c explained: in-kernel irqchip.\n (3) Introduce ins_len field in patch where actually used.\n (4) Remove outdated comment \"/* Definition of hvf_state is here */\"\n (5) Treat vcpu_dirty as boolean throughout.\n (6) checkpatch.pl patch only does styling changes.\n (8) Add comment documenting meaning of -1 returned by\n apic_get_highest_priority_irr.\n (9) Add TODO to move hvf's synchronization functions inside\n cpu_synchronize_*, along with kvm's and hax's.\n================\n\n================\nChanges in v3:\n (1) Fixed licensing issues in patch 3.\n (2) Revert to late adding of compilation rules in Makefile.objs (patch 8/14);\n files aren't ready to compile earlier.\n (3) Make a single patch just for fixing style (patch 4/14).\n (4) Fix data type for hvf_fd field.\n (5) Add comment that return value of -1 in apic function added in 7/14 means\n \"no interrupt\".\n================\n\n================\nChanges in v2:\n (1) Removed legacy option \"-enable-hvf\" in favor of \"-M accel=hvf\"\n (2) Added missing copyright headers; replace fprintfs for error_report;\n improved commit description.\n (3) Moved patch that adds compilation rules in Makefile.objs right after\n the patch that adds the new files from Google's repo.\n (4) Removed conditional macros from cpus.c and cpu.c\n (5) Moved patch that fixes coding style to patch # 3\n (6) Fix commit message in apic patch\n (7) Squash some commits to avoid code churn\n================\n\nThe following patchset adds to QEMU the supporting for macOS's native\nhypervisor, Hypervisor.framework (hvf). The code base is taken from\nGoogle's Android emulator at\nhttps://android.googlesource.com/platform/external/qemu/+/emu-master-dev.\nNotably missing is support for live migration.\n\nApart from general code refactoring, some additional features were implemented:\nretrieve the set of features supported by host cpu and hvf (cpuid);\ndirty page tracking for VGA memory area; reimplementation of the event\ninjection mechanism to allow injection of exceptions during vmexits, which is\nexemplified by the injection of a GP fault when the guest vmexits due to\nexecution of the vmcall instruction; changing the emulator's use of CPUState\nstructure in favor of CPUX86State, so as to in the future remove data structures\nthat are uselessly specific to hvf and unified some of the state between kvm/tcg\nand hvf.\nSome features initially planned to implement that didn't make it include:\npage fault handling in the emulator and implementing the dummy_signal to handle\nthe SIG_IPI signal without race conditions. Hopefully these can be implemented\nin the near future.\nEvent injection was tested through the eventinj kvm test. The latest Debian and\nDamn Small Linux images were executed, although by now ways it was an\nexhaustive testing.\n\nSergio Andres Gomez Del Real (14):\n hvf: add support for Hypervisor.framework in the configure script\n hvf: add code base from Google's QEMU repository\n hvf: fix licensing issues; isolate task handling code (GPL v2-only)\n hvf: run hvf code through checkpatch.pl and fix style issues\n hvf: add code to cpus.c and do refactoring in preparation for\n compiling\n hvf: handle fields from CPUState and CPUX86State\n apic: add function to apic that will be used by hvf\n hvf: add compilation rules to Makefile.objs\n hvf: use new helper functions for put/get xsave\n hvf: implement hvf_get_supported_cpuid\n hvf: refactor cpuid code\n hvf: implement vga dirty page tracking\n hvf: refactor event injection code for hvf\n hvf: inject General Protection Fault when vmexit through vmcall\n\n configure | 38 +\n cpus.c | 89 ++\n hw/intc/apic.c | 12 +\n include/hw/i386/apic.h | 1 +\n include/qom/cpu.h | 2 +\n include/sysemu/hvf.h | 181 +++\n qemu-options.hx | 10 +-\n target/i386/Makefile.objs | 1 +\n target/i386/cpu-qom.h | 4 +-\n target/i386/cpu.c | 80 +-\n target/i386/cpu.h | 34 +-\n target/i386/hvf-all.c | 963 +++++++++++++++\n target/i386/hvf-i386.h | 50 +\n target/i386/hvf-utils/Makefile.objs | 1 +\n target/i386/hvf-utils/README.md | 7 +\n target/i386/hvf-utils/vmcs.h | 371 ++++++\n target/i386/hvf-utils/vmx.h | 222 ++++\n target/i386/hvf-utils/x86.c | 184 +++\n target/i386/hvf-utils/x86.h | 394 +++++++\n target/i386/hvf-utils/x86_cpuid.c | 417 +++++++\n target/i386/hvf-utils/x86_cpuid.h | 52 +\n target/i386/hvf-utils/x86_decode.c | 2186 +++++++++++++++++++++++++++++++++++\n target/i386/hvf-utils/x86_decode.h | 325 ++++++\n target/i386/hvf-utils/x86_descr.c | 124 ++\n target/i386/hvf-utils/x86_descr.h | 55 +\n target/i386/hvf-utils/x86_emu.c | 1536 ++++++++++++++++++++++++\n target/i386/hvf-utils/x86_emu.h | 49 +\n target/i386/hvf-utils/x86_flags.c | 333 ++++++\n target/i386/hvf-utils/x86_flags.h | 243 ++++\n target/i386/hvf-utils/x86_gen.h | 53 +\n target/i386/hvf-utils/x86_mmu.c | 273 +++++\n target/i386/hvf-utils/x86_mmu.h | 45 +\n target/i386/hvf-utils/x86_task.c | 201 ++++\n target/i386/hvf-utils/x86_task.h | 18 +\n target/i386/hvf-utils/x86hvf.c | 463 ++++++++\n target/i386/hvf-utils/x86hvf.h | 39 +\n target/i386/kvm.c | 2 -\n 37 files changed, 9025 insertions(+), 33 deletions(-)\n create mode 100644 include/sysemu/hvf.h\n create mode 100644 target/i386/hvf-all.c\n create mode 100644 target/i386/hvf-i386.h\n create mode 100644 target/i386/hvf-utils/Makefile.objs\n create mode 100644 target/i386/hvf-utils/README.md\n create mode 100644 target/i386/hvf-utils/vmcs.h\n create mode 100644 target/i386/hvf-utils/vmx.h\n create mode 100644 target/i386/hvf-utils/x86.c\n create mode 100644 target/i386/hvf-utils/x86.h\n create mode 100644 target/i386/hvf-utils/x86_cpuid.c\n create mode 100644 target/i386/hvf-utils/x86_cpuid.h\n create mode 100644 target/i386/hvf-utils/x86_decode.c\n create mode 100644 target/i386/hvf-utils/x86_decode.h\n create mode 100644 target/i386/hvf-utils/x86_descr.c\n create mode 100644 target/i386/hvf-utils/x86_descr.h\n create mode 100644 target/i386/hvf-utils/x86_emu.c\n create mode 100644 target/i386/hvf-utils/x86_emu.h\n create mode 100644 target/i386/hvf-utils/x86_flags.c\n create mode 100644 target/i386/hvf-utils/x86_flags.h\n create mode 100644 target/i386/hvf-utils/x86_gen.h\n create mode 100644 target/i386/hvf-utils/x86_mmu.c\n create mode 100644 target/i386/hvf-utils/x86_mmu.h\n create mode 100644 target/i386/hvf-utils/x86_task.c\n create mode 100644 target/i386/hvf-utils/x86_task.h\n create mode 100644 target/i386/hvf-utils/x86hvf.c\n create mode 100644 target/i386/hvf-utils/x86hvf.h" }