{"id":816630,"url":"http://patchwork.ozlabs.org/api/covers/816630/?format=json","web_url":"http://patchwork.ozlabs.org/project/qemu-devel/cover/20170921012037.553-1-jsnow@redhat.com/","project":{"id":14,"url":"http://patchwork.ozlabs.org/api/projects/14/?format=json","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":"<20170921012037.553-1-jsnow@redhat.com>","list_archive_url":null,"date":"2017-09-21T01:20:35","name":"[0/2] IDE: combine portio r/w functions","submitter":{"id":64343,"url":"http://patchwork.ozlabs.org/api/people/64343/?format=json","name":"John Snow","email":"jsnow@redhat.com"},"mbox":"http://patchwork.ozlabs.org/project/qemu-devel/cover/20170921012037.553-1-jsnow@redhat.com/mbox/","series":[{"id":4269,"url":"http://patchwork.ozlabs.org/api/series/4269/?format=json","web_url":"http://patchwork.ozlabs.org/project/qemu-devel/list/?series=4269","date":"2017-09-21T01:20:35","name":"IDE: combine portio r/w functions","version":1,"mbox":"http://patchwork.ozlabs.org/series/4269/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/covers/816630/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=jsnow@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 3xyJjN6R72z9s4q\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu, 21 Sep 2017 11:21:27 +1000 (AEST)","from localhost ([::1]:51323 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 1duqBA-0008DH-5K\n\tfor incoming@patchwork.ozlabs.org; Wed, 20 Sep 2017 21:21:24 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:40965)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <jsnow@redhat.com>) id 1duqAZ-0008Ai-Mn\n\tfor qemu-devel@nongnu.org; Wed, 20 Sep 2017 21:20:48 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from <jsnow@redhat.com>) id 1duqAY-000738-Lb\n\tfor qemu-devel@nongnu.org; Wed, 20 Sep 2017 21:20:47 -0400","from mx1.redhat.com ([209.132.183.28]:54872)\n\tby eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32)\n\t(Exim 4.71) (envelope-from <jsnow@redhat.com>)\n\tid 1duqAT-0006vY-ET; Wed, 20 Sep 2017 21:20:41 -0400","from smtp.corp.redhat.com\n\t(int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12])\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 552B85D687;\n\tThu, 21 Sep 2017 01:20:40 +0000 (UTC)","from probe.bos.redhat.com (dhcp-17-130.bos.redhat.com\n\t[10.18.17.130])\n\tby smtp.corp.redhat.com (Postfix) with ESMTP id 0346C60BE6;\n\tThu, 21 Sep 2017 01:20:37 +0000 (UTC)"],"DMARC-Filter":"OpenDMARC Filter v1.3.2 mx1.redhat.com 552B85D687","From":"John Snow <jsnow@redhat.com>","To":"qemu-block@nongnu.org","Date":"Wed, 20 Sep 2017 21:20:35 -0400","Message-Id":"<20170921012037.553-1-jsnow@redhat.com>","X-Scanned-By":"MIMEDefang 2.79 on 10.5.11.12","X-Greylist":"Sender IP whitelisted, not delayed by milter-greylist-4.5.16\n\t(mx1.redhat.com [10.5.110.39]);\n\tThu, 21 Sep 2017 01:20:40 +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 0/2] IDE: combine portio r/w functions","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":"John Snow <jsnow@redhat.com>, mark.cave-ayland@ilande.co.uk,\n\tqemu-devel@nongnu.org","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":"Mark, here's a quick sketch for you. There are two things I don't like,\nbut didn't care enough to fix:\n\n(1) Restricting nbytes to 2 or 4 means some extra boilerplate\n    to quiet compilers who don't know it will only ever be 2 or 4\n(2) the address value is all-but-ignored, it carries over from the\n    portio signature and is useful primarily for tracing, but it's\n    a little ugly/deceiving to take a parameter and not use it.\n\nSuggested-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>\n\nJohn Snow (2):\n  ide: generic ide_data_read\n  ide: generic ide_data_write\n\n hw/ide/core.c             | 185 +++++++++++++++++++++-------------------------\n hw/ide/trace-events       |   7 +-\n include/hw/ide/internal.h |   4 +-\n 3 files changed, 89 insertions(+), 107 deletions(-)"}