[PATCH 0/1] [PULL] qemu-kvm.git uq/master queue

[PATCH 0/1] [PULL] qemu-kvm.git uq/master queue

[Marcelo Tosatti]

The following changes since commit d33ea50a958b2e050d2b28e5f17e3b55e91c6d74:

  scsi-disk: Fix immediate failure of bdrv_aio_* (2010-11-04 13:54:37 +0100)

are available in the git repository at:
  git://git.kernel.org/pub/scm/virt/kvm/qemu-kvm.git uq/master

Gleb Natapov (1):
      Add support for async page fault to qemu

 target-i386/cpu.h     |    1 +
 target-i386/cpuid.c   |    2 +-
 target-i386/kvm.c     |   14 ++++++++++++++
 target-i386/machine.c |   26 ++++++++++++++++++++++++++
 4 files changed, 42 insertions(+), 1 deletions(-)

[PATCH 0/1] [PULL] qemu-kvm.git uq/master queue

[Marcelo Tosatti]

The following changes since commit d33ea50a958b2e050d2b28e5f17e3b55e91c6d74:

  scsi-disk: Fix immediate failure of bdrv_aio_* (2010-11-04 13:54:37 +0100)

are available in the git repository at:
  git://git.kernel.org/pub/scm/virt/kvm/qemu-kvm.git uq/master

Gleb Natapov (1):
      Add support for async page fault to qemu

 target-i386/cpu.h     |    1 +
 target-i386/cpuid.c   |    2 +-
 target-i386/kvm.c     |   14 ++++++++++++++
 target-i386/machine.c |   26 ++++++++++++++++++++++++++
 4 files changed, 42 insertions(+), 1 deletions(-)

[PATCH 0/1] [PULL] qemu-kvm.git uq/master queue

[Marcelo Tosatti]

The following changes since commit d11cf8cc80d946dfc9a23597cd9a0bb1c487cfa7:

  etrax-dma: Remove bogus if statement (2011-10-03 10:20:13 +0200)

are available in the git repository at:
  git://github.com/avikivity/qemu.git uq/master

Liu, Jinsong (1):
      kvm: support TSC deadline MSR

 target-i386/cpu.h     |    4 +++-
 target-i386/kvm.c     |   14 ++++++++++++++
 target-i386/machine.c |    1 +
 3 files changed, 18 insertions(+), 1 deletions(-)

Re: [PATCH 0/1] [PULL] qemu-kvm.git uq/master queue

[Avi Kivity]

On 10/10/2011 04:41 PM, Anthony Liguori wrote:
> On 10/03/2011 03:55 PM, Marcelo Tosatti wrote:
>> The following changes since commit 
>> d11cf8cc80d946dfc9a23597cd9a0bb1c487cfa7:
>>
>>    etrax-dma: Remove bogus if statement (2011-10-03 10:20:13 +0200)
>>
>> are available in the git repository at:
>>    git://github.com/avikivity/qemu.git uq/master
>
> Pulled.  Thanks.
>

Um, this had a comment about it regarding s/version bump/subsection/

> Are ya'll planning on moving your repo back to kernel.org or sticking 
> with github?

We'll move back to kernel.org as soon as we sort around the keys.

-- 
error compiling committee.c: too many arguments to function

Re: [PATCH 0/1] [PULL] qemu-kvm.git uq/master queue

[Anthony Liguori]

On 10/10/2011 09:48 AM, Avi Kivity wrote:
> On 10/10/2011 04:41 PM, Anthony Liguori wrote:
>> On 10/03/2011 03:55 PM, Marcelo Tosatti wrote:
>>> The following changes since commit d11cf8cc80d946dfc9a23597cd9a0bb1c487cfa7:
>>>
>>> etrax-dma: Remove bogus if statement (2011-10-03 10:20:13 +0200)
>>>
>>> are available in the git repository at:
>>> git://github.com/avikivity/qemu.git uq/master
>>
>> Pulled. Thanks.
>>
>
> Um, this had a comment about it regarding s/version bump/subsection/

Hrm, sorry about that.  In the future, it would be helpful to explicitly 
withdrawal a PULL request.

Do you want me to revert?  FWIW, I think bumping the version is the right thing 
to do.

Regards,

Anthony Liguori

>
>> Are ya'll planning on moving your repo back to kernel.org or sticking with
>> github?
>
> We'll move back to kernel.org as soon as we sort around the keys.
>

Re: [PATCH 0/1] [PULL] qemu-kvm.git uq/master queue

[Avi Kivity]

