Re: [Xen-devel] [Qemu-devel] [PATCH 2/2] Xen: Add xen-apic support and hook it up.

[Peter Maydell <peter.maydell@xxxxxxxxxx>]

On 5 April 2012 10:35, Wei Liu <wei.liu2@xxxxxxxxxx> wrote:
>
> --- /dev/null
> +++ b/hw/xen_apic.c
> @@ -0,0 +1,90 @@
> +/*
> + * Xen basic APIC support
> + *
> + * Copyright (c) 2012 Citrix
> + *
> + * Authors:
> + * ÂWei Liu <wei.liu2@xxxxxxxxxx>
> + *
> + * 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 ?

> +#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;'

-- PMM

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel