Re: [PATCH] x86/mm: Drop TS_COMPAT on 64-bit exec() syscall
From: Dmitry Safonov
Date: Fri May 18 2018 - 18:14:55 EST
Hi Andy,
2018-05-18 23:03 GMT+01:00 Andy Lutomirski <luto@xxxxxxxxxx>:
> On Thu, May 17, 2018 at 4:40 PM Dmitry Safonov <dima@xxxxxxxxxx> wrote:
>> Some selftests are failing, but the same way as before the patch
>> (ITOW, it's not regression):
>> [root@localhost self]# grep FAIL out
>> [FAIL] Reg 1 mismatch: requested 0x0; got 0x3
>> [FAIL] Reg 15 mismatch: requested 0x8badf00d5aadc0de; got
>> 0xffffff425aadc0de
>> [FAIL] Reg 15 mismatch: requested 0x8badf00d5aadc0de; got
>> 0xffffff425aadc0de
>> [FAIL] Reg 15 mismatch: requested 0x8badf00d5aadc0de; got
>> 0xffffff425aadc0de
>
> Are you on AMD? Can you try this patch:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/luto/linux.git/commit/?h=x86/fixes&id=c88aa6d53840e48970c54f9ef70c79415033b32d
>
> and give me a Tested-by if it fixes it for you?
Sure.
I'm on Intel actually:
cpu family : 6
model : 142
model name : Intel(R) Core(TM) i7-7600U CPU @ 2.80GHz
But I usually test kernels in VM. So, I use virt-manager as it's
easier to manage
multiple VMs. The thing is that I've chosen "Copy host CPU configuration"
and for some reason, I don't quite follow virt-manager makes model "Opteron_G4".
I'm on Fedora 27, virt-manager 1.4.3, qemu 2.9.1(qemu-2.9.1-2.fc26).
So, cpuinfo in VM says:
cpu family : 21
model : 1
model name : AMD Opteron 62xx class CPU
What's worse than registers changes is that some selftests actually lead to
Oops's. The same reason for criu-ia32 fails.
I've tested so far v4.15 and v4.16 releases besides master (2c71d338bef2),
so it looks to be not a recent regression.
Full Oopses:
[ 189.100174] BUG: unable to handle kernel paging request at 00000000417bafe8
[ 189.100174] PGD 69ed4067 P4D 69ed4067 PUD 707fc067 PMD 6c535067 PTE 6991f067
[ 189.100174] Oops: 0001 [#3] SMP NOPTI
[ 189.100174] Modules linked in:
[ 189.100174] CPU: 0 PID: 2443 Comm: sysret_ss_attrs Tainted: G
D 4.17.0-rc5+ #11
[ 189.103187] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996),
BIOS 1.10.2-1.fc26 04/01/2014
[ 189.103187] RIP: 0033:0x40085a
[ 189.103187] RSP: 002b:00000000417bafe8 EFLAGS: 00000206
[ 189.103187] RAX: 0000000000000000 RBX: 00000000000003e8 RCX: 0000000000000000
[ 189.103187] RDX: 0000000000000000 RSI: 0000000000400830 RDI: 00000000417baff8
[ 189.103187] RBP: 00000000417baff8 R08: 0000000000000000 R09: 0000000000000077
[ 189.103187] R10: 0000000000000006 R11: 0000000000000000 R12: 00000000417ba000
[ 189.103187] R13: 00007ffc05207840 R14: 0000000000000000 R15: 0000000000000000
[ 189.103187] FS: 00007f98566ecb40(0000) GS:ffff9740ffc00000(0000)
knlGS:0000000000000000
[ 189.103187] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[ 189.103187] CR2: 00000000417bafe8 CR3: 0000000069dc4000 CR4: 00000000007406f0
[ 189.103187] PKRU: 55555554
[ 189.103187] RIP: 0x40085a RSP: 00000000417bafe8
[ 189.103187] CR2: 00000000417bafe8
[ 189.103187] ---[ end trace 8878c9a088d5f296 ]---
Killed
[ 219.366814] BUG: unable to handle kernel paging request at 00000000ffd2874c
[ 219.367040] PGD 69fbf067 P4D 69fbf067 PUD 69fa5067 PMD 69fa4067 PTE 6cb04067
[ 219.367040] Oops: 0001 [#4] SMP NOPTI
[ 219.367040] Modules linked in:
[ 219.367040] CPU: 1 PID: 2497 Comm: test_syscall_vd Tainted: G
D 4.17.0-rc5+ #11
[ 219.367040] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996),
BIOS 1.10.2-1.fc26 04/01/2014
[ 219.367040] RIP: 0033:0x8048e9d
[ 219.367040] RSP: 002b:00000000ffd2874c EFLAGS: 00000202
[ 219.367040] RAX: 0000000008048778 RBX: 0000000000000000 RCX: 000000000000003f
[ 219.367040] RDX: 0000000000000001 RSI: 00000000f7ff7b80 RDI: 0000000000000000
[ 219.367040] RBP: 00000000ffd287c8 R08: 7f7f7f7f7f7f7f7f R09: 7f7f7f7f7f7f7f80
[ 219.367040] R10: 7f7f7f7f7f7f7f81 R11: 7f7f7f7f7f7f7f82 R12: 7f7f7f7f7f7f7f83
[ 219.367040] R13: 7f7f7f7f7f7f7f84 R14: 7f7f7f7f7f7f7f85 R15: 7f7f7f7f7f7f7f86
[ 219.367040] FS: 0000000000000000(0000) GS:ffff9740ffd00000(0063)
knlGS:00000000f7fc6700
[ 219.367040] CS: 0010 DS: 002b ES: 002b CR0: 0000000080050033
[ 219.367040] CR2: 00000000ffd2874c CR3: 000000006c4ca000 CR4: 00000000007406e0
[ 219.367040] PKRU: 55555554
[ 219.367040] RIP: 0x8048e9d RSP: 00000000ffd2874c
[ 219.367040] CR2: 00000000ffd2874c
[ 219.367040] ---[ end trace 8878c9a088d5f297 ]---
Killed
When I choose kvm64 (or qemu64) as CPU model, Oops's are gone, but
tests still fail with registers mismatch the same way.
Possibly, Oops's are qemu faults?
>
>> [FAIL] f[u]comi[p] errors: 1
>> [FAIL] fisttp errors: 1'
>
> I don't know about these.
>
>> [FAIL] R8 has changed:0000000000000000
>> [FAIL] R9 has changed:0000000000000000
>> [FAIL] R10 has changed:0000000000000000
>> [FAIL] R11 has changed:0000000000000000
>> [FAIL] R8 has changed:0000000000000000
>> [FAIL] R9 has changed:0000000000000000
>> [FAIL] R10 has changed:0000000000000000
>> [FAIL] R11 has changed:0000000000000000
>
> The patch that added these test lines was the same patch that should have
> made them pass. Are you sure your tests match your running kernel? You
> need commit 8bb2610bc4967f19672444a7b0407367f1540028.
Yeah, it is already in the last master.
> If you still have failures, can you send me the complete output from the
> test_syscall_vdso test?
So, with such possibly loosy qemu (mis-)configuration that I have,
with your patch
applied on the top of the last master, it fixes "Reg 15 mismatch".
Still see the following faults:
======./sigreturn_32========
[OK] set_thread_area refused 16-bit data
[OK] set_thread_area refused 16-bit data
[RUN] Valid sigreturn: 64-bit CS (33), 32-bit SS (2b, GDT)
[FAIL] Reg 1 mismatch: requested 0x0; got 0x3
SP: 5aadc0de -> 5aadc0de
[RUN] Valid sigreturn: 32-bit CS (23), 32-bit SS (2b, GDT)
SP: 5aadc0de -> 5aadc0de
[OK] all registers okay
[RUN] Valid sigreturn: 16-bit CS (37), 32-bit SS (2b, GDT)
SP: 5aadc0de -> 5aadc0de
[OK] all registers okay
[RUN] Valid sigreturn: 64-bit CS (33), 16-bit SS (3f)
SP: 5aadc0de -> 5aadc0de
[OK] all registers okay
--
[RUN] Testing fcmovCC instructions
[OK] fcmovCC
======./test_syscall_vdso_32========
[RUN] Executing 6-argument 32-bit syscall via VDSO
[OK] Arguments are preserved across syscall
[NOTE] R11 has changed:0000000000200ed7 - assuming clobbered by SYSRET insn
[OK] R8..R15 did not leak kernel data
[RUN] Executing 6-argument 32-bit syscall via INT 80
[OK] Arguments are preserved across syscall
[FAIL] R8 has changed:0000000000000000
[FAIL] R9 has changed:0000000000000000
[FAIL] R10 has changed:0000000000000000
[FAIL] R11 has changed:0000000000000000
[RUN] Executing 6-argument 32-bit syscall via VDSO
[OK] Arguments are preserved across syscall
[NOTE] R11 has changed:0000000000200ed7 - assuming clobbered by SYSRET insn
[OK] R8..R15 did not leak kernel data
[RUN] Executing 6-argument 32-bit syscall via INT 80
[OK] Arguments are preserved across syscall
[FAIL] R8 has changed:0000000000000000
[FAIL] R9 has changed:0000000000000000
[FAIL] R10 has changed:0000000000000000
[FAIL] R11 has changed:0000000000000000
[RUN] Running tests under ptrace
Thanks,
Dmitry