El 20/05/15 a les 13.46, Jan Beulich ha escrit:
>>>> On 20.05.15 at 12:11, <roger....@citrix.com> wrote:
>> --- a/xen/common/domain.c
>> +++ b/xen/common/domain.c
>> @@ -42,6 +42,7 @@
>>  #include <xsm/xsm.h>
>>  #include <xen/trace.h>
>>  #include <xen/tmem.h>
>> +#include <asm/setup.h>
> 
> This caused the ARM build to fail. Instead of once again reverting I
> applied the trivial fix eab0647587 without asking for permission by
> you ARM maintainers - I hope that's okay in a case like this.

I'm sorry but I'm not sure what's wrong with this chunk, and AFAICT you
seem to have applied it unmodified. The code added in
xen/common/domain.c is protected with #ifdef CONFIG_X86 and ARM has a
setup.h header.

> But Roger, considering how many issues recent patches of yours
> introduced - please apply a little more care.

Yes, and I'm really sorry for that.

Roger.


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

Reply via email to