[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 1/4] s390x/tcg: Implement Miscellaneous-Instruction-Extens
From: |
David Miller |
Subject: |
Re: [PATCH v6 1/4] s390x/tcg: Implement Miscellaneous-Instruction-Extensions Facility 3 for the s390x |
Date: |
Wed, 23 Feb 2022 16:11:33 -0500 |
> Bit 0 controls this, and recall that IBM uses big-bit numbering, so "8".
> This stores the low part of r[23] in the high part of r1.
> You need to select the high part of r[23].
good catch, these are both fixed will update patch shortly.
Thanks for the review
- David Miller
On Wed, Feb 23, 2022 at 2:41 PM Richard Henderson
<richard.henderson@linaro.org> wrote:
>
> On 2/17/22 13:17, David Miller wrote:
> > +/* SELECT HIGH */
> > + C(0xb9c0, SELFHR, RRF_a, MIE3, r3, r2, new, r1_32h, loc, 0)
>
> This stores the low part of r[23] in the high part of r1.
> You need to select the high part of r[23].
>
> > static DisasJumpType op_popcnt(DisasContext *s, DisasOps *o)
> > {
> > - gen_helper_popcnt(o->out, o->in2);
> > + const uint8_t m3 = get_field(s, m3);
> > +
> > + if ((m3 & 1) && s390_has_feat(S390_FEAT_MISC_INSTRUCTION_EXT3)) {
>
> Bit 0 controls this, and recall that IBM uses big-bit numbering, so "8".
>
>
> r~
- Re: [PATCH v6 4/4] tests/tcg/s390x: changed to using .insn for tests requiring z15, (continued)
[PATCH v6 1/4] s390x/tcg: Implement Miscellaneous-Instruction-Extensions Facility 3 for the s390x, David Miller, 2022/02/17
Re: [PATCH v6 1/4] s390x/tcg: Implement Miscellaneous-Instruction-Extensions Facility 3 for the s390x, Richard Henderson, 2022/02/23
- Re: [PATCH v6 1/4] s390x/tcg: Implement Miscellaneous-Instruction-Extensions Facility 3 for the s390x,
David Miller <=