[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] SH4: Privilege check for instructions
From: |
Shin-ichiro KAWASAKI |
Subject: |
Re: [Qemu-devel] [PATCH] SH4: Privilege check for instructions |
Date: |
Sun, 14 Sep 2008 19:27:51 +0900 |
User-agent: |
Thunderbird 2.0.0.16 (Windows/20080708) |
Thank you for the comment!
Blue Swirl wrote:
On 9/14/08, Shin-ichiro KAWASAKI <address@hidden> wrote:
This patch adds check for all SH4 instructions which are
executed only in privileged mode.
The checks get the privileged mode status from translation context. In
theory, the same TB code block could be used in unprivileged and
privileged mode, so the status that was true at translation time may
no longer be correct at execution time. Of course normally kernel code
is not visible or executable to user processes.
As you say, this patch has the restriction that you pointed out : the
generated TB cannot used for both unprivileged and privileged.
I guess the codes generated by tcg_gen_qemu_st/ld() have the same
restriction, because those tcg_gen functions take the argument QEMU memory
index flags, which is decided at translation time. If it is true, the
restriction might be allowed for privilege check.
The TB flags are handled in cpu-exec.c:tb_find_fast(). If I understand
the SH part correctly, the flags copied from env->flags don't contain
the privileged mode bits, isn't that in env->sr & SR_MD?
Right. In target-sh4/translate.c:get_intermediate_code_internal(),
the value env->sr & SR_MD used to set ctx->memidx.
We can use ctx->memidx to check the privileged mode at translation time,
and can use env->sr to check at execution time. Both implementation
can be done, I guess.
Alternatively, the check could be made at execution time, but that's
less efficient.
If QEMU means *quick* emulator, more efficient way seems proper,
so my current opinion is that privilege check should be done at
translation time.
Anyway I'm not yet sure about QEMU internal.
More comments will be welcome!
Best Regards,
Shin-ichiro KAWASAKI
- [Qemu-devel] [PATCH] SH4: Privilege check for instructions, Shin-ichiro KAWASAKI, 2008/09/14
- Re: [Qemu-devel] [PATCH] SH4: Privilege check for instructions, Blue Swirl, 2008/09/14
- Re: [Qemu-devel] [PATCH] SH4: Privilege check for instructions,
Shin-ichiro KAWASAKI <=
- Re: [Qemu-devel] [PATCH] SH4: Privilege check for instructions, Blue Swirl, 2008/09/14
- Re: [Qemu-devel] [PATCH] SH4: Privilege check for instructions, Shin-ichiro KAWASAKI, 2008/09/14
- Re: [Qemu-devel] [PATCH] SH4: Privilege check for instructions, Aurelien Jarno, 2008/09/15
- Re: [Qemu-devel] [PATCH] SH4: Privilege check for instructions, Blue Swirl, 2008/09/15
- Re: [Qemu-devel] [PATCH] SH4: Privilege check for instructions, Shin-ichiro KAWASAKI, 2008/09/16
- Re: [Qemu-devel] [PATCH] SH4: Privilege check for instructions, Blue Swirl, 2008/09/16
- Re: [Qemu-devel] [PATCH] SH4: Privilege check for instructions, Shin-ichiro KAWASAKI, 2008/09/16
- Re: [Qemu-devel] [PATCH] SH4: Privilege check for instructions, Paul Mundt, 2008/09/17