From: Avi Kivity <avi@redhat.com>
To: Jason Wang <jasowang@redhat.com>
Cc: mtosatti@redhat.com, kvm@vger.kernel.org
Subject: Re: [PATCH kvm-unit-tests 07/10] Correct the tss size
Date: Wed, 25 Aug 2010 16:11:22 +0300 [thread overview]
Message-ID: <4C75167A.9080402@redhat.com> (raw)
In-Reply-To: <706252157.521401282741228357.JavaMail.root@zmail05.collab.prod.int.phx2.redhat.com>
On 08/25/2010 04:00 PM, Jason Wang wrote:
>
> 0000000000400bb8<__setup_args>:
> 400bb8: 41 55 push %r13
> 400bba: 41 54 push %r12
> 400bbc: 55 push %rbp
> 400bbd: 53 push %rbx
> 400bbe: 48 8b 1d db e7 00 00 mov 0xe7db(%rip),%rbx # 40f3a0<__args>
> 400bc5: 41 bc 80 ec 40 00 mov $0x40ec80,%r12d
> 400bcb: 41 bd 80 f0 40 00 mov $0x40f080,%r13d
> 400bd1: eb 42 jmp 400c15<__setup_args+0x5d>
> 400bd3: 4d 89 65 00 mov %r12,0x0(%r13)
> 400bd7: 0f b6 28 movzbl (%rax),%ebp
> 400bda: 40 84 ed test %bpl,%bpl
> 400bdd: 75 16 jne 400bf5<__setup_args+0x3d>
> 400bdf: eb 21 jmp 400c02<__setup_args+0x4a>
> 400be1: 41 88 2c 24 mov %bpl,(%r12)
> 400be5: 49 83 c4 01 add $0x1,%r12
> 400bed: 0f b6 2b movzbl (%rbx),%ebp
> 400bf0: 40 84 ed test %bpl,%bpl
> 400bf3: 74 0d je 400c02<__setup_args+0x4a>
> 400bf5: 40 0f be fd movsbl %bpl,%edi
> 400bf9: e8 a6 ff ff ff callq 400ba4<isblank>
> 400bfe: 84 c0 test %al,%al
> 400c00: 74 df je 400be1<__setup_args+0x29>
> 400c02: 49 83 c5 08 add $0x8,%r13
> 400c06: 41 c6 04 24 00 movb $0x0,(%r12)
> 400c0b: 49 83 c4 01 add $0x1,%r12
> 400c0f: eb 04 jmp 400c15<__setup_args+0x5d>
> 400c11: 48 83 c3 01 add $0x1,%rbx
>>> 400c15: 0f b6 2b movzbl (%rbx),%ebp
> 400c18: 40 0f be fd movsbl %bpl,%edi
> 400c1c: e8 83 ff ff ff callq 400ba4<isblank>
> ....
>
Looks completely unrelated. We access the command line string and fault.
You can hack qemu-kvm to vmstop() on KVM_EXIT_SHUTDOWN and examine
%rbx. It's probably corrupted. Maybe we need to copy the multiboot
command line early to prevent the rest of the code from stomping on it.
--
I have a truly marvellous patch that fixes the bug which this
signature is too narrow to contain.
next prev parent reply other threads:[~2010-08-25 13:11 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1917811696.509261282728232368.JavaMail.root@zmail05.collab.prod.int.phx2.redhat.com>
2010-08-25 9:40 ` [PATCH kvm-unit-tests 07/10] Correct the tss size Jason Wang
2010-08-25 9:55 ` Avi Kivity
2010-08-25 12:27 ` Jason Wang
2010-08-25 12:45 ` Avi Kivity
2010-08-25 13:00 ` Jason Wang
2010-08-25 13:11 ` Avi Kivity [this message]
[not found] <1014013070.604691282806259198.JavaMail.root@zmail05.collab.prod.int.phx2.redhat.com>
2010-08-26 7:05 ` Jason Wang
2010-08-26 8:06 ` Avi Kivity
2010-08-24 13:46 [PATCH kvm-unit-tests 00/10] Minor cleanup and fix for 32bit test Jason Wang
2010-08-24 13:47 ` [PATCH kvm-unit-tests 07/10] Correct the tss size Jason Wang
2010-08-24 13:57 ` Avi Kivity
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=4C75167A.9080402@redhat.com \
--to=avi@redhat.com \
--cc=jasowang@redhat.com \
--cc=kvm@vger.kernel.org \
--cc=mtosatti@redhat.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox