Re: [PATCH^2] gdb: mips: Add MIPSR6 support

[Simon Marchi <simon.marchi@polymtl.ca>]

On 8/31/23 17:36, Maciej W. Rozycki wrote:
> Hi Simon,
> 
>> I can do a superficial review (formatting / coding style / general
>> feeling), but can't speak about the behavior.  Ideally Maciej would look
>> at it, but if he's not available, I would be fine approving the patch,
>> since it's isolated to the MIPS support.  The code looks nice and
>> well-organized.
> 
>  I can try, but I have been rather overloaded these days, and the change 
> is sufficiently large it probably requires a number of working days to 
> review in detail just for the initial iteration, as there's a lot to 
> cross-check here with documentation (there's always a second pair of 
> educated eyes required for this kind of a change).

I agree, it's not something that can be properly reviewed with just a
few minutes (even hours) to spare.

>  Mind that it's something I'd have to do entirely in my free time, and 
> also that I have no MIPSr6 hardware available, so that would include time 
> required to set up QEMU (assuming that it works without glitches) to do 
> any R6 verification (and this change still has to be verified not to break 
> existing MIPS targets).  That's why I offer support primarily to other 
> enthusiasts running legacy MIPS hardware.

That's completely understandable.  That said, if somebody else that is
trustworthy has a long term interest in the MIPS support in GDB
(especially if it's a commercial interest), we could think about
onboarding them as co-maintainers for the port.

Simon