|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Minios-devel] [UNIKRAFT PATCHv4 22/43] plat/kvm: Allow access to floating-point and Advanced SIMD registers
Hello Wei Chen, On 07/23/2018 11:01 AM, Wei Chen wrote: Hi Sharan,-----Original Message----- From: Sharan Santhanam <sharan.santhanam@xxxxxxxxx> Sent: 2018年7月20日 21:28 To: Wei Chen <Wei.Chen@xxxxxxx>; minios-devel@xxxxxxxxxxxxxxxxxxxx; simon.kuenzer@xxxxxxxxx Cc: Julien Grall <Julien.Grall@xxxxxxx>; Kaly Xin <Kaly.Xin@xxxxxxx>; nd <nd@xxxxxxx>; Dave P Martin <Dave.Martin@xxxxxxx> Subject: Re: Re: [Minios-devel] [UNIKRAFT PATCHv4 22/43] plat/kvm: Allow access to floating-point and Advanced SIMD registers Hello, On 07/09/2018 12:48 PM, Julien Grall wrote:(+ Dave) On 09/07/18 10:47, Wei Chen wrote:Hi, Julien,Hi Wei,-----Original Message----- From: Julien Grall <julien.grall@xxxxxxx> Sent: 2018年7月8日 6:17 To: Wei Chen <Wei.Chen@xxxxxxx>; minios-devel@xxxxxxxxxxxxxxxxxxxx; simon.kuenzer@xxxxxxxxx Cc: Kaly Xin <Kaly.Xin@xxxxxxx>; nd <nd@xxxxxxx> Subject: Re: [Minios-devel] [UNIKRAFT PATCHv4 22/43] plat/kvm: Allowaccess tofloating-point and Advanced SIMD registers Hi, On 07/06/2018 10:03 AM, Wei Chen wrote:On Arm64, when we want to print out some message, the va_list will use the SIMD&FP registers (like q0, q1) to store parameters. So, we have to disable the trap of accessing floating-point and Advanced SIMD registers to make va_list use SIMD&FP registers properly.I don't think this is true. Xen and Linux don't use SIMD&FP registers as this would require to context switch at every entry. Yet we are able to use va_list without SIMD&FP registers.This is true, because Linux and Xen doesn't support float print. Unikraft kernel just a part of an application, its print library need To support float. Just like nolibc, when GCC compiling this library, it doesn't care you are running EL1 or EL0, the generated code will access q0, q1. When you're running this code in EL1 without SIMD&FP, system crash.According to the [2] and [4], Cortex-A53 could be configured and linux[3] has support for running the soft floating point. But in our description of Cortex-A53 in "arch/arm/arm64/Config.uk" we restrict it. Do we plan to support it in the future?Hmm, Both floating-point and NEON are required in all standard ARMv8 implementations. However, as the document[2] mentioned, implementations targeting specialized markets may support No NEON or floating-point. Although I don't really want to support it, but I think I don't have choice. I have to support such specialized users, I prefer to introduce a new target CPU cortex-A53+NOFP to support it in another patch series. I agree it would be better if we have it as a part of the new patch series. About document[3], I can't open the original URL, but I find it in http://lists-archives.com/linux-kernel/28688456-arm64-support-systems-without-fp-asimd.html This patch is not for soft floating. It's used to avoid save fp&simd registers in context switch when fpsimd feature is absent. And GCC for AArch64 doesn't support soft-float, if users select the +NOFP option, he must confirm the code don't contain any float type.When will q0, q1 will be accessed? Is it only when floating point will be printed?The document[1], mentions that register qN are used only for floating and vector operations. In appendix B, where they explain about theSorry, I haven't found "only" from [1], I just found "can" : ) "Stage C - Assignment ofarguments to registers and stack" in section 5.4.2 describes about the allocation of function parameters to the registers. C1-C6 describes the use of the qN registers. I used this as reference for the register assignment. variadic calling conventions it mentions prologue of the variadic function creates a separate variadic stack frame and if the Floating points arguments or vector arguments are not passed as a variadic argument qN register do not have to be saved and restored. For further reference, section 5.4[1] describe how the function parameters are passed and how the SIMD/floating point register are used. Appendix B of the document gives a description of variadic functions.So, without -mgeneral-regs-only flag, gcc va_start, va_list will use qN to pass parameters. As Unikraft is not a kernel, it's part of an application, we will transfer lots of userspace libraries and applications to unikriaft libraries and applications. And most applications on Cloud like DPDK or other network application, the already use SIMD and other vector operations to accelerate transfer speed. Not to mention the image processing applications, FP is mandatory. So I prefer to keep FP&SIMD enabled by default, except +NOFP be selected Explicitly. I agree, we need FP/SIMD for the use cases mentioned.In Unikraft we would have them as libraries which can be configured to be enabled or disabled. By default those libraries would also be disabled. Also, IIUC, the kernel code will be compiled with floating point, right? If so, it means the compiler will be able to use floating point for some optimization. This will become a problem when receiving a trap as you would need to context switch the registers (this assume handler may use FP). Cheers,[1] http://infocenter.arm.com/help/topic/com.arm.doc.ihi0055b/IHI0055B_aapcs64.pdf [2] https://developer.arm.com/products/architecture/cpu-architecture/a- profile/docs/112086438/latest/software-implications-for-v8-a-implementations- with-no-hardware-floating-point [3] https://patchwork.kernel.org/patch/9405787/ [4] http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.dui0773b/chr138314 3713787.html Thanks & Regards Sharan Thanks & Regards Sharan _______________________________________________ Minios-devel mailing list Minios-devel@xxxxxxxxxxxxxxxxxxxx https://lists.xenproject.org/mailman/listinfo/minios-devel
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |