Re: [Qemu-devel] [PATCH] qemu-kvm bugfix for IA32_FEATURE_CONTROL

2013-08-18 Thread Stefan Weil
Am 18.08.2013 20:23, schrieb Liu, Jinsong: > From 1273f8b2e5464ec987facf9942fd3ccc0b69087e Mon Sep 17 00:00:00 2001 > From: Liu Jinsong > Date: Mon, 19 Aug 2013 09:33:30 +0800 > Subject: [PATCH] qemu-kvm bugfix for IA32_FEATURE_CONTROL > > This patch is to fix the bug https://bugs.launchpad.net/qe

Re: [Qemu-devel] [PATCH] qemu-kvm bugfix for IA32_FEATURE_CONTROL

2013-08-19 Thread Andreas Färber
Am 19.08.2013 16:31, schrieb Liu, Jinsong: > Paolo Bonzini wrote: >> >> The patch looks good. Please repost it with checkpatch.pl failures >> fixed. >> >> Paolo > > Thanks Stefan and Paolo! Updated patch attached. > > Regards, > Jinsong > > === > From a0ddf948d40e42de862543157a5668

Re: [Qemu-devel] [PATCH] qemu-kvm bugfix for IA32_FEATURE_CONTROL

2013-08-19 Thread Paolo Bonzini
Il 19/08/2013 16:59, Andreas Färber ha scritto: > qemu-kvm is no longer maintained since 1.3 so it should not be occurring > any more. > > Please use a prefix of "target-i386: " (the directory name) to signal > where you are changing code, i.e. x86 only. > > "bugfix" is not a very telling descrip

RE: [Qemu-devel] [PATCH] qemu-kvm bugfix for IA32_FEATURE_CONTROL

2013-08-19 Thread Liu, Jinsong
Andreas Färber wrote: > Am 19.08.2013 16:31, schrieb Liu, Jinsong: >> Paolo Bonzini wrote: >>> >>> The patch looks good. Please repost it with checkpatch.pl failures >>> fixed. >>> >>> Paolo >> >> Thanks Stefan and Paolo! Updated patch attached. >> >> Regards, >> Jinsong >> >> ==

RE: [Qemu-devel] [PATCH] qemu-kvm bugfix for IA32_FEATURE_CONTROL

2013-08-19 Thread Liu, Jinsong
Paolo Bonzini wrote: > Il 19/08/2013 16:59, Andreas Färber ha scritto: >> qemu-kvm is no longer maintained since 1.3 so it should not be >> occurring any more. >> >> Please use a prefix of "target-i386: " (the directory name) to signal >> where you are changing code, i.e. x86 only. >> >> "bugfix

Re: [Qemu-devel] [PATCH] qemu-kvm bugfix for IA32_FEATURE_CONTROL

2013-08-20 Thread Paolo Bonzini
Il 20/08/2013 05:33, Liu, Jinsong ha scritto: > Thanks Andreas! > > This patch is for qemu-kvm. Even though the repository is still called qemu-kvm, the uq/master branch is the only active one and patches there will end up in upstream QEMU. There are no qemu-kvm releases anymore. I applied the