[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [v3 PATCH] target-mips: implement UserLocal Register
From: |
Aurelien Jarno |
Subject: |
Re: [Qemu-devel] [v3 PATCH] target-mips: implement UserLocal Register |
Date: |
Thu, 29 May 2014 15:08:18 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Thu, May 29, 2014 at 01:01:38PM +0000, Petar Jovanovic wrote:
> > While adding MIPS_HFLAG_HWRENA_ULR is a good idea for performance
> > reasons, I don't think we should add MIPS_HFLAG_CP0UL. This value is
> > always the same (true or false) during all the run time of the
> > qemu-system-mips binary, and thus we don't need to take care of code
> > generated with it being true or being false.
>
> Initial version of this patch did not have this flag, but it was added
> as requested in code review.
> What do you suggest?
Well maybe I missed something, but it looks to me the value can never
change as CP0_Config3 is a read-only register. If I am right, probably
the best is to check directly env->CP0_Config3.
--
Aurelien Jarno GPG: 4096R/1DDD8C9B
address@hidden http://www.aurel32.net
- [Qemu-devel] [v3 PATCH] target-mips: implement UserLocal Register, Petar Jovanovic, 2014/05/28
- Re: [Qemu-devel] [v3 PATCH] target-mips: implement UserLocal Register, Aurelien Jarno, 2014/05/29
- Re: [Qemu-devel] [v3 PATCH] target-mips: implement UserLocal Register, Petar Jovanovic, 2014/05/29
- Re: [Qemu-devel] [v3 PATCH] target-mips: implement UserLocal Register,
Aurelien Jarno <=
- Re: [Qemu-devel] [v3 PATCH] target-mips: implement UserLocal Register, James Hogan, 2014/05/29
- Re: [Qemu-devel] [v3 PATCH] target-mips: implement UserLocal Register, Peter Maydell, 2014/05/29
- Re: [Qemu-devel] [v3 PATCH] target-mips: implement UserLocal Register, James Hogan, 2014/05/29
- Re: [Qemu-devel] [v3 PATCH] target-mips: implement UserLocal Register, Peter Maydell, 2014/05/29
- Re: [Qemu-devel] [v3 PATCH] target-mips: implement UserLocal Register, James Hogan, 2014/05/29
- Re: [Qemu-devel] [v3 PATCH] target-mips: implement UserLocal Register, Petar Jovanovic, 2014/05/29
- Re: [Qemu-devel] [v3 PATCH] target-mips: implement UserLocal Register, Andreas Färber, 2014/05/29
- Re: [Qemu-devel] [v3 PATCH] target-mips: implement UserLocal Register, Aurelien Jarno, 2014/05/29
- Re: [Qemu-devel] [v3 PATCH] target-mips: implement UserLocal Register, Peter Maydell, 2014/05/29
- Re: [Qemu-devel] [v3 PATCH] target-mips: implement UserLocal Register, Aurelien Jarno, 2014/05/29