On 10/10/2011 05:12 PM, Anthony Liguori wrote:
> On 10/10/2011 10:00 AM, Avi Kivity wrote:
>> On 10/10/2011 04:55 PM, Anthony Liguori wrote:
>>>
>>> Hrm, sorry about that. In the future, it would be helpful to explicitly
>>> withdrawal a PULL request.
>>>
>>> Do you want me to revert?
>>
>> We'll send the revert together with the new patch.
>>
>>> FWIW, I think bumping the version is the right thing to do.
>>
>> Why?
>
> Because we still haven't fixed subsections.  Juan's patches help but 
> they can still result in sending bad data to and older QEMU that won't 
> handle it properly.
>
> We need to figure out a proper fix for subsections, either Paolo's 
> protocol change or moving subsections out to a detected section.

Ok.  So bumping the version is only right if we don't fix subsections.

-- 
error compiling committee.c: too many arguments to function

Re: [PATCH 0/1] [PULL] qemu-kvm.git uq/master queue

[Anthony Liguori]

On 10/10/2011 10:24 AM, Avi Kivity wrote:
> On 10/10/2011 05:12 PM, Anthony Liguori wrote:
>> On 10/10/2011 10:00 AM, Avi Kivity wrote:
>>> On 10/10/2011 04:55 PM, Anthony Liguori wrote:
>>>>
>>>> Hrm, sorry about that. In the future, it would be helpful to explicitly
>>>> withdrawal a PULL request.
>>>>
>>>> Do you want me to revert?
>>>
>>> We'll send the revert together with the new patch.
>>>
>>>> FWIW, I think bumping the version is the right thing to do.
>>>
>>> Why?
>>
>> Because we still haven't fixed subsections. Juan's patches help but they can
>> still result in sending bad data to and older QEMU that won't handle it properly.
>>
>> We need to figure out a proper fix for subsections, either Paolo's protocol
>> change or moving subsections out to a detected section.
>
> Ok. So bumping the version is only right if we don't fix subsections.

If we bump *any* version from 0.15 -> 1.0, then there's no point at all in 
having a subsection.  If we break compatibility by using Paolo's new protocol, 
or doing subsections as sections, then there's no point in making it a 
subsection either.

Regards,

Anthony Liguori

>

Re: [PATCH 0/1] [PULL] qemu-kvm.git uq/master queue

[Avi Kivity]

On 10/10/2011 05:36 PM, Anthony Liguori wrote:
>> Ok. So bumping the version is only right if we don't fix subsections.
>
>
> If we bump *any* version from 0.15 -> 1.0, then there's no point at 
> all in having a subsection. 

Did we bump versions of relevant devices?

> If we break compatibility by using Paolo's new protocol, or doing 
> subsections as sections, then there's no point in making it a 
> subsection either.

These are workaroundable.  For example if you migrate 0.15 to 1.0 you 
start the destination with -old-subsection-format.  Even if you don't, 
since subsections are rarely present, migration will succeed.

-- 
error compiling committee.c: too many arguments to function

[PATCH 0/1] [PULL] qemu-kvm.git uq/master queue

[Marcelo Tosatti]

The following changes since commit 1ccbc2851282564308f790753d7158487b6af8e2:

  qemu-sockets: Fix parsing of the inet option 'to'. (2012-11-21 12:07:59 +0400)

are available in the git repository at:
  git://git.kernel.org/pub/scm/virt/kvm/qemu-kvm.git uq/master

Bruce Rogers (1):
      Legacy qemu-kvm options have no argument

 qemu-options.hx |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

[PATCH 1/1] Legacy qemu-kvm options have no argument

[Marcelo Tosatti]

From: Bruce Rogers <brogers@suse.com>

The options no-kvm, no-kvm-pit, no-kvm-pit-reinjection, and no-kvm-irqchip
should be marked as having no argument.

Signed-off-by: Bruce Rogers <brogers@suse.com>
Reviewed-by: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
---
 qemu-options.hx |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/qemu-options.hx b/qemu-options.hx
index 9bb29d3..fbcf079 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -2918,17 +2918,17 @@ Enable FIPS 140-2 compliance mode.
 ETEXI
 
 HXCOMM Deprecated by -machine accel=tcg property
-DEF("no-kvm", HAS_ARG, QEMU_OPTION_no_kvm, "", QEMU_ARCH_I386)
+DEF("no-kvm", 0, QEMU_OPTION_no_kvm, "", QEMU_ARCH_I386)
 
 HXCOMM Deprecated by kvm-pit driver properties
