Re: [PATCH 12/12] i386 boot: Add an ELF header to bzImage

[Eric W. Biederman]

Vivek Goyal <vgoyal@xxxxxxxxxx> writes:

Ok.  I just noticed another piece that we want to change for
greater compatibility.  We should make the virtual and the physical
addresses the same.  Then there is no danger of some loader getting
them mixed up.

I.e. Not:
  
> +.org 80
> +phdr:
> +	.int PT_LOAD					# p_type
> +	.int (SETUPSECTS+1)*512				# p_offset
> +	.int LOAD_PHYSICAL_ADDR + __PAGE_OFFSET		# p_vaddr
> +	.int LOAD_PHYSICAL_ADDR				# p_paddr
> +	.int SYSSIZE*16					# p_filesz
> +	.int 0						# p_memsz
> +	.int PF_R | PF_W | PF_X				# p_flags
> +	.int CONFIG_PHYSICAL_ALIGN			# p_align
> +e_phdr:
> +

but
> +.org 80
> +phdr:
> +	.int PT_LOAD					# p_type
> +	.int (SETUPSECTS+1)*512				# p_offset
> +	.int LOAD_PHYSICAL_ADDR				# p_vaddr
> +	.int LOAD_PHYSICAL_ADDR				# p_paddr
> +	.int SYSSIZE*16					# p_filesz
> +	.int 0						# p_memsz
> +	.int PF_R | PF_W | PF_X				# p_flags
> +	.int CONFIG_PHYSICAL_ALIGN			# p_align
> +e_phdr:
> +
>  start2:
>  	movw	%cs, %ax
>  	movw	%ax, %ds
> @@ -78,11 +128,11 @@ die:


Eric
-
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/