This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PING] [PATCH 00/11] Regset rework preparations part 1
- From: Pedro Alves <palves at redhat dot com>
- To: Andreas Arnez <arnez at linux dot vnet dot ibm dot com>, gdb-patches at sourceware dot org
- Date: Wed, 21 May 2014 11:38:57 +0100
- Subject: Re: [PING] [PATCH 00/11] Regset rework preparations part 1
- Authentication-results: sourceware.org; auth=none
- References: <1400168975-3145-1-git-send-email-arnez at linux dot vnet dot ibm dot com> <8761l28bql dot fsf at br87z6lw dot de dot ibm dot com>
On 05/19/2014 08:32 AM, Andreas Arnez wrote:
> On Thu, May 15 2014, Andreas Arnez wrote:
>
>> This is the first half of the "regset rework preparations" series I've
>> posted as RFC:
>> https://sourceware.org/ml/gdb-patches/2014-04/msg00560.html
>
> On Wed, May 14 2014, Mark Kettenis wrote:
>
>>> From: Andreas Arnez <arnez@linux.vnet.ibm.com>
>>> Date: Wed, 14 May 2014 14:13:26 +0200
>>>
>>> Although there was feedback for some of the patches in this series
>>> (https://sourceware.org/ml/gdb-patches/2014-04/msg00560.html), many are
>>> still uncommented. I'd really appreciate more feedback, particularly
>>> for the architecture-specific changes.
>>
>> Sorry, I've not been able to find time to process your reply on my
>> review of patch 12 yet and won't be able to for the next 1.5 weeks or
>> so. But perhaps you could move ahead with the diffs that don't depend
>> on that one which seem useful in their own right.
>>
>>> Patches 3, 4, 6, 7, 8 (AARCH64, ARM, MIPS, MN10300, and SCORE): No
>>> comments yet.
>>>
>>>> * Patch 9 ("SPARC: Rename register maps from "*regset" to "*regmap"):
>>>> Adjusts naming, to avoid name clashes in patch 10.
>>>
>>> Approved by Mark Kettenis:
>>> https://sourceware.org/ml/gdb-patches/2014-05/msg00038.html
>>
>> I don't really expect any response for MN10300 and SCORE. I'd give it
>> a few days to give people with an interest in ARM and MIPS the
>> opportunity to repsond, but feel free to move ahead with 3, 4, 6, 7
>> and 8 after that.
>
> As suggested, I've waited a few additional days now for comments on
> patches 3, 4, 6, 7, and 8. Apart from those I believe everything else
> in this series was approved already.
>
> OK to push?
>
I've read the series, and it all looked fine to me.
Please push.
--
Pedro Alves