[PATCH unit-tests 06/16] Specify correct operand length for ltr and str.

2010-12-22 Thread Gleb Natapov
Signed-off-by: Gleb Natapov --- lib/x86/processor.h |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/x86/processor.h b/lib/x86/processor.h index c348808..c3ab109 100644 --- a/lib/x86/processor.h +++ b/lib/x86/processor.h @@ -193,13 +193,13 @@ static inline u16 sldt

Re: [PATCH unit-tests 06/16] Specify correct operand length for ltr and str.

2010-12-29 Thread Avi Kivity
On 12/22/2010 05:06 PM, Gleb Natapov wrote: Signed-off-by: Gleb Natapov --- lib/x86/processor.h |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/x86/processor.h b/lib/x86/processor.h index c348808..c3ab109 100644 --- a/lib/x86/processor.h +++ b/lib/x86/processor.

Re: [PATCH unit-tests 06/16] Specify correct operand length for ltr and str.

2010-12-29 Thread Gleb Natapov
On Wed, Dec 29, 2010 at 11:37:58AM +0200, Avi Kivity wrote: > On 12/22/2010 05:06 PM, Gleb Natapov wrote: > >Signed-off-by: Gleb Natapov > >--- > > lib/x86/processor.h |4 ++-- > > 1 files changed, 2 insertions(+), 2 deletions(-) > > > >diff --git a/lib/x86/processor.h b/lib/x86/processor.h >

Re: [PATCH unit-tests 06/16] Specify correct operand length for ltr and str.

2010-12-29 Thread Gleb Natapov
On Wed, Dec 29, 2010 at 11:40:15AM +0200, Gleb Natapov wrote: > On Wed, Dec 29, 2010 at 11:37:58AM +0200, Avi Kivity wrote: > > On 12/22/2010 05:06 PM, Gleb Natapov wrote: > > >Signed-off-by: Gleb Natapov > > >--- > > > lib/x86/processor.h |4 ++-- > > > 1 files changed, 2 insertions(+), 2 del

Re: [PATCH unit-tests 06/16] Specify correct operand length for ltr and str.

2010-12-29 Thread Avi Kivity
On 12/29/2010 11:43 AM, Gleb Natapov wrote: > > > > This looks completely unneeded, since val is already a u16. > > > IIRC I got errors from assembler. Will recheck. > You are correct. The error was only about ltr. Changing unsigned to u16 fixes it. Ok, I'll fix it. Applied all the other