-DEF("no-kvm-pit-reinjection", HAS_ARG, QEMU_OPTION_no_kvm_pit_reinjection,
+DEF("no-kvm-pit-reinjection", 0, QEMU_OPTION_no_kvm_pit_reinjection,
     "", QEMU_ARCH_I386)
 
 HXCOMM Deprecated (ignored)
-DEF("no-kvm-pit", HAS_ARG, QEMU_OPTION_no_kvm_pit, "", QEMU_ARCH_I386)
+DEF("no-kvm-pit", 0, QEMU_OPTION_no_kvm_pit, "", QEMU_ARCH_I386)
 
 HXCOMM Deprecated by -machine kernel_irqchip=on|off property
-DEF("no-kvm-irqchip", HAS_ARG, QEMU_OPTION_no_kvm_irqchip, "", QEMU_ARCH_I386)
+DEF("no-kvm-irqchip", 0, QEMU_OPTION_no_kvm_irqchip, "", QEMU_ARCH_I386)
 
 HXCOMM Deprecated (ignored)
 DEF("tdf", 0, QEMU_OPTION_tdf,"", QEMU_ARCH_ALL)
-- 
1.7.6.4

Re: [Qemu-devel] [PATCH 1/1] Legacy qemu-kvm options have no argument

[Andreas Färber]

Am 23.11.2012 13:41, schrieb Marcelo Tosatti:
> From: Bruce Rogers <brogers@suse.com>
> 
> The options no-kvm, no-kvm-pit, no-kvm-pit-reinjection, and no-kvm-irqchip
> should be marked as having no argument.
> 
> Signed-off-by: Bruce Rogers <brogers@suse.com>
> Reviewed-by: Jan Kiszka <jan.kiszka@siemens.com>

Missing a

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>

from earlier today.

Andreas

> Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
> ---
>  qemu-options.hx |    8 ++++----
>  1 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/qemu-options.hx b/qemu-options.hx
> index 9bb29d3..fbcf079 100644
> --- a/qemu-options.hx
> +++ b/qemu-options.hx
> @@ -2918,17 +2918,17 @@ Enable FIPS 140-2 compliance mode.
>  ETEXI
>  
>  HXCOMM Deprecated by -machine accel=tcg property
> -DEF("no-kvm", HAS_ARG, QEMU_OPTION_no_kvm, "", QEMU_ARCH_I386)
> +DEF("no-kvm", 0, QEMU_OPTION_no_kvm, "", QEMU_ARCH_I386)
>  
>  HXCOMM Deprecated by kvm-pit driver properties
> -DEF("no-kvm-pit-reinjection", HAS_ARG, QEMU_OPTION_no_kvm_pit_reinjection,
> +DEF("no-kvm-pit-reinjection", 0, QEMU_OPTION_no_kvm_pit_reinjection,
>      "", QEMU_ARCH_I386)
>  
>  HXCOMM Deprecated (ignored)
> -DEF("no-kvm-pit", HAS_ARG, QEMU_OPTION_no_kvm_pit, "", QEMU_ARCH_I386)
> +DEF("no-kvm-pit", 0, QEMU_OPTION_no_kvm_pit, "", QEMU_ARCH_I386)
>  
>  HXCOMM Deprecated by -machine kernel_irqchip=on|off property
> -DEF("no-kvm-irqchip", HAS_ARG, QEMU_OPTION_no_kvm_irqchip, "", QEMU_ARCH_I386)
> +DEF("no-kvm-irqchip", 0, QEMU_OPTION_no_kvm_irqchip, "", QEMU_ARCH_I386)
>  
>  HXCOMM Deprecated (ignored)
>  DEF("tdf", 0, QEMU_OPTION_tdf,"", QEMU_ARCH_ALL)
> 


-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

Re: [PATCH 0/1] [PULL] qemu-kvm.git uq/master queue

[Anthony Liguori]

Marcelo Tosatti <mtosatti@redhat.com> writes:

> The following changes since commit 1ccbc2851282564308f790753d7158487b6af8e2:
>
>   qemu-sockets: Fix parsing of the inet option 'to'. (2012-11-21 12:07:59 +0400)
>
> are available in the git repository at:
>   git://git.kernel.org/pub/scm/virt/kvm/qemu-kvm.git uq/master
>
> Bruce Rogers (1):
>       Legacy qemu-kvm options have no argument

Pulled. Thanks.

Regards,

Anthony Liguori

>
>  qemu-options.hx |    8 ++++----
>  1 files changed, 4 insertions(+), 4 deletions(-)
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 0/1] [PULL] qemu-kvm.git uq/master queue

[Gleb Natapov]

Anthony please pull if it is not too later for 1.5.

The following changes since commit 467b34689d277fa56c09ad07ca0f08d7d7539f6d:

  Update OpenBIOS images (2013-05-05 09:53:22 +0000)

are available in the git repository at:

  git://git.kernel.org/pub/scm/virt/kvm/qemu-kvm.git uq/master

for you to fetch changes up to 0c1cd0ae2a4faabeb948b9a07ea1696e853de174:

  kvmvapic: add ioport read accessor (2013-05-06 14:52:26 +0300)

----------------------------------------------------------------
Marcelo Tosatti (1):
      kvmvapic: add ioport read accessor

 hw/i386/kvmvapic.c |    6 ++++++
 1 file changed, 6 insertions(+)