[U-Boot] [PATCH 04/25] remoteproc: elf-loader: Add 64 bit elf loading support

[Fabien DESSENNE]

Hi Lokesh



On 28/08/2019 2:55 PM, Lokesh Vutla wrote:
> The current rproc-elf-loader supports loading of only 32 bit elf files.
> Introduce support for loading of 64 bit elf files in rproc-elf-loader.
>
> Signed-off-by: Lokesh Vutla <lokeshvutla at ti.com>
> ---
>   drivers/remoteproc/rproc-elf-loader.c | 109 ++++++++++++++++++++++++++
>   include/remoteproc.h                  |  12 +++
>   2 files changed, 121 insertions(+)
>
> diff --git a/drivers/remoteproc/rproc-elf-loader.c b/drivers/remoteproc/rproc-elf-loader.c
> index 149aeafb85..dff1873a51 100644
> --- a/drivers/remoteproc/rproc-elf-loader.c
> +++ b/drivers/remoteproc/rproc-elf-loader.c
> @@ -64,6 +64,61 @@ static int rproc_elf32_sanity_check(ulong addr, ulong size)
>   	return 0;
>   }
>   
> +static int rproc_elf64_sanity_check(ulong addr, ulong size)


The same remark as for rproc_elf32_sanity_check() applies : not static

BR

Fabien


> +{
> +	Elf64_Ehdr *ehdr = (Elf64_Ehdr *)addr;
> +	char class;
> +
> +	if (!addr) {
> +		pr_debug("Invalid fw address?\n");
> +		return -EFAULT;
> +	}
> +
> +	if (size < sizeof(Elf64_Ehdr)) {
> +		pr_debug("Image is too small\n");
> +		return -ENOSPC;
> +	}
> +
> +	class = ehdr->e_ident[EI_CLASS];
> +
> +	if (!IS_ELF(*ehdr) || ehdr->e_type != ET_EXEC || class != ELFCLASS64) {
> +		pr_debug("Not an executable ELF64 image\n");
> +		return -EPROTONOSUPPORT;
> +	}
> +
> +	/* We assume the firmware has the same endianness as the host */
> +# ifdef __LITTLE_ENDIAN
> +	if (ehdr->e_ident[EI_DATA] != ELFDATA2LSB) {
> +# else /* BIG ENDIAN */
> +	if (ehdr->e_ident[EI_DATA] != ELFDATA2MSB) {
> +# endif
> +		pr_debug("Unsupported firmware endianness\n");
> +		return -EILSEQ;
> +	}
> +
> +	if (size < ehdr->e_shoff + sizeof(Elf64_Shdr)) {
> +		pr_debug("Image is too small\n");
> +		return -ENOSPC;
> +	}
> +
> +	if (memcmp(ehdr->e_ident, ELFMAG, SELFMAG)) {
> +		pr_debug("Image is corrupted (bad magic)\n");
> +		return -EBADF;
> +	}
> +
> +	if (ehdr->e_phnum == 0) {
> +		pr_debug("No loadable segments\n");
> +		return -ENOEXEC;
> +	}
> +
> +	if (ehdr->e_phoff > size) {
> +		pr_debug("Firmware size is too small\n");
> +		return -ENOSPC;
> +	}
> +
> +	return 0;
> +}
> +
>   int rproc_elf32_load_image(struct udevice *dev, unsigned long addr, ulong size)
>   {
>   	Elf32_Ehdr *ehdr; /* Elf header structure pointer */
> @@ -110,3 +165,57 @@ int rproc_elf32_load_image(struct udevice *dev, unsigned long addr, ulong size)
>   
>   	return 0;
>   }
> +
> +int rproc_elf64_load_image(struct udevice *dev, ulong addr, ulong size)
> +{
> +	const struct dm_rproc_ops *ops = rproc_get_ops(dev);
> +	u64 da, memsz, filesz, offset;
> +	Elf64_Ehdr *ehdr;
> +	Elf64_Phdr *phdr;
> +	int i, ret = 0;
> +	void *ptr;
> +
> +	dev_dbg(dev, "%s: addr = 0x%lx size = 0x%lx\n", __func__, addr, size);
> +
> +	if (rproc_elf64_sanity_check(addr, size))
> +		return -EINVAL;
> +
> +	ehdr = (Elf64_Ehdr *)addr;
> +	phdr = (Elf64_Phdr *)(addr + (ulong)ehdr->e_phoff);
> +
> +	/* go through the available ELF segments */
> +	for (i = 0; i < ehdr->e_phnum; i++, phdr++) {
> +		da = phdr->p_paddr;
> +		memsz = phdr->p_memsz;
> +		filesz = phdr->p_filesz;
> +		offset = phdr->p_offset;
> +
> +		if (phdr->p_type != PT_LOAD)
> +			continue;
> +
> +		dev_dbg(dev, "%s:phdr: type %d da 0x%llx memsz 0x%llx filesz 0x%llx\n",
> +			__func__, phdr->p_type, da, memsz, filesz);
> +
> +		ptr = (void *)(uintptr_t)da;
> +		if (ops->device_to_virt) {
> +			ptr = ops->device_to_virt(dev, da, phdr->p_memsz);
> +			if (!ptr) {
> +				dev_err(dev, "bad da 0x%llx mem 0x%llx\n", da,
> +					memsz);
> +				ret = -EINVAL;
> +				break;
> +			}
> +		}
> +
> +		if (filesz)
> +			memcpy(ptr, (void *)addr + offset, filesz);
> +		if (filesz != memsz)
> +			memset(ptr + filesz, 0x00, memsz - filesz);
> +
> +		flush_cache(rounddown((ulong)ptr, ARCH_DMA_MINALIGN),
> +			    roundup((ulong)ptr + filesz, ARCH_DMA_MINALIGN) -
> +			    rounddown((ulong)ptr, ARCH_DMA_MINALIGN));
> +	}
> +
> +	return ret;
> +}
> diff --git a/include/remoteproc.h b/include/remoteproc.h
> index 6657b39723..f5d77c8c81 100644
> --- a/include/remoteproc.h
> +++ b/include/remoteproc.h
> @@ -210,6 +210,15 @@ int rproc_is_running(int id);
>    * @return 0 if the image is successfully loaded, else appropriate error value.
>    */
>   int rproc_elf32_load_image(struct udevice *dev, unsigned long addr, ulong size);
> +
> +/**
> + * rproc_elf64_load_image() - load an ELF64 image
> + * @dev:	device loading the ELF64 image
> + * @addr:	valid ELF64 image address
> + * @size:	size of the image
> + * @return 0 if the image is successfully loaded, else appropriate error value.
> + */
> +int rproc_elf64_load_image(struct udevice *dev, ulong addr, ulong size);
>   #else
>   static inline int rproc_init(void) { return -ENOSYS; }
>   static inline int rproc_dev_init(int id) { return -ENOSYS; }
> @@ -223,6 +232,9 @@ static inline int rproc_is_running(int id) { return -ENOSYS; }
>   static inline int rproc_elf32_load_image(struct udevice *dev,
>   					 unsigned long addr, ulong size)
>   { return -ENOSYS; }
> +static inline int rproc_elf64_load_image(struct udevice *dev, ulong addr,
> +					 ulong size)
> +{ return -ENOSYS; }
>   #endif
>   
>   #endif	/* _RPROC_H_ */