This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH 0/24] MIPS: Add support for reconfigurable FPR size, MIPS MSA and MIPSR6,support
- From: "Maciej W. Rozycki" <macro at imgtec dot com>
- To: Bhushan Attarde <Bhushan dot Attarde at imgtec dot com>
- Cc: "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>, Matthew Fortune <Matthew dot Fortune at imgtec dot com>, James Hogan <James dot Hogan at imgtec dot com>, Andrew Bennett <Andrew dot Bennett at imgtec dot com>, Jaydeep Patil <Jaydeep dot Patil at imgtec dot com>
- Date: Mon, 25 Jul 2016 14:58:19 +0100
- Subject: Re: [PATCH 0/24] MIPS: Add support for reconfigurable FPR size, MIPS MSA and MIPSR6,support
- Authentication-results: sourceware.org; auth=none
- References: <9D3E5517712EFA419C6B9C50EDC1282CAA1FA536@PUMAIL01.pu.imgtec.org>
Bhushan,
First of all thank you for your submission, and also apologies for the
long RTT.
Now a couple of general questions/notes:
1. Can you please reformat the e-mails with patches such that they can be
readily fed to Git? Using `git format-patch' would be the usual way,
although of course feel free to do it however you like. In the form
you posted they have leading blanks across descriptions which do not
work well with `git am' (see also
<https://sourceware.org/gdb/wiki/ContributionChecklist>).
2. You haven't told us how you verified these changes. Have you run any
regression testing at all? If so, the which targets have you tested?
Also please make sure that you at least build changes which affect
generic files with `--enable-targets=all'.
3. Some changes touch files in the include/ and opcodes/ subdirectories.
These (except for include/gdb/) are maintained as a part of the
binutils project and need to be posted to the relevant mailing list,
which is <binutils@sourceware.org>. I'll review the relevant parts in
this round with an appropriate `Cc' added, but please pay attention to
have the binutils mailing list included where appropriate in the
submission. In fact especially changes to opcodes/ and include/opcode/
appear self-contained and largely independent from the GDB part of the
containing patch, so they may best be split off and submitted to the
binutils project first. More on this in replies to individual patches.
> 1. Handles run-time reconfigurable FPR size. MIPS revision 2 onwards
> MIPS32
>
> and MIPS64 both can have either 32-bit or 64-bit FPRs. This is decided
>
> based on the values of CP1.FIR.F64, CP0.SR.FR and CP0.Config5.FRE bits.
>
> Patch 1 to patch 12 in the series handles this property. This work is
> based
>
> on the patch submitted earlier by Maciej W. Rozycki. Please refer to
>
> https://sourceware.org/ml/gdb-patches/2012-06/msg00201.html for details.
> So
>
> this series includes his patch (updated to GDB 7.11 release version) as
> the
>
> [patch 1] of the series.
For the record please note that the availability of CP0.Status.FR goes
back to the MIPS III ISA and therefore all 64-bit MIPS processors have it.
So if you need to determine the presence of this bit, then the algorithm
conceptually is (isa_is_64bit || CP1.FIR.F64).
This introduction is not going to end up in our Git repository, but for
the sake of people's confusion avoidance please update it in patch
resubmission.
Finally please note that I cannot approve changes outside the MIPS
target, but to speed up the process I'll check them regardless to see if
there are any obvious issues. You'll still have to seek a global
maintainer's approval for those changes though.
Maciej