Commit Message

On Mon, 8 Oct 2012, Andriy Gapon wrote:
> > I am running Qemu (plain, no kvm, etc) on an AMD 10h machine that> provides popcnt instruction. Qemu advertises availability of pocnt> to a guest as well. What I see in the guest that popcnt> 0x20(%r12),%r8 instruction actually placed its result into %rax.> With %rdi and %rax operands the instruction worked fine though.> >
Does following work?

On Thu, 11 Oct 2012, Andriy Gapon wrote:
> on 08/10/2012 12:02 Andriy Gapon said the following:> > on 08/10/2012 10:52 malc said the following:> >> On Mon, 8 Oct 2012, Andriy Gapon wrote:> >>> >>>> >>> I am running Qemu (plain, no kvm, etc) on an AMD 10h machine that> >>> provides popcnt instruction. Qemu advertises availability of pocnt> >>> to a guest as well. What I see in the guest that popcnt> >>> 0x20(%r12),%r8 instruction actually placed its result into %rax.> >>> With %rdi and %rax operands the instruction worked fine though.> >>>> >>>> >>> >> Does following work?> > > > It does! Thank you very much.> > Do you plan to commit this fix?> Is there anything that I should do to make that happen (sooner)?>
Submit a patch with a well thought out comment and it'll be commited.
[..snip..]

on 11/10/2012 00:09 malc said the following:
> On Thu, 11 Oct 2012, Andriy Gapon wrote:> >> on 08/10/2012 12:02 Andriy Gapon said the following:>>> on 08/10/2012 10:52 malc said the following:>>>> On Mon, 8 Oct 2012, Andriy Gapon wrote:>>>>>>>>>>>>>> I am running Qemu (plain, no kvm, etc) on an AMD 10h machine that>>>>> provides popcnt instruction. Qemu advertises availability of pocnt>>>>> to a guest as well. What I see in the guest that popcnt>>>>> 0x20(%r12),%r8 instruction actually placed its result into %rax.>>>>> With %rdi and %rax operands the instruction worked fine though.>>>>>>>>>>>>>>>>>> Does following work?>>>>>> It does! Thank you very much.>>>> Do you plan to commit this fix?>> Is there anything that I should do to make that happen (sooner)?>>> > Submit a patch with a well thought out comment and it'll be commited.> > [..snip..]>
Hmm... Since you are the author of the patch, wouldn't be more appropriate for
you to submit it? Besides, I can only mostly repeat the bug report as I do not
quite understand the code and can not properly describe what the patch does.
Sorry for not taking up the work, but I am really just a qemu user.

On Sun, 14 Oct 2012, Andriy Gapon wrote:
[..snip..]
>> Hmm... Since you are the author of the patch, wouldn't be more > appropriate for you to submit it? Besides, I can only mostly repeat the > bug report as I do not quite understand the code and can not properly > describe what the patch does.
Sigh, okay, commited.
> > Sorry for not taking up the work, but I am really just a qemu user.> >

on 14/10/2012 13:56 malc said the following:
> On Sun, 14 Oct 2012, Andriy Gapon wrote:> > [..snip..]> >>>> Hmm... Since you are the author of the patch, wouldn't be more >> appropriate for you to submit it? Besides, I can only mostly repeat the >> bug report as I do not quite understand the code and can not properly >> describe what the patch does.> > Sigh, okay, commited.
Thank you again!
>>>> Sorry for not taking up the work, but I am really just a qemu user.>>>>>