Re: [PATCH kvm-unit-tests] realmode: test AAM, XLAT, SALC

[Gleb Natapov <gleb@redhat.com>]

On Thu, May 09, 2013 at 11:18:07AM +0200, Paolo Bonzini wrote:
> Il 09/05/2013 11:13, Gleb Natapov ha scritto:
> > On Thu, May 09, 2013 at 10:53:37AM +0200, Paolo Bonzini wrote:
> >> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> >> ---
> >>  x86/realmode.c | 40 ++++++++++++++++++++++++++++++++++++++++
> >>  1 file changed, 40 insertions(+)
> >>
> >> diff --git a/x86/realmode.c b/x86/realmode.c
> >> index 91c93a9..35ace08 100644
> >> --- a/x86/realmode.c
> >> +++ b/x86/realmode.c
> >> @@ -1391,6 +1391,43 @@ static void test_aad(void)
> >>      report("aad", R_AX, outregs.eax == 0x123400d4);
> >>  }
> >>  
> >> +static void test_aam(void)
> >> +{
> >> +    MK_INSN(aam, "aam");
> >> +
> >> +    inregs.eax = 0x76543210;
> >> +    exec_in_big_real_mode(&insn_aam);
> >> +    report("aam", R_AX, outregs.eax == 0x76540106);
> >> +}
> >> +
> >> +static void test_xlat(void)
> >> +{
> >> +    MK_INSN(xlat, "xlat");
> >> +    u8 table[256];
> >> +    int i;
> >> +
> >> +    for (i = 0; i < 256; i++) {
> >> +        table[i] = i + 1;
> >> +    }
> >> +
> >> +    inregs.eax = 0x89abcdef;
> >> +    inregs.ebx = (u32)table;
> >> +    exec_in_big_real_mode(&insn_xlat);
> >> +    report("xlat", R_AX, outregs.eax == 0x89abcdf0);
> >> +}
> >> +
> >> +static void test_salc(void)
> >> +{
> >> +    MK_INSN(clc_salc, "clc; .byte 0xd6");
> >> +    MK_INSN(stc_salc, "stc; .byte 0xd6");
> > Heh, who uses that?
> 
> Some real-mode programs do.  Plus, the "N" in the opcode table looked
> really bad. :)
> 
Ah, I thought sgabios use that one too.

--
			Gleb.