This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] remote: allow aborting long operations (e.g., file transfers) (Re: [PATCH] Prelimit number of bytes to read in "vFile:pread:")
- From: Pedro Alves <palves at redhat dot com>
- To: Sandra Loosemore <sandra at codesourcery dot com>
- Cc: Gary Benson <gbenson at redhat dot com>, gdb-patches at sourceware dot org, Joel Brobecker <brobecker at adacore dot com>, Doug Evans <dje at google dot com>, Jan Kratochvil <jan dot kratochvil at redhat dot com>, André Pönitz <apoenitz at t-online dot de>, Paul Koning <Paul_Koning at dell dot com>
- Date: Fri, 21 Aug 2015 18:27:44 +0100
- Subject: Re: [PATCH] remote: allow aborting long operations (e.g., file transfers) (Re: [PATCH] Prelimit number of bytes to read in "vFile:pread:")
- Authentication-results: sourceware.org; auth=none
- References: <55D3DB83 dot 4050204 at redhat dot com> <1439980862-21305-1-git-send-email-gbenson at redhat dot com> <20150819105054 dot GA22009 at blade dot nx> <55D4B190 dot 6080700 at codesourcery dot com> <20150819172059 dot GA31845 at blade dot nx> <55D4F125 dot 4080409 at codesourcery dot com> <55D5F799 dot 9020700 at redhat dot com> <55D607B8 dot 70103 at redhat dot com> <55D61AA2 dot 5000502 at codesourcery dot com> <55D73B35 dot 8010008 at redhat dot com> <55D75B69 dot 1040408 at codesourcery dot com>
On 08/21/2015 06:10 PM, Sandra Loosemore wrote:
> On 08/21/2015 08:52 AM, Pedro Alves wrote:
>
>> But please try the new patch below.
>
> In combination with the two patches Gary just checked in, this is
> working fabulously. I see:
Ribbit! :-)
> I think this addresses all my concerns with the change in the default
> behavior. Pedro and Gary, thanks very much for your patience and hard
> work in getting this resolved! Between the speedup in reading the
> libraries, the messages to explain what is going on, and making
> transfers interruptible, this is a big improvement in usability. :-D
Thank you for your patience pushing us all in the right direction!
Does anyone want to comment on the patch, or shall I go ahead with it?
Thanks,
Pedro Alves