Re: [Xen-devel] [PATCH RFC 5/9] x86/traps: Functional prep work

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 15.05.14 at 11:48, <andrew.cooper3@xxxxxxxxxx> wrote:
> --- a/xen/arch/x86/setup.c
> +++ b/xen/arch/x86/setup.c
> @@ -558,6 +558,12 @@ void __init noreturn __start_xen(unsigned long mbi_p)
>          .stop_bits = 1
>      };
>  
> +    set_processor_id(0);
> +    set_current((struct vcpu *)0xfffff000); /* debug sanity */
> +    this_cpu(curr_vcpu) = idle_vcpu[0] = current;

The this_cpu() part wasn't there in the original code - is that really
needed, and ...

> +
> +    sort_exception_tables();
> +
>      percpu_init_areas();

... is that really safe/meaningful before this function got called?

Jan


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