[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/2] Xen: Add xen-apic support and hook it up.
From: |
Stefano Stabellini |
Subject: |
Re: [Qemu-devel] [PATCH 2/2] Xen: Add xen-apic support and hook it up. |
Date: |
Wed, 11 Apr 2012 17:17:15 +0100 |
User-agent: |
Alpine 2.00 (DEB 1167 2008-08-23) |
On Wed, 11 Apr 2012, Peter Maydell wrote:
> On 5 April 2012 10:35, Wei Liu <address@hidden> wrote:
> >
> > --- /dev/null
> > +++ b/hw/xen_apic.c
> > @@ -0,0 +1,90 @@
> > +/*
> > + * Xen basic APIC support
> > + *
> > + * Copyright (c) 2012 Citrix
> > + *
> > + * Authors:
> > + * Wei Liu <address@hidden>
> > + *
> > + * This work is licensed under the terms of the GNU GPL version 2.
> > + * See the COPYING file in the top-level directory.
> > + */
>
> Really 2-only, not 2-or-later ?
I am not a great fun of the 2-or-later clause and it doesn't look like a
requirement from the QEMU project POV. However if it is, I'll change it
to 2-or-later.
> > +#include "hw/apic_internal.h"
> > +#include "hw/msi.h"
> > +#include "xen.h"
> > +
> > +static uint64_t xen_apic_mem_read(void *opaque, target_phys_addr_t addr,
> > + unsigned size)
> > +{
> > + return -1U;
> > +}
>
> This seems a rather confusing way to write 'return 0xffffffff;'
Yep, I'll change it.
- [Qemu-devel] [PATCH 0/0] MSI/MSIX injection for Xen HVM guests, Wei Liu, 2012/04/05
- [Qemu-devel] [PATCH 1/2] Xen: basic HVM MSI injection support., Wei Liu, 2012/04/05
- [Qemu-devel] [PATCH 2/2] Xen: Add xen-apic support and hook it up., Wei Liu, 2012/04/05
- Re: [Qemu-devel] [PATCH 2/2] Xen: Add xen-apic support and hook it up., Stefano Stabellini, 2012/04/11
- Re: [Qemu-devel] [PATCH 2/2] Xen: Add xen-apic support and hook it up., Peter Maydell, 2012/04/11
- Re: [Qemu-devel] [PATCH 2/2] Xen: Add xen-apic support and hook it up.,
Stefano Stabellini <=
- Re: [Qemu-devel] [PATCH 2/2] Xen: Add xen-apic support and hook it up., Stefano Stabellini, 2012/04/11
- Re: [Qemu-devel] [PATCH 2/2] Xen: Add xen-apic support and hook it up., Eric Blake, 2012/04/11
- Re: [Qemu-devel] [PATCH 2/2] Xen: Add xen-apic support and hook it up., Wei Liu, 2012/04/12
- Re: [Qemu-devel] [PATCH 2/2] Xen: Add xen-apic support and hook it up., Paolo Bonzini, 2012/04/11
Re: [Qemu-devel] [PATCH 0/0] MSI/MSIX injection for Xen HVM guests, Paolo Bonzini, 2012/04/05