[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 07/10] target-s390x: enable fully implemented fa
From: |
Aurelien Jarno |
Subject: |
Re: [Qemu-devel] [PATCH 07/10] target-s390x: enable fully implemented facilities |
Date: |
Mon, 25 May 2015 23:13:47 +0200 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On 2015-05-25 23:04, Alexander Graf wrote:
>
>
> On 25.05.15 23:02, Aurelien Jarno wrote:
> > On 2015-05-25 22:39, Alexander Graf wrote:
> >>
> >>
> >> On 25.05.15 01:47, Aurelien Jarno wrote:
> >>> Cc: Alexander Graf <address@hidden>
> >>> Cc: Richard Henderson <address@hidden>
> >>> Signed-off-by: Aurelien Jarno <address@hidden>
> >>
> >> Shouldn't this get populated based on the selected -cpu type?
> >
> > In the long term yes, but given we only implement one CPU type (or
> > rather none) in TCG mode, we can consider that's already the case.
>
> There are patches coming from IBM to at least add a list of a good
> number of s390x cpu types. I'd really like to make use of that and have
> actual CPU types selectable.
I guess they are for the KVM mode. Do they provide the corresponding
facilities list? Probably otherwise that doesn't really differentiate
various CPUs. Please make sure of that when reviewing these patches.
> At least let's move towards that model. So the code in question should
> take the facility capabilities from the first cpu object (or the class?)
> for example and we bump it to the currently supported feature set in there.
Yes, that would work for STFL/STFLE, though we should have a list of
facilities implemented by TCG so we can mask out the non-implemented
facilities. This basically corresponds to the informations provided by
the current patch.
That said that won't work for actually disabling the corresponding
instructions as we don't have a 1 to 1 mapping between the facilities
and the group of instructions. Anyway we don't even check that right
now.
--
Aurelien Jarno GPG: 4096R/1DDD8C9B
address@hidden http://www.aurel32.net
- [Qemu-devel] [PATCH 00/10] target-s390x: TCG fixes and improvements, Aurelien Jarno, 2015/05/24
- [Qemu-devel] [PATCH 01/10] target-s390x: fix PSW value on dynamical exception from helpers, Aurelien Jarno, 2015/05/24
- [Qemu-devel] [PATCH 04/10] target-s390x: silence NaNs for LOAD LENGTHENED and LOAD ROUNDED, Aurelien Jarno, 2015/05/24
- [Qemu-devel] [PATCH 05/10] target-s390x: detect tininess before rounding for FP operations, Aurelien Jarno, 2015/05/24
- [Qemu-devel] [PATCH 07/10] target-s390x: enable fully implemented facilities, Aurelien Jarno, 2015/05/24
- Re: [Qemu-devel] [PATCH 07/10] target-s390x: enable fully implemented facilities, Alexander Graf, 2015/05/25
- Re: [Qemu-devel] [PATCH 07/10] target-s390x: enable fully implemented facilities, Aurelien Jarno, 2015/05/25
- Re: [Qemu-devel] [PATCH 07/10] target-s390x: enable fully implemented facilities, Alexander Graf, 2015/05/25
- Re: [Qemu-devel] [PATCH 07/10] target-s390x: enable fully implemented facilities,
Aurelien Jarno <=
- Re: [Qemu-devel] [PATCH 07/10] target-s390x: enable fully implemented facilities, Alexander Graf, 2015/05/25
- Re: [Qemu-devel] [PATCH 07/10] target-s390x: enable fully implemented facilities, Aurelien Jarno, 2015/05/26
- Re: [Qemu-devel] [PATCH 07/10] target-s390x: enable fully implemented facilities, Alexander Graf, 2015/05/26
- Re: [Qemu-devel] [PATCH 07/10] target-s390x: enable fully implemented facilities, Aurelien Jarno, 2015/05/26
- Re: [Qemu-devel] [PATCH 07/10] target-s390x: enable fully implemented facilities, Alexander Graf, 2015/05/26
[Qemu-devel] [PATCH 02/10] target-s390x: fix MMU index computation, Aurelien Jarno, 2015/05/24
[Qemu-devel] [PATCH 06/10] target-s390x: improve facilities list, Aurelien Jarno, 2015/05/24
[Qemu-devel] [PATCH 08/10] target-s390x: implement STFLE instruction, Aurelien Jarno, 2015/05/24