From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 20705 invoked by alias); 19 Feb 2014 20:29:43 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 20617 invoked by uid 89); 19 Feb 2014 20:29:42 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.0 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 19 Feb 2014 20:29:41 +0000 Received: from int-mx01.intmail.prod.int.phx2.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s1JKTdg9006386 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Wed, 19 Feb 2014 15:29:40 -0500 Received: from brno.lan (ovpn01.gateway.prod.ext.ams2.redhat.com [10.39.146.11]) by int-mx01.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id s1JKTamo025004 for ; Wed, 19 Feb 2014 15:29:38 -0500 From: Pedro Alves To: gdb-patches@sourceware.org Subject: [PATCH 2/7] remote-m32r-sdi.c: Don't install a deprecated_xfer_memory method Date: Wed, 19 Feb 2014 20:29:00 -0000 Message-Id: <1392841775-19126-3-git-send-email-palves@redhat.com> In-Reply-To: <1392841775-19126-1-git-send-email-palves@redhat.com> References: <1392841775-19126-1-git-send-email-palves@redhat.com> X-SW-Source: 2014-02/txt/msg00610.txt.bz2 This removes yet another instance of a deprecated_xfer_memory user. Tested by building a --enable-targets=all gdb, on x86-64 Fedora 17. gdb/ 2014-02-19 Pedro Alves * remote-m32r-sdi.c (send_data): Constify 'buf' parameter. (m32r_xfer_memory): Adjust as a to_xfer_partial helper. (m32r_xfer_partial): New function. (init_m32r_ops): Don't install a deprecated_xfer_memory hook. Install a to_xfer_partial hook. --- gdb/remote-m32r-sdi.c | 60 +++++++++++++++++++++++++++++++++++---------------- 1 file changed, 42 insertions(+), 18 deletions(-) diff --git a/gdb/remote-m32r-sdi.c b/gdb/remote-m32r-sdi.c index 02a812a..0c23403 100644 --- a/gdb/remote-m32r-sdi.c +++ b/gdb/remote-m32r-sdi.c @@ -179,7 +179,7 @@ get_ack (void) /* Send data to the target and check an ack packet. */ static int -send_data (void *buf, int len) +send_data (const void *buf, int len) { if (!sdi_desc) return -1; @@ -1032,11 +1032,12 @@ m32r_files_info (struct target_ops *target) } } -/* Read/Write memory. */ -static int -m32r_xfer_memory (CORE_ADDR memaddr, gdb_byte *myaddr, int len, - int write, - struct mem_attrib *attrib, struct target_ops *target) +/* Helper for m32r_xfer_partial that handles memory transfers. + Arguments are like target_xfer_partial. */ + +static enum target_xfer_status +m32r_xfer_memory (gdb_byte *readbuf, const gdb_byte *writebuf, + ULONGEST memaddr, ULONGEST len, ULONGEST *xfered_len) { unsigned long taddr; unsigned char buf[0x2000]; @@ -1052,22 +1053,24 @@ m32r_xfer_memory (CORE_ADDR memaddr, gdb_byte *myaddr, int len, if (remote_debug) { - if (write) - fprintf_unfiltered (gdb_stdlog, "m32r_xfer_memory(%s,%d,write)\n", - paddress (target_gdbarch (), memaddr), len); + if (writebuf != NULL) + fprintf_unfiltered (gdb_stdlog, "m32r_xfer_memory(%s,%s,write)\n", + paddress (target_gdbarch (), memaddr), + plongest (len)); else - fprintf_unfiltered (gdb_stdlog, "m32r_xfer_memory(%s,%d,read)\n", - paddress (target_gdbarch (), memaddr), len); + fprintf_unfiltered (gdb_stdlog, "m32r_xfer_memory(%s,%s,read)\n", + paddress (target_gdbarch (), memaddr), + plongest (len)); } - if (write) + if (writebuf != NULL) { buf[0] = SDI_WRITE_MEMORY; store_long_parameter (buf + 1, taddr); store_long_parameter (buf + 5, len); if (len < 0x1000) { - memcpy (buf + 9, myaddr, len); + memcpy (buf + 9, writebuf, len); ret = send_data (buf, len + 9) - 9; } else @@ -1079,7 +1082,7 @@ m32r_xfer_memory (CORE_ADDR memaddr, gdb_byte *myaddr, int len, "m32r_xfer_memory() failed\n"); return 0; } - ret = send_data (myaddr, len); + ret = send_data (writebuf, len); } } else @@ -1102,17 +1105,38 @@ m32r_xfer_memory (CORE_ADDR memaddr, gdb_byte *myaddr, int len, return 0; } - ret = recv_data (myaddr, len); + ret = recv_data (readbuf, len); } if (ret <= 0) { if (remote_debug) fprintf_unfiltered (gdb_stdlog, "m32r_xfer_memory() fails\n"); - return 0; + return TARGET_XFER_E_IO; } - return ret; + *xfered_len = ret; + return TARGET_XFER_OK; +} + +/* Target to_xfer_partial implementation. */ + +static enum target_xfer_status +m32r_xfer_partial (struct target_ops *ops, enum target_object object, + const char *annex, gdb_byte *readbuf, + const gdb_byte *writebuf, ULONGEST offset, ULONGEST len, + ULONGEST *xfered_len) +{ + switch (object) + { + case TARGET_OBJECT_MEMORY: + return m32r_xfer_memory (readbuf, writebuf, offset, len, xfered_len); + + default: + return ops->beneath->to_xfer_partial (ops->beneath, object, annex, + readbuf, writebuf, offset, len, + xfered_len); + } } static void @@ -1635,7 +1659,7 @@ init_m32r_ops (void) m32r_ops.to_fetch_registers = m32r_fetch_register; m32r_ops.to_store_registers = m32r_store_register; m32r_ops.to_prepare_to_store = m32r_prepare_to_store; - m32r_ops.deprecated_xfer_memory = m32r_xfer_memory; + m32r_ops.to_xfer_partial = m32r_xfer_partial; m32r_ops.to_files_info = m32r_files_info; m32r_ops.to_insert_breakpoint = m32r_insert_breakpoint; m32r_ops.to_remove_breakpoint = m32r_remove_breakpoint; -- 1.7.11.7