Re: [PATCH 0/3] fix bugs with stale or corrupt MMIO caches

2014-09-02 Thread Xiao Guangrong
On 09/02/2014 11:42 PM, Paolo Bonzini wrote:
> Il 29/08/2014 12:31, Paolo Bonzini ha scritto:
>> David and Xiao, here's my take on the MMIO generation patches.  Now
>> with documentation, too. :)  Please review!
>>
>> David Matlack (2):
>>   kvm: fix potentially corrupt mmio cache
>>   kvm: x86: fix stale mmio cache bug
>>
>> Paolo Bonzini (1):
>>   KVM: do not bias the generation number in kvm_current_mmio_generation
>>
>>  Documentation/virtual/kvm/mmu.txt | 14 ++
>>  arch/x86/include/asm/kvm_host.h   |  1 +
>>  arch/x86/kvm/mmu.c| 29 ++---
>>  arch/x86/kvm/x86.h| 20 +++-
>>  virt/kvm/kvm_main.c   | 30 +++---
>>  5 files changed, 67 insertions(+), 27 deletions(-)
>>
> 
> Ping?

Looks good to me.

Reviewed-by: Xiao Guangrong 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/3] fix bugs with stale or corrupt MMIO caches

2014-09-02 Thread David Matlack
On Tue, Sep 2, 2014 at 9:50 AM, Paolo Bonzini  wrote:
> Il 02/09/2014 18:47, David Matlack ha scritto:
>>> > Ping?
>> Sorry for the delay. I think the patches look good. And patch 3/3 still
>> fixes the bug I was originally seeing, so I'm happy :). I just had one
>> small comment (see my reply to patch 2/3).
>>
>
> I answered that question now.  Can I add your Reviewed-by and, for patch
> 3, Tested-by?

Please do. And thanks again for working on these!

>
> Paolo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/3] fix bugs with stale or corrupt MMIO caches

2014-09-02 Thread Paolo Bonzini
Il 02/09/2014 18:47, David Matlack ha scritto:
>> > Ping?
> Sorry for the delay. I think the patches look good. And patch 3/3 still
> fixes the bug I was originally seeing, so I'm happy :). I just had one
> small comment (see my reply to patch 2/3).
> 

I answered that question now.  Can I add your Reviewed-by and, for patch
3, Tested-by?

Paolo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/3] fix bugs with stale or corrupt MMIO caches

2014-09-02 Thread David Matlack
On Tue, Sep 2, 2014 at 8:42 AM, Paolo Bonzini  wrote:
> Il 29/08/2014 12:31, Paolo Bonzini ha scritto:
>> David and Xiao, here's my take on the MMIO generation patches.  Now
>> with documentation, too. :)  Please review!
>>
>> David Matlack (2):
>>   kvm: fix potentially corrupt mmio cache
>>   kvm: x86: fix stale mmio cache bug
>>
>> Paolo Bonzini (1):
>>   KVM: do not bias the generation number in kvm_current_mmio_generation
>>
>>  Documentation/virtual/kvm/mmu.txt | 14 ++
>>  arch/x86/include/asm/kvm_host.h   |  1 +
>>  arch/x86/kvm/mmu.c| 29 ++---
>>  arch/x86/kvm/x86.h| 20 +++-
>>  virt/kvm/kvm_main.c   | 30 +++---
>>  5 files changed, 67 insertions(+), 27 deletions(-)
>>
>
> Ping?

Sorry for the delay. I think the patches look good. And patch 3/3 still
fixes the bug I was originally seeing, so I'm happy :). I just had one
small comment (see my reply to patch 2/3).

>
> Paolo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/3] fix bugs with stale or corrupt MMIO caches

2014-09-02 Thread Paolo Bonzini
Il 29/08/2014 12:31, Paolo Bonzini ha scritto:
> David and Xiao, here's my take on the MMIO generation patches.  Now
> with documentation, too. :)  Please review!
> 
> David Matlack (2):
>   kvm: fix potentially corrupt mmio cache
>   kvm: x86: fix stale mmio cache bug
> 
> Paolo Bonzini (1):
>   KVM: do not bias the generation number in kvm_current_mmio_generation
> 
>  Documentation/virtual/kvm/mmu.txt | 14 ++
>  arch/x86/include/asm/kvm_host.h   |  1 +
>  arch/x86/kvm/mmu.c| 29 ++---
>  arch/x86/kvm/x86.h| 20 +++-
>  virt/kvm/kvm_main.c   | 30 +++---
>  5 files changed, 67 insertions(+), 27 deletions(-)
> 

Ping?

Paolo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/3] fix bugs with stale or corrupt MMIO caches

2014-09-02 Thread Paolo Bonzini
Il 29/08/2014 12:31, Paolo Bonzini ha scritto:
 David and Xiao, here's my take on the MMIO generation patches.  Now
 with documentation, too. :)  Please review!
 
 David Matlack (2):
   kvm: fix potentially corrupt mmio cache
   kvm: x86: fix stale mmio cache bug
 
 Paolo Bonzini (1):
   KVM: do not bias the generation number in kvm_current_mmio_generation
 
  Documentation/virtual/kvm/mmu.txt | 14 ++
  arch/x86/include/asm/kvm_host.h   |  1 +
  arch/x86/kvm/mmu.c| 29 ++---
  arch/x86/kvm/x86.h| 20 +++-
  virt/kvm/kvm_main.c   | 30 +++---
  5 files changed, 67 insertions(+), 27 deletions(-)
 

