ARM: RES0/RES1 SCTLR fields not read-only

Bug #1679358 reported by Yifan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
QEMU
Expired
Undecided
Unassigned

Bug Description

There are fields in SCTLR that are RAO/SBOP or WI or in the case of the RR field, accessible only in secure mode. Currently it seems that qemu just propagates any write to SCTLR to the register and this screwed up in a bootloader that I am debugging.

Tags: arm
Revision history for this message
Peter Maydell (pmaydell) wrote : Re: [Qemu-devel] [Bug 1679358] [NEW] ARM: SCTLR fields not being preserved

On 3 April 2017 at 23:17, Yifan <email address hidden> wrote:
> There are fields in SCTLR that are RAO/SBOP or WI or in the case of the
> RR field, accessible only in secure mode. Currently it seems that qemu
> just propagates any write to SCTLR to the register and this screwed up
> in a bootloader that I am debugging.

Yes, we're a bit loose in QEMU on the handling of reserved bits.

Note that most of the SCTLR bits like this are RAO/SBOP or RAZ/SBZP,
so the guest should not be writing wrong values to them.

thanks
-- PMM

Revision history for this message
Yifan (yifanlu) wrote : Re: ARM: SCTLR fields not being preserved

So there won't be a fix in the future? I'm working with debugging a proprietary bootloader that I do not have the source code for. I wonder if this becomes an issue for any other platform targets.

Revision history for this message
Peter Maydell (pmaydell) wrote :

Well, I wouldn't object to a patch to fix it (it would have to correctly handle the various different versions of the CPU architecture we implement, etc), but I'm not planning on writing one today myself.

Peter Maydell (pmaydell)
tags: added: arm
summary: - ARM: SCTLR fields not being preserved
+ ARM: RES0/RES1 SCTLR fields not read-only
Changed in qemu:
status: New → Confirmed
Revision history for this message
Thomas Huth (th-huth) wrote :

The QEMU project is currently moving its bug tracking to another system.
For this we need to know which bugs are still valid and which could be
closed already. Thus we are setting the bug state to "Incomplete" now.

If the bug has already been fixed in the latest upstream version of QEMU,
then please close this ticket as "Fix released".

If it is not fixed yet and you think that this bug report here is still
valid, then you have two options:

1) If you already have an account on gitlab.com, please open a new ticket
for this problem in our new tracker here:

    https://gitlab.com/qemu-project/qemu/-/issues

and then close this ticket here on Launchpad (or let it expire auto-
matically after 60 days). Please mention the URL of this bug ticket on
Launchpad in the new ticket on GitLab.

2) If you don't have an account on gitlab.com and don't intend to get
one, but still would like to keep this ticket opened, then please switch
the state back to "New" within the next 60 days (otherwise it will get
closed as "Expired"). We will then eventually migrate the ticket auto-
matically to the new system (but you won't be the reporter of the bug
in the new system and thus won't get notified on changes anymore).

Thank you and sorry for the inconvenience.

Changed in qemu:
status: Confirmed → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for QEMU because there has been no activity for 60 days.]

Changed in qemu:
status: Incomplete → Expired
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.