Re: [PATCH] arch/cris/arch-v10/vmlinux.lds.S fix boot problem

[Sam Ravnborg]

Hi Jesper.

>  		__initramfs_end = .;
> -		/* We fill to the next page, so we can discard all init
> -		   pages without needing to consider what payload might be
> -		   appended to the kernel image.  */
> -		FILL (0); 
> -		. = ALIGN (8192);
>  	}
>  #endif
> -	
>  	__vmlinux_end = .;            /* last address of the physical file */
> -  	__init_end = .;
> +
> +	/*
> +	 * We fill to the next page, so we can discard all init
> +	 * pages without needing to consider what payload might be
> +	 * appended to the kernel image.
> +	 */
> +	.init.fill : {
> +		FILL (0);
> +		. = ALIGN (8192);
> +	}
> +
> +	__init_end = .;

Can we please have the hardcoded 8192 replaced with a proper constant.
I assume this is PAGESIZE?
See hwat other archs does to make PAGESIZE available for assembly
files and ldscripts. The macro AC is the key point here.

And then please explain why it make a diffrence to put the
alignment in a section like you do.
I do not see it used in the other archs.

	Sam
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/