diff mbox

x86/lapic: remove unwanted debugging messages

Message ID 1473949287-31122-1-git-send-email-pbonzini@redhat.com
State New
Headers show

Commit Message

Paolo Bonzini Sept. 15, 2016, 2:21 p.m. UTC
/me hides in the corner...

Cc: Dave Gilbert <dgilbert@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
 hw/i386/kvm/apic.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Dr. David Alan Gilbert Sept. 15, 2016, 3:07 p.m. UTC | #1
* Paolo Bonzini (pbonzini@redhat.com) wrote:
> /me hides in the corner...
> 
> Cc: Dave Gilbert <dgilbert@redhat.com>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>  hw/i386/kvm/apic.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/hw/i386/kvm/apic.c b/hw/i386/kvm/apic.c
> index 5d140b9..feb0002 100644
> --- a/hw/i386/kvm/apic.c
> +++ b/hw/i386/kvm/apic.c
> @@ -141,7 +141,6 @@ static void kvm_apic_put(void *data)
>  
>  static void kvm_apic_post_load(APICCommonState *s)
>  {
> -    fprintf(stderr, "%s: Yeh\n", __func__);
>      run_on_cpu(CPU(s->cpu), kvm_apic_put, s);
>  }

Hmm, I think you picked up the debug version we exchanged
privately rather than the one I actually posted which
I'd excised the debug from:
   https://lists.gnu.org/archive/html/qemu-devel/2016-09/msg02658.html

I think the only other change was that the one I posted I'd
reformatted kvm_put_apic_state so it wasn't 81 characters long.

Dave

>  
> -- 
> 1.8.3.1
> 
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
Paolo Bonzini Sept. 15, 2016, 3:40 p.m. UTC | #2
On 15/09/2016 17:07, Dr. David Alan Gilbert wrote:
> * Paolo Bonzini (pbonzini@redhat.com) wrote:
>> /me hides in the corner...
>>
>> Cc: Dave Gilbert <dgilbert@redhat.com>
>> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
>> ---
>>  hw/i386/kvm/apic.c | 1 -
>>  1 file changed, 1 deletion(-)
>>
>> diff --git a/hw/i386/kvm/apic.c b/hw/i386/kvm/apic.c
>> index 5d140b9..feb0002 100644
>> --- a/hw/i386/kvm/apic.c
>> +++ b/hw/i386/kvm/apic.c
>> @@ -141,7 +141,6 @@ static void kvm_apic_put(void *data)
>>  
>>  static void kvm_apic_post_load(APICCommonState *s)
>>  {
>> -    fprintf(stderr, "%s: Yeh\n", __func__);
>>      run_on_cpu(CPU(s->cpu), kvm_apic_put, s);
>>  }
> 
> Hmm, I think you picked up the debug version we exchanged
> privately rather than the one I actually posted which
> I'd excised the debug from:
>    https://lists.gnu.org/archive/html/qemu-devel/2016-09/msg02658.html
> 
> I think the only other change was that the one I posted I'd
> reformatted kvm_put_apic_state so it wasn't 81 characters long.

Yes, I did.  It's me who hides in the corner, not you. :)

Paolo

> 
> Dave
> 
>>  
>> -- 
>> 1.8.3.1
>>
> --
> Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
>
diff mbox

Patch

diff --git a/hw/i386/kvm/apic.c b/hw/i386/kvm/apic.c
index 5d140b9..feb0002 100644
--- a/hw/i386/kvm/apic.c
+++ b/hw/i386/kvm/apic.c
@@ -141,7 +141,6 @@  static void kvm_apic_put(void *data)
 
 static void kvm_apic_post_load(APICCommonState *s)
 {
-    fprintf(stderr, "%s: Yeh\n", __func__);
     run_on_cpu(CPU(s->cpu), kvm_apic_put, s);
 }