From patchwork Thu Mar 1 13:25:38 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Harsh Prateek Bora X-Patchwork-Id: 144005 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 4FAD8B6EF3 for ; Fri, 2 Mar 2012 00:26:38 +1100 (EST) Received: from localhost ([::1]:50147 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S361j-00037Y-O3 for incoming@patchwork.ozlabs.org; Thu, 01 Mar 2012 08:26:35 -0500 Received: from eggs.gnu.org ([208.118.235.92]:51136) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S361U-00030t-H4 for qemu-devel@nongnu.org; Thu, 01 Mar 2012 08:26:22 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1S3614-0000UF-79 for qemu-devel@nongnu.org; Thu, 01 Mar 2012 08:26:20 -0500 Received: from e28smtp04.in.ibm.com ([122.248.162.4]:37577) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S3613-0000RO-Gj for qemu-devel@nongnu.org; Thu, 01 Mar 2012 08:25:54 -0500 Received: from /spool/local by e28smtp04.in.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Thu, 1 Mar 2012 18:55:47 +0530 Received: from d28relay05.in.ibm.com (9.184.220.62) by e28smtp04.in.ibm.com (192.168.1.134) with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted; Thu, 1 Mar 2012 18:55:44 +0530 Received: from d28av02.in.ibm.com (d28av02.in.ibm.com [9.184.220.64]) by d28relay05.in.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id q21DPhVt3330276 for ; Thu, 1 Mar 2012 18:55:43 +0530 Received: from d28av02.in.ibm.com (loopback [127.0.0.1]) by d28av02.in.ibm.com (8.14.4/8.13.1/NCO v10.0 AVout) with ESMTP id q21DPfqZ013281 for ; Fri, 2 Mar 2012 00:25:43 +1100 Received: from harshbora.in.ibm.com ([9.124.35.242]) by d28av02.in.ibm.com (8.14.4/8.13.1/NCO v10.0 AVin) with ESMTP id q21DPfJq013178; Fri, 2 Mar 2012 00:25:41 +1100 From: Harsh Prateek Bora To: qemu-devel@nongnu.org Date: Thu, 1 Mar 2012 18:55:38 +0530 Message-Id: <1330608340-9515-13-git-send-email-harsh@linux.vnet.ibm.com> X-Mailer: git-send-email 1.7.1.1 In-Reply-To: <1330608340-9515-1-git-send-email-harsh@linux.vnet.ibm.com> References: <1330608340-9515-1-git-send-email-harsh@linux.vnet.ibm.com> x-cbid: 12030113-5564-0000-0000-0000019F169D X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 122.248.162.4 Cc: stefanha@gmail.com, vilanova@ac.upc.edu, aneesh.kumar@linux.vnet.ibm.com Subject: [Qemu-devel] [RFC PATCH v5 12/14] monitor: remove unused do_info_trace X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Going forward with simpletrace v2 variable size trace records, we cannot have a generic function to print trace event info and therefore this interface becomes invalid. As per Stefan Hajnoczi: "This command is only available from the human monitor. It's not very useful because it historically hasn't been able to pretty-print events or show them in the right order (we use a ringbuffer but it prints them out from index 0). Therefore, I don't think we're under any obligation to keep this command around. No one has complained about it's limitations - I think this is a sign that no one has used it. I'd be okay with a patch that removes it." Ref: http://lists.gnu.org/archive/html/qemu-devel/2012-01/msg01268.html Signed-off-by: Harsh Prateek Bora --- monitor.c | 16 ---------------- 1 files changed, 0 insertions(+), 16 deletions(-) diff --git a/monitor.c b/monitor.c index ffda0fe..365ce47 100644 --- a/monitor.c +++ b/monitor.c @@ -942,13 +942,6 @@ static void do_info_cpu_stats(Monitor *mon) } #endif -#if defined(CONFIG_TRACE_SIMPLE) -static void do_info_trace(Monitor *mon) -{ - st_print_trace((FILE *)mon, &monitor_fprintf); -} -#endif - static void do_trace_print_events(Monitor *mon) { trace_print_events((FILE *)mon, &monitor_fprintf); @@ -3033,15 +3026,6 @@ static const mon_cmd_t info_cmds[] = { .help = "show roms", .mhandler.info = do_info_roms, }, -#if defined(CONFIG_TRACE_SIMPLE) - { - .name = "trace", - .args_type = "", - .params = "", - .help = "show current contents of trace buffer", - .mhandler.info = do_info_trace, - }, -#endif { .name = "trace-events", .args_type = "",