some instructions translate error in x86

Bug #1751422 reported by yabi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
QEMU
Fix Released
Undecided
Unassigned

Bug Description

There is some instructions translation error on target i386 in many versions, such as 2.11.1, 2.10.2, 2.7.1 and so on.
The error translation instructions include les, lds. I has got a patch, but I have no idea how to apply it.

Revision history for this message
Thomas Huth (th-huth) wrote :

Could you please provide some more information about the problem? What's exactly the error? If you've already got a patch, please have a look at https://wiki.qemu.org/Contribute/SubmitAPatch to get some information how to submit it.

Changed in qemu:
status: New → Incomplete
Revision history for this message
yabi (yabi) wrote : Re:[Bug 1751422] Re: some instructions translate error in x86

The patch is In this mail attachments, which is patch for version 2.11.1 target/i386/translate.c.
The patch is created by diff.
my English is so poor to explain how the error come, but you can see the patch result to get it.

At 2018-02-25 17:41:15, "Thomas Huth" <email address hidden> wrote:
>Could you please provide some more information about the problem? What's
>exactly the error? If you've already got a patch, please have a look at
>https://wiki.qemu.org/Contribute/SubmitAPatch to get some information
>how to submit it.
>
>** Changed in: qemu
> Status: New => Incomplete
>
>--
>You received this bug notification because you are subscribed to the bug
>report.
>https://bugs.launchpad.net/bugs/1751422
>
>Title:
> some instructions translate error in x86
>
>Status in QEMU:
> Incomplete
>
>Bug description:
> There is some instructions translation error on target i386 in many versions, such as 2.11.1, 2.10.2, 2.7.1 and so on.
> The error translation instructions include les, lds. I has got a patch, but I have no idea how to apply it.
>
>To manage notifications about this bug go to:
>https://bugs.launchpad.net/qemu/+bug/1751422/+subscriptions

Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for QEMU because there has been no activity for 60 days.]

Changed in qemu:
status: Incomplete → Expired
Revision history for this message
Peter Maydell (pmaydell) wrote :

We shouldn't really have let this expire, the submitter has a patch attached to the bug.

Yabi: do you have a simple test program which fails without this patch and works with it? If so can you attach it to the bug ?

Changed in qemu:
status: Expired → New
Revision history for this message
Richard Henderson (rth) wrote :

I believe this to be fixed by cfcca361d77, which is present in 2.12 but not 2.11.

Revision history for this message
Peter Maydell (pmaydell) wrote :

Since Richard pointed out a commit which fixed this in 2.12 and we haven't heard back from the submitter, I'm going to close this bug as fixed.

Changed in qemu:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.