arm64: uprobe: Return EOPNOTSUPP for AARCH32 instruction probing
authorHe Zhe <zhe.he@windriver.com>
Tue, 23 Feb 2021 08:25:34 +0000 (16:25 +0800)
committerWill Deacon <will@kernel.org>
Tue, 23 Feb 2021 10:38:27 +0000 (10:38 +0000)
As stated in linux/errno.h, ENOTSUPP should never be seen by user programs.
When we set up uprobe with 32-bit perf and arm64 kernel, we would see the
following vague error without useful hint.

The sys_perf_event_open() syscall returned with 524 (INTERNAL ERROR:
strerror_r(524, [buf], 128)=22)

Use EOPNOTSUPP instead to indicate such cases.

Signed-off-by: He Zhe <zhe.he@windriver.com>
Link: https://lore.kernel.org/r/20210223082535.48730-1-zhe.he@windriver.com
Cc: <stable@vger.kernel.org>
Signed-off-by: Will Deacon <will@kernel.org>
arch/arm64/kernel/probes/uprobes.c

index a412d8e..2c24763 100644 (file)
@@ -38,7 +38,7 @@ int arch_uprobe_analyze_insn(struct arch_uprobe *auprobe, struct mm_struct *mm,
 
        /* TODO: Currently we do not support AARCH32 instruction probing */
        if (mm->context.flags & MMCF_AARCH32)
-               return -ENOTSUPP;
+               return -EOPNOTSUPP;
        else if (!IS_ALIGNED(addr, AARCH64_INSN_SIZE))
                return -EINVAL;