From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 112183 invoked by alias); 4 Oct 2016 18:14:52 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 112169 invoked by uid 89); 4 Oct 2016 18:14:51 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-4.5 required=5.0 tests=BAYES_00,RP_MATCHES_RCVD,SPF_PASS autolearn=ham version=3.3.2 spammy=unnecessarily, Parts, submission, arrived X-HELO: mailapp01.imgtec.com Received: from mailapp02.imgtec.com (HELO mailapp01.imgtec.com) (217.156.133.132) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 04 Oct 2016 18:14:41 +0000 Received: from HHMAIL03.hh.imgtec.org (unknown [10.44.0.21]) by Forcepoint Email with ESMTPS id 26765996477FE; Tue, 4 Oct 2016 19:14:35 +0100 (IST) Received: from HHMAIL01.hh.imgtec.org (10.100.10.19) by HHMAIL03.hh.imgtec.org (10.44.0.21) with Microsoft SMTP Server (TLS) id 14.3.294.0; Tue, 4 Oct 2016 19:14:38 +0100 Received: from [10.20.78.69] (10.20.78.69) by HHMAIL01.hh.imgtec.org (10.100.10.21) with Microsoft SMTP Server id 14.3.294.0; Tue, 4 Oct 2016 19:14:37 +0100 Date: Tue, 04 Oct 2016 18:14:00 -0000 From: "Maciej W. Rozycki" To: Pedro Alves CC: Bhushan Attarde , "gdb-patches@sourceware.org" , Matthew Fortune , James Hogan , Andrew Bennett , Jaydeep Patil Subject: Re: [PATCH 0/24] MIPS: Add support for reconfigurable FPR size, MIPS MSA and MIPSR6,support In-Reply-To: Message-ID: References: <9D3E5517712EFA419C6B9C50EDC1282CAA1FA536@PUMAIL01.pu.imgtec.org> User-Agent: Alpine 2.00 (DEB 1167 2008-08-23) MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" X-SW-Source: 2016-10/txt/msg00057.txt.bz2 On Tue, 4 Oct 2016, Pedro Alves wrote: > > 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. > > I can't speak for others, but at least I was waiting for a > repost and for Maciej to validate the overall idea. After numerous hiccups and distractions I am about to get back to this review. For the record -- I have come across an inconsistency in FCR handling, coming from a regression introduced with XML description support for the MIPS target, affecting this patch series. I am about to commit a fix, after the final testing I'm still working on; I'd like to see it there in 7.12 so that we have a fixed proper release before MSA goes in. Code in the patch set will then have to be adjusted accordingly. Parts of some patches apply to binutils rather than GDB and need to be split, so I'll go through the series as they are so that there is no need to proliferate reposts unnecessarily. > Also, for some reason, neither I nor the mailing list archives > seem to have received the whole patch set: > > https://sourceware.org/ml/gdb-patches/2016-06/msg00510.html There has been an issue with this submission indeed in that it arrived as two separate threads, but the remaining patches do have made it there: https://sourceware.org/ml/gdb-patches/2016-06/msg00425.html > FYI, in case a global-maintainer review was the blocker here. I don't think so, not at this stage. We need to sort out the general structure of the patch set first. Please feel free, or indeed anyone else, to chime in if you like, of course. Maciej