>>> On 11.09.16 at 22:35, <konrad.w...@oracle.com> wrote:
> Those symbols are used to help final linkers to replace insn.
> The ARM ELF specification mandates that they are present
> to denote the start of certain CPU features. There are two
> variants of it - short and long format.
> 
> Either way - we can ignore these symbols.
> 
> Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com> [x86 bits]
> Signed-off-by: Konrad Rzeszutek Wilk <konrad.w...@oracle.com>
> 
> ---
> Cc: Konrad Rzeszutek Wilk <konrad.w...@oracle.com>
> Cc: Ross Lagerwall <ross.lagerw...@citrix.com>
> Cc: Stefano Stabellini <sstabell...@kernel.org>
> Cc: Julien Grall <julien.gr...@arm.com 
> Cc: Jan Beulich <jbeul...@suse.com>
> Cc: Andrew Cooper <andrew.coop...@citrix.com>
> 
> v1: First submission
> v2: Update the order of symbols, fix title
>     Add {} in after the first if - per Jan's recommendation.
> v3: Add Andrew's Review tag
>     Make the function return an bool_t.

Yet it should have been bool.

Jan


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

Reply via email to