Skip to contentSkip to conversation

um: Remove dead usage of TIF_IA32

PATCH In review Gabriel Krisman Bertazi 3 participants 3 messages Oct 4, 2020

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 um: Remove dead usage of TIF_IA32

This seems like a dead artifact since TIF_IA32 is not even defined as a TI flag for UM. Looking back in git history, it made sense in the old days, but it is apparently not used since UM was split out of the x86 arch/. It is also going away from the x86 tree soon.

Also, I think the variable clean up it performs is not needed as 64-bit UML doesn't run 32-bit binaries as far as I can tell, and 32-bit UML has 32-bit ulong.

---
 arch/x86/um/ptrace_64.c | 13 +------------
 1 file changed, 1 insertion(+), 12 deletions(-)
arch/x86/um/ptrace_64.c +1 -12
index 09a085bde0d4..1401899dee9b 100644
@@ -52,14 +52,6 @@ static const int reg_offsets[] =
 
 int putreg(struct task_struct *child, int regno, unsigned long value)
 {
-#ifdef TIF_IA32
-	/*
-	 * Some code in the 64bit emulation may not be 64bit clean.
-	 * Don't take any chances.
-	 */
-	if (test_tsk_thread_flag(child, TIF_IA32))
-		value &= 0xffffffff;
-#endif
 	switch (regno) {
 	case R8:
 	case R9:
@@ -137,10 +129,7 @@ int poke_user(struct task_struct *child, long addr, long data)
 unsigned long getreg(struct task_struct *child, int regno)
 {
 	unsigned long mask = ~0UL;
-#ifdef TIF_IA32
-	if (test_tsk_thread_flag(child, TIF_IA32))
-		mask = 0xffffffff;
-#endif
+
 	switch (regno) {
 	case R8:
 	case R9:
Signature · 7 lines
2.28.0


_______________________________________________
linux-um mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/linux-um
#2 lore raw

----- Ursprüngliche Mail -----

Quoted 7 lines

Von: "Johannes Berg" <[email protected]>
An: "Gabriel Krisman Bertazi" <[email protected]>, "Jeff Dike" <[email protected]>, "richard" <[email protected]>, "anton ivanov" <[email protected]>
CC: [email protected], "linux-um" <[email protected]>, [email protected]
Gesendet: Sonntag, 4. Oktober 2020 22:10:17
Betreff: Re: [PATCH] um: Remove dead usage of TIF_IA32

Quoted 6 lines
Quoted 3 lines · Gabriel Krisman Bertazi

Also, I think the variable clean up it performs is not needed as 64-bit UML doesn't run 32-bit binaries as far as I can tell

Yep, that's true. I was looking into it once _very_ briefly, but no :)

Yep. UML has no IA32 emulation and never had.
Looks like a copy&paste leftover.

Thanks,
//richard

_______________________________________________
linux-um mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/linux-um