All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew Cooper <andrew.cooper3@citrix.com>
To: Fabio Fantoni <fabio.fantoni@m2r.biz>, Jan Beulich <JBeulich@suse.com>
Cc: xen-devel <xen-devel@lists.xenproject.org>, Keir Fraser <keir@xen.org>
Subject: Re: [PATCH] x86_emulate: properly do IP updates and other side effects on success
Date: Mon, 11 Aug 2014 14:32:12 +0100	[thread overview]
Message-ID: <53E8C5DC.6080306@citrix.com> (raw)
In-Reply-To: <53E8C225.9070705@m2r.biz>

On 11/08/14 14:16, Fabio Fantoni wrote:
> Il 11/08/2014 14:53, Fabio Fantoni ha scritto:
>> Il 07/08/2014 16:57, Fabio Fantoni ha scritto:
>>> Il 07/08/2014 16:38, Jan Beulich ha scritto:
>>>>>>> On 07.08.14 at 15:24, <fabio.fantoni@m2r.biz> wrote:
>>>>> Il 07/08/2014 10:30, Jan Beulich ha scritto:
>>>>>> The two MMX/SSE/AVX code blocks failed to update IP properly, and
>>>>>> these
>>>>>> as well as get_reg_refix(), which "manually" updated IP so far,
>>>>>> failed
>>>>>> to do the TF and RF processing needed at the end of successfully
>>>>>> emulated instructions.
>>>>>>
>>>>>> Reported-by: Andrei LUTAS <vlutas@bitdefender.com>
>>>>>> Signed-off-by: Jan Beulich <jbeulich@suse.com>
>>>>>> Tested-by: Razvan Cojocaru <rcojocaru@bitdefender.com>
>>>>> Thanks for the patch, I tested it to see if solves also qxl on linux
>>>>> domUs problem but with this patch the domU crashes and I cannot get a
>>>>> backtrace or specific errors.
>>>> Are you saying that this is different from without that patch?
>>>> Considering the lack of EIP update, the emulation of the
>>>> instructions in question can't have done much good before,
>>>> and - just like the reporters of the issue saw - the guest should
>>>> have hung.
>>>>
>>>> Jan
>>>>
>>>
>>> Before this patch the domU's org was always 100% cpu, spice show
>>> black screen and is possible a xl shutdown, after at xorg start,
>>> show domU's mouse in center in spice and domU crash.
>>> I'm unable to debug it, can you tell me what I must do to debug it
>>> and give you all possibile information?
>>>
>>> This is the exact git I used for my tests:
>>> https://github.com/Fantu/Xen/tree/rebase/m2r-staging
>>> Contains some libxl patch about spice used for a long time and with
>>> only qxl problem on linux domUs (with windows domUs is working good
>>> since your patch of long time ago about similar istructions and same
>>> performance as kvm after one or more patches in xen-unstable that I
>>> not know exactly and is still working also with this patch).
>>>
>>> Thanks for any reply and sorry for my bad english.
>>
>> Someone can tell me how to debug the domU's crash please?
>>
>> Thanks for any reply and sorry for my bad english.
>
> Now I did also a vcpus dump on domU preserved after crash if can be
> useful for this problem probably about cpu's instructions emulation:
> gdbsx -c 1 64
> ===> Context for DOMID:1
>
> --> VCPU:0

SHEDOP_shutdown.  Shutdown code in a structure at ffff880074803b3c

Looks like the VM has crashed.

~Andrew

> rip:ffffffff810013a8 rsp:ffff880074803b30 flags:0000000000000092
> rax:0000000000000000 rbx:00000000fffffffe rcx:00000000ffffffff
> rdx:ffffffff81edc300 rsi:ffff880074803b3c rdi:0000000000000002
> r08:0000000000000000 r09:0000000000000002 r10:ffffffff81d0a5e0
> r11:ffff8800748038c6 r12:ffffffff81c515c0 r13:0000000000000000
> r14:ffffffff81edc300 r15:0000000000000000 rbp:ffff880074803b40
> cs:0000000000000010 ds:0000000000000000 fs:0000000000000000
> gs:0000000000000000
>
> Call Trace:
>    [ffffffff810013a8]
>    [ffffffff8100357e]
>    [ffffffff810035a3]
>    [ffffffff816fa43c]
>    [ffffffff81a07727]
>    [ffffffff816fa49a]
>    [ffffffff816eb027]
>    [ffffffff810e17c8]
>    [ffffffff816f7603]
>    [ffffffff81a073db]
>    [ffffffff81016f9b]
>
> --> VCPU:1
> rip:ffffffff81053585 rsp:ffff880074823f88 flags:0000000000000046
> rax:ffffffff81c2a160 rbx:ffff88007342ffd8 rcx:00000006867023fa
> rdx:0000000000bad748 rsi:00000000000000ff rdi:00000000000000f0
> r08:00000000000000ac r09:0000000000000000 r10:0000000000000400
> r11:0000000000000001 r12:0000000000000001 r13:0000000000000000
> r14:ffff88007342c000 r15:ffff88007342ffd8 rbp:ffff880074823f88
> cs:0000000000000010 ds:0000000000000000 fs:0000000000000000
> gs:0000000000000000
>
> Call Trace:
>    [ffffffff81053585]
>    [ffffffff8101d70d]
>    [ffffffff810433b9]
>    [ffffffff8170005d]
>    [ffffffffffffffff]
>    [ffffffff810afd40]
>    [ffffffff8100a410]
>    [ffffffff8100a020]
>    [ffffffff81a06d7f]
>    [ffffffff8180b180]
>    [ffffffff81c2a858]
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel

  reply	other threads:[~2014-08-11 13:32 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-07  8:30 [PATCH] x86_emulate: properly do IP updates and other side effects on success Jan Beulich
2014-08-07 11:40 ` Andrew Cooper
2014-08-07 13:24 ` Fabio Fantoni
2014-08-07 14:38   ` Jan Beulich
2014-08-07 14:57     ` Fabio Fantoni
2014-08-11 12:53       ` Fabio Fantoni
2014-08-11 13:16         ` Fabio Fantoni
2014-08-11 13:32           ` Andrew Cooper [this message]
2014-08-11 14:14           ` Jan Beulich
2014-08-11 14:31             ` Fabio Fantoni
2014-08-07 14:40   ` Fabio Fantoni

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=53E8C5DC.6080306@citrix.com \
    --to=andrew.cooper3@citrix.com \
    --cc=JBeulich@suse.com \
    --cc=fabio.fantoni@m2r.biz \
    --cc=keir@xen.org \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.