[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(-)

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(+)

[Qemu-devel] [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(+)

[PATCH 1/1] kvmvapic: add ioport read accessor

[Gleb Natapov]

From: Marcelo Tosatti <mtosatti@redhat.com>

Necessary since memory region accessor assumes read and write
methods are registered. Otherwise reading I/O port 0x7e segfaults.

https://bugzilla.redhat.com/show_bug.cgi?id=954306

Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
Reviewed-by: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: Gleb Natapov <gleb@redhat.com>
---
 hw/i386/kvmvapic.c |    6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/hw/i386/kvmvapic.c b/hw/i386/kvmvapic.c
index 5b558aa..655483b 100644
--- a/hw/i386/kvmvapic.c
+++ b/hw/i386/kvmvapic.c
@@ -687,8 +687,14 @@ static void vapic_write(void *opaque, hwaddr addr, uint64_t data,
     }
 }
 
+static uint64_t vapic_read(void *opaque, hwaddr addr, unsigned size)
+{
+    return 0xffffffff;
+}
+
 static const MemoryRegionOps vapic_ops = {
     .write = vapic_write,
+    .read = vapic_read,
     .endianness = DEVICE_NATIVE_ENDIAN,
 };
 
-- 
1.7.10.4

[Qemu-devel] [PATCH 1/1] kvmvapic: add ioport read accessor

[Gleb Natapov]

From: Marcelo Tosatti <mtosatti@redhat.com>

Necessary since memory region accessor assumes read and write
methods are registered. Otherwise reading I/O port 0x7e segfaults.

https://bugzilla.redhat.com/show_bug.cgi?id=954306

Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
Reviewed-by: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: Gleb Natapov <gleb@redhat.com>
---
 hw/i386/kvmvapic.c |    6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/hw/i386/kvmvapic.c b/hw/i386/kvmvapic.c
index 5b558aa..655483b 100644
--- a/hw/i386/kvmvapic.c
+++ b/hw/i386/kvmvapic.c
@@ -687,8 +687,14 @@ static void vapic_write(void *opaque, hwaddr addr, uint64_t data,
     }
 }
 
+static uint64_t vapic_read(void *opaque, hwaddr addr, unsigned size)
+{
+    return 0xffffffff;
+}
+
 static const MemoryRegionOps vapic_ops = {
     .write = vapic_write,
+    .read = vapic_read,
     .endianness = DEVICE_NATIVE_ENDIAN,
 };
 
-- 
1.7.10.4