Ping?

Paolo
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/3] fix bugs with stale or corrupt MMIO caches

2014-09-02 Thread David Matlack
On Tue, Sep 2, 2014 at 8:42 AM, Paolo Bonzini pbonz...@redhat.com wrote:
 Il 29/08/2014 12:31, Paolo Bonzini ha scritto:
 David and Xiao, here's my take on the MMIO generation patches.  Now
 with documentation, too. :)  Please review!

 David Matlack (2):
   kvm: fix potentially corrupt mmio cache
   kvm: x86: fix stale mmio cache bug

 Paolo Bonzini (1):
   KVM: do not bias the generation number in kvm_current_mmio_generation

  Documentation/virtual/kvm/mmu.txt | 14 ++
  arch/x86/include/asm/kvm_host.h   |  1 +
  arch/x86/kvm/mmu.c| 29 ++---
  arch/x86/kvm/x86.h| 20 +++-
  virt/kvm/kvm_main.c   | 30 +++---
  5 files changed, 67 insertions(+), 27 deletions(-)


 Ping?

Sorry for the delay. I think the patches look good. And patch 3/3 still
fixes the bug I was originally seeing, so I'm happy :). I just had one
small comment (see my reply to patch 2/3).


 Paolo
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/3] fix bugs with stale or corrupt MMIO caches

2014-09-02 Thread Paolo Bonzini
Il 02/09/2014 18:47, David Matlack ha scritto:
  Ping?
 Sorry for the delay. I think the patches look good. And patch 3/3 still
 fixes the bug I was originally seeing, so I'm happy :). I just had one
 small comment (see my reply to patch 2/3).
 

I answered that question now.  Can I add your Reviewed-by and, for patch
3, Tested-by?

Paolo
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/3] fix bugs with stale or corrupt MMIO caches

2014-09-02 Thread David Matlack
On Tue, Sep 2, 2014 at 9:50 AM, Paolo Bonzini pbonz...@redhat.com wrote:
 Il 02/09/2014 18:47, David Matlack ha scritto:
  Ping?
 Sorry for the delay. I think the patches look good. And patch 3/3 still
 fixes the bug I was originally seeing, so I'm happy :). I just had one
 small comment (see my reply to patch 2/3).


 I answered that question now.  Can I add your Reviewed-by and, for patch
 3, Tested-by?

Please do. And thanks again for working on these!


 Paolo
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 0/3] fix bugs with stale or corrupt MMIO caches

2014-09-02 Thread Xiao Guangrong
On 09/02/2014 11:42 PM, Paolo Bonzini wrote:
 Il 29/08/2014 12:31, Paolo Bonzini ha scritto:
 David and Xiao, here's my take on the MMIO generation patches.  Now
 with documentation, too. :)  Please review!

 David Matlack (2):
   kvm: fix potentially corrupt mmio cache
   kvm: x86: fix stale mmio cache bug

 Paolo Bonzini (1):
   KVM: do not bias the generation number in kvm_current_mmio_generation

  Documentation/virtual/kvm/mmu.txt | 14 ++
  arch/x86/include/asm/kvm_host.h   |  1 +
  arch/x86/kvm/mmu.c| 29 ++---
  arch/x86/kvm/x86.h| 20 +++-
  virt/kvm/kvm_main.c   | 30 +++---
  5 files changed, 67 insertions(+), 27 deletions(-)

 
 Ping?

Looks good to me.

Reviewed-by: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH 0/3] fix bugs with stale or corrupt MMIO caches

2014-08-29 Thread Paolo Bonzini
David and Xiao, here's my take on the MMIO generation patches.  Now
with documentation, too. :)  Please review!

David Matlack (2):
  kvm: fix potentially corrupt mmio cache
  kvm: x86: fix stale mmio cache bug

Paolo Bonzini (1):
  KVM: do not bias the generation number in kvm_current_mmio_generation

 Documentation/virtual/kvm/mmu.txt | 14 ++
 arch/x86/include/asm/kvm_host.h   |  1 +
 arch/x86/kvm/mmu.c| 29 ++---
 arch/x86/kvm/x86.h| 20 +++-
 virt/kvm/kvm_main.c   | 30 +++---
 5 files changed, 67 insertions(+), 27 deletions(-)

-- 
1.8.3.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH 0/3] fix bugs with stale or corrupt MMIO caches

2014-08-29 Thread Paolo Bonzini
David and Xiao, here's my take on the MMIO generation patches.  Now
with documentation, too. :)  Please review!

David Matlack (2):
  kvm: fix potentially corrupt mmio cache
  kvm: x86: fix stale mmio cache bug

Paolo Bonzini (1):
  KVM: do not bias the generation number in kvm_current_mmio_generation

 Documentation/virtual/kvm/mmu.txt | 14 ++
 arch/x86/include/asm/kvm_host.h   |  1 +
 arch/x86/kvm/mmu.c| 29 ++---
 arch/x86/kvm/x86.h| 20 +++-
 virt/kvm/kvm_main.c   | 30 +++---
 5 files changed, 67 insertions(+), 27 deletions(-)

-- 
1.8.3.1

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/