Re: [Xen-devel] [PATCH] fix EFI part of "symbols: Generate an xen-sym.map"

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 21.09.16 at 18:17, <konrad.wilk@xxxxxxxxxx> wrote:
> On Wed, Sep 21, 2016 at 10:04:21AM -0600, Jan Beulich wrote:
>> >>> On 21.09.16 at 17:59, <konrad.wilk@xxxxxxxxxx> wrote:
>> > The fix can be done two ways:
>> >  a) See if xen.efi.map exists and then copy it
>> >  b) Or link xen.efi.map to xen-syms.map (similar to how xen.efi is linked
>> >     against xen).
>> > 
>> > The patch chooses the latter.
>> 
>> Well, if the ARM maintainers like that ... I don't really see a point in
>> installing the same file twice without its second incarnation having a
>> specific purpose.
> 
> I also have the a) part ready, which is simple:

Ping? I'd really like to see this bogus error gone from the build.

Jan

> diff --git a/xen/Makefile b/xen/Makefile
> index e989a20..678f188 100644
> --- a/xen/Makefile
> +++ b/xen/Makefile
> @@ -67,7 +67,9 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX)
>       if [ -r $(TARGET).efi -a -n '$(EFI_DIR)' ]; then \
>               [ -d $(D)$(EFI_DIR) ] || $(INSTALL_DIR) $(D)$(EFI_DIR); \
>               $(INSTALL_DATA) $(TARGET).efi 
> $(D)$(EFI_DIR)/$(T)-$(XEN_FULLVERSION).efi; \
> -             $(INSTALL_DATA) $(TARGET).efi.map 
> $(D)$(DEBUG_DIR)/$(T)-$(XEN_FULLVERSION).efi.map; \
> +                if [ -e $(TARGET).efi.map ]; then \
> +                     $(INSTALL_DATA) $(TARGET).efi.map 
> $(D)$(DEBUG_DIR)/$(T)-$(XEN_FULLVERSION).efi.map; \
> +                fi; \
>               ln -sf $(T)-$(XEN_FULLVERSION).efi 
> $(D)$(EFI_DIR)/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION).efi; \
>               ln -sf $(T)-$(XEN_FULLVERSION).efi 
> $(D)$(EFI_DIR)/$(T)-$(XEN_VERSION).efi; \
>               ln -sf $(T)-$(XEN_FULLVERSION).efi $(D)$(EFI_DIR)/$(T).efi; \
> 
> Either fix works.




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