Re: [PATCH 0/6] kvm: Emulate MOVBE, v3

2013-10-30 Thread Borislav Petkov
On Wed, Oct 30, 2013 at 07:10:56PM +0100, Paolo Bonzini wrote: > I tested this patch manually with a small C test program and asm. > > Would you please prepare a patch to kvm-unit-tests that stresses the new > codepath? Ha, now that you bring that up, I think Andre already had something ready.

Re: [PATCH 0/6] kvm: Emulate MOVBE, v3

2013-10-30 Thread Paolo Bonzini
Il 22/09/2013 16:44, Borislav Petkov ha scritto: > From: Borislav Petkov > > Alriiight, > > here's another version of the patchset, hopefully addressing all review > feedback from last time. 6/6 is the respective qemu patch to handle > emulated features query, etc. > > It is still a lot of fun

Re: [PATCH 0/6] kvm: Emulate MOVBE, v3

2013-10-30 Thread Paolo Bonzini
Il 22/09/2013 16:44, Borislav Petkov ha scritto: From: Borislav Petkov b...@suse.de Alriiight, here's another version of the patchset, hopefully addressing all review feedback from last time. 6/6 is the respective qemu patch to handle emulated features query, etc. It is still a lot of

Re: [PATCH 0/6] kvm: Emulate MOVBE, v3

2013-10-30 Thread Borislav Petkov
On Wed, Oct 30, 2013 at 07:10:56PM +0100, Paolo Bonzini wrote: I tested this patch manually with a small C test program and asm. Would you please prepare a patch to kvm-unit-tests that stresses the new codepath? Ha, now that you bring that up, I think Andre already had something ready.

Re: [PATCH 0/6] kvm: Emulate MOVBE, v3

2013-10-29 Thread Borislav Petkov
On Tue, Oct 29, 2013 at 01:36:03PM +0200, Gleb Natapov wrote: > git://git.kernel.org/pub/scm/virt/kvm/kvm.git queue branch. Ok, they still apply. I'm sending 4 and 5 (5 needed changes too) as a reply to this message. > That's why we got "max users limit is reached" error! (only for the > first

Re: [PATCH 0/6] kvm: Emulate MOVBE, v3

2013-10-29 Thread Gleb Natapov
On Tue, Oct 29, 2013 at 12:28:24PM +0100, Borislav Petkov wrote: > On Tue, Oct 29, 2013 at 12:35:24PM +0200, Gleb Natapov wrote: > > If everything still applies patch 4 resend should be fine. > > Which tree should they still apply to? > git://git.kernel.org/pub/scm/virt/kvm/kvm.git queue branch.

Re: [PATCH 0/6] kvm: Emulate MOVBE, v3

2013-10-29 Thread Borislav Petkov
On Tue, Oct 29, 2013 at 12:35:24PM +0200, Gleb Natapov wrote: > If everything still applies patch 4 resend should be fine. Which tree should they still apply to? > Yeah, kvm forum was not at the same building as LinuxCon :( Yep, I know. We did go to the kvm forum to use the wifi there because

Re: [PATCH 0/6] kvm: Emulate MOVBE, v3

2013-10-29 Thread Gleb Natapov
On Tue, Oct 29, 2013 at 11:30:03AM +0100, Borislav Petkov wrote: > On Tue, Oct 29, 2013 at 11:53:00AM +0200, Gleb Natapov wrote: > > Except a small nitpick for patch 4 this looks good to me. No need to > > wait for QEMU cpu parsing code to change to apply it. > > Ok, want a resend or rather a

Re: [PATCH 0/6] kvm: Emulate MOVBE, v3

2013-10-29 Thread Borislav Petkov
On Tue, Oct 29, 2013 at 11:53:00AM +0200, Gleb Natapov wrote: > Except a small nitpick for patch 4 this looks good to me. No need to > wait for QEMU cpu parsing code to change to apply it. Ok, want a resend or rather a pull request or only patch 4 resend? Btw, we kinda missed each other in

Re: [PATCH 0/6] kvm: Emulate MOVBE, v3

2013-10-29 Thread Gleb Natapov
On Sun, Sep 22, 2013 at 04:44:49PM +0200, Borislav Petkov wrote: > From: Borislav Petkov > > Alriiight, > > here's another version of the patchset, hopefully addressing all review > feedback from last time. 6/6 is the respective qemu patch to handle > emulated features query, etc. > Except a

Re: [PATCH 0/6] kvm: Emulate MOVBE, v3

2013-10-29 Thread Gleb Natapov
On Sun, Sep 22, 2013 at 04:44:49PM +0200, Borislav Petkov wrote: From: Borislav Petkov b...@suse.de Alriiight, here's another version of the patchset, hopefully addressing all review feedback from last time. 6/6 is the respective qemu patch to handle emulated features query, etc. Except

Re: [PATCH 0/6] kvm: Emulate MOVBE, v3

2013-10-29 Thread Borislav Petkov
On Tue, Oct 29, 2013 at 11:53:00AM +0200, Gleb Natapov wrote: Except a small nitpick for patch 4 this looks good to me. No need to wait for QEMU cpu parsing code to change to apply it. Ok, want a resend or rather a pull request or only patch 4 resend? Btw, we kinda missed each other in

Re: [PATCH 0/6] kvm: Emulate MOVBE, v3

2013-10-29 Thread Gleb Natapov
On Tue, Oct 29, 2013 at 11:30:03AM +0100, Borislav Petkov wrote: On Tue, Oct 29, 2013 at 11:53:00AM +0200, Gleb Natapov wrote: Except a small nitpick for patch 4 this looks good to me. No need to wait for QEMU cpu parsing code to change to apply it. Ok, want a resend or rather a pull

Re: [PATCH 0/6] kvm: Emulate MOVBE, v3

2013-10-29 Thread Borislav Petkov
On Tue, Oct 29, 2013 at 12:35:24PM +0200, Gleb Natapov wrote: If everything still applies patch 4 resend should be fine. Which tree should they still apply to? Yeah, kvm forum was not at the same building as LinuxCon :( Yep, I know. We did go to the kvm forum to use the wifi there because the

Re: [PATCH 0/6] kvm: Emulate MOVBE, v3

2013-10-29 Thread Gleb Natapov
On Tue, Oct 29, 2013 at 12:28:24PM +0100, Borislav Petkov wrote: On Tue, Oct 29, 2013 at 12:35:24PM +0200, Gleb Natapov wrote: If everything still applies patch 4 resend should be fine. Which tree should they still apply to? git://git.kernel.org/pub/scm/virt/kvm/kvm.git queue branch.

Re: [PATCH 0/6] kvm: Emulate MOVBE, v3

2013-10-29 Thread Borislav Petkov
On Tue, Oct 29, 2013 at 01:36:03PM +0200, Gleb Natapov wrote: git://git.kernel.org/pub/scm/virt/kvm/kvm.git queue branch. Ok, they still apply. I'm sending 4 and 5 (5 needed changes too) as a reply to this message. That's why we got max users limit is reached error! (only for the first half

[PATCH 0/6] kvm: Emulate MOVBE, v3

2013-09-22 Thread Borislav Petkov
From: Borislav Petkov Alriiight, here's another version of the patchset, hopefully addressing all review feedback from last time. 6/6 is the respective qemu patch to handle emulated features query, etc. It is still a lot of fun to generate fast! Atom 32-bit SMP guests like this: [

[PATCH 0/6] kvm: Emulate MOVBE, v3

2013-09-22 Thread Borislav Petkov
From: Borislav Petkov b...@suse.de Alriiight, here's another version of the patchset, hopefully addressing all review feedback from last time. 6/6 is the respective qemu patch to handle emulated features query, etc. It is still a lot of fun to generate fast! Atom 32-bit SMP guests like this: [