On 2015-06-19 11:08, Leon Alrae wrote:
> From: Matthew Fortune <matthew.fort...@imgtec.com>
>
> The BEV flag controls whether the boot exception vector is still
> in place when starting a kernel. When cleared the exception vector
> at EBASE (or hard coded address of 0x80000000) is used instead.
>
> The early stages of the linux kernel would benefit from BEV still
> being set to ensure any faults get handled by the boot rom exception
> handlers. This is a moot point for system qemu as there aren't really
> any BEV handlers, but there are other good reasons to change this...
The fake boot loader doesn't provide any exception handler at address
0xbcf00200. On the other hand the kernel might provide an exception
handler at address 0x80000000 through the ELF file (this doesn't seem to
be the case currently).
Is there a clear interface which defines the value of this bit when
booting a kernel. If not, it would be a good idea to look what YAMON or
U-Boot do and mimic that.
> The UHI (semi-hosting interface) defines special behaviours depending
> on whether an application starts in an environment with BEV set or
> cleared. When BEV is set then UHI assumes that a bootloader is
> relatively dumb and has no advanced exception handling logic.
> However, when BEV is cleared then UHI assumes that the bootloader
> has the ability to handle UHI exceptions with its exception handlers
> and will unwind and forward UHI SYSCALL exceptions to the exception
> vector that was installed prior to running the application.
In UHI mode that indeed makes sense.
> Signed-off-by: Matthew Fortune <matthew.fort...@imgtec.com>
> Signed-off-by: Leon Alrae <leon.al...@imgtec.com>
> ---
> hw/mips/mips_malta.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/mips/mips_malta.c b/hw/mips/mips_malta.c
> index 786a8f0..a5d64d5 100644
> --- a/hw/mips/mips_malta.c
> +++ b/hw/mips/mips_malta.c
> @@ -887,7 +887,7 @@ static void main_cpu_reset(void *opaque)
> read only location. The kernel location and the arguments table
> location does not change. */
> if (loaderparams.kernel_filename) {
> - env->CP0_Status &= ~((1 << CP0St_BEV) | (1 << CP0St_ERL));
> + env->CP0_Status &= ~(1 << CP0St_ERL);
> }
>
> malta_mips_config(cpu);
>
--
Aurelien Jarno GPG: 4096R/1DDD8C9B
aurel...@aurel32.net http://www.aurel32.net
