Cover Letter Detail
Show a cover letter.
GET /api/covers/811138/?format=api
{ "id": 811138, "url": "http://patchwork.ozlabs.org/api/covers/811138/?format=api", "web_url": "http://patchwork.ozlabs.org/project/qemu-devel/cover/20170907201335.13956-1-david@redhat.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": "<20170907201335.13956-1-david@redhat.com>", "list_archive_url": null, "date": "2017-09-07T20:13:14", "name": "[v3,00/21] s390x cleanups and CPU hotplug via device_add", "submitter": { "id": 70402, "url": "http://patchwork.ozlabs.org/api/people/70402/?format=api", "name": "David Hildenbrand", "email": "david@redhat.com" }, "mbox": "http://patchwork.ozlabs.org/project/qemu-devel/cover/20170907201335.13956-1-david@redhat.com/mbox/", "series": [ { "id": 2053, "url": "http://patchwork.ozlabs.org/api/series/2053/?format=api", "web_url": "http://patchwork.ozlabs.org/project/qemu-devel/list/?series=2053", "date": "2017-09-07T20:13:14", "name": "s390x cleanups and CPU hotplug via device_add", "version": 3, "mbox": "http://patchwork.ozlabs.org/series/2053/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/covers/811138/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>)", "ext-mx10.extmail.prod.ext.phx2.redhat.com;\n\tdmarc=none (p=none dis=none) header.from=redhat.com", "ext-mx10.extmail.prod.ext.phx2.redhat.com;\n\tspf=fail smtp.mailfrom=david@redhat.com" ], "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 3xpBWQ1DK7z9sBW\n\tfor <incoming@patchwork.ozlabs.org>;\n\tFri, 8 Sep 2017 06:14:40 +1000 (AEST)", "from localhost ([::1]:42093 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 1dq3C9-00040I-Ka\n\tfor incoming@patchwork.ozlabs.org; Thu, 07 Sep 2017 16:14:37 -0400", "from eggs.gnu.org ([2001:4830:134:3::10]:38748)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <david@redhat.com>) id 1dq3BN-0003zp-9a\n\tfor qemu-devel@nongnu.org; Thu, 07 Sep 2017 16:13:54 -0400", "from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from <david@redhat.com>) id 1dq3BI-0001zr-Bh\n\tfor qemu-devel@nongnu.org; Thu, 07 Sep 2017 16:13:49 -0400", "from mx1.redhat.com ([209.132.183.28]:42458)\n\tby eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32)\n\t(Exim 4.71) (envelope-from <david@redhat.com>) id 1dq3BI-0001z4-1o\n\tfor qemu-devel@nongnu.org; Thu, 07 Sep 2017 16:13:44 -0400", "from smtp.corp.redhat.com\n\t(int-mx06.intmail.prod.int.phx2.redhat.com [10.5.11.16])\n\t(using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby mx1.redhat.com (Postfix) with ESMTPS id D75505F7AE;\n\tThu, 7 Sep 2017 20:13:42 +0000 (UTC)", "from t460s.redhat.com (ovpn-116-60.ams2.redhat.com [10.36.116.60])\n\tby smtp.corp.redhat.com (Postfix) with ESMTP id 5409C5C88F;\n\tThu, 7 Sep 2017 20:13:36 +0000 (UTC)" ], "DMARC-Filter": "OpenDMARC Filter v1.3.2 mx1.redhat.com D75505F7AE", "From": "David Hildenbrand <david@redhat.com>", "To": "qemu-devel@nongnu.org", "Date": "Thu, 7 Sep 2017 22:13:14 +0200", "Message-Id": "<20170907201335.13956-1-david@redhat.com>", "X-Scanned-By": "MIMEDefang 2.79 on 10.5.11.16", "X-Greylist": "Sender IP whitelisted, not delayed by milter-greylist-4.5.16\n\t(mx1.redhat.com [10.5.110.39]);\n\tThu, 07 Sep 2017 20:13:43 +0000 (UTC)", "X-detected-operating-system": "by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic]\n\t[fuzzy]", "X-Received-From": "209.132.183.28", "Subject": "[Qemu-devel] [PATCH v3 00/21] s390x cleanups and CPU hotplug via\n\tdevice_add", "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": "Matthew Rosato <mjrosato@linux.vnet.ibm.com>, thuth@redhat.com,\n\tEduardo Habkost <ehabkost@redhat.com>, david@redhat.com,\n\tcohuck@redhat.com, Richard Henderson <richard.henderson@linaro.org>,\n\tAlexander Graf <agraf@suse.de>, borntraeger@de.ibm.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": "The first patches are a bunch of cleanups. I decided to go the\nextra mile and implement CPU hotplug via \"device_add\", as well as\n\"query-hotpluggable-cpus\".\n\nOn s390x, only complete cores can be plugged. CPU hot unplug is currently\nnot supported by the architecture.\n\nIn contrast to v2, this series now also adds support for hotplugging CPUs\nin random core-id order (last two patches). We once hat a KVM bug\npreventing this, but the stable patch should now be included in relevant\nplaces - 152e9f65d66f (\"KVM: s390: fix wrong lookup of VCPUs by array\nindex\"). Current tooling will plug them in sequential order anyway, so\nit should not hurt.\n\nI am pretty sure now that this patch series won't grow even more :)\n\n\nv2 -> v3:\n* \"exec,dump,i386,ppc,s390x: don't include exec/cpu-all.h explicitly\"\n - dropped removal of one unrelated include\n* \"s390x: move subsystem_reset() to s390-virtio-ccw.h\"\n - previously \"s390x: move two function declarations to s390-virtio-ccw.h\"\n - as Thomas requested, s390_cpu_addr2state() will be later moved to cpu.c\n* \"s390x: move sclp_service_call() to sclp.h\"\n - previously \"s390x: move sclp_service_call() to interrupt.c\"\n - only move the declaration and make it compile\n* \"s390x: allow only 1 CPU with TCG\"\n - switch to simple ifdef. Let's see what people think about this one :)\n* \"s390x: print CPU definitions in sorted order\"\n - fix host model getting reported as static :(\n* \"s390x: CPU hot unplug via device_del cannot work\"\n - use the term \"currently\" instead of \"never\"\n* \"s390x: implement query-hotpluggable-cpus\"\n - as requested by Thomas, moving s390_cpu_addr2state() to cpu.c\n* added \"s390x: generate sclp cpu information from possible_cpus\"\n* added \"s390x: allow CPU hotplug in random core-id order\"\n\n\nDavid Hildenbrand (21):\n exec,dump,i386,ppc,s390x: don't include exec/cpu-all.h explicitly\n cpu: drop old comments describing members\n s390x: get rid of s390-virtio.c\n s390x: rename s390-virtio.h to s390-virtio-hcall.h\n target/s390x: move typedef of S390CPU to its definition\n s390x: move s390_virtio_hypercall() to s390-virtio-hcall.h\n s390x: move subsystem_reset() to s390-virtio-ccw.h\n s390x: move sclp_service_call() to sclp.h\n target/s390x: use trigger_pgm_exception() in\n s390_cpu_handle_mmu_fault()\n target/s390x: use program_interrupt() in per_check_exception()\n s390x: allow only 1 CPU with TCG\n target/s390x: set cpu->id for linux user when realizing\n target/s390x: use \"core-id\" for cpu number/address/id handling\n target/s390x: rename next_cpu_id to next_core_id\n s390x: print CPU definitions in sorted order\n s390x: allow cpu hotplug via device_add\n s390x: CPU hot unplug via device_del cannot work for now\n s390x: implement query-hotpluggable-cpus\n s390x: get rid of cpu_s390x_create()\n s390x: generate sclp cpu information from possible_cpus\n s390x: allow CPU hotplug in random core-id order\n\n dump.c | 1 -\n exec.c | 1 -\n hw/s390x/Makefile.objs | 1 -\n hw/s390x/s390-virtio-ccw.c | 245 +++++++++++++++++++++++++++++++++++--\n hw/s390x/s390-virtio-hcall.c | 2 +-\n hw/s390x/s390-virtio-hcall.h | 21 ++++\n hw/s390x/s390-virtio.c | 37 ------\n hw/s390x/s390-virtio.h | 35 ------\n hw/s390x/sclp.c | 34 +++--\n include/hw/s390x/s390-virtio-ccw.h | 2 +\n include/hw/s390x/sclp.h | 2 +\n include/qom/cpu.h | 6 +-\n qapi-schema.json | 16 +++\n target/i386/arch_dump.c | 1 -\n target/i386/arch_memory_mapping.c | 1 -\n target/i386/svm_helper.c | 1 -\n target/ppc/arch_dump.c | 1 -\n target/s390x/arch_dump.c | 1 -\n target/s390x/cpu-qom.h | 4 +-\n target/s390x/cpu.c | 101 ++++++---------\n target/s390x/cpu.h | 17 +--\n target/s390x/cpu_models.c | 58 ++++++---\n target/s390x/diag.c | 1 +\n target/s390x/excp_helper.c | 5 +-\n target/s390x/helper.c | 47 +------\n target/s390x/internal.h | 1 -\n target/s390x/kvm.c | 1 +\n target/s390x/misc_helper.c | 21 ++--\n target/s390x/translate.c | 5 +-\n 29 files changed, 400 insertions(+), 269 deletions(-)\n create mode 100644 hw/s390x/s390-virtio-hcall.h\n delete mode 100644 hw/s390x/s390-virtio.h" }