target-i386: Fix addr16 prefix
authorPaolo Bonzini <pbonzini@redhat.com>
Wed, 2 Mar 2016 15:04:38 +0000 (16:04 +0100)
committerRichard Henderson <rth@twiddle.net>
Mon, 14 Mar 2016 17:52:48 +0000 (10:52 -0700)
commite2e02a820741ec4d96b8f313b06a2a7ed5e94fbd
tree6e0df6c61fbf4b602e6bccc175830d95222c6319
parenta657f79e32422634415c09f3f15c73d610297af5
target-i386: Fix addr16 prefix

While ADDSEG will only be false in 16-bit mode for LEA, it can be
false even in other cases when 16-bit addresses are obtained via
the 67h prefix in 32-bit mode.  In this case, gen_lea_v_seg forgets
to add a nonzero FS or GS base if CS/DS/ES/SS are all zero.  This
case is pretty rare but happens when booting Windows 95/98, and
this patch fixes it.

The bug is visible since commit d6a291498, but it was introduced
together with gen_lea_v_seg and it probably could be reproduced
with a "addr16 gs movsb" instruction as early as in commit
ca2f29f555805d07fb0b9ebfbbfc4e3656530977.

Reported-by: Hervé Poussineau <hpoussin@reactos.org>
Tested-by: Hervé Poussineau <hpoussin@reactos.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <1456931078-21635-1-git-send-email-pbonzini@redhat.com>
Signed-off-by: Richard Henderson <rth@twiddle.net>
target-i386/translate.c