[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:19:09 +0100 |
User-agent: |
Alpine 2.00 (DEB 1167 2008-08-23) |
On Wed, 11 Apr 2012, Stefano Stabellini wrote:
> 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.
Actually it is a uint64 so it is a lot of f's
- [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, 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., 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