PCR 15035 - Proposed update BootSRD/PALSRD requirements
Summary: Proposed update BootSRD/PALSRD requirements
Status: ASSIGNED
Alias: None
Product: Kernel
Classification: Deos
Component: Kernel (show other PCRs)
Version: mainline
Hardware: ARM Deos
: Any Upcoming
: Enhancement
Target Milestone: mainline
Assignee: .Kernel
URL:
Whiteboard:
Depends on:
Blocks:
 
Reported: 2023-06-27 15:44 MST by Adina Roffelsen
Modified: 2024-08-30 06:55 MST (History)
0 users

See Also:
Impact Assessment: Trivial
Organization: DDC-I, Inc.
aroffelsen: Requirements?
aroffelsen: Code-
aroffelsen: TestCases-
aroffelsen: TestProcedures-
aroffelsen: Other?


Attachments

Note You need to log in before you can comment on or make changes to this PCR.
Description Adina Roffelsen 2023-06-27 15:44:59 MST
Currently the BOOTSRD says,
```Boot must ensure all of the following before enabling floating point:[3]
    FPSCR.Stride is RAZ
    FPSCR.Len is RAZ
    MVFR0.FPDivide is 0x1 "Supported"
    MVFR0.FPTrap is 0x0 "Not supported"
    MVFR0.FPDP is 0x2 "Supported, VFPv3 or greater"
    MVFR.SIMDLS is 0x1 "Implemented"```
This is answered at the A53/A72 level in the FPU TRM, so the kernel should not levy the requirement on boot to determine this since it is part of the kernel's evaluation of the A53/A72 processor manuals.
Having this in the manual requires each boot to evaluate this unnecessarily.
Comment 1 Adina Roffelsen 2023-06-28 07:47:37 MST
Additionally, DDD_DEOS_ARMCPU_321 should have a domain1="DDD".
Comment 2 Adina Roffelsen 2023-06-28 10:26:57 MST
Additional change requested:
cpu-asm.sgm says, "The following ensures that all cores see consistent ASID values for each possible address
  translation during startup.  Prior to entering the kernel <xref linkend="DEBOOT"/> establishes address
  translation using the BOOT VAS ASID value." but the "following" paragraph is only visible in the DDD, so it does not make sense in the boot SRD.
Comment 3 Adina Roffelsen 2023-06-28 12:09:48 MST
Suggest rearranging the document so that the "3.6.1. Processor State Table Column Definitions" is immediately before the 3.6.2.2.3. ARMv8 Processor States. It used to immediately precede the tables, which made sense. Now it is completely disassociated.
Comment 4 Adina Roffelsen 2023-06-29 10:03:05 MST
The PAL SRD says, "The PAL must not permit any user mode application to have modify access to any hardware configuration "registers" that might alter the physical memory map or in some other way bypass the protection of the memory management unit (e.g., permit an application to set up DMA)." 
The ability of an application to set up DMA is entirely outside the control of the PAL. Propose "The PAL must not enable..."
Comment 5 Adina Roffelsen 2023-06-29 10:04:41 MST
The PALSRD says, "The maximum processor wait time must be specified." There is no way for the PAL to document the maximum wait time on PCI as it is unbounded. This may need to be moved into a higher level document about the architecture of a system.
Comment 6 Adina Roffelsen 2023-06-29 10:18:52 MST
In the PALSRD: " The PAL cannot use the thread coordination mechanisms available to applications and to prevent non-AIB the kernel does not provide any facility for privileged mode platform software (i.e., the PAL or a PRL) to establish cross-core critical sections."
This sentence is incoherent. I think you mean, " The PAL cannot use the thread coordination mechanisms available to applications. To prevent non-AIB the kernel does not provide any facility for privileged mode platform software (i.e., the PAL or a PRL) to establish cross-core critical sections.
Comment 7 Adina Roffelsen 2023-06-29 14:40:03 MST
SRS_PAL_KernelExtensions_20.1 says nothing but the return type of the function, which is already part of the function interface SRS_PAL_KernelExtensions_10.1. It should be deleted.
Comment 8 Adina Roffelsen 2023-07-07 12:18:03 MST
"There must be a user mode and kernel mode writable virtual address range of RAM. The RAM must not be used for any other purpose, e.g., as DEOS_RAM. This memory is used for ABCHitMap and TestResults." would make more sense worded as "There must be a user mode and kernel mode writable virtual address range of RAM reserved for ABCHitMap and TestResults. The RAM must not be used for any other purpose, e.g., as DEOS_RAM."
Comment 9 deosbugs.ccb 2023-07-11 14:50:31 MST
CCB visited this PCR on 2023-07-11-76177
Comment 10 deosbugs.ccb 2023-08-14 11:04:28 MST
CCB visited this PCR on 2023-08-14-64795
Comment 11 deosbugs.ccb 2023-08-23 09:44:56 MST
PCR to be worked on kismet verf if time permits.
Comment 12 deosbugs.ccb 2024-08-30 06:55:27 MST
CCB visited this PCR on 2024-08-30-49739