Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/808492/?format=api
{ "id": 808492, "url": "http://patchwork.ozlabs.org/api/patches/808492/?format=api", "web_url": "http://patchwork.ozlabs.org/project/qemu-devel/patch/20170901001502.29915-9-jsnow@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": "<20170901001502.29915-9-jsnow@redhat.com>", "list_archive_url": null, "date": "2017-09-01T00:15:01", "name": "[v3,8/9] AHCI: pretty-print FIS to buffer instead of stderr", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "5399da28ad3c37889c6c771ea28dafff24f8f87a", "submitter": { "id": 64343, "url": "http://patchwork.ozlabs.org/api/people/64343/?format=api", "name": "John Snow", "email": "jsnow@redhat.com" }, "delegate": null, "mbox": "http://patchwork.ozlabs.org/project/qemu-devel/patch/20170901001502.29915-9-jsnow@redhat.com/mbox/", "series": [ { "id": 931, "url": "http://patchwork.ozlabs.org/api/series/931/?format=api", "web_url": "http://patchwork.ozlabs.org/project/qemu-devel/list/?series=931", "date": "2017-09-01T00:14:53", "name": "IDE: replace printfs with tracing", "version": 3, "mbox": "http://patchwork.ozlabs.org/series/931/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/808492/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/808492/checks/", "tags": {}, "related": [], "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-mx06.extmail.prod.ext.phx2.redhat.com;\n\tdmarc=none (p=none dis=none) header.from=redhat.com", "ext-mx06.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 3xk0N71tjtz9s81\n\tfor <incoming@patchwork.ozlabs.org>;\n\tFri, 1 Sep 2017 10:23:51 +1000 (AEST)", "from localhost ([::1]:34849 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 1dnZkT-00076w-DR\n\tfor incoming@patchwork.ozlabs.org; Thu, 31 Aug 2017 20:23:49 -0400", "from eggs.gnu.org ([2001:4830:134:3::10]:51806)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <jsnow@redhat.com>) id 1dnZco-0000Qa-Gv\n\tfor qemu-devel@nongnu.org; Thu, 31 Aug 2017 20:15:55 -0400", "from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from <jsnow@redhat.com>) id 1dnZcn-00050n-6t\n\tfor qemu-devel@nongnu.org; Thu, 31 Aug 2017 20:15:54 -0400", "from mx1.redhat.com ([209.132.183.28]:48250)\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 1dnZcj-0004sF-DU; Thu, 31 Aug 2017 20:15:49 -0400", "from smtp.corp.redhat.com\n\t(int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13])\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 8019F1E2F8;\n\tFri, 1 Sep 2017 00:15:48 +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 B19ACC14AB;\n\tFri, 1 Sep 2017 00:15:43 +0000 (UTC)" ], "DMARC-Filter": "OpenDMARC Filter v1.3.2 mx1.redhat.com 8019F1E2F8", "From": "John Snow <jsnow@redhat.com>", "To": "qemu-block@nongnu.org", "Date": "Thu, 31 Aug 2017 20:15:01 -0400", "Message-Id": "<20170901001502.29915-9-jsnow@redhat.com>", "In-Reply-To": "<20170901001502.29915-1-jsnow@redhat.com>", "References": "<20170901001502.29915-1-jsnow@redhat.com>", "X-Scanned-By": "MIMEDefang 2.79 on 10.5.11.13", "X-Greylist": "Sender IP whitelisted, not delayed by milter-greylist-4.5.16\n\t(mx1.redhat.com [10.5.110.30]);\n\tFri, 01 Sep 2017 00:15:48 +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 8/9] AHCI: pretty-print FIS to buffer\n\tinstead of stderr", "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>, qemu-devel@nongnu.org, stefanha@redhat.com,\n\tf4bug@amsat.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": "The current FIS printing routines dump the FIS to screen. adjust this\nsuch that it dumps to buffer instead, then use this ability to have\nFIS dump mechanisms via trace-events instead of compiled defines.\n\nSigned-off-by: John Snow <jsnow@redhat.com>\n---\n hw/ide/ahci.c | 28 ++++++++++++++++++----------\n hw/ide/trace-events | 4 ++++\n 2 files changed, 22 insertions(+), 10 deletions(-)", "diff": "diff --git a/hw/ide/ahci.c b/hw/ide/ahci.c\nindex 82cc196..6c98047 100644\n--- a/hw/ide/ahci.c\n+++ b/hw/ide/ahci.c\n@@ -644,20 +644,21 @@ static void ahci_reset_port(AHCIState *s, int port)\n ahci_init_d2h(d);\n }\n \n-static void debug_print_fis(uint8_t *fis, int cmd_len)\n+/* Buffer pretty output based on a raw FIS structure. */\n+static char *ahci_pretty_buffer_fis(uint8_t *fis, int cmd_len)\n {\n-#if DEBUG_AHCI\n int i;\n+ GString *s = g_string_new(\"FIS:\");\n \n- fprintf(stderr, \"fis:\");\n for (i = 0; i < cmd_len; i++) {\n if ((i & 0xf) == 0) {\n- fprintf(stderr, \"\\n%02x:\",i);\n+ g_string_append_printf(s, \"\\n0x%02x: \", i);\n }\n- fprintf(stderr, \"%02x \",fis[i]);\n+ g_string_append_printf(s, \"%02x \", fis[i]);\n }\n- fprintf(stderr, \"\\n\");\n-#endif\n+ g_string_append_c(s, '\\n');\n+\n+ return g_string_free(s, FALSE);\n }\n \n static bool ahci_map_fis_address(AHCIDevice *ad)\n@@ -1201,7 +1202,11 @@ static void handle_reg_h2d_fis(AHCIState *s, int port,\n * table to ide_state->io_buffer */\n if (opts & AHCI_CMD_ATAPI) {\n memcpy(ide_state->io_buffer, &cmd_fis[AHCI_COMMAND_TABLE_ACMD], 0x10);\n- debug_print_fis(ide_state->io_buffer, 0x10);\n+ if (trace_event_get_state_backends(TRACE_HANDLE_REG_H2D_FIS_DUMP)) {\n+ char *pretty_fis = ahci_pretty_buffer_fis(ide_state->io_buffer, 0x10);\n+ trace_handle_reg_h2d_fis_dump(s, port, pretty_fis);\n+ g_free(pretty_fis);\n+ }\n s->dev[port].done_atapi_packet = false;\n /* XXX send PIO setup FIS */\n }\n@@ -1256,8 +1261,11 @@ static int handle_cmd(AHCIState *s, int port, uint8_t slot)\n trace_handle_cmd_badmap(s, port, cmd_len);\n goto out;\n }\n- debug_print_fis(cmd_fis, 0x80);\n-\n+ if (trace_event_get_state_backends(TRACE_HANDLE_CMD_FIS_DUMP)) {\n+ char *pretty_fis = ahci_pretty_buffer_fis(cmd_fis, 0x80);\n+ trace_handle_cmd_fis_dump(s, port, pretty_fis);\n+ g_free(pretty_fis);\n+ }\n switch (cmd_fis[0]) {\n case SATA_FIS_TYPE_REGISTER_H2D:\n handle_reg_h2d_fis(s, port, slot, cmd_fis);\ndiff --git a/hw/ide/trace-events b/hw/ide/trace-events\nindex e15fd77..601bd97 100644\n--- a/hw/ide/trace-events\n+++ b/hw/ide/trace-events\n@@ -105,3 +105,7 @@ ahci_cmd_done(void *s, int port) \"ahci(%p)[%d]: cmd done\"\n ahci_reset(void *s) \"ahci(%p): HBA reset\"\n allwinner_ahci_mem_read(void *s, void *a, uint64_t addr, uint64_t val, unsigned size) \"ahci(%p): read a=%p addr=0x%\"HWADDR_PRIx\" val=0x%\"PRIx64\", size=%d\"\n allwinner_ahci_mem_write(void *s, void *a, uint64_t addr, uint64_t val, unsigned size) \"ahci(%p): write a=%p addr=0x%\"HWADDR_PRIx\" val=0x%\"PRIx64\", size=%d\"\n+\n+# Warning: Verbose\n+handle_reg_h2d_fis_dump(void *s, int port, const char *fis) \"ahci(%p)[%d]: %s\"\n+handle_cmd_fis_dump(void *s, int port, const char *fis) \"ahci(%p)[%d]: %s\"\n", "prefixes": [ "v3", "8/9" ] }