Re: [Xen-devel] [PATCH v5 05/12] x86: provide executable fixmap facility

[Wei Liu <wl@xxxxxxx>]

On Thu, Jan 30, 2020 at 12:30:47AM +0000, Wei Liu wrote:
> On Wed, Jan 29, 2020 at 08:20:27PM +0000, Wei Liu wrote:
> >  
> > +void __set_fixmap_x(
> > +    enum fixed_addresses_x idx, unsigned long mfn, unsigned long flags)
> > +{
> > +    BUG_ON(idx >= __end_of_fixed_addresses_x || idx <= FIX_X_RESERVED);
> > +    map_pages_to_xen(__fix_x_to_virt(idx), _mfn(mfn), 1, flags);
> > +
> > +    /* Generate a symbol to be used in linker script */
> > +    asm ( ".equ FIXADDR_X_SIZE, %c0; .global FIXADDR_X_SIZE"
> > +          :: "i" (__end_of_fixed_addresses_x << PAGE_SHIFT) );
> 
> The (__end << SHIFT) part can be replaced with FIXADDR_X_SIZE (the macro
> defined in fixmap.h) directly.

I also discover that this snippet to generate symbol should be moved
else where because if Hyper-V support is compiled out, this function has
no user. That causes it to be DCE'd. FIXADDR_X_SIZE would be gone and
linking would fail.

I have moved this to arch_init_memory. Its storage will be reclaimed
during runtime, but this symbol is not used anywhere else in code, so we
should be fine.

Wei.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel