Re: [PATCH kvm-unit-tests 07/10] Correct the tss size

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



----- "Avi Kivity" <avi@xxxxxxxxxx> wrote:

> On 08/24/2010 04:47 PM, Jason Wang wrote:
> > TSS size should be 104 byte.
> >
> > Signed-off-by: Jason Wang<jasowang@xxxxxxxxxx>
> > ---
> >   x86/cstart64.S |    2 +-
> >   1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/x86/cstart64.S b/x86/cstart64.S
> > index 5d358ad..b871153 100644
> > --- a/x86/cstart64.S
> > +++ b/x86/cstart64.S
> > @@ -69,7 +69,7 @@ tss:
> >   	.long 0
> >   	.quad ring0stacktop - i * 4096
> 
> ring 0 stack
> 
> >   	.quad 0, 0, 0
> 
> rings 1, 2, 3 stack

Hello avi:

Rechek with the manual, there's no filed of RSP3. So this patch may
make sense. But unfortunately it breaks 64bit vmexit test. Triple
fault happens in setup_args(). Any suggestions or is there any thing I
missed?

> 
> > -	.quad 0, 0, 0, 0, 0, 0, 0, 0
> 
> 1 qword reserved, 7 qwords IST
> 
> > +	.quad 0, 0, 0, 0, 0, 0, 0
> >   	.long 0, 0, 0
> 
> 3 dwords reserved + I/O map base address - so this looks correct?
> 
> >   i = i + 1
> >   	.endr
> >
> 
> 
> -- 
> error compiling committee.c: too many arguments to function
> 
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux