On 11.08.20 10:07, Peng Fan wrote: >> Subject: Re: [PATCH 08/10] arm64: sysregs: correct SCTLR_EL1_RES >> >> On 11.08.20 09:49, Peng Fan wrote: >>>> Subject: Re: [PATCH 08/10] arm64: sysregs: correct SCTLR_EL1_RES >>>> >>>> On 07.08.20 05:06, [email protected] wrote: >>>>> From: Peng Fan <[email protected]> >>>>> >>>>> bit23 is SPAN, not reserved bit. >>>>> >>>> >>>> This is not correct, see D7.2.81 in the ARMv8 manual. >>>> >>>> Where did you get this from? What is it trying to fix? >>> >>> DDI0487D_a_armv8_arm, >>> " D12.2.100 SCTLR_EL1, System Control Register (EL1)" >>> >>> This is just to align with Linux Kernel, since the head.S in the >>> bootloader needs this. >> >> OK, this is ARMv8.1+, I was looking at 8.0. Are you sure the reset value >> changed from 8.0 to 8.1? We definitely need to remain compatible to 8.0. > > See > https://elixir.bootlin.com/linux/latest/source/arch/arm64/include/asm/sysreg.h#L577 > > It not have bit23 reserved. >
...but set. That is broken with your patch. Jan -- Siemens AG, Corporate Technology, CT RDA IOT SES-DE Corporate Competence Center Embedded Linux -- You received this message because you are subscribed to the Google Groups "Jailhouse" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/jailhouse-dev/91ede2a2-f658-7588-7add-fd93554e66d3%40siemens.com.
