[CRIU] [PATCH 7/8] arm: kill fpu_state_t

Alexander Kartashov alekskartashov at parallels.com
Thu Oct 10 23:48:04 PDT 2013


It has been a mistake to introduce this struct since the FPU state
is stored deep inside an ARM sigframe. A remnant of the struct
is left to make the code compatible with the universal declaration
of the routine sigreturn_prep_fpu_frame().

Signed-off-by: Alexander Kartashov <alekskartashov at parallels.com>
---
 arch/arm/include/asm/restorer.h |    4 ++--
 arch/arm/include/asm/types.h    |    7 -------
 2 files changed, 2 insertions(+), 9 deletions(-)

diff --git a/arch/arm/include/asm/restorer.h b/arch/arm/include/asm/restorer.h
index 74df4cf..181744d 100644
--- a/arch/arm/include/asm/restorer.h
+++ b/arch/arm/include/asm/restorer.h
@@ -42,6 +42,8 @@ struct vfp_sigframe {
 	struct user_vfp_exc     ufp_exc;
 };
 
+typedef struct vfp_sigframe fpu_state_t;
+
 struct aux_sigframe {
 	/*
 	struct crunch_sigframe  crunch;
@@ -62,8 +64,6 @@ struct sigframe {
 struct rt_sigframe {
 	struct rt_siginfo info;
 	struct sigframe sig;
-
-	fpu_state_t fpu_state;
 };
 
 
diff --git a/arch/arm/include/asm/types.h b/arch/arm/include/asm/types.h
index 3d917f8..d5d2d56 100644
--- a/arch/arm/include/asm/types.h
+++ b/arch/arm/include/asm/types.h
@@ -228,11 +228,4 @@ typedef uint32_t auxv_t;
 static inline void *decode_pointer(uint64_t v) { return (void*)(uint32_t)v; }
 static inline uint64_t encode_pointer(void *p) { return (uint32_t)p; }
 
-typedef struct {
-	struct user_vfp		ufp;
-	struct user_vfp_exc	ufp_exc;
-
-	bool has_fpu;
-} fpu_state_t;
-
 #endif /* __CR_ASM_TYPES_H__ */
-- 
1.7.9.5



More information about the CRIU mailing list