From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 745 invoked by alias); 1 Aug 2006 05:34:10 -0000 Received: (qmail 737 invoked by uid 22791); 1 Aug 2006 05:34:09 -0000 X-Spam-Check-By: sourceware.org Received: from potter.codesourcery.com (HELO mail.codesourcery.com) (65.74.133.4) by sourceware.org (qpsmtpd/0.31) with ESMTP; Tue, 01 Aug 2006 05:34:05 +0000 Received: (qmail 13071 invoked from network); 1 Aug 2006 05:34:04 -0000 Received: from unknown (HELO zigzag.lvk.cs.msu.su) (vladimir@127.0.0.2) by mail.codesourcery.com with ESMTPA; 1 Aug 2006 05:34:04 -0000 From: Vladimir Prus To: Eli Zaretskii Subject: Re: [PATCH] remove download_write_size variable Date: Tue, 01 Aug 2006 05:34:00 -0000 User-Agent: KMail/1.7.2 Cc: gdb-patches@sources.redhat.com References: <200607181648.45180.vladimir@codesourcery.com> <200607311622.58927.vladimir@codesourcery.com> In-Reply-To: MIME-Version: 1.0 Content-Type: Multipart/Mixed; boundary="Boundary-00=_HfuzEVmMoRR/j/N" Message-Id: <200608010933.59270.vladimir@codesourcery.com> Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2006-08/txt/msg00005.txt.bz2 --Boundary-00=_HfuzEVmMoRR/j/N Content-Type: text/plain; charset="koi8-r" Content-Transfer-Encoding: 7bit Content-Disposition: inline Content-length: 1751 On Monday 31 July 2006 21:16, Eli Zaretskii wrote: > > From: Vladimir Prus > > Date: Mon, 31 Jul 2006 16:22:57 +0400 > > Cc: gdb-patches@sources.redhat.com > > > > > > * symfile.c (download_write_size): Remove. > > > > (show_download_write_size): Remove. > > > > (load_section_callback): Don't use download_write_size. > > > > (_initialize_symfile): Don't register download_write_size. > > > > > > I can find references to download-write-size on the Internet, e.g. in > > > FAQs suggesting how to get better write performance. They all seem to > > > push it as high as memory-write-packet-size, though, which suggests we > > > do not need both limits. > > > > > > I think it's reasonable to remove it. However, you'll have to also > > > remove it from the manual (it's a documented command). > > > > Are you sure? Grep does not find any occurrence in docs, on current > > mainline. > > It's in the node called "Target Commands". Thanks. I was searching with underscores in names, not dashes ;-) > > > === gdb/NEWS > > ================================================================== > > --- gdb/NEWS (/mirrors/gdb) (revision 315) > > +++ gdb/NEWS (/patches/download_write_size/gdb) (revision 315) > > @@ -18,6 +18,8 @@ > > Kernel Object Display, an embedded debugging feature which only worked > > with an obsolete version of Cisco IOS. > > > > +The 'download_write_size' variable. > > + > > This variable is not user-visible. You need to mention the command, > not the C variable. Please find the new patch attached. Doc dir changelog is: 2006-08-01 Vladimir Prus * gdb.texinfo (Target Commands): Remove 'set download-write-size' and 'show download-write-size'. - Volodya --Boundary-00=_HfuzEVmMoRR/j/N Content-Type: text/x-diff; charset="koi8-r"; name="download_write_size__gdb.diff" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="download_write_size__gdb.diff" Content-length: 3785 === gdb/doc/gdb.texinfo ================================================================== --- gdb/doc/gdb.texinfo (/mirrors/gdb) (revision 338) +++ gdb/doc/gdb.texinfo (/patches/download_write_size/gdb) (revision 338) @@ -12133,23 +12133,10 @@ Many remote targets require you to download the executable's code once you've successfully established a connection. You may wish to control -various aspects of this process, such as the size of the data chunks -used by @value{GDBN} to download program parts to the remote target. +various aspects of this process. @table @code -@kindex set download-write-size -@item set download-write-size @var{size} -Set the write size used when downloading a program. Only used when -downloading a program onto a remote target. Specify zero or a -negative value to disable blocked writes. The actual size of each -transfer is also limited by the size of the target packet and the -memory cache. -@kindex show download-write-size -@item show download-write-size -@kindex show download-write-size -Show the current value of the write size. - @item set hash @kindex set hash@r{, for remote monitors} @cindex hash mark while downloading === gdb/symfile.c ================================================================== --- gdb/symfile.c (/mirrors/gdb) (revision 338) +++ gdb/symfile.c (/patches/download_write_size/gdb) (revision 338) @@ -1521,15 +1521,6 @@ we don't want to run a subprocess. On the other hand, I'm not sure how performance compares. */ -static int download_write_size = 512; -static void -show_download_write_size (struct ui_file *file, int from_tty, - struct cmd_list_element *c, const char *value) -{ - fprintf_filtered (file, _("\ -The write size used when downloading a program is %s.\n"), - value); -} static int validate_download = 0; /* Callback service function for generic_load (bfd_map_over_sections). */ @@ -1570,11 +1561,6 @@ const char *sect_name = bfd_get_section_name (abfd, asec); bfd_size_type sent; - if (download_write_size > 0 && size > download_write_size) - block_size = download_write_size; - else - block_size = size; - buffer = xmalloc (size); old_chain = make_cleanup (xfree, buffer); @@ -1591,8 +1577,6 @@ int len; bfd_size_type this_transfer = size - sent; - if (this_transfer >= block_size) - this_transfer = block_size; len = target_write_memory_partial (lma, buffer, this_transfer, &err); if (err) @@ -3806,19 +3790,6 @@ add_info ("extensions", info_ext_lang_command, _("All filename extensions associated with a source language.")); - add_setshow_integer_cmd ("download-write-size", class_obscure, - &download_write_size, _("\ -Set the write size used when downloading a program."), _("\ -Show the write size used when downloading a program."), _("\ -Only used when downloading a program onto a remote\n\ -target. Specify zero, or a negative value, to disable\n\ -blocked writes. The actual size of each transfer is also\n\ -limited by the size of the target packet and the memory\n\ -cache."), - NULL, - show_download_write_size, - &setlist, &showlist); - debug_file_directory = xstrdup (DEBUGDIR); add_setshow_optional_filename_cmd ("debug-file-directory", class_support, &debug_file_directory, _("\ === gdb/NEWS ================================================================== --- gdb/NEWS (/mirrors/gdb) (revision 338) +++ gdb/NEWS (/patches/download_write_size/gdb) (revision 338) @@ -18,6 +18,8 @@ Kernel Object Display, an embedded debugging feature which only worked with an obsolete version of Cisco IOS. +The 'set download-write-size' and 'show download-write-size' commands. + * New remote packets qSupported: --Boundary-00=_HfuzEVmMoRR/j/N--