From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54415) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gMx8H-00054o-0R for qemu-devel@nongnu.org; Wed, 14 Nov 2018 10:31:09 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gMx8E-0002aa-CI for qemu-devel@nongnu.org; Wed, 14 Nov 2018 10:31:08 -0500 Received: from ste-pvt-msa1.bahnhof.se ([213.80.101.70]:58290) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1gMx8C-0002VU-CT for qemu-devel@nongnu.org; Wed, 14 Nov 2018 10:31:06 -0500 Date: Wed, 14 Nov 2018 16:30:46 +0100 From: Fredrik Noring Message-ID: <20181114153046.GA2359@sx9> References: <1542135956-3780-1-git-send-email-aleksandar.markovic@rt-rk.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH] target/mips: Disable R5900 support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Philippe =?utf-8?Q?Mathieu-Daud=C3=A9?= Cc: Aleksandar Markovic , qemu-devel@nongnu.org, Stefan Markovic , Aleksandar Markovic , Aurelien Jarno , "Maciej W. Rozycki" , =?utf-8?Q?J=C3=BCrgen?= Urban Hi Philippe, On Tue, Nov 13, 2018 at 11:51:54PM +0100, Philippe Mathieu-Daud=C3=A9 wro= te: > On Tue, Nov 13, 2018 at 8:29 PM Philippe Mathieu-Daud=C3=A9 > wrote: > > On Tue, Nov 13, 2018 at 8:08 PM Aleksandar Markovic > > wrote: > > > > > > From: Aleksandar Markovic > > > > > > Disable R5900 support. There are some outstanding issues related > > > to ABI support and emulation accuracy, that were not understood > > > well during review process. Disable to avoid backward compatibility > > > issues. >=20 > If the issues you mentioned are "the R5900 tcg opcodes are not > implemented correctly", then this patch is OK, because no cpu can use > the R5900 opcodes. The issue is that the Linux kernel emulates certain opcodes (e.g. LL, SC, RDHWR, DMULT, etc.) by psABI requirements. This is a common requirement for many other MIPS implementations, not only the R5900. It is trivial to do this with QEMU user mode, but evidently for whatever reason Aleksandar does not permit this emulation. > At some point while reading your reviews, I understood the R5900 > patches introduced incorrect behaviors for the non-R5900 cpus. In this > case this patch wouldn't suffice. No, that was never the case and I'm not aware of any such problems. However, there is a refactoring series, and we have observed preexisting bugs in the MIPS emulation, unrelated to the R5900. The opcode decoder could also be improved, such as in asserting reserved instructions in more cases where opcodes are invalid, etc. > Hoping I misinterpreted your reviews, then this patch is OK. > With one of the suggested comments: > Reviewed-by: Philippe Mathieu-Daud=C3=A9 >=20 > Then we will fix this for the 4.0 release. What exactly needs to be fixed regarding the psABI? The relevant opcodes would need to stay, and not be prohibited and removed as Aleksandar has suggested, since such opcode removal breaks the psABI requirements. Finally, as Maciej explained in some detail, the document that Aleksandar just recently requested is known to not exist, for any MIPS implementatio= n, so we are not going to make any progress on that either. Fredrik